fix events logic

app/Commands/SendExamDraftDeadlineReminders.php

@@ -0,0 +1,197 @@
+<?php
+
+namespace App\Commands;
+
+use App\Controllers\View\EmailController;
+use App\Models\ClassSectionModel;
+use App\Models\ConfigurationModel;
+use App\Models\TeacherSubmissionNotificationHistoryModel;
+use App\Models\UserModel;
+use CodeIgniter\CLI\BaseCommand;
+use CodeIgniter\CLI\CLI;
+use Config\Database;
+
+class SendExamDraftDeadlineReminders extends BaseCommand
+{
+    protected $group = 'Exam Drafts';
+    protected $name = 'exam-drafts:deadline-reminders';
+    protected $description = 'Sends exam draft deadline reminders to teachers who have not submitted drafts.';
+
+    public function run(array $params)
+    {
+        helper('date');
+
+        $configModel = new ConfigurationModel();
+        $schoolYear = (string) ($configModel->getConfig('school_year') ?? '');
+        $semester = (string) ($configModel->getConfig('semester') ?? '');
+        $deadlineValue = trim((string) ($configModel->getConfig('exam_draft_deadline') ?? ''));
+        if ($deadlineValue === '') {
+            CLI::write('exam_draft_deadline is not configured.', 'yellow');
+            return;
+        }
+
+        $tz = new \DateTimeZone(config('App')->appTimezone ?? 'UTC');
+        try {
+            $deadline = new \DateTimeImmutable($deadlineValue, $tz);
+        } catch (\Throwable $e) {
+            CLI::write('Invalid exam_draft_deadline value.', 'red');
+            return;
+        }
+        $deadline = $deadline->setTime(0, 0, 0);
+        $today = new \DateTimeImmutable('now', $tz);
+        $today = $today->setTime(0, 0, 0);
+
+        if ($today > $deadline) {
+            CLI::write('Deadline has passed. No reminders sent.', 'yellow');
+            return;
+        }
+
+        $daysToDeadline = (int) $today->diff($deadline)->format('%r%a');
+        if (!$this->shouldSendOnDay($daysToDeadline)) {
+            CLI::write('No reminder scheduled for today.', 'yellow');
+            return;
+        }
+
+        $db = Database::connect();
+        $teacherClassRows = $db->table('teacher_class')
+            ->select('teacher_id, class_section_id, school_year, semester')
+            ->when($schoolYear !== '', static function ($builder) use ($schoolYear) {
+                return $builder->where('school_year', $schoolYear);
+            })
+            ->when($semester !== '', static function ($builder) use ($semester) {
+                return $builder->where('semester', $semester);
+            })
+            ->get()
+            ->getResultArray();
+
+        if (empty($teacherClassRows)) {
+            CLI::write('No teacher assignments found.', 'yellow');
+            return;
+        }
+
+        $draftTable = 'exam_drafts';
+        $fields = $db->getFieldNames($draftTable);
+        $teacherColumn = in_array('teacher_id', $fields, true) ? 'teacher_id' : 'author_id';
+        $hasStatusColumn = in_array('status', $fields, true);
+        $hasIsLegacyColumn = in_array('is_legacy', $fields, true);
+
+        $draftsQuery = $db->table($draftTable)
+            ->select("{$teacherColumn} AS teacher_id, class_section_id")
+            ->when($schoolYear !== '', static function ($builder) use ($schoolYear) {
+                return $builder->where('school_year', $schoolYear);
+            })
+            ->when($semester !== '', static function ($builder) use ($semester) {
+                return $builder->where('semester', $semester);
+            });
+
+        if ($hasStatusColumn) {
+            $draftsQuery = $draftsQuery->where('status !=', 'legacy');
+        }
+        if ($hasIsLegacyColumn) {
+            $draftsQuery = $draftsQuery->where('is_legacy', 0);
+        }
+
+        $draftRows = $draftsQuery->get()->getResultArray();
+        $submittedMap = [];
+        foreach ($draftRows as $row) {
+            $key = (int) ($row['teacher_id'] ?? 0) . '|' . (int) ($row['class_section_id'] ?? 0);
+            $submittedMap[$key] = true;
+        }
+
+        $missingByTeacher = [];
+        foreach ($teacherClassRows as $row) {
+            $teacherId = (int) ($row['teacher_id'] ?? 0);
+            $classSectionId = (int) ($row['class_section_id'] ?? 0);
+            if ($teacherId <= 0 || $classSectionId <= 0) {
+                continue;
+            }
+            $key = $teacherId . '|' . $classSectionId;
+            if (!isset($submittedMap[$key])) {
+                $missingByTeacher[$teacherId][] = $classSectionId;
+            }
+        }
+
+        if (empty($missingByTeacher)) {
+            CLI::write('All teachers have submitted drafts.', 'green');
+            return;
+        }
+
+        $userModel = new UserModel();
+        $classSectionModel = new ClassSectionModel();
+        $historyModel = new TeacherSubmissionNotificationHistoryModel();
+        $mailer = new EmailController();
+
+        $classSections = $classSectionModel
+            ->select('class_section_id, class_section_name')
+            ->findAll();
+        $classLookup = [];
+        foreach ($classSections as $section) {
+            $classLookup[(int) ($section['class_section_id'] ?? 0)] = $section['class_section_name'] ?? '';
+        }
+
+        $todayStamp = $today->format('Y-m-d');
+        foreach ($missingByTeacher as $teacherId => $sectionIds) {
+            $teacher = $userModel->find($teacherId);
+            $email = $teacher['email'] ?? '';
+            if ($email === '' || !filter_var($email, FILTER_VALIDATE_EMAIL)) {
+                continue;
+            }
+
+            $alreadySent = $historyModel
+                ->where('teacher_id', $teacherId)
+                ->where('notification_category', 'exam_draft_deadline')
+                ->where('school_year', $schoolYear)
+                ->where('semester', $semester)
+                ->like('sent_at', $todayStamp)
+                ->countAllResults() > 0;
+            if ($alreadySent) {
+                continue;
+            }
+
+            $classNames = array_map(static function ($id) use ($classLookup) {
+                return $classLookup[(int) $id] ?? "Class {$id}";
+            }, $sectionIds);
+            $classList = implode(', ', $classNames);
+            $teacherName = trim(($teacher['firstname'] ?? '') . ' ' . ($teacher['lastname'] ?? ''));
+            if ($teacherName === '') {
+                $teacherName = 'Teacher';
+            }
+
+            $subject = 'Reminder: Exam draft submission deadline';
+            $body = '<p>Dear ' . esc($teacherName) . ',</p>'
+                . '<p>This is a reminder to submit your exam draft(s) for: ' . esc($classList) . '.</p>'
+                . '<p>Deadline: <strong>' . esc($deadline->format('Y-m-d')) . '</strong></p>'
+                . '<p>Please submit your draft at <a href="' . esc(base_url('teacher/exam-drafts')) . '">Teacher Exam Drafts</a>.</p>'
+                . '<p>Thank you.</p>';
+
+            $status = 'failed';
+            if ($mailer->sendEmail($email, $subject, $body, 'notifications')) {
+                $status = 'sent';
+                CLI::write("Sent reminder to {$email}", 'green');
+            }
+
+            $historyModel->insert([
+                'teacher_id' => $teacherId,
+                'class_section_id' => 0,
+                'admin_id' => null,
+                'notification_category' => 'exam_draft_deadline',
+                'message' => strip_tags($body),
+                'status' => $status,
+                'school_year' => $schoolYear,
+                'semester' => $semester,
+                'sent_at' => utc_now(),
+            ]);
+        }
+    }
+
+    private function shouldSendOnDay(int $daysToDeadline): bool
+    {
+        if ($daysToDeadline < 0) {
+            return false;
+        }
+        if ($daysToDeadline <= 4) {
+            return true;
+        }
+        return in_array($daysToDeadline, [28, 21, 14, 7], true);
+    }
+}

app/Config/App.php

@@ -82,7 +82,7 @@ class App extends BaseConfig
     | DO NOT CHANGE THIS UNLESS YOU FULLY UNDERSTAND THE REPERCUSSIONS!!
     |
     */
-    public string $permittedURIChars = 'a-z 0-9~%.:_\-';
+    public string $permittedURIChars = 'a-z 0-9~%.:_\-,';
 
     /**
      * --------------------------------------------------------------------------

app/Config/Routes.php

@@ -363,6 +363,7 @@ $routes->get('/teacher/teacher_contactus', 'View\TeacherController::contactusTea
 
 $routes->get('/teacher/exam-drafts', 'View\ExamDraftController::teacherIndex', ['filter' => 'auth:teacher,teacher_assistant,teacher_dashboard,read']);
 $routes->post('/teacher/exam-drafts', 'View\ExamDraftController::teacherStore', ['filter' => 'auth:teacher,teacher_assistant,teacher_dashboard,read']);
+$routes->get('/teacher/exam-drafts/status', 'View\ExamDraftController::teacherStatusFeed', ['filter' => 'auth:teacher,teacher_assistant,teacher_dashboard,read']);
 
 
 

app/Controllers/ClassProgressController.php

@@ -28,6 +28,10 @@ class ClassProgressController extends BaseController
             'db_subject' => 'Quran/Arabic',
         ],
     ];
+
+    /** Unit column for teacher custom Islamic / Quran rows; must match teacher form JS. Segment: "Custom / {text}". */
+    public const CUSTOM_UNIT_ROW_LABEL = 'Custom';
+
     protected ClassProgressReportModel $reportModel;
     protected ClassProgressAttachmentModel $attachmentModel;
     protected TeacherClassModel $teacherClassModel;
@@ -383,13 +387,18 @@ class ClassProgressController extends BaseController
             }
         }
 
+        $sundayOptions = $this->buildSundayOptions();
+        if (!in_array($row['week_start'], $sundayOptions, true)) {
+            array_unshift($sundayOptions, $row['week_start']);
+        }
+
         return view('teacher/class_progress_submit', [
             'subjectSections' => self::SUBJECT_SECTIONS,
             'subjectCurriculum' => $subjectCurriculum,
             'classSectionId' => $row['class_section_id'],
             'classSectionName' => $classSectionName,
             'classId' => $classId,
-            'sundayOptions' => [$row['week_start']],
+            'sundayOptions' => $sundayOptions,
             'defaultWeekStart' => $row['week_start'],
             'existingWeekEnd' => $row['week_end'],
             'existingReports' => $subjectReports,
@@ -458,6 +467,32 @@ class ClassProgressController extends BaseController
             return redirect()->back()->withInput()->with('error', 'No class assignment found for this report.');
         }
 
+        $confirmOverwrite = (bool) $this->request->getPost('confirm_overwrite');
+        if ($weekStart && $weekStart !== (string) ($row['week_start'] ?? '')) {
+            $conflicts = $this->reportModel
+                ->select('id')
+                ->where('class_section_id', $classSectionId)
+                ->where('week_start', $weekStart)
+                ->where('teacher_id', $teacherId)
+                ->findAll();
+            if (! $confirmOverwrite && ! empty($conflicts)) {
+                return redirect()->back()
+                    ->withInput()
+                    ->with('warning', 'A progress report already exists for this week, are you sure you want to override it?')
+                    ->with('confirm_overwrite', true);
+            }
+            if ($confirmOverwrite && ! empty($conflicts)) {
+                $conflictIds = array_values(array_filter(array_map(
+                    static fn (array $row): int => (int) ($row['id'] ?? 0),
+                    $conflicts
+                )));
+                if (! empty($conflictIds)) {
+                    $this->attachmentModel->whereIn('report_id', $conflictIds)->delete();
+                    $this->reportModel->whereIn('id', $conflictIds)->delete();
+                }
+            }
+        }
+
         $weeklyReports = $this->reportModel
             ->select('class_progress_reports.*')
             ->whereIn('teacher_id', $allowedTeacherIds)
@@ -674,6 +709,37 @@ class ClassProgressController extends BaseController
         return $flags ? json_encode($flags) : null;
     }
 
+    /**
+     * Split stored unit_title into curriculum lines vs custom teacher-typed subjects ("Custom / …").
+     * Keep in sync with teacher form, {@see buildUnitChapterSummary()}, and admin views.
+     *
+     * @return array{curriculum: list<string>, custom: list<string>}
+     */
+    public static function splitUnitTitleForDisplay(string $unitTitle): array
+    {
+        $unitTitle = trim($unitTitle);
+        if ($unitTitle === '') {
+            return ['curriculum' => [], 'custom' => []];
+        }
+
+        $segments = preg_split('/\s*;\s*/', $unitTitle, -1, PREG_SPLIT_NO_EMPTY);
+        $curriculum = [];
+        $custom = [];
+        foreach ($segments as $seg) {
+            $seg = trim((string) $seg);
+            if ($seg === '') {
+                continue;
+            }
+            if (preg_match('/^Custom\s*\/\s*(.+)$/iu', $seg, $m)) {
+                $custom[] = trim($m[1]);
+            } else {
+                $curriculum[] = $seg;
+            }
+        }
+
+        return ['curriculum' => $curriculum, 'custom' => $custom];
+    }
+
     protected function buildUnitChapterSummary(string $slug): ?string
     {
         $unitValues = array_map('trim', (array) $this->request->getPost("unit_$slug"));
@@ -686,9 +752,12 @@ class ClassProgressController extends BaseController
             if ($unit === '' && $chapter === '') {
                 continue;
             }
+            if (strcasecmp($unit, self::CUSTOM_UNIT_ROW_LABEL) === 0 && $chapter !== '') {
+                $unit = self::CUSTOM_UNIT_ROW_LABEL;
+            }
             $segment = $unit;
             if ($chapter !== '') {
-                $segment = $segment !== '' ? $segment . ' / ' . $chapter : $chapter;
+                $segment = $segment !== '' ? $unit . ' / ' . $chapter : $chapter;
             }
             if ($segment === '') {
                 continue;
@@ -699,17 +768,23 @@ class ClassProgressController extends BaseController
             return null;
         }
         $summary = implode(' ; ', $parts);
+
         return mb_strlen($summary) > 120 ? mb_substr($summary, 0, 120) : $summary;
     }
 
     protected function hasIslamicUnitSelection(): bool
     {
         $unitValues = array_map('trim', (array) $this->request->getPost('unit_islamic'));
-        foreach ($unitValues as $value) {
+        $chapterValues = array_map('trim', (array) $this->request->getPost('chapter_islamic'));
-            if ($value !== '') {
+        $count = max(count($unitValues), count($chapterValues));
+        for ($i = 0; $i < $count; $i++) {
+            $unit = $unitValues[$i] ?? '';
+            $chapter = $chapterValues[$i] ?? '';
+            if ($unit !== '' || $chapter !== '') {
                 return true;
             }
         }
+
         return false;
     }
 
@@ -728,9 +803,19 @@ class ClassProgressController extends BaseController
             if ($segment === '') {
                 continue;
             }
+            if (preg_match('/^Custom\s*\/\s*(.+)$/iu', $segment, $m)) {
+                $units[] = self::CUSTOM_UNIT_ROW_LABEL;
+                $chapters[] = trim($m[1]);
+
+                continue;
+            }
             $parts = preg_split('/\s*\/\s*/', $segment, 2);
             if (count($parts) === 2) {
-                $units[] = trim($parts[0]);
+                $u = trim($parts[0]);
+                if (strcasecmp($u, self::CUSTOM_UNIT_ROW_LABEL) === 0) {
+                    $u = self::CUSTOM_UNIT_ROW_LABEL;
+                }
+                $units[] = $u;
                 $chapters[] = trim($parts[1]);
             } else {
                 $units[] = $segment;

app/Controllers/View/AdministratorController.php

@@ -763,7 +763,13 @@ class AdministratorController extends BaseController
 
         [$progressExpectedWeeks, $progressSubmittedBySection] = $this->buildClassProgressStats($sectionIds);
         $examDraftCounts = [];
-        $examDraftDeadline = $this->resolveExamDraftDeadline($semester, $schoolYear);
+        $examDraftDeadline = $this->resolveTeacherDashboardExamDraftDeadline($semester, $schoolYear);
+        $examDraftDeadlineConfig = trim((string) ($this->configModel->getConfig('exam_draft_deadline') ?? ''));
+        $examDraftDeadlineFormatted = '';
+        if ($examDraftDeadlineConfig !== '') {
+            $parsedUi = $this->parseExamDraftDeadlineConfigValue();
+            $examDraftDeadlineFormatted = $parsedUi !== null ? $parsedUi->format('M j, Y') : '';
+        }
         $homeworkCounts = [];
         if (! empty($sectionIds)) {
             $draftBuilder = $examDrafts
@@ -1066,6 +1072,8 @@ class AdministratorController extends BaseController
             'notificationHistory' => $historyMap,
             'summary' => $summary,
             'lowProgressSectionIds' => $lowProgressSectionIds,
+            'examDraftDeadlineConfig' => $examDraftDeadlineConfig,
+            'examDraftDeadlineFormatted' => $examDraftDeadlineFormatted,
         ]);
     }
 
@@ -1299,6 +1307,7 @@ class AdministratorController extends BaseController
         $progressUrl = site_url('teacher/progress/history');
         $examDraftUrl = site_url('teacher/exam-drafts');
         $homeworkUrl = site_url('teacher/addHomework');
+        $examDraftDeadlineEmailHtml = $this->buildExamDraftDeadlineEmailHtml();
         $sentCount = 0;
         $failCount = 0;
 
@@ -1347,7 +1356,8 @@ class AdministratorController extends BaseController
                 } else {
                     $draftLabel = 'exam draft';
                 }
-                $examDraftNote = "<p>" . ucfirst($draftLabel) . " submissions can be updated at <a href=\"{$examDraftUrl}\">Teacher Exam Drafts</a>.</p>";
+                $examDraftNote = "<p>" . ucfirst($draftLabel) . " submissions can be updated at <a href=\"{$examDraftUrl}\">Teacher Exam Drafts</a>.</p>"
+                    . $examDraftDeadlineEmailHtml;
             }
             $homeworkNote = '';
             if (in_array('homework', $missingItems, true)) {
@@ -1485,6 +1495,60 @@ class AdministratorController extends BaseController
         ];
     }
 
+    /**
+     * Exam draft due date for the teacher submissions dashboard: prefers the configuration key
+     * `exam_draft_deadline` (same as automated reminders); otherwise fall/spring exam deadlines.
+     */
+    private function resolveTeacherDashboardExamDraftDeadline(string $semester, string $schoolYear): ?\DateTimeImmutable
+    {
+        $fromExamDraftKey = $this->parseExamDraftDeadlineConfigValue();
+        if ($fromExamDraftKey !== null) {
+            return $fromExamDraftKey;
+        }
+
+        return $this->resolveExamDraftDeadline($semester, $schoolYear);
+    }
+
+    /**
+     * Parses the `exam_draft_deadline` configuration value using the application timezone (midnight that calendar day).
+     */
+    private function parseExamDraftDeadlineConfigValue(): ?\DateTimeImmutable
+    {
+        $raw = trim((string) ($this->configModel->getConfig('exam_draft_deadline') ?? ''));
+        if ($raw === '') {
+            return null;
+        }
+        $tz = new \DateTimeZone(config('App')->appTimezone ?? 'UTC');
+        try {
+            $deadline = new \DateTimeImmutable($raw, $tz);
+        } catch (\Throwable $e) {
+            return null;
+        }
+
+        return $deadline->setTime(0, 0, 0);
+    }
+
+    /**
+     * HTML snippet for reminder emails when exam draft is included (deadline from exam_draft_deadline config).
+     */
+    private function buildExamDraftDeadlineEmailHtml(): string
+    {
+        $raw = trim((string) ($this->configModel->getConfig('exam_draft_deadline') ?? ''));
+        if ($raw === '') {
+            return '';
+        }
+        $parsed = $this->parseExamDraftDeadlineConfigValue();
+        $display = $parsed !== null
+            ? htmlspecialchars($parsed->format('l, F j, Y'), ENT_QUOTES, 'UTF-8')
+            : htmlspecialchars($raw, ENT_QUOTES, 'UTF-8');
+        $rawEsc = htmlspecialchars($raw, ENT_QUOTES, 'UTF-8');
+
+        return '<p><strong>Exam draft submission deadline</strong> (<code>exam_draft_deadline</code>): '
+            . "<strong>{$display}</strong>"
+            . ($parsed !== null && $rawEsc !== $display ? " <span style=\"color:#555;\">(configured value: {$rawEsc})</span>" : '')
+            . '.</p>';
+    }
+
     private function resolveExamDraftDeadline(string $semester, string $schoolYear): ?\DateTimeImmutable
     {
         $semesterKey = strtolower(trim($semester));

app/Controllers/View/EventController.php

@@ -93,58 +93,6 @@ class EventController extends ResourceController
                 'created_by' => session()->get('user_id'),
             ]);
 
-            if ($eventId) {
-                $amount = (float) $this->request->getPost('amount');
-                $semester = (string) $this->request->getPost('semester');
-                $schoolYear = (string) $this->request->getPost('school_year');
-                $userId = (int) (session()->get('user_id') ?? 0);
-
-                $enrollments = $this->enrollmentModel
-                    ->select('enrollments.student_id, students.parent_id')
-                    ->join('students', 'students.id = enrollments.student_id', 'left')
-                    ->where('enrollments.school_year', $schoolYear)
-                    ->whereIn('enrollments.enrollment_status', ['enrolled', 'payment pending'])
-                    ->findAll();
-
-                $parentIds = [];
-                foreach ($enrollments as $row) {
-                    $studentId = (int) ($row['student_id'] ?? 0);
-                    $parentId = (int) ($row['parent_id'] ?? 0);
-                    if ($studentId <= 0 || $parentId <= 0) {
-                        continue;
-                    }
-
-                    $exists = $this->eventChargesModel
-                        ->where('event_id', $eventId)
-                        ->where('student_id', $studentId)
-                        ->where('school_year', $schoolYear)
-                        ->where('semester', $semester)
-                        ->first();
-
-                    if ($exists) {
-                        continue;
-                    }
-
-                    $this->eventChargesModel->insert([
-                        'event_id' => $eventId,
-                        'parent_id' => $parentId,
-                        'student_id' => $studentId,
-                        'participation' => 'yes',
-                        'charged' => $amount,
-                        'school_year' => $schoolYear,
-                        'semester' => $semester,
-                        'updated_by' => $userId ?: null,
-                    ]);
-
-                    $parentIds[] = $parentId;
-                }
-
-                $parentIds = array_unique($parentIds);
-                foreach ($parentIds as $pid) {
-                    $this->invoiceController->generateInvoice((string) $pid);
-                }
-            }
-
             return redirect()->to('/administrator/events')->with('success', 'Event created successfully');
         }
 
@@ -240,17 +188,24 @@ class EventController extends ResourceController
 
         $parents = $this->userModel->getParents();
         $events = $this->eventModel->getActiveEvents($this->schoolYear);
+        $filterEventId = (int) ($this->request->getGet('event_id') ?? 0);
 
-        $charges = $this->eventChargesModel
+        $chargesBuilder = $this->eventChargesModel
             ->select('event_charges.*, 
                     users.firstname AS parent_firstname, users.lastname AS parent_lastname, 
                     students.firstname AS student_firstname, students.lastname AS student_lastname,
-                    events.event_name')
+                    events.event_name, events.description AS event_description, events.amount AS event_amount')
             ->join('users', 'users.id = event_charges.parent_id', 'left')
             ->join('students', 'students.id = event_charges.student_id', 'left')
             ->join('events', 'events.id = event_charges.event_id', 'left')
             ->where('event_charges.school_year', $schoolYear)
-            ->where('event_charges.semester', $semester)
+            ->where('event_charges.semester', $semester);
+
+        if ($filterEventId > 0) {
+            $chargesBuilder->where('event_charges.event_id', $filterEventId);
+        }
+
+        $charges = $chargesBuilder
             ->orderBy('event_charges.created_at', 'DESC')
             ->findAll();
 
@@ -260,6 +215,7 @@ class EventController extends ResourceController
             'events'  => $events,
             'school_year' => $schoolYear,
             'semester' => $semester,
+            'filterEventId' => $filterEventId,
         ]);
     }
 

app/Controllers/View/FilesController.php

@@ -271,8 +271,8 @@ class FilesController extends Controller
 
     private function buildDraftDownloadName(string $filename, string $subdir): string
     {
-        $column = $subdir === 'finals' ? 'final_file' : 'teacher_file';
         $db = Database::connect();
+        $column = $subdir === 'finals' ? 'final_file' : $this->resolveExamDraftFileColumn($db);
         $row = $db->table('exam_drafts ed')
             ->select('ed.version, ed.exam_type, ed.class_section_id, cs.class_section_name')
             ->join('classSection cs', 'cs.class_section_id = ed.class_section_id', 'left')
@@ -301,4 +301,20 @@ class FilesController extends Controller
         $value = trim($value, '_');
         return $value === '' ? 'Exam' : mb_strtolower($value);
     }
+
+    private function resolveExamDraftFileColumn($db): string
+    {
+        try {
+            $fields = $db->getFieldNames('exam_drafts');
+            if (in_array('teacher_file', $fields, true)) {
+                return 'teacher_file';
+            }
+            if (in_array('author_file', $fields, true)) {
+                return 'author_file';
+            }
+        } catch (\Throwable $e) {
+            log_message('error', 'FilesController::resolveExamDraftFileColumn error: ' . $e->getMessage());
+        }
+        return 'teacher_file';
+    }
 }

app/Controllers/View/FinancialController.php

@@ -216,25 +216,28 @@ public function financialReport()
         $schoolYears[] = (string)$schoolYear;
     }
     
+    $eventFeesTotal = $this->getEventFeesTotal($schoolYear, $dateFrom, $dateTo);
+
     // JSON API support
     if ($this->wantsJson() || strtolower((string)($this->request->getGet('format') ?? '')) === 'json') {
-        return $this->response->setJSON([
+      return $this->response->setJSON([
-            'ok'               => true,
+        'ok'               => true,
-            'selectedYear'     => $schoolYear,
+        'selectedYear'     => $schoolYear,
-            'dateFrom'         => $dateFrom,
+        'dateFrom'         => $dateFrom,
-            'dateTo'           => $dateTo,
+        'dateTo'           => $dateTo,
             'schoolYears'      => $schoolYears,
             'invoices'         => $invoices,
             'payments'         => $payments,
             'paymentBreakdown' => $paymentBreakdown,
             'paymentTotals'    => $paymentTotals,
-            'refunds'          => $refunds,
+        'refunds'          => $refunds,
-            'expenses'         => $expenses,
+        'expenses'         => $expenses,
-            'reimbursements'   => $reimbursements,
+        'reimbursements'   => $reimbursements,
-            'discounts'        => $discounts,
+        'discounts'        => $discounts,
-            'csrf_token'       => csrf_token(),
+        'eventFeesTotal'   => $eventFeesTotal,
-            'csrf_hash'        => csrf_hash(),
+        'csrf_token'       => csrf_token(),
-        ]);
+        'csrf_hash'        => csrf_hash(),
+      ]);
     }
 
     return view('payment/financial_report', [
@@ -246,6 +249,7 @@ public function financialReport()
         'expenses'         => $expenses,
         'reimbursements'   => $reimbursements,
         'discounts'        => $discounts,
+        'eventFeesTotal'   => $eventFeesTotal,
         'selectedYear'     => $schoolYear,
         'schoolYears'      => $schoolYears,
         'dateFrom'         => $dateFrom,
@@ -1058,6 +1062,7 @@ public function financialReport()
         $amountCollected = $totalPaid;
         $netAmount = ($totalCharges - $totalDiscounts - $totalRefunds);
 
+        $totalEventFees = $this->getEventFeesTotal($schoolYear, $invoiceDateFrom, $invoiceDateTo);
         return [
             'schoolYear' => $schoolYear,
             'dateFrom' => $dateFrom,
@@ -1073,9 +1078,28 @@ public function financialReport()
             'amountCollected' => $amountCollected,
             'totalUnpaid' => $totalUnpaid,
             'netAmount' => $netAmount,
+            'totalEventFees' => $totalEventFees,
         ];
     }
 
+    private function getEventFeesTotal(?string $schoolYear, ?string $dateFrom, ?string $dateTo): float
+    {
+        $db = \Config\Database::connect();
+        $builder = $db->table('event_charges ec')
+            ->select('COALESCE(SUM(ec.charged),0) AS amount', false);
+        if (!empty($schoolYear)) {
+            $builder->where('ec.school_year', $schoolYear);
+        }
+        if (!empty($dateFrom)) {
+            $builder->where('DATE(ec.created_at) >=', $dateFrom);
+        }
+        if (!empty($dateTo)) {
+            $builder->where('DATE(ec.created_at) <=', $dateTo);
+        }
+        $row = $builder->get()->getRowArray();
+        return $row ? (float)($row['amount'] ?? 0) : 0.0;
+    }
+
 
     /**
      * Management page: list parents with outstanding balances (> 0) for a school year.
@@ -1226,6 +1250,23 @@ public function financialReport()
             $byParent[$pid]['total_balance'] += $extra;
         }
 
+        $eventFeesPerParent = [];
+        try {
+            $eventFeesRows = $db->table('event_charges ec')
+                ->select('ec.parent_id, COALESCE(SUM(ec.charged),0) AS event_fees', false)
+                ->where('ec.school_year', $schoolYear)
+                ->groupBy('ec.parent_id')
+                ->get()
+                ->getResultArray();
+            foreach ($eventFeesRows as $row) {
+                $pid = (int)($row['parent_id'] ?? 0);
+                if ($pid <= 0) continue;
+                $eventFeesPerParent[$pid] = (float)($row['event_fees'] ?? 0);
+            }
+        } catch (\Throwable $e) {
+            // ignore, fallback to no event fees data
+        }
+
         // Reduce into rows list; only parents with positive balance
         // Also compute remaining installments and suggested monthly amount
         $rows = [];
@@ -1305,7 +1346,7 @@ public function financialReport()
             }
         }
 
-        $dataRows = array_map(function(array $r) use ($hasPayments, $paidTotals, $paymentCounts, $nextInstallmentYmd) {
+        $dataRows = array_map(function(array $r) use ($hasPayments, $paidTotals, $paymentCounts, $nextInstallmentYmd, $eventFeesPerParent) {
             $pid = (int)($r['parent_id'] ?? 0);
             $name = trim((string)($r['firstname'] ?? '') . ' ' . (string)($r['lastname'] ?? ''));
             return [
@@ -1322,6 +1363,7 @@ public function financialReport()
                 'payment_count' => isset($paymentCounts[$pid]) ? (int)$paymentCounts[$pid] : 0,
                 'has_installment'=> isset($hasPayments[$pid]) ? 1 : 0,
                 'next_installment' => $nextInstallmentYmd,
+                'event_fees'     => (float)($eventFeesPerParent[$pid] ?? 0),
             ];
         }, $rows);
 

app/Controllers/View/HomeworkTrackingController.php

@@ -86,6 +86,7 @@ class HomeworkTrackingController extends BaseController
 
         $hasHomework = [];
         $hwEnteredAt = [];
+        $homeworkSubmissionCounts = [];
         foreach ($rows as $r) {
             $csid = (int)($r['class_section_id'] ?? 0);
             $hi   = (int)($r['homework_index'] ?? 0);
@@ -94,6 +95,7 @@ class HomeworkTrackingController extends BaseController
                 $hasHomework[$csid][$hi] = true;
                 $dateStr = substr((string)($r['first_created'] ?? ''), 0, 10);
                 $hwEnteredAt[$csid][$hi] = $dateStr ?: null;
+                $homeworkSubmissionCounts[$csid] = ($homeworkSubmissionCounts[$csid] ?? 0) + 1;
             }
         }
 
@@ -217,6 +219,7 @@ class HomeworkTrackingController extends BaseController
             'teachers'     => $teachersPage,
             'hasHomework'  => $hasHomework,
             'hwEnteredAt'  => $hwEnteredAt,
+            'homeworkSubmissionCounts' => $homeworkSubmissionCounts,
             'hasHomeworkByDate' => $hasHomeworkByDate,
             'hwEnteredAtByDate' => $hwEnteredAtByDate,
             'page'         => $page,

app/Controllers/View/LandingPageController.php

@@ -95,6 +95,11 @@ class LandingPageController extends BaseController
             return null;
         }
 
+        // Teacher class sections only apply to the teacher role; other roles have no teacher_class rows.
+        if (strtolower(trim((string) $this->getUserRole())) !== 'teacher') {
+            return null;
+        }
+
         // Get all class assignments for this teacher in the current term
         $assignments = $this->teacherClassModel->getClassAssignmentsByUserId(
             (int)$user_id,
@@ -106,7 +111,7 @@ class LandingPageController extends BaseController
         $ids = array_values(array_filter(array_unique($ids)));
 
         if (empty($ids)) {
-            log_message('error', "No class section found for user ID: $user_id");
+            log_message('warning', "No class section found for user ID: $user_id");
             return null;
         }
 
@@ -868,8 +873,12 @@ class LandingPageController extends BaseController
 
     protected function getUserRole()
     {
-        // Assuming you have a session variable storing the user role
+        $active = session()->get('active_role');
-        return session()->get('user_role', 'guest'); // Default to guest if not set
+        if ($active !== null && $active !== '') {
+            return (string) $active;
+        }
+
+        return (string) (session()->get('role') ?? 'guest');
     }
 
     protected function getUserRoleFromDatabase($user_id)

app/Controllers/View/ScoreController.php

@@ -1287,14 +1287,14 @@ class ScoreController extends Controller
 public function viewStudentScore()
 {
     $parentId = session()->get('user_id');
-    $userType = $_SESSION['user_type'];
+    $userType = session()->get('user_type') ?? '';
     $firstParentId = null;
     $releaseFall = $this->getParentScoresReleasedForSemester('Fall');
     $releaseSpring = $this->getParentScoresReleasedForSemester('Spring');
     $releaseAny = $releaseFall || $releaseSpring;
 
     // Identify the firstparent based on user type
-    if ($userType === 'primary') {
+    if ($userType === 'primary' || $userType === '') {
         $firstParentId = $parentId;
     } elseif ($userType === 'secondary') {
         $parentData = $this->db->table('parents')

app/Controllers/View/TeacherController.php

@@ -297,12 +297,16 @@ class TeacherController extends BaseController
         $schoolYear = $forYear ?: ((string)($this->schoolYear ?? 'Not Set'));
 
         $teachers = $this->teacherModel->getTeachersAndTAs();
-        // Prefer class sections for the selected year, fallback to all if none
+        // Prefer class sections for the selected year if the column exists, otherwise fallback to all.
-        $classSections = $classSectionModel
+        if ($this->db->fieldExists('school_year', 'classSection')) {
-            ->where('school_year', $schoolYear)
+            $classSections = $classSectionModel
-            ->orderBy('class_section_name', 'ASC')
+                ->where('school_year', $schoolYear)
-            ->findAll();
+                ->orderBy('class_section_name', 'ASC')
-        if (empty($classSections)) {
+                ->findAll();
+            if (empty($classSections)) {
+                $classSections = $classSectionModel->orderBy('class_section_name', 'ASC')->findAll();
+            }
+        } else {
             $classSections = $classSectionModel->orderBy('class_section_name', 'ASC')->findAll();
         }
 

app/Database/Migrations/2026-01-27-220000_AddReviewRevisionToExamDrafts.php

@@ -0,0 +1,30 @@
+<?php
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+
+class AddReviewRevisionToExamDrafts extends Migration
+{
+    public function up()
+    {
+        if (! $this->db->fieldExists('review_revision', 'exam_drafts')) {
+            $this->forge->addColumn('exam_drafts', [
+                'review_revision' => [
+                    'type' => 'INT',
+                    'constraint' => 11,
+                    'null' => false,
+                    'default' => 0,
+                    'after' => 'version',
+                ],
+            ]);
+        }
+    }
+
+    public function down()
+    {
+        if ($this->db->fieldExists('review_revision', 'exam_drafts')) {
+            $this->forge->dropColumn('exam_drafts', 'review_revision');
+        }
+    }
+}

app/Models/ClassProgressReportModel.php

@@ -4,6 +4,15 @@ namespace App\Models;
 
 use CodeIgniter\Model;
 
+/**
+ * Weekly class progress reports (one row per subject per week).
+ *
+ * unit_title stores a compact summary of unit/chapter rows: segments joined with " ; ".
+ * Teacher-entered custom Islamic or Quran topics use the prefix "Custom / {text}"
+ * (see {@see \App\Controllers\ClassProgressController::CUSTOM_UNIT_ROW_LABEL} and
+ * {@see \App\Controllers\ClassProgressController::splitUnitTitleForDisplay()}).
+ * DB column is VARCHAR(120); controller truncates to match.
+ */
 class ClassProgressReportModel extends Model
 {
     protected $table = 'class_progress_reports';
@@ -27,7 +36,42 @@ class ClassProgressReportModel extends Model
         'flags_json',
         'attachment_path',
     ];
+
     protected $useTimestamps = true;
     protected $createdField = 'created_at';
     protected $updatedField = 'updated_at';
+
+    /**
+     * Validation is performed in {@see \App\Controllers\ClassProgressController} on HTTP input.
+     * Rules below document schema limits and can be enabled if you set {@see $skipValidation} to false.
+     */
+    protected $skipValidation = true;
+
+    protected $validationRules = [
+        'class_section_id' => 'required|integer',
+        'teacher_id' => 'required|integer',
+        'week_start' => 'required|valid_date[Y-m-d]',
+        'week_end' => 'required|valid_date[Y-m-d]',
+        'subject' => 'required|string|max_length[160]',
+        'unit_title' => 'permit_empty|string|max_length[120]',
+        'covered' => 'permit_empty|string',
+        'homework' => 'permit_empty|string',
+        'assessment' => 'permit_empty|string',
+        'status' => 'permit_empty|in_list[on_track,slightly_behind,behind]',
+        'status_notes' => 'permit_empty|string|max_length[200]',
+        'class_notes' => 'permit_empty|string',
+        'next_week_plan' => 'permit_empty|string',
+        'support_needed' => 'permit_empty|string',
+        'flags_json' => 'permit_empty|string',
+        'attachment_path' => 'permit_empty|string|max_length[255]',
+    ];
+
+    protected $validationMessages = [
+        'unit_title' => [
+            'max_length' => 'Unit and chapter summary cannot exceed 120 characters.',
+        ],
+        'subject' => [
+            'max_length' => 'Subject cannot exceed 160 characters.',
+        ],
+    ];
 }

app/Models/EventChargesModel.php

@@ -30,7 +30,7 @@ class EventChargesModel extends Model
 
     public function getChargesWithEventInfo($parentId = null, $schoolYear = null, $semester = null)
     {
-        $builder = $this->select('event_charges.*, events.event_name, events.amount')
+        $builder = $this->select('event_charges.*, events.event_name, events.amount AS event_amount, events.description AS event_description')
                         ->join('events', 'events.id = event_charges.event_id', 'left');
 
         if ($parentId) {

app/Models/ExamDraftModel.php

@@ -8,19 +8,51 @@ class ExamDraftModel extends Model
 {
     protected $table = 'exam_drafts';
     protected $primaryKey = 'id';
+    protected $returnType = 'array';
+    protected $useTimestamps = true;
+    protected $createdField = 'created_at';
+    protected $updatedField = 'updated_at';
+    protected bool $updateOnlyChanged = false;
+
+    /** @var list<string> Stored in `status` column */
+    public const STATUSES = [
+        'submitted',
+        'accepted',
+        'review needed',
+        'rejected',
+        'canceled',
+        'under review',
+        'legacy',
+    ];
+
+    /** @var list<string> Stored in `acceptance_type` when status is finalized */
+    public const ACCEPTANCE_TYPES = [
+        'as_is',
+        'minor_edits',
+    ];
+
     protected $allowedFields = [
         'teacher_id',
+        'author_id',
         'class_section_id',
         'semester',
         'school_year',
         'exam_type',
         'draft_title',
+        'author_comment',
         'description',
         'teacher_file',
         'teacher_filename',
+        'author_file',
+        'author_filename',
         'status',
+        'acceptance_type',
+        'review_revision',
+        'reviewer_id',
         'admin_id',
         'is_legacy',
+        'reviewer_comment',
+        'reviewer_comments',
         'admin_comments',
         'reviewed_at',
         'final_file',
@@ -29,9 +61,40 @@ class ExamDraftModel extends Model
         'version',
         'previous_draft_id',
     ];
-    protected $returnType = 'array';
+
-    protected $useTimestamps = true;
+    /**
-    protected $createdField = 'created_at';
+     * Applied on insert/update when validation runs (e.g. $model->insert($data, true)).
-    protected $updatedField = 'updated_at';
+     * Uses if_exist so partial updates still validate only keys present in $data.
-    protected bool $updateOnlyChanged = false; // force updates even if CI thinks nothing changed
+     */
+    protected $validationRules = [
+        'status' => 'if_exist|in_list[submitted,accepted,review needed,rejected,canceled,under review,legacy]',
+        'acceptance_type' => 'if_exist|permit_empty|in_list[as_is,minor_edits]',
+        'author_id' => 'if_exist|is_natural_no_zero',
+        'class_section_id' => 'if_exist|is_natural_no_zero',
+        'version' => 'if_exist|is_natural_no_zero',
+    ];
+
+    protected $validationMessages = [
+        'status' => [
+            'in_list' => 'Invalid exam draft status.',
+        ],
+        'acceptance_type' => [
+            'in_list' => 'Acceptance must be as_is or minor_edits.',
+        ],
+    ];
+
+    /**
+     * @var array<string, string>
+     */
+    protected array $casts = [
+        'teacher_id' => 'int',
+        'author_id' => 'int',
+        'class_section_id' => 'int',
+        'reviewer_id' => '?int',
+        'admin_id' => '?int',
+        'review_revision' => 'int',
+        'version' => 'int',
+        'previous_draft_id' => '?int',
+        'is_legacy' => 'boolean',
+    ];
 }

app/Models/TeacherClassModel.php

@@ -71,7 +71,7 @@ class TeacherClassModel extends Model
         $builder = $this->db->table('teacher_class tc')
             ->select([
                 'cs.class_section_name',
-                'cs.id AS class_section_pk',
+                'cs.class_section_id AS class_section_pk',
                 'cs.class_section_id',
                 'c.id AS class_id',
                 'c.class_name',
@@ -181,10 +181,10 @@ class TeacherClassModel extends Model
     {
         $db = \Config\Database::connect();
 
-        // Adjust table/column names if yours differ (e.g., cs.id vs cs.class_section_id)
+        // Adjust table/column names if yours differ (e.g., cs.class_section_id vs cs.id)
         $row = $db->table('classSection cs')
             ->select('u.id AS teacher_id, u.firstname, u.lastname')
-            ->join('teacher_class tc', 'tc.class_section_id = cs.id', 'inner')
+            ->join('teacher_class tc', 'tc.class_section_id = cs.class_section_id', 'inner')
             ->join('users u', 'u.id = tc.teacher_id', 'inner')
             ->where('cs.class_section_name', $classSectionName)
             ->where('tc.school_year', $schoolYear)

app/Models/TeacherModel.php

@@ -34,11 +34,11 @@ class TeacherModel extends Model
 public function getTeachersAndTAs(): array
 {
     return $this->db->table('users u')
-        ->select('u.id, u.firstname, u.lastname, u.email, u.cellphone, r.name as role')
+        ->select('u.id, u.firstname, u.lastname, u.email, u.cellphone, MIN(r.name) as role')
         ->join('user_roles ur', 'ur.user_id = u.id')
         ->join('roles r', 'r.id = ur.role_id')
         ->whereIn('r.name', ['teacher', 'teacher_assistant']) // ✅ Filter only relevant roles
-        ->groupBy('u.id')
+        ->groupBy('u.id, u.firstname, u.lastname, u.email, u.cellphone')
         ->orderBy('u.lastname', 'ASC')
         ->get()
         ->getResultArray();
@@ -72,4 +72,4 @@ public function getTeachersAndTAs(): array
             ->get()
             ->getResultArray();
     }
-}
+}

app/Views/admin/class_progress_list.php

@@ -214,7 +214,17 @@
                                           <strong class="small mb-0"><?= esc($section['label'] ?? $subjectName) ?></strong>
                                           <span class="badge <?= $badgeClass ?>"><?= esc($statusTag) ?></span>
                                         </div>
-                                        <div class="small text-muted"><?= $report ? esc($report['unit_title'] ?: '-') : 'No submission' ?></div>
+                                        <div class="small">
+                                          <?php if ($report): ?>
+                                            <?= view('admin/partials/class_progress_unit_display', [
+                                              'unitTitle' => (string) ($report['unit_title'] ?? ''),
+                                              'isQuran' => ($subjectName === 'Quran/Arabic'),
+                                              'compact' => true,
+                                            ]) ?>
+                                          <?php else: ?>
+                                            <span class="text-muted">No submission</span>
+                                          <?php endif; ?>
+                                        </div>
                                       </div>
                                     <?php endforeach; ?>
                                   </div>

app/Views/admin/class_progress_view.php

@@ -39,7 +39,6 @@
         <?php
           $subjectName = $section['db_subject'] ?? $section['label'] ?? $slug;
           $isQuran = $subjectName === 'Quran/Arabic';
-          $unitLabel = $isQuran ? 'Surah / Custom Arabic' : 'Unit / Chapter';
           $homeworkLabel = $isQuran ? 'Arabic Practice / Homework' : 'Assigned Homework';
           $report = $reportsBySubject[$subjectName] ?? null;
         ?>
@@ -51,7 +50,13 @@
             <?php if (! $report): ?>
               <div class="text-muted">No entry submitted for this subject this week.</div>
             <?php else: ?>
-              <div class="mb-2"><strong><?= $unitLabel ?>:</strong> <?= esc($report['unit_title'] ?: '-') ?></div>
+              <div class="mb-2">
+                <?= view('admin/partials/class_progress_unit_display', [
+                  'unitTitle' => (string) ($report['unit_title'] ?? ''),
+                  'isQuran' => $isQuran,
+                  'compact' => false,
+                ]) ?>
+              </div>
               <?php if (!empty($report['materials'])): ?>
                 <div class="mb-2"><strong>Materials:</strong> <?= esc($report['materials']) ?></div>
               <?php endif; ?>

app/Views/admin/partials/class_progress_unit_display.php

@@ -0,0 +1,48 @@
+<?php
+/**
+ * Renders class progress unit_title with explicit curriculum vs custom subject lines.
+ *
+ * @var string $unitTitle
+ * @var bool   $isQuran
+ * @var bool   $compact  If true, tighter layout for the list accordion.
+ */
+
+use App\Controllers\ClassProgressController;
+
+$unitTitle = trim((string) ($unitTitle ?? ''));
+$compact = ! empty($compact);
+$isQuran = ! empty($isQuran);
+
+$split = ClassProgressController::splitUnitTitleForDisplay($unitTitle);
+$curriculumLines = $split['curriculum'];
+$customTopics = $split['custom'];
+
+$labelCurriculum = $isQuran ? 'Surah / curriculum' : 'Unit / chapter';
+$labelCustom = $isQuran ? 'Custom Surah / Arabic' : 'Custom subject(s)';
+?>
+
+<?php if ($unitTitle === ''): ?>
+  <span class="text-muted">-</span>
+<?php elseif ($compact): ?>
+  <?php if (! empty($customTopics)): ?>
+    <div class="small">
+      <span class="badge text-bg-info me-1"><?= $isQuran ? 'Custom' : 'Custom subject' ?></span><?= esc(implode(', ', $customTopics)) ?>
+    </div>
+  <?php endif; ?>
+  <?php if (! empty($curriculumLines)): ?>
+    <div class="small <?= ! empty($customTopics) ? 'text-muted mt-1' : 'text-muted' ?>"><?= esc(implode(' · ', $curriculumLines)) ?></div>
+  <?php endif; ?>
+  <?php if (empty($customTopics) && empty($curriculumLines)): ?>
+    <div class="small text-muted"><?= esc($unitTitle) ?></div>
+  <?php endif; ?>
+<?php else: ?>
+  <?php if (! empty($curriculumLines)): ?>
+    <div class="mb-2"><strong><?= esc($labelCurriculum) ?>:</strong> <?= esc(implode(' ; ', $curriculumLines)) ?></div>
+  <?php endif; ?>
+  <?php if (! empty($customTopics)): ?>
+    <div class="mb-2"><strong><?= esc($labelCustom) ?>:</strong> <?= esc(implode(' ; ', $customTopics)) ?></div>
+  <?php endif; ?>
+  <?php if (empty($curriculumLines) && empty($customTopics)): ?>
+    <div class="mb-2"><strong><?= esc($labelCurriculum) ?>:</strong> <?= esc($unitTitle) ?></div>
+  <?php endif; ?>
+<?php endif; ?>

app/Views/administrator/events/create_event.php

@@ -4,6 +4,11 @@
 <div class="container mt-4">
     <h2>Create Event</h2>
 
+    <?php
+    $defaultCategories = ['Fun Event-1', 'Fun Event-2', 'Fun Event-3'];
+    $existingCategories = array_map('strval', $categories ?? []);
+    $allCategories = array_unique(array_merge($defaultCategories, $existingCategories));
+    ?>
     <form method="post" action="<?= site_url('administrator/events/create') ?>" enctype="multipart/form-data">
         <?= csrf_field() ?>
         <div class="mb-3">
@@ -15,7 +20,7 @@
             <label class="form-label">Category</label>
             <select name="event_category" class="form-control" required>
                 <option value="" selected disabled>Select category</option>
-                <?php foreach (($categories ?? []) as $category): ?>
+                <?php foreach ($allCategories as $category): ?>
                     <option value="<?= esc($category) ?>"><?= esc(ucwords($category)) ?></option>
                 <?php endforeach; ?>
             </select>

app/Views/administrator/events/event_charges.php

@@ -11,44 +11,85 @@
         <div class="alert alert-danger"><?= session()->getFlashdata('error') ?></div>
     <?php endif; ?>
 
+    <?php
+        $selectedEvent = null;
+        if (!empty($filterEventId)) {
+            foreach ($events as $event) {
+                if ((int)$event['id'] === (int)$filterEventId) {
+                    $selectedEvent = $event;
+                    break;
+                }
+            }
+        }
+    ?>
     <!-- Add New Charge Form -->
-    <form action="<?= site_url('payment/event_charges') ?>" method="post">
+    <div class="card mb-4">
-        <?= csrf_field() ?>
+        <div class="card-body">
-        <div class="row">
+            <form action="<?= site_url('payment/event_charges') ?>" method="post">
-            <div class="col-md-3 mt-3">
+                <?= csrf_field() ?>
-                <label for="event_id" class="form-label">Select Event</label>
+                <div class="row g-3">
-                <select name="event_id" id="event_id" class="form-select" required>
+                    <div class="col-md-4">
-                    <option value="">-- Select Event --</option>
+                        <label for="event_id" class="form-label">Select Event</label>
-                    <?php foreach ($events as $event): ?>
+                        <select name="event_id" id="event_id" class="form-select" required>
-                        <option value="<?= esc($event['id']) ?>">
+                            <option value="">-- All events --</option>
-                            <?= esc($event['event_name']) ?>
+                            <?php foreach ($events as $event): ?>
-                        </option>
+                                <option value="<?= esc($event['id']) ?>" <?= isset($filterEventId) && $filterEventId == $event['id'] ? 'selected' : '' ?>>
-                    <?php endforeach; ?>
+                                    <?= esc($event['event_name']) ?>
-                </select>
+                                </option>
-            </div>
+                            <?php endforeach; ?>
+                        </select>
+                    </div>
 
-            <div class="col-md-3 mt-3">
+                    <?php if ($selectedEvent): ?>
-                <label for="parent_id" class="form-label">Parent</label>
+                        <div class="col-md-8">
-                <select id="parent_id" name="parent_id" class="form-select" required>
+                            <div class="border rounded bg-light p-3 h-100">
-                    <option value="">-- Select Parent --</option>
+                                <div class="d-flex justify-content-between align-items-center mb-2">
-                    <?php foreach ($parents as $parent): ?>
+                                    <h6 class="m-0"><?= esc($selectedEvent['event_name']) ?></h6>
-                        <option value="<?= $parent['id'] ?>">
+                                    <span class="badge bg-info text-dark">
-                            <?= esc($parent['firstname'] . ' ' . $parent['lastname']) ?> (<?= esc($parent['school_id']) ?>)
+                                        $<?= esc(number_format($selectedEvent['amount'] ?? 0, 2)) ?> fee
-                        </option>
+                                    </span>
-                    <?php endforeach; ?>
+                                </div>
-                </select>
+                                <p class="mb-1 text-muted small">
-            </div>
+                                    <?= esc($selectedEvent['description'] ?: 'No description provided for this event.') ?>
+                                </p>
+                                <?php if (!empty($selectedEvent['expiration_date'])): ?>
+                                    <small class="text-secondary">
+                                        Expires: <?= esc(local_date($selectedEvent['expiration_date'], 'm-d-Y')) ?>
+                                    </small>
+                                <?php endif; ?>
+                            </div>
+                        </div>
+                    <?php endif; ?>
 
-            <div class="col-12 mt-3" id="studentCheckboxContainer" style="display: none;">
+                </div>
-                <label>Select Students:</label>
+
-                <div id="studentList" class="row"></div>
+                <div class="row g-3 mt-2">
-            </div>           
+                    <div class="col-md-4">
+                        <label for="parent_id" class="form-label">Parents List</label>
+                        <select id="parent_id" name="parent_id" class="form-select" required>
+                            <option value="">-- Select Parent --</option>
+                            <?php foreach ($parents as $parent): ?>
+                                <option value="<?= $parent['id'] ?>">
+                                    <?= esc($parent['firstname'] . ' ' . $parent['lastname']) ?> (<?= esc($parent['school_id']) ?>)
+                                </option>
+                            <?php endforeach; ?>
+                        </select>
+                    </div>
+                </div>
+
+                <div class="mt-4" id="studentCheckboxContainer" style="display: none;">
+                    <label class="form-label">Select Students:</label>
+                    <div id="studentList" class="row g-3"></div>
+                </div>
+
+                <div class="d-flex flex-wrap gap-2 mt-3">
+                    <button type="submit" class="btn btn-primary">Submit</button>
+                    <a href="<?= site_url('administrator/events') ?>" class="btn btn-success">Event List</a>
+                </div>
+            </form>
         </div>
+    </div>
 
-<button type="submit" class="btn btn-primary mt-3">Submit</button>
-    <a href="<?= site_url('administrator/events') ?>" class="btn btn-success mt-3">Event List</a>
-    </form>
-<br>
     <!-- Charge Tables grouped by event -->
     <?php
         $grouped = [];
@@ -70,16 +111,18 @@
                     <div class="table-responsive">
                         <table class="table table-bordered table-striped mb-0 no-mgmt-sticky">
                             <thead class="table-light">
-                                <tr>
+                            <tr>
-                                    <th>ID</th>
+                                <th>ID</th>
-                                    <th>Parent Name</th>
+                                <th>Parent Name</th>
-                                    <th>Student Name</th>
+                                <th>Student Name</th>
-                                    <th>Charged Amount</th>
+                                <th>Charged Amount</th>
-                                    <th>Is Participating</th>
+                                <th>Is Participating</th>
-                                    <th>Semester</th>
+                                <th>Semester</th>
-                                    <th>Year</th>
+                                <th>Year</th>
-                                    <th>Created</th>
+                                <th>Created</th>
-                                </tr>
+                                <th>Description</th>
+                                <th>Event Fees</th>
+                            </tr>
                             </thead>
                             <tbody>
                                 <?php foreach ($rows as $charge): ?>
@@ -100,6 +143,8 @@
                                         <td><?= esc($charge['semester'] ?? '-') ?></td>
                                         <td><?= esc($charge['school_year'] ?? '-') ?></td>
                                         <td><?= esc(!empty($charge['created_at']) ? local_datetime($charge['created_at'], 'm-d-Y H:i') : '') ?></td>
+                                        <td><?= esc($charge['event_description'] ?? '—') ?></td>
+                                        <td>$<?= esc(number_format($charge['event_amount'] ?? 0, 2)) ?></td>
                                     </tr>
                                 <?php endforeach; ?>
                             </tbody>
@@ -156,7 +201,19 @@ function loadStudentsWithCharges() {
     }
 }
 
-$('#parent_id').on('change', loadStudentsWithCharges);
+$(function() {
-$('#event_id').on('change', loadStudentsWithCharges);
+    $('#parent_id').on('change', loadStudentsWithCharges);
+
+    $('#event_id').on('change', function() {
+        let eventId = $(this).val();
+        let url = new URL(window.location.href);
+        if (eventId) {
+            url.searchParams.set('event_id', eventId);
+        } else {
+            url.searchParams.delete('event_id');
+        }
+        window.location.href = url.toString();
+    });
+});
 </script>
 <?= $this->endSection() ?>

app/Views/administrator/events/event_list.php

@@ -21,7 +21,8 @@
                 <tr>
                     <th>Event Name</th>
                     <th>Category</th>
-                    <th>Amount</th>
+                    <th>Description</th>
+                    <th>Event Fees</th>
                     <th>Expiration Date</th>
                     <th>Semester</th>
                     <th>School Year</th>
@@ -33,7 +34,8 @@
                     <tr>
                         <td><?= esc($event['event_name']) ?></td>
                         <td><?= esc($event['event_category'] ?? '—') ?></td>
-                        <td><?= esc($event['amount']) ?></td>
+                        <td><?= esc(!empty($event['description']) ? $event['description'] : '—') ?></td>
+                        <td>$<?= esc(number_format($event['amount'], 2)) ?></td>
                         <td><?= esc(!empty($event['expiration_date']) ? local_date($event['expiration_date'], 'm-d-Y') : '') ?></td>
                         <td><?= esc($event['semester']) ?></td>
                         <td><?= esc($event['school_year']) ?></td>

app/Views/administrator/exam_drafts.php

@@ -1,9 +1,31 @@
 <?= $this->extend('layout/management_layout') ?>
 <?= $this->section('content') ?>
+<?php
+$statusBadges = $statusBadges ?? [];
+$drafts = $drafts ?? [];
+$legacyByClass = $legacyByClass ?? [];
+$classSections = $classSections ?? [];
+$examTypes = $examTypes ?? [];
+$visibleClasses = $visibleClasses ?? [];
+$classDraftGroups = $classDraftGroups ?? [];
+$newSubmissionClasses = $newSubmissionClasses ?? [];
+$schoolYear = $schoolYear ?? '';
+$semester = $semester ?? '';
+$maxUploadBytes = $maxUploadBytes ?? (12 * 1024 * 1024);
+$allowedExtensions = $allowedExtensions ?? ['doc', 'docx', 'pdf'];
+
+$renderBadge = static function (string $status, array $badges): string {
+    $b = $badges[$status] ?? ['label' => $status, 'class' => 'bg-secondary text-white'];
+    $style = !empty($b['style']) ? ' style="' . esc($b['style']) . '"' : '';
+    return '<span class="badge ' . esc($b['class']) . '"' . $style . '>' . esc($b['label']) . '</span>';
+};
+
+$fileAccept = implode(',', array_map(static fn ($x) => '.' . $x, $allowedExtensions));
+?>
 <div class="container-fluid px-4 py-4">
     <div class="d-flex flex-column flex-lg-row justify-content-between align-items-lg-center gap-3 mb-4">
         <div>
-            <h1 class="h3 mb-1">Exam Draft Submissions</h1>
+            <h1 class="h3 mb-1">Exam draft submissions</h1>
             <p class="text-muted mb-0 small">
                 <?= esc($semester ?: 'Semester') ?> <?= esc($schoolYear ?: '') ?>
             </p>
@@ -15,13 +37,9 @@
     </div>
 
     <?php if (session()->getFlashdata('success')): ?>
-        <div class="alert alert-success">
+        <div class="alert alert-success"><?= esc(session()->getFlashdata('success')) ?></div>
-            <?= esc(session()->getFlashdata('success')) ?>
-        </div>
     <?php elseif (session()->getFlashdata('error')): ?>
-        <div class="alert alert-danger">
+        <div class="alert alert-danger"><?= esc(session()->getFlashdata('error')) ?></div>
-            <?= esc(session()->getFlashdata('error')) ?>
-        </div>
     <?php endif; ?>
 
     <ul class="nav nav-pills mb-3" id="examDraftTabs" role="tablist">
@@ -32,177 +50,253 @@
         </li>
         <li class="nav-item" role="presentation">
             <button class="nav-link" id="legacy-tab" data-bs-toggle="pill" data-bs-target="#legacy" type="button" role="tab" aria-controls="legacy" aria-selected="false">
-                Legacy Exams
+                Legacy exams
             </button>
         </li>
     </ul>
+
     <div class="tab-content" id="examDraftTabsContent">
         <div class="tab-pane fade show active" id="submissions" role="tabpanel" aria-labelledby="submissions-tab">
             <div class="card mb-4">
                 <div class="card-body">
-                    <?php if (empty($drafts)): ?>
+                    <?php if (empty($visibleClasses)): ?>
-                        <p class="text-muted mb-0">No exam drafts have been submitted yet.</p>
+                        <p class="text-muted mb-0">No classes with enrolled students are available for this term.</p>
                     <?php else: ?>
-                        <div class="table-responsive">
+                        <div class="accordion exam-drafts-accordion" id="examDraftsAccordion">
-                            <table class="table table-striped table-bordered align-middle mb-0 exam-drafts-table no-mgmt-sticky">
+                            <?php foreach ($visibleClasses as $classSectionId => $classInfo): ?>
-                                <thead class="table-light">
+                                <?php $classDraftsForSection = $classDraftGroups[$classSectionId] ?? []; ?>
-                                    <tr>
+                                <?php $collapseId = 'classDraftGroup_' . $classSectionId; ?>
-                                        <th>Teacher</th>
+                                <div class="accordion-item mb-3">
-                                        <th>Class</th>
+                                    <h2 class="accordion-header" id="heading_<?= esc($collapseId) ?>">
-                                        <th>Title / Type</th>
+                                        <button class="accordion-button collapsed px-3" type="button" data-bs-toggle="collapse" data-bs-target="#<?= esc($collapseId) ?>" aria-expanded="false" aria-controls="<?= esc($collapseId) ?>">
-                                        <th>Version</th>
+                                            <div class="d-flex flex-column flex-lg-row w-100 justify-content-between gap-2">
-                                        <th>Submitted</th>
+                                                <div>
-                                        <th>Status</th>
+                                                    <strong><?= esc($classInfo['class_section_name'] ?? ('Class ' . $classSectionId)) ?></strong>
-                                        <th>Files</th>
+                                                    <div class="small text-muted">
-                                        <th>PDF</th>
+                                                        <?= esc($classInfo['student_count'] ?? 0) ?> students · <?= esc(count($classDraftsForSection)) ?> submissions
-                                        <th>Review</th>
+                                                        <?php if (!empty($newSubmissionClasses[$classSectionId])): ?>
-                                    </tr>
+                                                            <span class="badge bg-info text-dark ms-2">New submission</span>
-                                </thead>
-                                <tbody>
-                                    <?php foreach ($drafts as $draft): ?>
-                                        <?php
-                                            $teacherName = trim(($draft['teacher_first'] ?? '') . ' ' . ($draft['teacher_last'] ?? ''));
-                                            if ($teacherName === '') {
-                                                $teacherName = ($draft['admin_id'] ?? null) === ($draft['teacher_id'] ?? null)
-                                                    ? 'Admin Upload'
-                                                    : 'User #' . ($draft['teacher_id'] ?? 'N/A');
-                                            }
-                                            $statusInfo = $statusBadges[$draft['status'] ?? 'pending'] ?? ['label' => 'Unknown', 'class' => 'bg-secondary text-white'];
-                                            $adminName = trim(($draft['admin_first'] ?? '') . ' ' . ($draft['admin_last'] ?? ''));
-                                        ?>
-                                        <tr>
-                                            <td><?= esc($teacherName) ?></td>
-                                            <td><?= esc($draft['class_section_name'] ?? 'Unknown') ?></td>
-                                            <td>
-                                                <strong><?= esc($draft['draft_title'] ?? 'Untitled') ?></strong>
-                                                <div class="small text-muted"><?= esc($draft['exam_type'] ?? 'N/A') ?></div>
-                                                <?php if (!empty($draft['description'])): ?>
-                                                    <div class="small text-muted"><?= esc($draft['description']) ?></div>
-                                                <?php endif; ?>
-                                            </td>
-                                            <td class="text-nowrap">v<?= esc($draft['version'] ?? 1) ?></td>
-                                            <td class="text-nowrap"><?= esc((!empty($draft['created_at']) ? local_datetime($draft['created_at'], 'M j, Y g:i A') : '—')) ?></td>
-                                            <td>
-                                                <span class="badge <?= esc($statusInfo['class']) ?>">
-                                                    <?= esc($statusInfo['label']) ?>
-                                                </span>
-                                                <?php if (!empty($draft['reviewed_at'])): ?>
-                                                    <div class="small text-muted mt-1">
-                                                        Reviewed <?= esc(local_datetime($draft['reviewed_at'], 'M j, Y g:i A')) ?>
-                                                        <?php if ($adminName !== ''): ?>
-                                                            by <?= esc($adminName) ?>
                                                         <?php endif; ?>
                                                     </div>
-                                                <?php endif; ?>
+                                                </div>
-                                            </td>
+                                                <span class="badge bg-secondary align-self-start">
-                                            <td>
+                                                    <?= esc(count($classDraftsForSection)) ?>
-                                                <?php if (!empty($draft['teacher_file'])): ?>
+                                                    <?= count($classDraftsForSection) === 1 ? 'submission' : 'submissions' ?>
-                                                    <div>
+                                                </span>
-                                                        <a href="<?= base_url('exam-drafts/files/teacher/' . $draft['teacher_file']) ?>" target="_blank">
+                                            </div>
-                                                            <?= esc($draft['teacher_filename'] ?? 'Submitted draft') ?>
+                                        </button>
-                                                        </a>
+                                    </h2>
-                                                    </div>
+                                    <div id="<?= esc($collapseId) ?>" class="accordion-collapse collapse" aria-labelledby="heading_<?= esc($collapseId) ?>">
-                                                <?php else: ?>
+                                        <div class="accordion-body pt-0 px-3">
-                                                    <span class="text-muted small">No teacher file</span>
+                                            <?php if (empty($classDraftsForSection)): ?>
-                                                <?php endif; ?>
+                                                <p class="text-muted mb-0">No submissions yet for this class.</p>
-                                                <?php if (!empty($draft['final_file'])): ?>
+                                            <?php else: ?>
-                                                    <div class="mt-2">
+                                                <div class="table-responsive">
-                                                        <a href="<?= base_url('exam-drafts/files/final/' . $draft['final_file']) ?>" target="_blank" class="link-success small">
+                                                    <table class="table table-striped table-bordered align-middle mb-0 exam-drafts-table no-mgmt-sticky">
-                                                            <?= esc($draft['final_filename'] ?? 'Final draft') ?>
+                                                        <thead class="table-light">
-                                                        </a>
+                                                            <tr>
-                                                    </div>
+                                                                <th>Teacher</th>
-                                                <?php endif; ?>
+                                                                <th>Class</th>
-                                            </td>
+                                                                <th>Title & AuthorNote</th>
-                                            <td class="text-nowrap">
+                                                                <th>Version</th>
-                                                <?php
+                                                                <th>Date-Time</th>
-                                                    $pdfFile = $draft['final_pdf_file'] ?? null;
+                                                                <th>Status</th>
-                                                    // Fallback: if final_file itself is a pdf, use it
+                                                                <th>Files</th>
-                                                    if (empty($pdfFile) && !empty($draft['final_file']) && strtolower(pathinfo($draft['final_file'], PATHINFO_EXTENSION)) === 'pdf') {
+                                                                <th>Reviewer Action</th>
-                                                        $pdfFile = $draft['final_file'];
+                                                                <th>Final version</th>
-                                                    }
+                                                            </tr>
-                                                ?>
+                                                        </thead>
-                                                <?php if (!empty($pdfFile)): ?>
+                                                        <tbody>
-                                                    <a href="<?= base_url('exam-drafts/files/final/' . $pdfFile) ?>" target="_blank" class="link-success small">
+                                                            <?php foreach ($classDraftsForSection as $draft): ?>
-                                                        PDF
+                                                                <?php
-                                                    </a>
+                                                                    $teacherName = trim(($draft['teacher_first'] ?? '') . ' ' . ($draft['teacher_last'] ?? ''));
-                                                <?php else: ?>
+                                                                    if ($teacherName === '') {
-                                                    <span class="text-muted small">—</span>
+                                                                        $teacherName = ($draft['admin_id'] ?? null) === ($draft['teacher_id'] ?? null)
-                                                <?php endif; ?>
+                                                                            ? 'Admin upload'
-                                            </td>
+                                                                            : 'User #' . ($draft['teacher_id'] ?? 'N/A');
-                                            <td>
+                                                                    }
-                                                <form method="post" action="<?= base_url('/administrator/exam-drafts/review') ?>" enctype="multipart/form-data">
+                                                                    $st = strtolower((string) ($draft['status'] ?? ''));
-                                                    <?= csrf_field() ?>
+                                                                    $adminName = trim(($draft['admin_first'] ?? '') . ' ' . ($draft['admin_last'] ?? ''));
-                                                    <input type="hidden" name="draft_id" value="<?= esc($draft['id']) ?>">
+                                                                    $authorNote = $draft['author_comment'] ?? $draft['description'] ?? '';
-                                                    <div class="mb-2">
+                                                                    $reviewerNote = $draft['reviewer_comment'] ?? $draft['admin_comments'] ?? '';
-                                                        <textarea
+                                                                ?>
-                                                            name="admin_comments"
+                                                                <tr>
-                                                            rows="2"
+                                                                    <td><?= esc($teacherName) ?></td>
-                                                            class="form-control form-control-sm"
+                                                                    <td><?= esc($draft['class_section_name'] ?? 'Unknown') ?></td>
-                                                            placeholder="Optional note for the teacher"
+                                                                    <td>
-                                                        ><?= esc($draft['admin_comments'] ?? '') ?></textarea>
+                                                                        <strong><?= esc($draft['draft_title'] ?? 'Untitled') ?></strong>
-                                                    </div>
+                                                                        <div class="small text-muted"><?= esc($draft['exam_type'] ?? 'N/A') ?></div>
-                                                    <div class="mb-2">
+                                                                        <?php if ($authorNote !== ''): ?>
-                                                        <label class="form-label small mb-1">Status</label>
+                                                                            <div class="small mt-1">
-                                                        <select name="review_status" class="form-select form-select-sm">
+                                                                                <span class="text-muted fw-semibold">Author:</span>
-                                                            <?php foreach ($statusOptions as $option): ?>
+                                                                                <?= esc($authorNote) ?>
-                                                                <option value="<?= esc($option) ?>" <?= ($option === ($draft['status'] ?? '')) ? 'selected' : '' ?>>
+                                                                            </div>
-                                                                    <?= esc(ucfirst($option)) ?>
+                                                                        <?php endif; ?>
-                                                                </option>
+                                                                        <?php if ($reviewerNote !== ''): ?>
+                                                                            <div class="small mt-1 border-top pt-1">
+                                                                                <span class="text-muted fw-semibold">Reviewer:</span>
+                                                                                <?= esc($reviewerNote) ?>
+                                                                            </div>
+                                                                        <?php endif; ?>
+                                                                    </td>
+                                                                    <td class="text-nowrap">v<?= esc((string) ($draft['version'] ?? 1)) ?></td>
+                                                                    <td class="text-nowrap"><?= esc(!empty($draft['created_at']) ? local_datetime($draft['created_at'], 'M j, Y g:i A') : '—') ?></td>
+                                                                    <td>
+                                                                        <?php
+                                                                            $badgeData = $statusBadges[$st] ?? ['label' => $st, 'class' => 'bg-secondary text-white'];
+                                                                            $badgeStyle = !empty($badgeData['style']) ? ' style="' . esc($badgeData['style']) . '"' : '';
+                                                                        ?>
+                                                                        <span class="badge <?= esc($badgeData['class']) ?> js-status-badge" data-status="<?= esc($st) ?>"<?= $badgeStyle ?>>
+                                                                            <?= esc($badgeData['label']) ?>
+                                                                        </span>
+                                                                        <div class="small text-muted mt-1 js-acceptance-note">
+                                                                            <?php if ($st === 'accepted' && !empty($draft['acceptance_type'])): ?>
+                                                                                <?= $draft['acceptance_type'] === 'minor_edits' ? 'Accepted w/ minor edits' : 'Accepted as is' ?>
+                                                                            <?php endif; ?>
+                                                                        </div>
+                                                                        <?php if (!empty($draft['reviewed_at'])): ?>
+                                                                            <div class="small text-muted mt-1">
+                                                                                Reviewed <?= esc(local_datetime($draft['reviewed_at'], 'M j, Y g:i A')) ?>
+                                                                                <?php if ($adminName !== ''): ?>
+                                                                                    by <?= esc($adminName) ?>
+                                                                                <?php endif; ?>
+                                                                            </div>
+                                                                        <?php endif; ?>
+                                                                    </td>
+                                                                    <td>
+                                                                        <?php $formId = 'review_form_' . (int) ($draft['id'] ?? 0); ?>
+                                                                        <?php $revNumber = max(1, (int) ($draft['version'] ?? 1)); ?>
+                                                                        <?php if (!empty($draft['teacher_file'])): ?>
+                                                                            <div>
+                                                                                <a href="<?= base_url('exam-drafts/files/teacher/' . $draft['teacher_file']) ?>" target="_blank" rel="noopener">
+                                                                                    <?= esc('Ver' . $revNumber . ' ' . ($draft['teacher_filename'] ?? 'Submitted draft')) ?>
+                                                                                </a>
+                                                                            </div>
+                                                                        <?php else: ?>
+                                                                            <span class="text-muted small">No teacher file</span>
+                                                                        <?php endif; ?>
+                                                                        <?php if (!empty($draft['final_file']) && strtolower((string) ($draft['status'] ?? '')) === 'accepted'): ?>
+                                                                            <div class="mt-2">
+                                                                                <a href="<?= base_url('exam-drafts/files/final/' . $draft['final_file']) ?>" target="_blank" rel="noopener" class="link-success small">
+                                                                                    <?= esc('Final ' . ($draft['final_filename'] ?? 'Final draft')) ?>
+                                                                                </a>
+                                                                            </div>
+                                                                        <?php endif; ?>
+                                                                        <?php if (!empty($draft['review_files'])): ?>
+                                                                            <div class="mt-2">
+                                                                                <?php
+                                                                                    $reviewLinks = [];
+                                                                                    foreach ($draft['review_files'] as $rf) {
+                                                                                        $rev = max(1, (int) ($rf['review_revision'] ?? 1));
+                                                                                        $name = $rf['final_filename'] ?? 'Review file';
+                                                                                        $file = $rf['final_file'] ?? '';
+                                                                                        if ($file !== '') {
+                                                                                            $reviewLinks[] = '<a href="' . esc(base_url('exam-drafts/files/final/' . $file)) . '" target="_blank" rel="noopener" class="link-success small">' . esc('Ver' . $revNumber . '_' . $rev . ' ' . $name) . '</a>';
+                                                                                        }
+                                                                                    }
+                                                                                ?>
+                                                                                <?= !empty($reviewLinks) ? implode(' | ', $reviewLinks) : '' ?>
+                                                                            </div>
+                                                                        <?php endif; ?>
+                                                                        <div class="mt-2">
+                                                                            <?php $fileInputId = 'review_file_' . (int) ($draft['id'] ?? 0); ?>
+                                                                            <input type="file" name="final_file" id="<?= esc($fileInputId) ?>" class="form-control form-control-sm" accept="<?= esc($fileAccept) ?>" form="<?= esc($formId) ?>">
+                                                                            <button type="submit" class="btn btn-sm btn-outline-primary mt-2" form="<?= esc($formId) ?>">Upload review file</button>
+                                                                        </div>
+                                                                    </td>
+                                                                    <td style="min-width: 280px;">
+                                                                        <?= form_open_multipart(base_url('administrator/exam-drafts/review'), ['class' => 'vstack gap-2', 'id' => $formId]) ?>
+                                                                            <?= csrf_field() ?>
+                                                                            <input type="hidden" name="draft_id" value="<?= (int) ($draft['id'] ?? 0) ?>">
+                                                                            <?php $selectedStatus = ''; ?>
+                                                                            <select name="review_status" class="form-select form-select-sm" required>
+                                                                                <option value="" <?= $selectedStatus === '' ? 'selected' : '' ?> disabled>— Select status —</option>
+                                                                                <option value="accepted" <?= $selectedStatus === 'accepted' ? 'selected' : '' ?>>Accepted</option>
+                                                                                <option value="legacy" <?= $selectedStatus === 'legacy' ? 'selected' : '' ?>>Legacy</option>
+                                                                                <option value="canceled" <?= $selectedStatus === 'canceled' ? 'selected' : '' ?>>Canceled</option>
+                                                                                <option value="rejected" <?= $selectedStatus === 'rejected' ? 'selected' : '' ?>>Rejected</option>
+                                                                                <option value="review needed" <?= $selectedStatus === 'review needed' ? 'selected' : '' ?>>Review needed</option>
+                                                                                <option value="under review" <?= $selectedStatus === 'under review' ? 'selected' : '' ?>>Under review</option>
+                                                                            </select>
+                                                                            <div class="small js-acceptance-group d-none">
+                                                                                <span class="d-block mb-1">As is / Minor edits</span>
+                                                                                <?php $acc = (string) ($draft['acceptance_type'] ?? ''); ?>
+                                                                                <div class="form-check form-check-inline">
+                                                                                    <input class="form-check-input" type="radio" name="acceptance_type" value="as_is" id="as_is_<?= (int) ($draft['id'] ?? 0) ?>" <?= $acc !== 'minor_edits' ? 'checked' : '' ?>>
+                                                                                    <label class="form-check-label" for="as_is_<?= (int) ($draft['id'] ?? 0) ?>">As is</label>
+                                                                                </div>
+                                                                                <div class="form-check form-check-inline">
+                                                                                    <input class="form-check-input" type="radio" name="acceptance_type" value="minor_edits" id="minor_<?= (int) ($draft['id'] ?? 0) ?>" <?= $acc === 'minor_edits' ? 'checked' : '' ?>>
+                                                                                    <label class="form-check-label" for="minor_<?= (int) ($draft['id'] ?? 0) ?>">Minor edits</label>
+                                                                                </div>
+                                                                            </div>
+                                                                            <textarea name="reviewer_comment" class="form-control form-control-sm js-review-comment" rows="3" placeholder="Feedback for the teacher"><?= esc($draft['reviewer_comment'] ?? $draft['reviewer_comments'] ?? $draft['admin_comments'] ?? '') ?></textarea>
+                                                                            <div class="d-flex flex-wrap gap-2">
+                                                                                <span class="text-muted small js-review-status"></span>
+                                                                            </div>
+                                                                        <?= form_close() ?>
+                                                                    </td>
+                                                                    <td class="text-nowrap" style="min-width: 220px;">
+                                                                        <?php
+                                                                            $pdfFile = $draft['final_pdf_file'] ?? null;
+                                                                            if (empty($pdfFile) && !empty($draft['final_file']) && strtolower(pathinfo($draft['final_file'], PATHINFO_EXTENSION)) === 'pdf') {
+                                                                                $pdfFile = $draft['final_file'];
+                                                                            }
+                                                                        ?>
+                                                                        <?php if (!empty($pdfFile)): ?>
+                                                                            <div class="mt-2 d-flex flex-wrap gap-2">
+                                                                                <a class="btn btn-sm btn-outline-primary" href="<?= base_url('exam-drafts/files/final/' . $pdfFile) ?>" target="_blank" rel="noopener">View</a>
+                                                                                <a class="btn btn-sm btn-outline-secondary" href="<?= base_url('exam-drafts/files/final/' . $pdfFile) ?>" target="_blank" rel="noopener" download>Download</a>
+                                                                            </div>
+                                                                        <?php endif; ?>
+                                                                    </td>
+                                                                </tr>
                                                             <?php endforeach; ?>
-                                                        </select>
+                                                        </tbody>
-                                                    </div>
+                                                    </table>
-                                                    <div class="mb-2">
+                                                </div>
-                                                        <label class="form-label small mb-1">Upload final draft</label>
+                                            <?php endif; ?>
-                                                        <input type="file" name="final_file" class="form-control form-control-sm">
+                                        </div>
-                                                    </div>
+                                    </div>
-                                                    <button type="submit" class="btn btn-sm btn-primary w-100">
+                                </div>
-                                                        Save review
+                            <?php endforeach; ?>
-                                                    </button>
-                                                </form>
-                                            </td>
-                                        </tr>
-                                    <?php endforeach; ?>
-                                </tbody>
-                            </table>
                         </div>
                     <?php endif; ?>
                 </div>
             </div>
         </div>
+
         <div class="tab-pane fade" id="legacy" role="tabpanel" aria-labelledby="legacy-tab">
             <div class="card border-primary-subtle mb-3">
                 <div class="card-header bg-light d-flex justify-content-between align-items-center">
                     <div>
-                        <strong>Upload Old / Legacy Exam</strong>
+                        <strong>Upload old / legacy exam</strong>
-                        <div class="small text-muted">Store historic exams as finalized records.</div>
+                        <div class="small text-muted">Store historic exams as accepted records.</div>
                     </div>
                     <span class="badge bg-primary-subtle text-primary">Admin only</span>
                 </div>
                 <div class="card-body">
-                    <form method="post" action="<?= base_url('/administrator/exam-drafts/upload-legacy') ?>" enctype="multipart/form-data" class="row g-3">
+                    <?= form_open_multipart(base_url('administrator/exam-drafts/upload-legacy'), ['class' => 'row g-3']) ?>
                         <?= csrf_field() ?>
                         <div class="col-md-4">
-                            <label class="form-label small">Class Sections</label>
+                            <label class="form-label small">Class sections</label>
-                            <select name="class_section_ids[]" class="form-select" multiple required>
+                            <select name="class_section_ids[]" class="form-select" multiple required size="6">
                                 <?php foreach ($classSections as $cs): ?>
-                                    <option value="<?= esc($cs['class_section_id']) ?>"><?= esc($cs['class_section_name']) ?></option>
+                                    <option value="<?= esc($cs['class_section_id'] ?? '') ?>"><?= esc($cs['class_section_name'] ?? '') ?></option>
                                 <?php endforeach; ?>
                             </select>
                             <div class="form-text">Hold Ctrl (Windows) or Command (Mac) to select multiple sections.</div>
                         </div>
                         <div class="col-md-2">
-                            <label class="form-label small">School Year</label>
+                            <label class="form-label small">School year</label>
                             <input type="text" name="school_year" class="form-control" value="<?= esc($schoolYear) ?>" placeholder="2025-2026" required>
                         </div>
                         <div class="col-md-2">
                             <label class="form-label small">Semester</label>
-                            <select name="semester" class="form-select" required>
+                            <input type="text" name="semester" class="form-control" value="<?= esc($semester) ?>" placeholder="Fall / Spring" required>
-                                <option value="Fall" <?= ($semester === 'Fall') ? 'selected' : '' ?>>Fall</option>
-                                <option value="Spring" <?= ($semester === 'Spring') ? 'selected' : '' ?>>Spring</option>
-                            </select>
                         </div>
                         <div class="col-md-4">
-                            <label class="form-label small">Exam Type</label>
+                            <label class="form-label small">Exam type</label>
                             <select name="exam_type" class="form-select">
                                 <option value="">— Select type —</option>
                                 <?php foreach ($examTypes as $type): ?>
@@ -211,14 +305,14 @@
                             </select>
                         </div>
                         <div class="col-12">
-                            <label class="form-label small">Upload File</label>
+                            <label class="form-label small">Upload file</label>
-                            <input type="file" name="old_exam_file" class="form-control" accept=".doc,.docx,.pdf" required>
+                            <input type="file" name="old_exam_file" class="form-control" accept="<?= esc($fileAccept) ?>" required>
                             <div class="form-text">Allowed: <?= esc(implode(', ', $allowedExtensions)) ?> • Max <?= number_format($maxUploadBytes / 1024 / 1024, 0) ?> MB</div>
                         </div>
                         <div class="col-12">
-                            <button type="submit" class="btn btn-primary">Upload Legacy Exam</button>
+                            <button type="submit" class="btn btn-primary">Upload legacy exam</button>
                         </div>
-                    </form>
+                    <?= form_close() ?>
                 </div>
             </div>
 
@@ -231,20 +325,21 @@
                             <div class="mb-4">
                                 <h6 class="mb-2"><?= esc($group['class_section_name'] ?? 'Class') ?></h6>
                                 <div class="list-group">
-                                    <?php foreach ($group['items'] as $item): ?>
+                                    <?php foreach ($group['items'] ?? [] as $item): ?>
-                                        <div class="list-group-item d-flex justify-content-between align-items-start">
+                                        <?php $lst = strtolower((string) ($item['status'] ?? '')); ?>
+                                        <div class="list-group-item d-flex justify-content-between align-items-start flex-wrap gap-2">
                                             <div class="me-3">
-                                                <div class="fw-semibold"><?= esc($item['draft_title'] ?? 'Legacy Exam') ?></div>
+                                                <div class="fw-semibold"><?= esc($item['draft_title'] ?? 'Legacy exam') ?></div>
                                                 <div class="small text-muted">
                                                     <?= esc($item['exam_type'] ?? 'N/A') ?> •
                                                     <?= esc($item['semester'] ?? '') ?> <?= esc($item['school_year'] ?? '') ?>
                                                 </div>
+                                                <div class="small mt-1"><?= $renderBadge($lst, $statusBadges) ?></div>
                                             </div>
-                                            <div class="text-end">
+                                            <div class="text-end d-flex flex-wrap gap-2 justify-content-end">
                                                 <?php if (!empty($item['final_file'])): ?>
-                                                    <a href="<?= base_url('exam-drafts/files/final/' . $item['final_file']) ?>" target="_blank" class="btn btn-sm btn-outline-primary">
+                                                    <a class="btn btn-sm btn-outline-primary" href="<?= base_url('exam-drafts/files/final/' . $item['final_file']) ?>" target="_blank" rel="noopener">View</a>
-                                                        Download
+                                                    <a class="btn btn-sm btn-outline-secondary" href="<?= base_url('exam-drafts/files/final/' . $item['final_file']) ?>" target="_blank" rel="noopener" download>Download</a>
-                                                    </a>
                                                 <?php else: ?>
                                                     <span class="text-muted small">File missing</span>
                                                 <?php endif; ?>
@@ -260,6 +355,208 @@
         </div>
     </div>
 </div>
+<script>
+document.addEventListener('DOMContentLoaded', () => {
+    const submissionsTab = document.getElementById('submissions-tab');
+    if (submissionsTab) {
+        submissionsTab.addEventListener('click', () => {
+            window.location.reload();
+        });
+    }
+    const statusBadges = <?= json_encode($statusBadges, JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+    const csrfTokenName = <?= json_encode(csrf_token(), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+    const csrfCookieNames = <?= json_encode(array_values(array_filter([
+        config('Security')->csrfCookieName ?? null,
+        config('Security')->cookieName ?? null,
+        'csrf_cookie_name',
+    ])), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+
+    const readCookie = (name) => {
+        const match = document.cookie.match(new RegExp('(?:^|; )' + name.replace(/[$()*+.?[\\\]^{|}-]/g, '\\$&') + '=([^;]*)'));
+        return match ? decodeURIComponent(match[1]) : '';
+    };
+
+    const syncCsrfToken = (form) => {
+        let tokenValue = '';
+        csrfCookieNames.some((name) => {
+            tokenValue = readCookie(name);
+            return tokenValue !== '';
+        });
+        if (!tokenValue && form) {
+            const tokenInput = form.querySelector(`input[name="${csrfTokenName}"]`);
+            if (tokenInput && tokenInput.value) {
+                tokenValue = tokenInput.value;
+            }
+        }
+        if (!tokenValue || !form) {
+            return tokenValue;
+        }
+        const tokenInput = form.querySelector(`input[name="${csrfTokenName}"]`);
+        if (tokenInput) {
+            tokenInput.value = tokenValue;
+        }
+        return tokenValue;
+    };
+    const updateRow = (row) => {
+        const statusSelect = row.querySelector('select[name="review_status"]');
+        const acceptanceGroup = row.querySelector('.js-acceptance-group');
+        if (!statusSelect || !acceptanceGroup) {
+            return;
+        }
+        acceptanceGroup.classList.toggle('d-none', statusSelect.value !== 'accepted');
+    };
+
+    const updateStatusUI = (row) => {
+        const statusSelect = row.querySelector('select[name="review_status"]');
+        const badge = row.querySelector('.js-status-badge');
+        if (!statusSelect || !badge || statusSelect.value === '') {
+            return;
+        }
+        const status = statusSelect.value;
+        const badgeData = statusBadges[status] || { label: status, class: 'bg-secondary text-white' };
+        badge.textContent = badgeData.label || status;
+        badge.className = `badge ${badgeData.class} js-status-badge`;
+        if (badgeData.style) {
+            badge.setAttribute('style', badgeData.style);
+        } else {
+            badge.removeAttribute('style');
+        }
+        badge.dataset.status = status;
+
+        const acceptanceNote = row.querySelector('.js-acceptance-note');
+        if (acceptanceNote) {
+            if (status === 'accepted') {
+                const acc = row.querySelector('input[name="acceptance_type"]:checked');
+                acceptanceNote.textContent = acc && acc.value === 'minor_edits'
+                    ? 'Accepted w/ minor edits'
+                    : 'Accepted as is';
+            } else {
+                acceptanceNote.textContent = '';
+            }
+        }
+    };
+
+    const saveRow = (row, options = {}) => {
+        const form = row.querySelector('form');
+        if (!form) {
+            return;
+        }
+        const statusSelect = form.querySelector('select[name="review_status"]');
+        if (!statusSelect) {
+            return;
+        }
+        const statusLabel = form.querySelector('.js-review-status');
+        const data = new FormData(form);
+        data.delete('final_file');
+        data.delete('old_exam_file');
+        if (options.commitComment) {
+            const commentInput = form.querySelector('textarea[name="reviewer_comment"]');
+            if (commentInput) {
+                const raw = commentInput.value || '';
+                const committed = raw.endsWith('\n') ? raw : raw + '\n';
+                data.set('reviewer_comment', committed);
+            }
+        }
+        const csrfToken = syncCsrfToken(form);
+        if (csrfToken) {
+            data.set(csrfTokenName, csrfToken);
+        }
+        if (statusLabel) {
+            statusLabel.textContent = 'Saving...';
+        }
+        fetch(form.action, {
+            method: 'POST',
+            body: data,
+            credentials: 'same-origin',
+            headers: {
+                'X-Requested-With': 'XMLHttpRequest',
+                'X-CSRF-TOKEN': csrfToken,
+            },
+        })
+            .then((resp) => {
+                if (!resp.ok) {
+                    throw new Error('Save failed');
+                }
+                syncCsrfToken(form);
+                updateStatusUI(row);
+                if (statusLabel) {
+                    statusLabel.textContent = 'Saved';
+                }
+                setTimeout(() => {
+                    if (statusLabel && statusLabel.textContent === 'Saved') {
+                        statusLabel.textContent = '';
+                    }
+                }, 1500);
+            })
+            .catch(() => {
+                if (statusLabel) {
+                    statusLabel.textContent = 'Error saving';
+                }
+            });
+    };
+
+    const debounce = (fn, wait) => {
+        let timer;
+        return (...args) => {
+            clearTimeout(timer);
+            timer = setTimeout(() => fn(...args), wait);
+        };
+    };
+
+    const debouncedSave = debounce(saveRow, 500);
+
+    document.querySelectorAll('.exam-drafts-table tbody tr').forEach(updateRow);
+
+    document.addEventListener('change', (event) => {
+        const target = event.target;
+        if (!(target instanceof HTMLElement)) {
+            return;
+        }
+        const row = target.closest('tr');
+        if (!row) {
+            return;
+        }
+        if (target instanceof HTMLSelectElement && target.name === 'review_status') {
+            updateRow(row);
+            debouncedSave(row);
+            return;
+        }
+        if (target instanceof HTMLInputElement && target.name === 'acceptance_type') {
+            debouncedSave(row);
+        }
+    });
+
+    document.addEventListener('input', (event) => {
+        const target = event.target;
+        if (!(target instanceof HTMLTextAreaElement) || target.name !== 'reviewer_comment') {
+            return;
+        }
+        const row = target.closest('tr');
+        if (row && target.value.endsWith('\n')) {
+            const lastCommitted = target.dataset.lastCommitted || '';
+            if (target.value !== lastCommitted) {
+                target.dataset.lastCommitted = target.value;
+                debouncedSave(row, { commitComment: true });
+            }
+        }
+    });
+
+    document.addEventListener('blur', (event) => {
+        const target = event.target;
+        if (!(target instanceof HTMLTextAreaElement) || target.name !== 'reviewer_comment') {
+            return;
+        }
+        const row = target.closest('tr');
+        if (row) {
+            const lastCommitted = target.dataset.lastCommitted || '';
+            if (target.value !== lastCommitted) {
+                target.dataset.lastCommitted = target.value;
+                debouncedSave(row, { commitComment: true });
+            }
+        }
+    }, true);
+});
+</script>
 <?= $this->endSection() ?>
 
 <?= $this->section('styles') ?>
@@ -270,13 +567,15 @@
     word-break: break-word;
   }
   .exam-drafts-table th {
-    min-width: 120px;
+    min-width: 0;
+    white-space: nowrap;
   }
   .exam-drafts-table td {
-    max-width: 220px;
+    max-width: none;
   }
   .exam-drafts-table {
     table-layout: auto;
+    width: max-content;
   }
   .exam-drafts-table thead th {
     background-color: #f8f9fa;

app/Views/administrator/subject_curriculum.php

@@ -144,58 +144,86 @@
           <span class="text-muted small"><?= count($entries) ?> record<?= count($entries) === 1 ? '' : 's' ?></span>
         </div>
         <div class="card-body">
-          <div class="table-responsive">
+          <?php if (empty($entries)): ?>
-            <table class="table table-bordered table-hover align-middle mb-0" data-no-mgmt-sticky>
+            <div class="text-center text-muted">No curriculum records yet.</div>
-              <thead class="table-light">
+          <?php else: ?>
-                <tr>
+            <?php
-                  <th>Class</th>
+              $entriesByClass = [];
-                  <th>Subject</th>
+              foreach ($entries as $entry) {
-                  <th>Unit</th>
+                $className = ($entry['class_name'] ?? '') ?: '—';
-                  <th>Unit title</th>
+                if (!isset($entriesByClass[$className])) {
-                  <th>Chapter / Surah</th>
+                  $entriesByClass[$className] = [];
-                  <th>Updated</th>
+                }
-                  <th style="min-width: 170px;">Actions</th>
+                $entriesByClass[$className][] = $entry;
-                </tr>
+              }
-              </thead>
+            ?>
-              <tbody>
+            <div class="accordion" id="curriculumAccordion">
-                <?php if (empty($entries)): ?>
+              <?php $accIndex = 0; ?>
-                  <tr>
+              <?php foreach ($entriesByClass as $className => $classEntries): ?>
-                    <td colspan="7" class="text-center text-muted">No curriculum records yet.</td>
+                <?php
-                  </tr>
+                  $accIndex++;
-                <?php else: ?>
+                  $collapseId = 'curriculum-class-' . $accIndex;
-                  <?php foreach ($entries as $entry): ?>
+                  $headingId = 'curriculum-heading-' . $accIndex;
-                    <?php
+                ?>
-                      $subjectLabel = $subjectLabels[$entry['subject']] ?? ucfirst($entry['subject'] ?? '');
+                <div class="accordion-item mb-2">
-                      $updatedAt = $entry['updated_at'] ?? $entry['created_at'] ?? '';
+                  <h2 class="accordion-header" id="<?= esc($headingId) ?>">
-                      $updatedDisplay = '';
+                    <button class="accordion-button collapsed" type="button" data-bs-toggle="collapse" data-bs-target="#<?= esc($collapseId) ?>" aria-expanded="false" aria-controls="<?= esc($collapseId) ?>">
-                      if ($updatedAt) {
+                      <?= esc($className) ?>
-                        try {
+                      <span class="badge bg-secondary ms-2"><?= count($classEntries) ?> entries</span>
-                          $updatedDisplay = (new \DateTime($updatedAt))->format('M d, Y H:i');
+                    </button>
-                        } catch (\Exception $e) {
+                  </h2>
-                          $updatedDisplay = $updatedAt;
+                  <div id="<?= esc($collapseId) ?>" class="accordion-collapse collapse" aria-labelledby="<?= esc($headingId) ?>" data-bs-parent="#curriculumAccordion">
-                        }
+                    <div class="accordion-body">
-                      }
+                      <div class="table-responsive">
-                    ?>
+                        <table class="table table-bordered table-hover align-middle mb-0" data-no-mgmt-sticky>
-                    <tr>
+                          <thead class="table-light">
-                      <td><?= esc(($entry['class_name'] ?? '') ?: '—') ?></td>
+                            <tr>
-                      <td><?= esc($subjectLabel) ?></td>
+                              <th>Subject</th>
-                      <td><?= $entry['unit_number'] ? esc((string)$entry['unit_number']) : '—' ?></td>
+                              <th>Unit</th>
-                      <td><?= esc(($entry['unit_title'] ?? '') ?: '—') ?></td>
+                              <th>Unit title</th>
-                      <td><?= esc(($entry['chapter_name'] ?? '') ?: '—') ?></td>
+                              <th>Chapter / Surah</th>
-                      <td><?= esc($updatedDisplay ?: '—') ?></td>
+                              <th>Updated</th>
-                      <td class="d-flex gap-2 flex-wrap">
+                              <th style="min-width: 170px;">Actions</th>
-                        <a href="<?= site_url('administrator/subject-curriculum/edit/' . $entry['id']) ?>" class="btn btn-sm btn-outline-primary">Edit</a>
+                            </tr>
-                        <form action="<?= site_url('administrator/subject-curriculum/delete/' . $entry['id']) ?>" method="post" onsubmit="return confirm('Remove this curriculum entry?');">
+                          </thead>
-                          <?= csrf_field() ?>
+                          <tbody>
-                          <button type="submit" class="btn btn-sm btn-outline-danger">Delete</button>
+                            <?php foreach ($classEntries as $entry): ?>
-                        </form>
+                              <?php
-                      </td>
+                                $subjectLabel = $subjectLabels[$entry['subject']] ?? ucfirst($entry['subject'] ?? '');
-                    </tr>
+                                $updatedAt = $entry['updated_at'] ?? $entry['created_at'] ?? '';
-                  <?php endforeach; ?>
+                                $updatedDisplay = '';
-                <?php endif; ?>
+                                if ($updatedAt) {
-              </tbody>
+                                  try {
-            </table>
+                                    $updatedDisplay = (new \DateTime($updatedAt))->format('M d, Y H:i');
-          </div>
+                                  } catch (\Exception $e) {
+                                    $updatedDisplay = $updatedAt;
+                                  }
+                                }
+                              ?>
+                              <tr>
+                                <td><?= esc($subjectLabel) ?></td>
+                                <td><?= $entry['unit_number'] ? esc((string)$entry['unit_number']) : '—' ?></td>
+                                <td><?= esc(($entry['unit_title'] ?? '') ?: '—') ?></td>
+                                <td><?= esc(($entry['chapter_name'] ?? '') ?: '—') ?></td>
+                                <td><?= esc($updatedDisplay ?: '—') ?></td>
+                                <td class="d-flex gap-2 flex-wrap">
+                                  <a href="<?= site_url('administrator/subject-curriculum/edit/' . $entry['id']) ?>" class="btn btn-sm btn-outline-primary">Edit</a>
+                                  <form action="<?= site_url('administrator/subject-curriculum/delete/' . $entry['id']) ?>" method="post" onsubmit="return confirm('Remove this curriculum entry?');">
+                                    <?= csrf_field() ?>
+                                    <button type="submit" class="btn btn-sm btn-outline-danger">Delete</button>
+                                  </form>
+                                </td>
+                              </tr>
+                            <?php endforeach; ?>
+                          </tbody>
+                        </table>
+                      </div>
+                    </div>
+                  </div>
+                </div>
+              <?php endforeach; ?>
+            </div>
+          <?php endif; ?>
         </div>
       </div>
     </div>

app/Views/administrator/teacher_class_assignment.php

@@ -63,7 +63,15 @@
                 </thead>
                 <tbody>
                     <tr>
-                        <td colspan="9" class="text-center text-muted">Loading...</td>
+                        <td class="text-center text-muted">Loading...</td>
+                        <td></td>
+                        <td></td>
+                        <td></td>
+                        <td></td>
+                        <td></td>
+                        <td></td>
+                        <td></td>
+                        <td></td>
                     </tr>
                 </tbody>
             </table>
@@ -198,10 +206,12 @@ document.addEventListener('DOMContentLoaded', function () {
         if (!teachers || teachers.length === 0) {
             var emptyRow = document.createElement('tr');
             var emptyCell = document.createElement('td');
-            emptyCell.colSpan = 9;
             emptyCell.className = 'text-center text-muted';
             emptyCell.textContent = 'No teachers found.';
             emptyRow.appendChild(emptyCell);
+            for (var i = 0; i < 8; i++) {
+                emptyRow.appendChild(document.createElement('td'));
+            }
             tbody.appendChild(emptyRow);
         } else {
             teachers.forEach(function (teacher, index) {
@@ -312,7 +322,10 @@ document.addEventListener('DOMContentLoaded', function () {
             return;
         }
 
-        tbody.innerHTML = '<tr><td colspan="9" class="text-center text-muted">Loading...</td></tr>';
+        tbody.innerHTML = '<tr>'
+            + '<td class="text-center text-muted">Loading...</td>'
+            + '<td></td><td></td><td></td><td></td><td></td><td></td><td></td><td></td>'
+            + '</tr>';
 
         var url = apiList + (selectedYear ? ('?schoolYear=' + encodeURIComponent(selectedYear)) : '');
         fetch(url, {

app/Views/administrator/teacher_submissions.php

@@ -23,6 +23,41 @@
     <div class="card-body">
       <?php
         $termExamLabel = (isset($semester) && strtolower((string) $semester) === 'spring') ? 'Final' : 'Midterm';
+        $lowProgressSectionIds = $lowProgressSectionIds ?? [];
+        $flaggedClasses = count($lowProgressSectionIds);
+        $pageNotifications = [];
+        if ($missingItemsCount > 0) {
+            $pageNotifications[] = [
+                'level' => 'danger',
+                'message' => "{$missingItemsCount} missing item" . ($missingItemsCount === 1 ? '' : 's') . " awaiting teacher uploads.",
+            ];
+        }
+        if ($completionPercent < 70) {
+            $pageNotifications[] = [
+                'level' => 'warning',
+                'message' => "Submission completion is below 70% — follow up with remaining teachers.",
+            ];
+        }
+        if ($flaggedClasses > 0) {
+            $pageNotifications[] = [
+                'level' => 'warning',
+                'message' => "Progress is under 50% for {$flaggedClasses} class section" . ($flaggedClasses === 1 ? '' : 's') . ".",
+            ];
+        }
+        if (empty($rows)) {
+            $pageNotifications[] = [
+                'level' => 'info',
+                'message' => 'No class-section assignments submitted yet; encourage teachers to upload drafts.',
+            ];
+        }
+        if (empty($pageNotifications)) {
+            $pageNotifications[] = [
+                'level' => 'info',
+                'message' => 'All tracked classes are current. Use the controls below to send reminders.',
+            ];
+        }
+        $examDraftDeadlineConfig = $examDraftDeadlineConfig ?? '';
+        $examDraftDeadlineFormatted = $examDraftDeadlineFormatted ?? '';
       ?>
       <?php if (session()->getFlashdata('success')): ?>
         <div class="alert alert-success">
@@ -37,203 +72,278 @@
           <?= esc(session()->getFlashdata('info')) ?>
         </div>
       <?php endif; ?>
-      <?php $lowProgressSectionIds = $lowProgressSectionIds ?? []; ?>
+      <?php
-      <?php if (!empty($lowProgressSectionIds)): ?>
+        $groupedRows = [];
-        <div class="alert alert-warning">
+        foreach ($rows as $idx => $row) {
-          Showing teachers for class sections with progress submissions below 50%.
+          $classKey = (string) ($row['class_section_id'] ?? $row['class_section'] ?? 'class_' . $idx);
-        </div>
+          $groupedRows[$classKey][] = $row;
-      <?php endif; ?>
+        }
-      <div class="border rounded-3 p-3 mb-4 bg-light">
+      ?>
-        <div class="d-flex flex-wrap gap-4 align-items-center">
+      <div class="row g-3 mb-4 align-items-stretch">
-          <div>
+        <div class="col-lg-8">
-            <div class="text-uppercase small text-muted">Submission completion</div>
+          <div class="row g-3 summary-row">
-            <div class="h4 fw-semibold mb-1"><?= esc($completionPercent) ?>%</div>
+            <div class="col-12 col-sm-6 col-xl-3">
-            <div class="progress" style="height:6px;">
+              <div class="summary-card h-100 shadow-sm border-0">
-              <div
+                <div class="summary-card-body">
-                class="progress-bar bg-primary"
+                  <div class="summary-label text-uppercase">Completion</div>
-                role="progressbar"
+                  <div class="summary-value"><?= esc($completionPercent) ?>%</div>
-                style="width: <?= esc($completionPercent) ?>%;"
+                  <div class="progress summary-progress">
-                aria-valuenow="<?= esc($completionPercent) ?>"
+                    <div
-                aria-valuemin="0"
+                      class="progress-bar"
-                aria-valuemax="100"
+                      role="progressbar"
-              ></div>
+                      style="width: <?= esc($completionPercent) ?>%;"
+                      aria-valuenow="<?= esc($completionPercent) ?>"
+                      aria-valuemin="0"
+                      aria-valuemax="100"
+                    ></div>
+                  </div>
+                </div>
+              </div>
+            </div>
+            <div class="col-12 col-sm-6 col-xl-3">
+              <div class="summary-card h-100 shadow-sm border-0">
+                <div class="summary-card-body">
+                  <div class="summary-label text-uppercase">Missing items</div>
+                  <div class="summary-value <?= $missingItemsCount > 0 ? 'text-danger' : '' ?>"><?= esc($missingItemsCount) ?></div>
+                  <div class="summary-note"><?= esc($submittedItems) ?> submitted / <?= esc($totalItems) ?> total</div>
+                </div>
+              </div>
+            </div>
+            <div class="col-12 col-sm-6 col-xl-3">
+              <div class="summary-card h-100 shadow-sm border-0">
+                <div class="summary-card-body">
+                  <div class="summary-label text-uppercase">Submissions</div>
+                  <div class="summary-value"><?= esc($submittedItems) ?></div>
+                  <div class="summary-note"><?= esc($totalItems) ?> teachers tracked</div>
+                </div>
+              </div>
+            </div>
+            <div class="col-12 col-sm-6 col-xl-3">
+              <div class="summary-card h-100 shadow-sm border-0">
+                <div class="summary-card-body">
+                  <div class="summary-label text-uppercase">Flagged</div>
+                  <div class="summary-value"><?= esc($flaggedClasses) ?></div>
+                  <div class="summary-note">Sections under 50% complete</div>
+                </div>
+              </div>
             </div>
           </div>
-          <div>
+        </div>
-            <div class="text-uppercase small text-muted">Missing items</div>
+        <div class="col-lg-4">
-            <div class="h4 fw-semibold text-danger mb-0"><?= esc($missingItemsCount) ?></div>
+          <div class="card page-notifications-card h-100 shadow-sm">
-          </div>
+            <div class="card-header d-flex justify-content-between align-items-center">
-          <div class="text-muted small">
+              <span>Page notifications</span>
-            <?= esc($submittedItems) ?> submitted / <?= esc($totalItems) ?> total items
+              <span class="badge bg-light text-dark"><?= count($pageNotifications) ?> alerts</span>
+            </div>
+            <div class="card-body">
+              <ul class="page-notifications-list mb-0">
+                <?php foreach ($pageNotifications as $notification): ?>
+                  <li class="page-notification-item page-notification-<?= esc($notification['level']) ?>">
+                    <span class="notification-indicator" aria-hidden="true"></span>
+                    <span><?= esc($notification['message']) ?></span>
+                  </li>
+                <?php endforeach; ?>
+              </ul>
+            </div>
           </div>
         </div>
       </div>
       <form method="post" action="<?= site_url('administrator/teacher-submissions/notify') ?>">
         <?= csrf_field() ?>
-        <div class="table-responsive">
+        <?php if (empty($rows)): ?>
-          <table
+          <p class="text-center text-muted mt-4">No teacher-class assignments found for this term.</p>
-            class="table table-striped table-bordered m-0 align-middle teacher-submissions-table"
+        <?php else: ?>
-            data-no-mgmt-sticky
+          <div class="table-control-bar mb-3 d-flex flex-wrap align-items-center gap-3">
-            data-no-dt-fixedheader
+            <div class="d-flex flex-wrap align-items-center gap-3">
-          >
+              <label class="form-check form-check-inline mb-0">
-            <thead class="table-light">
+                <input class="form-check-input" type="checkbox" name="notify_midterm_score" value="1">
-              <tr>
+                <span class="form-check-label small">Include <?= esc($termExamLabel) ?> score</span>
-                <th>Class-Section</th>
+              </label>
-                <th>Teachers Name</th>
+              <label class="form-check form-check-inline mb-0">
-                <th class="text-center">
+                <input class="form-check-input" type="checkbox" name="notify_midterm_comment" value="1">
-                  <?= esc($termExamLabel) ?> Score
+                <span class="form-check-label small">Include <?= esc($termExamLabel) ?> comment</span>
-                  <div class="form-check d-inline-flex align-items-center gap-1 ms-2">
+              </label>
-                    <input class="form-check-input" type="checkbox" name="notify_midterm_score" id="notifyMidtermScore" value="1">
+              <label class="form-check form-check-inline mb-0">
-                    <label class="form-check-label small" for="notifyMidtermScore">Include</label>
+                <input class="form-check-input" type="checkbox" name="notify_participation" value="1">
-                  </div>
+                <span class="form-check-label small">Include participation</span>
-                </th>
+              </label>
-                <th class="text-center">
+              <label class="form-check form-check-inline mb-0">
-                  <?= esc($termExamLabel) ?> Comment
+                <input class="form-check-input" type="checkbox" name="notify_ptap_comment" value="1">
-                  <div class="form-check d-inline-flex align-items-center gap-1 ms-2">
+                <span class="form-check-label small">Include PTAP comment</span>
-                    <input class="form-check-input" type="checkbox" name="notify_midterm_comment" id="notifyMidtermComment" value="1">
+              </label>
-                    <label class="form-check-label small" for="notifyMidtermComment">Include</label>
+              <label class="form-check form-check-inline mb-0">
-                  </div>
+                <input class="form-check-input" type="checkbox" name="notify_class_progress" value="1">
-                </th>
+                <span class="form-check-label small">Include class progress</span>
-                <th class="text-center">
+              </label>
-                  Participation
+              <label class="form-check form-check-inline mb-0">
-                  <div class="form-check d-inline-flex align-items-center gap-1 ms-2">
+                <input class="form-check-input" type="checkbox" name="notify_exam_draft" value="1">
-                    <input class="form-check-input" type="checkbox" name="notify_participation" id="notifyParticipation" value="1">
+                <span class="form-check-label small">Include exam draft</span>
-                    <label class="form-check-label small" for="notifyParticipation">Include</label>
+              </label>
-                  </div>
+              <label class="form-check form-check-inline mb-0">
-                </th>
+                <input class="form-check-input" type="checkbox" name="homework_notify_all" value="1">
-                <th class="text-center">
+                <span class="form-check-label small">Include homework</span>
-                  PTAP Comment
+              </label>
-                  <div class="form-check d-inline-flex align-items-center gap-1 ms-2">
+            </div>
-                    <input class="form-check-input" type="checkbox" name="notify_ptap_comment" id="notifyPtapComment" value="1">
+            <div class="small text-muted">
-                    <label class="form-check-label small" for="notifyPtapComment">Include</label>
+              <span class="text-uppercase" style="font-size:0.65rem;">Exam draft deadline</span><br>
-                  </div>
+              <?php if ($examDraftDeadlineConfig !== ''): ?>
-                </th>
+                <?= esc($examDraftDeadlineConfig) ?>
-                <th class="text-center">
+                <?php if ($examDraftDeadlineFormatted !== ''): ?>
-                  Class Progress
+                  <span class="text-muted"> → <?= esc($examDraftDeadlineFormatted) ?></span>
-                  <div class="form-check d-inline-flex align-items-center gap-1 ms-2">
+                <?php endif; ?>
-                    <input class="form-check-input" type="checkbox" name="notify_class_progress" id="notifyClassProgress" value="1">
-                    <label class="form-check-label small" for="notifyClassProgress">Include</label>
-                  </div>
-                </th>
-                <th class="text-center">
-                  Exam Draft
-                  <div class="form-check d-inline-flex align-items-center gap-1 ms-2">
-                    <input class="form-check-input" type="checkbox" name="notify_exam_draft" id="notifyExamDraft" value="1">
-                    <label class="form-check-label small" for="notifyExamDraft">Include</label>
-                  </div>
-                </th>
-                <th class="text-center">
-                  Homework
-                  <div class="form-check d-inline-flex align-items-center gap-1 ms-2">
-                    <input class="form-check-input" type="checkbox" name="homework_notify_all" id="homeworkNotifyAll" value="1">
-                    <label class="form-check-label small" for="homeworkNotifyAll">Include</label>
-                  </div>
-                </th>
-                <th class="text-center">Notifications</th>
-              </tr>
-            </thead>
-            <tbody>
-              <?php if (!empty($rows)): ?>
-                <?php $homeworkToggleRendered = false; ?>
-                <?php foreach ($rows as $row): ?>
-                  <tr>
-                    <td><?= esc($row['class_section']) ?></td>
-                    <td>
-                      <?php if (!empty($row['teachers'])): ?>
-                        <?php foreach ($row['teachers'] as $teacher): ?>
-                          <div><?= esc($teacher['label'] ?? 'Teacher') ?></div>
-                        <?php endforeach; ?>
-                      <?php else: ?>
-                        <span class="text-muted small">Unassigned</span>
-                      <?php endif; ?>
-                    </td>
-                    <?php foreach ([
-                      'midterm_score_status',
-                      'midterm_comment_status',
-                      'participation_status',
-                      'ptap_comment_status',
-                      'class_progress_status',
-                      'exam_draft_status',
-                    ] as $statusKey): ?>
-                      <?php $status = $row[$statusKey] ?? ['label' => 'N/A', 'badge' => 'bg-secondary']; ?>
-                      <td class="text-center">
-                        <span class="badge <?= esc($status['badge'] ?? 'bg-secondary') ?>">
-                          <?= esc($status['label'] ?? 'N/A') ?>
-                        </span>
-                        <?php if (!empty($status['detail'])): ?>
-                          <div class="small text-muted"><?= esc($status['detail']) ?></div>
-                        <?php endif; ?>
-                      </td>
-                    <?php endforeach; ?>
-                    <?php $homeworkStatus = $row['homework_status'] ?? ['label' => 'N/A', 'badge' => 'bg-secondary']; ?>
-                    <td class="text-center">
-                      <span class="badge <?= esc($homeworkStatus['badge'] ?? 'bg-secondary') ?>">
-                        <?= esc($homeworkStatus['label'] ?? 'N/A') ?>
-                      </span>
-                      <?php if (!empty($homeworkStatus['detail'])): ?>
-                        <div class="small text-muted"><?= esc($homeworkStatus['detail']) ?></div>
-                      <?php endif; ?>
-                    </td>
-                    <td>
-                      <?php if (!empty($row['teachers'])): ?>
-                        <?php $missingPayload = base64_encode(json_encode($row['missing_items'] ?? [])); ?>
-                        <?php foreach ($row['teachers'] as $teacher): ?>
-                          <?php $history = $notificationHistory[$row['class_section_id']][$teacher['id']] ?? []; ?>
-                          <?php $lastEntry = $history[0] ?? null; ?>
-                          <div class="mb-2">
-                            <label class="d-flex align-items-center gap-2 mb-1">
-                              <input
-                                type="checkbox"
-                                class="form-check-input"
-                                name="notify[<?= esc($row['class_section_id']) ?>][<?= esc($teacher['id']) ?>]"
-                                value="1"
-                              />
-                              <span class="fw-semibold"><?= esc($teacher['label'] ?? 'Teacher') ?></span>
-                            </label>
-                            <input
-                              type="hidden"
-                              name="missing_items[<?= esc($row['class_section_id']) ?>][<?= esc($teacher['id']) ?>]"
-                              value="<?= esc($missingPayload) ?>"
-                            />
-                            <div class="small text-muted">
-                              <?php if ($lastEntry !== null): ?>
-                                <span>
-                                  Last <?= esc($lastEntry['status'] === 'sent' ? 'sent' : 'attempted') ?> on <?= esc($lastEntry['sent_at_text'] ?: 'N/A') ?>
-                                  by <?= esc($lastEntry['admin_name'] ?? '') ?>
-                                </span>
-                                <?php if (count($history) > 1): ?>
-                                  <div>History: <?= count($history) ?> entries</div>
-                                <?php endif; ?>
-                              <?php else: ?>
-                                <span>No notifications sent yet</span>
-                              <?php endif; ?>
-                            </div>
-                          </div>
-                        <?php endforeach; ?>
-                      <?php else: ?>
-                        <span class="text-muted small">No teacher assigned</span>
-                      <?php endif; ?>
-                      <?php if (!empty($row['missing_items'])): ?>
-                        <div class="small text-danger mt-1">
-                          Outstanding: <?= esc(implode(', ', $row['missing_items'])) ?>
-                        </div>
-                      <?php endif; ?>
-                    </td>
-                  </tr>
-                <?php endforeach; ?>
               <?php else: ?>
-                <tr>
+                <span class="text-warning">Not set</span>
-                  <td colspan="7" class="text-center">No teacher-class assignments found for this term.</td>
-                </tr>
               <?php endif; ?>
-            </tbody>
+            </div>
-          </table>
+          </div>
-        </div>
+          <div class="accordion" id="teacherSubmissionAccordion">
-        <div class="mt-3 text-end">
+            <?php foreach ($groupedRows as $sectionKey => $sectionRows): ?>
-          <button type="submit" class="btn btn-primary" <?= empty($rows) ? 'disabled' : '' ?>>
+              <?php
-            Send notifications to selected teachers
+                $firstRow = $sectionRows[0] ?? [];
-          </button>
+                $sectionLabel = esc($firstRow['class_section'] ?? 'Class section');
-        </div>
+                $collapseId = 'teacherSectionCollapse_' . md5($sectionKey);
+                $badgeStatus = count($sectionRows) === 1 ? 'submission' : 'submissions';
+              ?>
+              <div class="accordion-item">
+                <h2 class="accordion-header" id="heading_<?= esc($collapseId) ?>">
+                  <button
+                    class="accordion-button collapsed d-flex justify-content-between align-items-center"
+                    type="button"
+                    data-bs-toggle="collapse"
+                    data-bs-target="#<?= esc($collapseId) ?>"
+                    aria-expanded="false"
+                    aria-controls="<?= esc($collapseId) ?>"
+                  >
+                    <div>
+                      <strong><?= $sectionLabel ?></strong>
+                      <div class="small text-muted"><?= esc(count($sectionRows)) ?> <?= $badgeStatus ?></div>
+                    </div>
+                    <span class="badge bg-primary-subtle text-primary"><?= count($sectionRows) ?> rows</span>
+                  </button>
+                </h2>
+                <div id="<?= esc($collapseId) ?>" class="accordion-collapse collapse" aria-labelledby="heading_<?= esc($collapseId) ?>">
+                  <div class="accordion-body p-0">
+                    <div class="table-responsive">
+                      <table
+                        class="table table-striped table-bordered m-0 align-middle teacher-submissions-table mb-0"
+                        data-no-mgmt-sticky
+                        data-no-dt-fixedheader
+                      >
+                        <thead class="table-light">
+                          <tr>
+                            <th>Class-Section</th>
+                            <th>Teachers Name</th>
+                            <th class="text-center"><?= esc($termExamLabel) ?> Score</th>
+                            <th class="text-center"><?= esc($termExamLabel) ?> Comment</th>
+                            <th class="text-center">Participation</th>
+                            <th class="text-center">PTAP Comment</th>
+                            <th class="text-center">Class Progress</th>
+                            <th class="text-center">Exam Draft</th>
+                            <th class="text-center">Homework</th>
+                            <th class="text-center">Notifications</th>
+                          </tr>
+                        </thead>
+                        <tbody>
+                          <?php foreach ($sectionRows as $row): ?>
+                            <tr>
+                              <td><?= esc($row['class_section']) ?></td>
+                              <td>
+                                <?php if (!empty($row['teachers'])): ?>
+                                  <?php foreach ($row['teachers'] as $teacher): ?>
+                                    <div><?= esc($teacher['label'] ?? 'Teacher') ?></div>
+                                  <?php endforeach; ?>
+                                <?php else: ?>
+                                  <span class="text-muted small">Unassigned</span>
+                                <?php endif; ?>
+                              </td>
+                              <?php foreach ([
+                                'midterm_score_status',
+                                'midterm_comment_status',
+                                'participation_status',
+                                'ptap_comment_status',
+                                'class_progress_status',
+                                'exam_draft_status',
+                              ] as $statusKey): ?>
+                                <?php $status = $row[$statusKey] ?? ['label' => 'N/A', 'badge' => 'bg-secondary']; ?>
+                                <td class="text-center">
+                                  <span class="badge <?= esc($status['badge'] ?? 'bg-secondary') ?>">
+                                    <?= esc($status['label'] ?? 'N/A') ?>
+                                  </span>
+                                  <?php if (!empty($status['detail'])): ?>
+                                    <div class="small text-muted"><?= esc($status['detail']) ?></div>
+                                  <?php endif; ?>
+                                </td>
+                              <?php endforeach; ?>
+                              <?php $homeworkStatus = $row['homework_status'] ?? ['label' => 'N/A', 'badge' => 'bg-secondary']; ?>
+                              <td class="text-center">
+                                <span class="badge <?= esc($homeworkStatus['badge'] ?? 'bg-secondary') ?>">
+                                  <?= esc($homeworkStatus['label'] ?? 'N/A') ?>
+                                </span>
+                                <?php if (!empty($homeworkStatus['detail'])): ?>
+                                  <div class="small text-muted"><?= esc($homeworkStatus['detail']) ?></div>
+                                <?php endif; ?>
+                              </td>
+                              <td>
+                                <?php if (!empty($row['teachers'])): ?>
+                                  <?php $missingPayload = base64_encode(json_encode($row['missing_items'] ?? [])); ?>
+                                  <?php foreach ($row['teachers'] as $teacher): ?>
+                                    <?php $history = $notificationHistory[$row['class_section_id']][$teacher['id']] ?? []; ?>
+                                    <?php $lastEntry = $history[0] ?? null; ?>
+                                    <div class="mb-2">
+                                      <label class="d-flex align-items-center gap-2 mb-1">
+                                        <input
+                                          type="checkbox"
+                                          class="form-check-input"
+                                          name="notify[<?= esc($row['class_section_id']) ?>][<?= esc($teacher['id']) ?>]"
+                                          value="1"
+                                        />
+                                        <span class="fw-semibold"><?= esc($teacher['label'] ?? 'Teacher') ?></span>
+                                      </label>
+                                      <input
+                                        type="hidden"
+                                        name="missing_items[<?= esc($row['class_section_id']) ?>][<?= esc($teacher['id']) ?>]"
+                                        value="<?= esc($missingPayload) ?>"
+                                      />
+                                      <div class="small text-muted">
+                                        <?php if ($lastEntry !== null): ?>
+                                          <span>
+                                            Last <?= esc($lastEntry['status'] === 'sent' ? 'sent' : 'attempted') ?> on <?= esc($lastEntry['sent_at_text'] ?: 'N/A') ?>
+                                            by <?= esc($lastEntry['admin_name'] ?? '') ?>
+                                          </span>
+                                          <?php if (count($history) > 1): ?>
+                                            <div>History: <?= count($history) ?> entries</div>
+                                          <?php endif; ?>
+                                        <?php else: ?>
+                                          <span>No notifications sent yet</span>
+                                        <?php endif; ?>
+                                      </div>
+                                    </div>
+                                  <?php endforeach; ?>
+                                <?php else: ?>
+                                  <span class="text-muted small">No teacher assigned</span>
+                                <?php endif; ?>
+                                <?php if (!empty($row['missing_items'])): ?>
+                                  <div class="small text-danger mt-1">
+                                    Outstanding: <?= esc(implode(', ', $row['missing_items'])) ?>
+                                  </div>
+                                <?php endif; ?>
+                              </td>
+                            </tr>
+                          <?php endforeach; ?>
+                        </tbody>
+                      </table>
+                    </div>
+                  </div>
+                </div>
+              </div>
+            <?php endforeach; ?>
+          </div>
+          <div class="mt-3 text-end">
+            <button type="submit" class="btn btn-primary">
+              Send notifications to selected teachers
+            </button>
+          </div>
+        <?php endif; ?>
       </form>
     </div>
   </div>
@@ -265,5 +375,77 @@
   .card-body .table-responsive .teacher-submissions-table td {
     white-space: normal;
   }
+
+  .summary-card {
+    background: var(--bs-white);
+    border-radius: 0.75rem;
+  }
+  .summary-card-body {
+    padding: 1.25rem;
+  }
+  .summary-label {
+    font-size: 0.7rem;
+    letter-spacing: 0.08em;
+    color: #6c757d;
+  }
+  .summary-value {
+    font-size: 1.75rem;
+    font-weight: 600;
+    margin-top: 0.35rem;
+  }
+  .summary-note {
+    font-size: 0.85rem;
+    color: #6c757d;
+  }
+  .summary-progress {
+    height: 6px;
+    margin-top: 0.65rem;
+  }
+  .summary-progress .progress-bar {
+    background: var(--bs-primary);
+  }
+  .page-notifications-card {
+    border-radius: 0.75rem;
+  }
+  .page-notifications-list {
+    list-style: none;
+    padding: 0;
+    margin: 0;
+    display: flex;
+    flex-direction: column;
+    gap: 0.5rem;
+  }
+  .page-notification-item {
+    display: flex;
+    align-items: center;
+    gap: 0.5rem;
+    font-size: 0.85rem;
+  }
+  .page-notification-info .notification-indicator {
+    background: #0dcaf0;
+  }
+  .page-notification-warning .notification-indicator {
+    background: #ffc107;
+  }
+  .page-notification-danger .notification-indicator {
+    background: #dc3545;
+  }
+  .notification-indicator {
+    width: 0.75rem;
+    height: 0.75rem;
+    border-radius: 50%;
+    display: inline-block;
+  }
+  .table-control-bar {
+    border: 1px solid rgba(0,0,0,0.08);
+    border-radius: 0.75rem;
+    padding: 0.75rem 1rem;
+    background: #f8f9fa;
+  }
+  .table-control-bar .form-check-label {
+    font-size: 0.8rem;
+    margin-left: 0.15rem;
+    text-transform: none;
+  }
 </style>
 <?= $this->endSection() ?>

app/Views/grading/homework_tracking.php

@@ -70,7 +70,8 @@ $todayYmd = local_date(utc_now(), 'Y-m-d');
             <tr>
               <th class="sticky-col">Grade</th>
               <th class="sticky-col-2" style="min-width:300px; width:300px;">Teacher &amp; TAs</th>
-              <?php foreach (($headerDates ?? []) as $i => $label): $ymd = $sundays[$i] ?? ''; ?>
+              <th class="text-center">HW Submitted</th>
+                <?php foreach (($headerDates ?? []) as $i => $label): $ymd = $sundays[$i] ?? ''; ?>
                 <?php $isCal = !empty($eventDays[$ymd]); $isFuture = ($ymd > $todayYmd); ?>
                 <th class="text-center <?= $isCal ? 'bg-warning' : ($isFuture ? 'bg-future' : '') ?>" title="<?= esc($ymd) ?>"><?= esc($label) ?></th>
               <?php endforeach; ?>
@@ -100,6 +101,9 @@ $todayYmd = local_date(utc_now(), 'Y-m-d');
                   <?php endif; ?>
                   <div><strong>TA<?= count($taNames) !== 1 ? 's' : '' ?>:</strong> <?= !empty($taNames) ? esc(implode(', ', $taNames)) : '—' ?></div>
                 </td>
+                <td class="text-center">
+                  <?= (int)($homeworkSubmissionCounts[$csid] ?? 0) ?>
+                </td>
                 <?php foreach (($sundays ?? []) as $ymd): ?>
                   <?php if (!empty($eventDays[$ymd])): ?>
                     <td class="bg-warning text-center">—</td>

app/Views/parent/event_participation.php

@@ -42,9 +42,6 @@
                         <?php endif; ?>
                     </div>
 
-                    <!-- Event Description -->
-                    <p class="card-text"><?= esc($event['description']) ?></p>
-
                     <!-- Participation Table -->
                     <form method="post" action="<?= site_url('parent/updateParticipation') ?>">
                         <?= csrf_field() ?>
@@ -53,11 +50,13 @@
                         <div class="table-responsive">
                             <table class="table table-sm table-bordered">
                                 <thead class="table-light">
-                                    <tr>
+                            <tr>
-                                        <th>Student First Name</th>
+                                <th>Student First Name</th>
-                                        <th>Student Last Name</th>
+                                <th>Student Last Name</th>
-                                        <th class="text-center">Participate</th>
+                                <th class="text-center">Participate</th>
-                                    </tr>
+                                <th>Description</th>
+                                <th>Event Fees</th>
+                            </tr>
                                 </thead>
                                 <tbody>
                                     <?php foreach ($yourStudents as $student): ?>
@@ -84,6 +83,8 @@
                                                     </div>
                                                 </div>
                                             </td>
+                                            <td><?= esc($event['description'] ?: 'No description') ?></td>
+                                            <td class="text-nowrap">$<?= esc(number_format((float) ($event['amount'] ?? 0), 2)) ?></td>
                                         </tr>
                                     <?php endforeach; ?>
                                 </tbody>

app/Views/payment/financial_report.php

@@ -54,6 +54,11 @@
     <button class="btn btn-primary" onclick="window.print()">Print Report</button>
     <a id="summaryReportLink" href="<?= base_url('financial-report/financialReportSummary') ?>" class="btn btn-info">Display Summary Report</a>
   </div>
+  <?php if (isset($eventFeesTotal)): ?>
+    <div class="alert alert-info mb-3">
+      <strong>Event fees total:</strong> $<?= number_format((float)$eventFeesTotal, 2) ?>
+    </div>
+  <?php endif; ?>
   <div class="table-responsive">
     <table id="invoicesTable" class="table table-bordered table-striped align-middle w-100">
       <thead>

app/Views/payment/financial_report_summary.php

@@ -32,6 +32,7 @@
   </thead>
   <tbody id="summaryBody">
     <tr><td>Total Charges</td><td class="text-right" id="sumCharges">$0.00</td></tr>
+    <tr><td>Event Fees</td><td class="text-right" id="sumEventFees">$0.00</td></tr>
     <tr><td>Total Extra Charges</td><td class="text-right" id="sumExtraCharges">$0.00</td></tr>
     <tr><td>Total Discounts</td><td class="text-right" id="sumDiscounts">$0.00</td></tr>
     <tr><td>Total Refunds</td><td class="text-right" id="sumRefunds">$0.00</td></tr>
@@ -98,6 +99,9 @@ function loadSummary(){
       if (!d || d.ok !== true) return;
       document.getElementById('summaryPeriod').textContent = 'Report for School Year: ' + (d.schoolYear||'');
       document.getElementById('sumCharges').textContent    = fmt(d.totalCharges);
+      if (document.getElementById('sumEventFees')) {
+        document.getElementById('sumEventFees').textContent = fmt(d.totalEventFees || 0);
+      }
       if (document.getElementById('sumExtraCharges')) {
         document.getElementById('sumExtraCharges').textContent = fmt(d.totalExtraCharges || 0);
       }
@@ -127,10 +131,10 @@ function renderCharts(d){
   const summaryCtx = document.getElementById('summaryChart').getContext('2d');
   window._summaryChart = new Chart(summaryCtx, {
     type: 'bar',
-    data: { labels: ['Charges','Paid','Unpaid','Discounts','Refunds','Expenses','Reimbursements','Net'],
+    data: { labels: ['Charges','Event Fees','Paid','Unpaid','Discounts','Refunds','Expenses','Reimbursements','Net'],
       datasets: [{ label:'Amount (USD)', data:[
-        d.totalCharges||0, d.totalPaid||0, d.totalUnpaid||0, d.totalDiscounts||0, d.totalRefunds||0, d.totalExpenses||0, d.totalReimbursements||0, d.netAmount||0
+        d.totalCharges||0, d.totalEventFees||0, d.totalPaid||0, d.totalUnpaid||0, d.totalDiscounts||0, d.totalRefunds||0, d.totalExpenses||0, d.totalReimbursements||0, d.netAmount||0
-      ], backgroundColor: ['#007bff','#28a745','#ffc107','#17a2b8','#ffc107','#dc3545','#6f42c1','#20c997']}] },
+      ], backgroundColor: ['#007bff','#6610f2','#28a745','#ffc107','#17a2b8','#ffc107','#dc3545','#6f42c1','#20c997']}] },
     options: { responsive:true, scales:{ y:{ beginAtZero:true }}}
   });
 
@@ -177,11 +181,12 @@ document.addEventListener("DOMContentLoaded", function() {
     window._summaryChart = new Chart(summaryCtx, {
         type: 'bar',
         data: {
-            labels: ['Charges', 'Paid', 'Unpaid', 'Discounts', 'Refunds', 'Expenses', 'Reimbursements', 'Net'],
+            labels: ['Charges', 'Event Fees', 'Paid', 'Unpaid', 'Discounts', 'Refunds', 'Expenses', 'Reimbursements', 'Net'],
             datasets: [{
                 label: 'Amount (USD)',
                 data: [
                     <?= (float)$totalCharges ?>,
+                    <?= (float)($totalEventFees ?? 0) ?>,
                     <?= (float)$totalPaid ?>,
                     <?= (float)$totalUnpaid ?>,
                     <?= (float)$totalDiscounts ?>,
@@ -191,7 +196,7 @@ document.addEventListener("DOMContentLoaded", function() {
                     <?= (float)$netAmount ?>
                 ],
                 backgroundColor: [
-                    '#007bff', '#28a745', '#ffc107', '#17a2b8', '#ffc107', '#dc3545', '#6f42c1', '#20c997'
+                    '#007bff', '#6610f2', '#28a745', '#ffc107', '#17a2b8', '#ffc107', '#dc3545', '#6f42c1', '#20c997'
                 ]
             }]
         },

app/Views/payment/unpaid_parents.php

@@ -47,6 +47,7 @@
           <button class="btn btn-primary btn-sm" type="submit">Send Reminders (All)</button>
         </form>
       </div>
+      <?php $sumEventFees = 0.0; ?>
       <div class="table-responsive">
         <table id="unpaidTable" class="table table-sm align-middle no-mgmt-sticky" data-no-mgmt-sticky>
           <thead>
@@ -54,6 +55,7 @@
               <th>Parent</th>
               <th class="text-center">Nbr of Installements</th>
               <th>Type</th>
+              <th class="text-end">Event Fees</th>
               <th class="text-end">Invoice Amount</th>
               <th class="text-end">Applied Discount</th>
               <th class="text-end">Paid Amount</th>
@@ -67,7 +69,7 @@
           </thead>
           <tbody>
             <?php if (empty($rows)): ?>
-              <tr><td colspan="12" class="text-center text-muted py-4">No parents with outstanding balance.</td></tr>
+              <tr><td colspan="13" class="text-center text-muted py-4">No parents with outstanding balance.</td></tr>
             <?php else: ?>
               <?php
                 $sumInvoice = 0.0;
@@ -86,18 +88,20 @@
                   </td>
                   <td class="text-center"><?= (int)($r['payment_count'] ?? 0) ?></td>
                   <td>
-                    <?php if (($r['type'] ?? '') === 'no_payment'): ?>
+                  <?php if (($r['type'] ?? '') === 'no_payment'): ?>
-                      <span class="badge bg-danger badge-type">no payment</span>
+                    <span class="badge bg-danger badge-type">no payment</span>
-                    <?php else: ?>
+                  <?php else: ?>
-                      <span class="badge bg-success badge-type">installment</span>
+                    <span class="badge bg-success badge-type">installment</span>
-                    <?php endif; ?>
+                  <?php endif; ?>
-                  </td>
+                </td>
-                  <?php $sumInvoice += (float)($r['total_invoice'] ?? 0); ?>
+                <?php $sumInvoice += (float)($r['total_invoice'] ?? 0); ?>
-                  <?php $sumPaid    += (float)($r['total_paid']    ?? 0); ?>
+                <?php $sumEventFees += (float)($r['event_fees'] ?? 0); ?>
+                <?php $sumPaid    += (float)($r['total_paid']    ?? 0); ?>
                   <?php $sumDisc    += (float)($r['total_discount'] ?? 0); ?>
                   <?php $sumInstAmt += (float)($r['installment_amount'] ?? 0); ?>
                   <?php $sumBal     += (float)($r['total_balance'] ?? 0); ?>
-                  <td class="text-end">$<?= number_format((float)($r['total_invoice'] ?? 0), 2) ?></td>
+                <td class="text-end">$<?= number_format((float)($r['event_fees'] ?? 0), 2) ?></td>
+                <td class="text-end">$<?= number_format((float)($r['total_invoice'] ?? 0), 2) ?></td>
                   <td class="text-end text-success">-$<?= number_format((float)($r['total_discount'] ?? 0), 2) ?></td>
                   <td class="text-end">$<?= number_format((float)($r['total_paid'] ?? 0), 2) ?></td>
                   <td class="text-center"><?= (int)($r['remaining_installments'] ?? 0) ?></td>
@@ -129,6 +133,7 @@
           <tfoot>
             <tr>
               <th colspan="3" class="text-end">Totals:</th>
+              <th class="text-end">$<?= number_format($sumEventFees, 2) ?></th>
               <th class="text-end">$<?= number_format($sumInvoice, 2) ?></th>
               <th class="text-end text-success">-$<?= number_format($sumDisc, 2) ?></th>
               <th class="text-end">$<?= number_format($sumPaid, 2) ?></th>

app/Views/teacher/class_progress_submit.php

@@ -77,39 +77,25 @@
         <div class="card shadow-sm mb-3">
           <div class="card-header bg-white d-flex flex-wrap align-items-center justify-content-between gap-3">
             <strong class="mb-0">Date Selection</strong>
-            <?php if ($isEdit): ?>
+            <div class="d-flex align-items-center gap-2">
-              <div class="text-muted small">
+                <select id="weekStartSelect" name="week_start" class="form-select form-select-sm" required <?= $isEdit ? 'data-original-week="' . esc($weekStartSelected) . '"' : '' ?>>
-                <?php
+                <option value="">Select week</option>
-                  try {
+                <?php foreach ($sundayOptions as $sunday): ?>
-                    $displayStart = (new \DateTime($weekStartSelected))->format('M d, Y');
+                  <?php
-                  } catch (\Exception $e) {
+                    try {
-                    $displayStart = $weekStartSelected;
+                      $startDt = new \DateTime($sunday);
-                  }
+                      $displayStart = $startDt->format('M d, Y');
-                ?>
+                    } catch (\Exception $e) {
-                Week of <?= esc($displayStart ?: 'N/A') ?>
+                      $displayStart = $sunday;
-              </div>
+                    }
-              <input type="hidden" name="week_start" value="<?= esc($weekStartSelected) ?>">
+                  ?>
-            <?php else: ?>
+                  <option value="<?= esc($sunday) ?>" <?= $sunday === $weekStartSelected ? 'selected' : '' ?>>
-              <div class="d-flex align-items-center gap-2">
+                    <?= esc($displayStart) ?>
-                  <select id="weekStartSelect" name="week_start" class="form-select form-select-sm" required>
+                  </option>
-                  <option value="">Select week</option>
+                <?php endforeach; ?>
-                  <?php foreach ($sundayOptions as $sunday): ?>
+              </select>
-                    <?php
+              <div class="invalid-feedback">Week start is required.</div>
-                      try {
+            </div>
-                        $startDt = new \DateTime($sunday);
-                        $displayStart = $startDt->format('M d, Y');
-                      } catch (\Exception $e) {
-                        $displayStart = $sunday;
-                      }
-                    ?>
-                    <option value="<?= esc($sunday) ?>" <?= $sunday === $weekStartSelected ? 'selected' : '' ?>>
-                      <?= esc($displayStart) ?>
-                    </option>
-                  <?php endforeach; ?>
-                </select>
-                <div class="invalid-feedback">Week start is required.</div>
-              </div>
-            <?php endif; ?>
           </div>
           <div class="card-body">
             <input type="hidden" name="week_end" id="weekEndInput" value="<?= esc($weekEndValue) ?>" required>
@@ -197,6 +183,14 @@
                         </div>
                         <div class="form-text small text-muted">Add a custom Surah or Arabic target.</div>
                       </div>
+                    <?php elseif ($slug === 'islamic'): ?>
+                      <div class="px-2 py-2 border-top">
+                        <div class="input-group input-group-sm">
+                          <input type="text" class="form-control" placeholder="Type subject or topic" data-custom-input data-subject="<?= esc($slug) ?>">
+                          <button type="button" class="btn btn-outline-secondary" data-custom-entry data-subject="<?= esc($slug) ?>">Add</button>
+                        </div>
+                        <div class="form-text small text-muted">Add a subject or unit not listed above (e.g. Seerah, Fiqh, Akhlaq).</div>
+                      </div>
                     <?php endif; ?>
                   </div>
                 </div>
@@ -306,6 +300,19 @@
   const forms = document.querySelectorAll('.needs-validation');
   Array.from(forms).forEach(form => {
     form.addEventListener('submit', event => {
+      const originalWeek = weekStartSelect?.dataset.originalWeek || '';
+      if (originalWeek && weekStartSelect && weekStartSelect.value && weekStartSelect.value !== originalWeek) {
+        const ok = confirm('A progress report already exists for the original week. Change the week and override any existing report for the new date?');
+        if (!ok) {
+          event.preventDefault();
+          event.stopPropagation();
+          return;
+        }
+        const confirmInput = document.getElementById('confirmOverwriteInput');
+        if (confirmInput) {
+          confirmInput.value = '1';
+        }
+      }
       const islamicUnits = form.querySelectorAll('input[name="unit_islamic[]"]');
       const hasIslamicUnit = Array.from(islamicUnits).some(input => input.value.trim() !== '');
       if (!hasIslamicUnit) {
@@ -386,6 +393,9 @@
     if (isQuran) {
       return isCustom ? 'Custom' : 'Surah';
     }
+    if (subject === 'islamic' && isCustom) {
+      return 'Custom';
+    }
     return parts.join(' – ');
   };
 
@@ -446,7 +456,10 @@
       if (!input) return;
       const customValue = input.value.trim();
       if (customValue === '') return;
-      const unitValue = buildUnitDisplay(subject, '', '', { isQuran: subject === 'quran', isCustom: true });
+      const unitValue = buildUnitDisplay(subject, '', '', {
+        isQuran: subject === 'quran',
+        isCustom: subject === 'quran' || subject === 'islamic',
+      });
       appendUnitChapterRow(subject, unitValue, customValue);
       input.value = '';
       hideMenus();

app/Views/teacher/drafts.php

@@ -1,34 +1,307 @@
 <?= $this->extend('layout/main_layout') ?>
 <?= $this->section('content') ?>
-<div class="container-xxl py-5">
+<?php
+$statusBadges = $statusBadges ?? [];
+$drafts = $drafts ?? [];
+$legacyExams = $legacyExams ?? [];
+$assignments = $assignments ?? [];
+$selectedClassSection = $selectedClassSection ?? 0;
+$examTypes = $examTypes ?? [];
+$schoolYear = $schoolYear ?? '';
+$semester = $semester ?? '';
+$maxUploadBytes = $maxUploadBytes ?? (12 * 1024 * 1024);
+?>
+<style>
+  .teacher-drafts-page,
+  .teacher-drafts-page * {
+    font-family: Arial, sans-serif !important;
+  }
+  .teacher-drafts-table {
+    table-layout: auto;
+    width: max-content;
+  }
+  .teacher-drafts-table th {
+    min-width: 0;
+  }
+  .teacher-drafts-table td {
+    max-width: none;
+  }
+  .teacher-drafts-table th {
+    white-space: nowrap;
+  }
+  .teacher-drafts-table td.title-cell {
+    white-space: nowrap;
+  }
+</style>
+<div class="container-xxl py-5 teacher-drafts-page">
     <div class="container">
-        <h1>Drafts</h1>
+        <div class="d-flex flex-wrap justify-content-between align-items-center mb-4">
-        <?php if (!empty($draftMessages)): ?>
+            <h1 class="mb-0">Exam drafts</h1>
+            <?php if (!empty($legacyExams)): ?>
+                <button type="button" class="btn btn-sm btn-outline-secondary" id="legacyToggle">Legacy Exams</button>
+            <?php endif; ?>
+        </div>
+
+        <?php if (session()->getFlashdata('success')): ?>
+            <div class="alert alert-success"><?= esc(session()->getFlashdata('success')) ?></div>
+        <?php endif; ?>
+        <?php if (session()->getFlashdata('error')): ?>
+            <div class="alert alert-danger"><?= esc(session()->getFlashdata('error')) ?></div>
+        <?php endif; ?>
+
+        <div class="card mb-4">
+            <div class="card-body">
+                <?= form_open_multipart(base_url('teacher/exam-drafts'), ['class' => 'row g-3', 'id' => 'examDraftForm']) ?>
+                    <?= csrf_field() ?>
+                    <div class="col-md-6">
+                        <label class="form-label" for="exam_type">Exam type <span class="text-danger">*</span></label>
+                        <select name="exam_type" id="exam_type" class="form-select" required>
+                            <option value="">— Select —</option>
+                            <?php foreach ($examTypes as $t): ?>
+                                <option value="<?= esc($t) ?>" <?= old('exam_type') === $t ? 'selected' : '' ?>><?= esc($t) ?></option>
+                            <?php endforeach; ?>
+                        </select>
+                    </div>
+                    <div class="col-8">
+                        <label class="form-label" for="author_comment">Author comment</label>
+                        <textarea name="author_comment" id="author_comment" class="form-control" rows="2" placeholder="Optional note for reviewers"><?= esc(old('author_comment') ?? '') ?></textarea>
+                    </div>
+                    <div class="col-md-8">
+                        <label class="form-label" for="draft_file">File (Word) <span class="text-danger">*</span></label>
+                        <input type="file" name="draft_file" id="draft_file" class="form-control" accept=".doc,.docx" required>
+                        <div class="form-text">Max <?= esc(number_format($maxUploadBytes / 1048576, 1)) ?> MB.</div>
+                    </div>
+                    <div class="col-12 d-flex flex-wrap gap-2">
+                        <button type="submit" class="btn btn-primary">Submit for review</button>
+                    </div>
+                <?= form_close() ?>
+                <?php if ($schoolYear !== '' || $semester !== ''): ?>
+                    <p class="text-muted small mb-0 mt-2">School year: <?= esc($schoolYear) ?> · Semester: <?= esc($semester) ?></p>
+                <?php endif; ?>
+            </div>
+        </div>
+
+        <?php
+        $renderBadge = static function (string $status, array $badges): string {
+            $b = $badges[$status] ?? ['label' => $status, 'class' => 'bg-secondary text-white'];
+            $style = !empty($b['style']) ? ' style="' . esc($b['style']) . '"' : '';
+            return '<span class="badge ' . esc($b['class']) . ' js-status-badge" data-status="' . esc($status) . '"' . $style . '>' . esc($b['label']) . '</span>';
+        };
+        ?>
+
+        <h2 class="h5 mb-3">Your submissions</h2>
+        <?php if (!empty($drafts)): ?>
             <div class="table-responsive">
-                <table class="table table-striped">
+                <table class="table table-striped align-middle teacher-drafts-table">
                     <thead>
                         <tr>
-                            <th>Subject</th>
+                            <th>Class</th>
-                            <th>Date</th>
+                            <th>Title / type</th>
-                            <th>Actions</th>
+                            <th>Ver.</th>
+                            <th>Status</th>
+                            <th>File link</th>
+                            <th>Reviewer comment</th>
+                            <th>Last Update</th>
                         </tr>
                     </thead>
                     <tbody>
-                        <?php foreach ($draftMessages as $message): ?>
+                        <?php foreach ($drafts as $d): ?>
-                            <tr>
+                            <?php
-                                <td><?= esc($message['subject']) ?></td>
+                            $st = strtolower((string) ($d['status'] ?? ''));
-                                <td><?= esc(!empty($message['created_at']) ? local_datetime($message['created_at'], 'm-d-Y H:i') : '') ?></td>
+                            $badgeHtml = $renderBadge($st, $statusBadges);
+                            ?>
+                            <tr data-draft-id="<?= (int) ($d['id'] ?? 0) ?>">
+                                <td><?= esc($d['class_section_name'] ?? '') ?></td>
+                                <td class="title-cell"><?= esc($d['draft_title'] ?? $d['exam_type'] ?? '') ?></td>
+                                <td><?= esc((string) ($d['version'] ?? '')) ?></td>
+                                <td><?= $badgeHtml ?></td>
                                 <td>
-                                    <a href="<?= base_url('messages/edit/' . $message['id']) ?>">Edit</a> |
+                                    <?php $revNumber = max(1, (int) ($d['version'] ?? 1)); ?>
-                                    <a href="<?= base_url('messages/delete/' . $message['id']) ?>">Delete</a>
+                                    <?php if (!empty($d['teacher_file'])): ?>
+                                        <div>
+                                            <a href="<?= base_url('exam-drafts/files/teacher/' . $d['teacher_file']) ?>" target="_blank" rel="noopener">
+                                                <?= esc('Ver' . $revNumber . ' ' . ($d['teacher_filename'] ?? 'Submitted draft')) ?>
+                                            </a>
+                                        </div>
+                                    <?php endif; ?>
+                                    <?php if (!empty($d['final_file'])): ?>
+                                        <div class="mt-1">
+                                            <a href="<?= base_url('exam-drafts/files/final/' . $d['final_file']) ?>" target="_blank" rel="noopener" class="link-success small">
+                                                <?= esc('Final ' . ($d['final_filename'] ?? 'Final draft')) ?>
+                                            </a>
+                                        </div>
+                                    <?php endif; ?>
+                                    <?php if (!empty($d['review_files'])): ?>
+                                        <div class="mt-1">
+                                            <?php
+                                                $reviewLinks = [];
+                                                foreach ($d['review_files'] as $rf) {
+                                                    $rev = max(1, (int) ($rf['review_revision'] ?? 1));
+                                                    $name = $rf['final_filename'] ?? 'Review file';
+                                                    $file = $rf['final_file'] ?? '';
+                                                    if ($file !== '') {
+                                                        $reviewLinks[] = '<a href="' . esc(base_url('exam-drafts/files/final/' . $file)) . '" target="_blank" rel="noopener" class="link-success small">' . esc('Ver' . $revNumber . '_' . $rev . ' ' . $name) . '</a>';
+                                                    }
+                                                }
+                                            ?>
+                                            <?= !empty($reviewLinks) ? implode(' | ', $reviewLinks) : '' ?>
+                                        </div>
+                                    <?php endif; ?>
+                                    <?php if (empty($d['teacher_file']) && empty($d['final_file'])): ?>
+                                        —
+                                    <?php endif; ?>
                                 </td>
+                                <td><?= nl2br(esc($d['reviewer_comment'] ?? $d['reviewer_comments'] ?? $d['admin_comments'] ?? '—')) ?></td>
+                                <td><?= esc(!empty($d['updated_at']) ? local_datetime($d['updated_at'], 'm-d-Y H:i') : '') ?></td>
+                                
                             </tr>
                         <?php endforeach; ?>
                     </tbody>
                 </table>
             </div>
         <?php else: ?>
-            <p>No drafts found.</p>
+            <p class="text-muted">No submissions yet.</p>
+        <?php endif; ?>
+
+        <?php if (!empty($legacyExams)): ?>
+            <div id="legacySection" class="table-responsive d-none mt-4">
+                <table class="table table-sm table-striped align-middle teacher-drafts-table">
+                    <thead>
+                        <tr>
+                            <th>Class</th>
+                            <th>Title</th>
+                            <th>Ver.</th>
+                            <th>Status</th>
+                            <th>School year</th>
+                            <th>Semester</th>
+                            <th>Exam type</th>
+                            <th>Updated</th>
+                            <th>Actions</th>
+                        </tr>
+                    </thead>
+                    <tbody>
+                        <?php foreach ($legacyExams as $d): ?>
+                            <?php $st = strtolower((string) ($d['status'] ?? '')); ?>
+                            <tr>
+                                <td><?= esc($d['class_section_name'] ?? '') ?></td>
+                                <td class="title-cell"><?= esc($d['draft_title'] ?? $d['exam_type'] ?? '') ?></td>
+                                <td><?= esc((string) ($d['version'] ?? '')) ?></td>
+                                <td><?= $renderBadge($st, $statusBadges) ?></td>
+                                <td><?= esc($d['school_year'] ?? '') ?></td>
+                                <td><?= esc($d['semester'] ?? '') ?></td>
+                                <td><?= esc($d['exam_type'] ?? '') ?></td>
+                                <td><?= esc(!empty($d['updated_at']) ? local_datetime($d['updated_at'], 'm-d-Y H:i') : '') ?></td>
+                                <td>
+                                    <?php if (!empty($d['final_file'])): ?>
+                                        <a class="btn btn-sm btn-outline-primary" href="<?= base_url('exam-drafts/files/final/' . $d['final_file']) ?>" target="_blank" rel="noopener">View</a>
+                                    <?php endif; ?>
+                                </td>
+                            </tr>
+                        <?php endforeach; ?>
+                    </tbody>
+                </table>
+            </div>
         <?php endif; ?>
     </div>
+</div>
+<?= $this->endSection() ?>
+
+<?= $this->section('scripts') ?>
+<script>
+document.addEventListener('DOMContentLoaded', () => {
+    const csrfTokenName = <?= json_encode(csrf_token(), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+    const csrfCookieNames = <?= json_encode(array_values(array_filter([
+        config('Security')->csrfCookieName ?? null,
+        config('Security')->cookieName ?? null,
+        'csrf_cookie_name',
+    ])), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+
+    const readCookie = (name) => {
+        const match = document.cookie.match(new RegExp('(?:^|; )' + name.replace(/[$()*+.?[\\\]^{|}-]/g, '\\$&') + '=([^;]*)'));
+        return match ? decodeURIComponent(match[1]) : '';
+    };
+
+    const syncCsrfToken = (form) => {
+        let tokenValue = '';
+        csrfCookieNames.some((name) => {
+            tokenValue = readCookie(name);
+            return tokenValue !== '';
+        });
+        if (!tokenValue || !form) {
+            return;
+        }
+        const tokenInput = form.querySelector(`input[name="${csrfTokenName}"]`);
+        if (tokenInput) {
+            tokenInput.value = tokenValue;
+        }
+    };
+
+    const draftForm = document.getElementById('examDraftForm');
+    if (draftForm) {
+        draftForm.addEventListener('submit', () => syncCsrfToken(draftForm));
+    }
+
+    const statusBadges = <?= json_encode($statusBadges, JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+    const rows = new Map();
+    document.querySelectorAll('tr[data-draft-id]').forEach((row) => {
+        const id = row.getAttribute('data-draft-id');
+        if (id) {
+            rows.set(id, row);
+        }
+    });
+
+    const updateRow = (row, status, acceptanceType) => {
+        const badge = row.querySelector('.js-status-badge');
+        if (badge) {
+            const badgeData = statusBadges[status] || { label: status, class: 'bg-secondary text-white' };
+            badge.textContent = badgeData.label || status;
+            badge.className = `badge ${badgeData.class} js-status-badge`;
+            if (badgeData.style) {
+                badge.setAttribute('style', badgeData.style);
+            } else {
+                badge.removeAttribute('style');
+            }
+            badge.dataset.status = status;
+        }
+        const note = row.querySelector('.js-acceptance-note');
+        if (note) {
+            if (status === 'accepted' && acceptanceType) {
+                note.textContent = acceptanceType === 'minor_edits' ? 'With minor edits' : 'As is';
+            } else {
+                note.textContent = '—';
+            }
+        }
+    };
+
+    const poll = () => {
+        fetch('<?= base_url('teacher/exam-drafts/status') ?>', { headers: { 'X-Requested-With': 'XMLHttpRequest' } })
+            .then((resp) => resp.ok ? resp.json() : null)
+            .then((data) => {
+                if (!data || !Array.isArray(data.drafts)) {
+                    return;
+                }
+                data.drafts.forEach((draft) => {
+                    const row = rows.get(String(draft.id));
+                    if (row) {
+                        updateRow(row, draft.status, draft.acceptance_type);
+                    }
+                });
+            })
+            .catch(() => {});
+    };
+
+    if (rows.size > 0) {
+        poll();
+        setInterval(poll, 15000);
+    }
+
+    const legacyToggle = document.getElementById('legacyToggle');
+    const legacySection = document.getElementById('legacySection');
+    if (legacyToggle && legacySection) {
+        legacyToggle.addEventListener('click', () => {
+            legacySection.classList.toggle('d-none');
+        });
+    }
+});
+</script>
 <?= $this->endSection() ?>

app/Views/teacher/scores.php

@@ -308,6 +308,7 @@
                 id="submitScoresLockBtn"
                 formaction="<?= base_url('/teacher/submit-scores-lock') ?>"
                 formmethod="post"
+                data-confirm-message="Once you submit, you cannot add any scores or comments. Do you want to continue?"
                 <?= $scoresLocked ? 'disabled' : '' ?>>
                 <?= $scoresLocked ? 'Scores Locked' : 'Submit Semester Scores' ?>
             </button>
@@ -430,6 +431,14 @@
 
             form.addEventListener('submit', function(event) {
                 syncCsrfToken();
+                const submitter = event.submitter;
+                if (submitter && submitter.id === 'submitScoresLockBtn') {
+                    const message = submitter.dataset.confirmMessage || 'Submit semester scores?';
+                    if (!confirm(message)) {
+                        event.preventDefault();
+                        return;
+                    }
+                }
                 const errors = [];
                 form.querySelectorAll('textarea[data-first-name]').forEach(function(field) {
                     const value = field.value.trim();