admin/35791-vm
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

End is very near

Browse Source
This commit is contained in:
Flatlogic Bot 2025-12-07 10:47:12 +00:00
parent 403c8e4f27
commit 9ab9dfefba
2 changed files with 57 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
teacher_timetable.php
View File

@ -104,25 +104,23 @@ foreach ($teacher_schedule_raw as $lesson) {
if (isset($timeslot_id_to_period_idx[$lesson['timeslot_id']])) {
$period_idx = $timeslot_id_to_period_idx[$lesson['timeslot_id']];
if (isset($teacher_timetable_by_period[$day_idx][$period_idx])) {
// This slot is already filled, potentially by a multi-class elective.
// Create an array if it's not already one.
if (!is_array($teacher_timetable_by_period[$day_idx][$period_idx])) {
$teacher_timetable_by_period[$day_idx][$period_idx] = [$teacher_timetable_by_period[$day_idx][$period_idx]];
}
$teacher_timetable_by_period[$day_idx][$period_idx][] = $lesson;
} else {
$teacher_timetable_by_period[$day_idx][$period_idx] = $lesson;
// Skip if this slot is already filled by a continuation marker
if (isset($teacher_timetable_by_period[$day_idx][$period_idx]) && isset($teacher_timetable_by_period[$day_idx][$period_idx]['continuation'])) {
continue;
}
if (!empty($lesson['is_double']) && isset($teacher_timetable_by_period[$day_idx][$period_idx + 1])) {
if (isset($teacher_timetable_by_period[$day_idx][$period_idx + 1])) {
if (!is_array($teacher_timetable_by_period[$day_idx][$period_idx + 1])) {
$teacher_timetable_by_period[$day_idx][$period_idx + 1] = [$teacher_timetable_by_period[$day_idx][$period_idx + 1]];
}
$teacher_timetable_by_period[$day_idx][$period_idx + 1][] = $lesson;
// Place the lesson. Handle co-teaching by making it an array.
if (!isset($teacher_timetable_by_period[$day_idx][$period_idx]) || $teacher_timetable_by_period[$day_idx][$period_idx] === null) {
$teacher_timetable_by_period[$day_idx][$period_idx] = [$lesson];
} else {
$teacher_timetable_by_period[$day_idx][$period_idx + 1] = $lesson;
$teacher_timetable_by_period[$day_idx][$period_idx][] = $lesson;
}
if (!empty($lesson['is_double'])) {
$next_period_idx = $period_idx + 1;
if (isset($non_break_periods[$next_period_idx])) {
$continuation_marker = ['continuation' => true, 'original_lesson' => $lesson];
$teacher_timetable_by_period[$day_idx][$next_period_idx] = $continuation_marker;
}
}
}
@ -234,13 +232,27 @@ error_log("Final teacher_timetable_by_period structure: " . print_r($teacher_tim
<small class="text-muted"><?php echo date("g:i A", strtotime($timeslot_info['start_time'])); ?> - <?php echo date("g:i A", strtotime($timeslot_info['end_time'])); ?></small>
</td>
<?php foreach ($days_of_week as $day_idx => $day): ?>
<td class="timetable-slot align-middle">
<?php
$lesson = $teacher_timetable_by_period[$day_idx][$period_idx] ?? null;
if ($lesson) {
// If it's an array of lessons (co-teaching), display them all
$lessons_to_display = is_array($lesson) && !isset($lesson['id']) ? $lesson : [$lesson];
foreach ($lessons_to_display as $single_lesson) {
$lessons = $teacher_timetable_by_period[$day_idx][$period_idx] ?? null;
if (isset($lessons['continuation'])) {
// This is a continuation of a double lesson, so we skip rendering the cell.
} else {
$rowspan = 1;
// Check if any lesson in this slot is a double lesson
if (is_array($lessons)) {
foreach ($lessons as $l) {
if (!empty($l['is_double'])) {
$rowspan = 2;
break;
}
}
}
?>
<td class="timetable-slot align-middle" rowspan="<?php echo $rowspan; ?>">
<?php
if (is_array($lessons)) {
foreach ($lessons as $single_lesson) {
if ($single_lesson) { // Check not null
$display_name = $single_lesson['lesson_display_name'];
// For electives, the teacher should see their specific subject.
@ -256,6 +268,9 @@ error_log("Final teacher_timetable_by_period structure: " . print_r($teacher_tim
}
?>
</td>
<?php
}
?>
<?php endforeach; ?>
</tr>
<?php

8
timetable.php
View File

@ -464,7 +464,13 @@ function get_timetable_from_db($pdo, $classes, $timeslots, $days_of_week) {
$class_timetables[$class_id][$day_idx][$period_idx] = $lesson_data;
if ($lesson_data['is_double'] && isset($class_timetables[$class_id][$day_idx][$period_idx + 1])) {
$class_timetables[$class_id][$day_idx][$period_idx + 1] = $lesson_data;
// For the second part of a double lesson, create a new array
// that is marked as a continuation, but does NOT have is_double = true.
// This prevents the rendering logic from trying to start a new rowspan.
$continuation_data = $lesson_data;
$continuation_data['is_double'] = false;
$continuation_data['is_continuation'] = true;
$class_timetables[$class_id][$day_idx][$period_idx + 1] = $continuation_data;
}
}