Skip to content
Snippets Groups Projects
Commit b987d545 authored by Moritz Strohm's avatar Moritz Strohm
Browse files

removed occurrences of "weekdays" key in schedule settings, fixes #5032

Closes #5032

Merge request studip/studip!3773
parent c2d4d914
No related branches found
No related tags found
No related merge requests found
...@@ -652,13 +652,11 @@ class Calendar_ScheduleController extends AuthenticatedController ...@@ -652,13 +652,11 @@ class Calendar_ScheduleController extends AuthenticatedController
$default_config = [ $default_config = [
'start_time' => '08:00', 'start_time' => '08:00',
'end_time' => '20:00', 'end_time' => '20:00',
'weekdays' => 5,
'visible_days' => [1, 2, 3, 4, 5] 'visible_days' => [1, 2, 3, 4, 5]
]; ];
if ( if (
empty($this->schedule_settings['start_time']) empty($this->schedule_settings['start_time'])
&& empty($this->schedule_settings['end_time']) && empty($this->schedule_settings['end_time'])
&& empty($this->schedule_settings['weekdays'])
&& empty($this->schedule_settings['visible_days']) && empty($this->schedule_settings['visible_days'])
) { ) {
//Use the defaults: //Use the defaults:
...@@ -675,18 +673,12 @@ class Calendar_ScheduleController extends AuthenticatedController ...@@ -675,18 +673,12 @@ class Calendar_ScheduleController extends AuthenticatedController
$start_time = Request::get('start_time', '08:00'); $start_time = Request::get('start_time', '08:00');
$end_time = Request::get('end_time', '20:00'); $end_time = Request::get('end_time', '20:00');
$weekdays = Request::int('weekdays', 5);
$visible_days = Request::intArray('visible_days'); $visible_days = Request::intArray('visible_days');
if ($start_time >= $end_time) { if ($start_time >= $end_time) {
PageLayout::postError(_('Die Startuhrzeit muss vor der Enduhrzeit liegen.')); PageLayout::postError(_('Die Startuhrzeit muss vor der Enduhrzeit liegen.'));
$this->redirect('calendar/schedule/settings'); $this->redirect('calendar/schedule/settings');
return; return;
} }
if (!in_array($weekdays, [5, 7])) {
PageLayout::postError(_('Der Stundenplan kann nur 5 oder 7 Tage anzeigen.'));
$this->redirect('calendar/schedule/settings');
return;
}
if (empty($visible_days)) { if (empty($visible_days)) {
PageLayout::postError(_('Es wurde kein Wochentag ausgewählt.')); PageLayout::postError(_('Es wurde kein Wochentag ausgewählt.'));
$this->redirect('calendar/schedule/settings'); $this->redirect('calendar/schedule/settings');
...@@ -697,7 +689,6 @@ class Calendar_ScheduleController extends AuthenticatedController ...@@ -697,7 +689,6 @@ class Calendar_ScheduleController extends AuthenticatedController
$schedule_settings = [ $schedule_settings = [
'start_time' => $start_time, 'start_time' => $start_time,
'end_time' => $end_time, 'end_time' => $end_time,
'weekdays' => $weekdays,
'visible_days' => $visible_days 'visible_days' => $visible_days
]; ];
......
...@@ -33,16 +33,6 @@ ...@@ -33,16 +33,6 @@
<? endfor ?> <? endfor ?>
</select> </select>
</label> </label>
<label>
<input type="radio" name="weekdays" value="7"
<?= $schedule_settings['weekdays'] === 7 ? 'checked' : '' ?>>
<?= _('Alle Wochentage im Stundenplan anzeigen.') ?>
</label>
<label>
<input type="radio" name="weekdays" value="5"
<?= $schedule_settings['weekdays'] === 5 ? 'checked' : '' ?>>
<?= _('Nur Montag bis Freitag im Stundenplan anzeigen.') ?>
</label>
</fieldset> </fieldset>
<fieldset> <fieldset>
<legend><?= _('Wochentage') ?></legend> <legend><?= _('Wochentage') ?></legend>
......
...@@ -49,11 +49,6 @@ class NewScheduleImprovements extends Migration ...@@ -49,11 +49,6 @@ class NewScheduleImprovements extends Migration
'end_time' => sprintf('%02u:00', $old_config['glb_end_time']), 'end_time' => sprintf('%02u:00', $old_config['glb_end_time']),
'semester_id' => $old_config['semester_id'] ?? $old_config['glb_sem'] ?? null, 'semester_id' => $old_config['semester_id'] ?? $old_config['glb_sem'] ?? null,
]; ];
if (!empty($old_config['glb_days']) && count($old_config['glb_days']) === 7) {
$new_config['weekdays'] = 7;
} else {
$new_config['weekdays'] = 5;
}
//Convert the visible days array: //Convert the visible days array:
$visible_days = []; $visible_days = [];
if (is_array($old_config['glb_days'])) { if (is_array($old_config['glb_days'])) {
......
...@@ -170,7 +170,6 @@ class Helper ...@@ -170,7 +170,6 @@ class Helper
'views' => [ 'views' => [
'timeGridWeek' => [ 'timeGridWeek' => [
'columnHeaderFormat' => ['weekday' => 'short'], 'columnHeaderFormat' => ['weekday' => 'short'],
'weekends' => !empty($schedule_settings['weekdays']) && $schedule_settings['weekdays'] === 7,
'slotDuration' => $slot_duration 'slotDuration' => $slot_duration
] ]
], ],
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment