fix attendance coemment in report card

app/Config/Routes.php

@@ -528,10 +528,17 @@ $routes->post('grading/decisions/generate', 'View\GradingController::generateAll
 $routes->get('grading/below-60/decisions', 'View\GradingController::belowSixtyDecisions', ['filter' => 'auth:read']);
 $routes->post('grading/below-60/decisions/save', 'View\GradingController::saveBelowSixtyDecision', ['filter' => 'auth:read']);
 $routes->get('grading/below-60/decisions/student-details', 'View\GradingController::studentDecisionDetails', ['filter' => 'auth:read']);
-$routes->get('grading/below-60/decisions/email/preview', 'View\GradingController::previewDecisionEmail', ['filter' => 'auth:read']);
+$routes->get(
-$routes->get('grading/below-60/decisions/email/edit', 'View\GradingController::editDecisionEmail', ['filter' => 'auth:read']);
+    'grading/below-60/decisions/email/preview',
-$routes->post('grading/below-60/decisions/email', 'View\GradingController::sendDecisionEmail', ['filter' => 'auth:read']);
+    'View\GradingController::previewBelowSixtyDecisionEmail',
+    ['filter' => 'auth:read']
+);
 
+$routes->post(
+    'grading/below-60/decisions/email',
+    'View\GradingController::sendBelowSixtyDecisionEmail',
+    ['filter' => 'auth:read']
+);
 
 // Final part
 $routes->get('grading/(:segment)/(:num)/(:num)', 'View\GradingController::show/$1/$2/$3');

app/Controllers/View/CertificateController.php

@@ -6,7 +6,6 @@ use App\Controllers\BaseController;
 use App\Models\ClassSectionModel;
 use App\Models\ConfigurationModel;
 use App\Models\CertificateRecordModel;
-use App\Models\StudentDecisionModel;
 
 class CertificateController extends BaseController
 {
@@ -27,9 +26,9 @@ class CertificateController extends BaseController
 
     public function index()
     {
-        $db             = \Config\Database::connect();
+        $db           = \Config\Database::connect();
-        $selectedCsid   = $this->request->getGet('class_section_id');
+        $selectedCsid = $this->request->getGet('class_section_id');
-        $schoolYear     = $this->request->getGet('school_year') ?? $this->schoolYear;
+        $schoolYear   = $this->request->getGet('school_year') ?? $this->schoolYear;
 
         // ── All enrolled students across every class section ───────────────────
         $allEnrolled = $db->table('student_class sc')
@@ -40,96 +39,126 @@ class CertificateController extends BaseController
             ->where('sc.school_year', $schoolYear)
             ->orderBy('s.firstname', 'ASC')
             ->orderBy('s.lastname', 'ASC')
-            ->get()->getResultArray();
+            ->get()
+            ->getResultArray();
 
-        $allIds = array_unique(array_column($allEnrolled, 'student_id'));
+        $allIds = array_values(array_unique(array_map('intval', array_column($allEnrolled, 'student_id'))));
+
+        // ── Saved generated YEAR decisions from student_decisions ──────────────
+        //
+        // Source of truth:
+        // student_decisions.year_score
+        // student_decisions.decision
+        //
+        // Do NOT recalculate certificate decisions here from semester_scores.
+        $decisionsByStudent = [];
 
-        // ── Semester scores ────────────────────────────────────────────────────
-        $allScoreMap = [];
         if (!empty($allIds)) {
-            foreach ($db->table('semester_scores')
+            $decisionRows = $db->table('student_decisions')
-                ->select('student_id, semester, semester_score')
                 ->whereIn('student_id', $allIds)
                 ->where('school_year', $schoolYear)
-                ->where('semester_score IS NOT NULL', null, false)
+                ->get()
-                ->get()->getResultArray() as $sr) {
+                ->getResultArray();
-                $allScoreMap[(int)$sr['student_id']][ucfirst(strtolower($sr['semester']))] =
+
-                    is_numeric($sr['semester_score']) ? (float)$sr['semester_score'] : null;
+            foreach ($decisionRows as $d) {
+                $sid = (int)($d['student_id'] ?? 0);
+
+                if ($sid <= 0) {
+                    continue;
+                }
+
+                $decision = trim((string)($d['decision'] ?? ''));
+                $source   = trim((string)($d['source'] ?? ''));
+
+                if ($source === '') {
+                    $source = $decision === '' ? 'pending' : 'manual';
+                }
+
+                $decisionsByStudent[$sid]['Year'] = [
+                    'decision'   => $decision,
+                    'source'     => $source,
+                    'notes'      => (string)($d['notes'] ?? ''),
+                    'year_score' => $d['year_score'] ?? null,
+                ];
             }
         }
 
-        // ── Below-60 manual decisions ──────────────────────────────────────────
+        // ── Certificate records: most recent per student ───────────────────────
-        $allBelowMap = [];
-        if (!empty($allIds)) {
-            foreach ($db->table('below_sixty_decisions')
-                ->whereIn('student_id', $allIds)
-                ->where('school_year', $schoolYear)
-                ->get()->getResultArray() as $b) {
-                $allBelowMap[(int)$b['student_id']][ucfirst(strtolower($b['semester']))] = (string)$b['decision'];
-            }
-        }
-
-        // ── Certificate records (most recent per student) ──────────────────────
         $certsByStudent = [];
+
         if (!empty($allIds)) {
             foreach ($db->table('certificate_records')
                 ->select('student_id, certificate_number, issued_at')
                 ->whereIn('student_id', $allIds)
                 ->where('school_year', $schoolYear)
                 ->orderBy('issued_at', 'DESC')
-                ->get()->getResultArray() as $c) {
+                ->get()
+                ->getResultArray() as $c) {
                 $sid = (int)$c['student_id'];
+
                 if (!isset($certsByStudent[$sid])) {
                     $certsByStudent[$sid] = $c['certificate_number'];
                 }
             }
         }
 
-        // ── Build per-student decisions + per-class buckets ────────────────────
+        // ── Build per-class buckets and stats ──────────────────────────────────
-        $decisionsByStudent = [];
+        $studentsByClass = [];
-        $studentsByClass    = [];   // [csid => [student rows...]]
+        $statsPerClass   = [];
-        $statsPerClass      = [];   // [csid => {name, total, pass, cert}]
 
         foreach ($allEnrolled as $row) {
             $sid  = (int)$row['student_id'];
             $csid = (int)$row['class_section_id'];
 
-            // Decisions per semester
+            if (!isset($statsPerClass[$csid])) {
-            foreach ($allScoreMap[$sid] ?? [] as $sem => $score) {
+                $statsPerClass[$csid] = [
-                if ($score === null) continue;
+                    'name'  => $row['class_section_name'],
-                if ($score >= 60) {
+                    'total' => 0,
-                    $dec = 'Pass'; $src = 'auto';
+                    'pass'  => 0,
-                } elseif (!empty($allBelowMap[$sid][$sem])) {
+                    'cert'  => 0,
-                    $dec = $allBelowMap[$sid][$sem]; $src = 'manual';
+                ];
-                } else {
-                    $dec = ''; $src = 'pending';
-                }
-                $decisionsByStudent[$sid][$sem] = ['decision' => $dec, 'source' => $src];
             }
 
-            // Per-class stats
-            if (!isset($statsPerClass[$csid])) {
-                $statsPerClass[$csid] = ['name' => $row['class_section_name'], 'total' => 0, 'pass' => 0, 'cert' => 0];
-            }
             $statsPerClass[$csid]['total']++;
 
-            $sems   = $allScoreMap[$sid] ?? [];
+            // If no generated decision exists yet, explicitly mark pending.
-            $isPass = !empty($sems);
+            if (!isset($decisionsByStudent[$sid])) {
-            foreach ($sems as $sem => $score) {
+                $decisionsByStudent[$sid]['Decision'] = [
-                if ($score === null) { $isPass = false; break; }
+                    'decision'   => '',
-                if ($score >= 60) continue;
+                    'source'     => 'pending',
-                $md = $allBelowMap[$sid][$sem] ?? '';
+                    'notes'      => '',
-                if ($md === '' || $md !== 'Pass') { $isPass = false; break; }
+                    'year_score' => null,
+                ];
+            }
+
+            // Certificate eligibility:
+            // A student is eligible only if the saved final generated decision is Pass.
+            $studentDecisions = $decisionsByStudent[$sid] ?? [];
+            $isPass           = !empty($studentDecisions);
+
+            foreach ($studentDecisions as $decisionRow) {
+                $decision = trim((string)($decisionRow['decision'] ?? ''));
+
+                if (strcasecmp($decision, 'Pass') !== 0) {
+                    $isPass = false;
+                    break;
+                }
+            }
+
+            if ($isPass) {
+                $statsPerClass[$csid]['pass']++;
+            }
+
+            if (isset($certsByStudent[$sid])) {
+                $statsPerClass[$csid]['cert']++;
             }
-            if ($isPass)                    $statsPerClass[$csid]['pass']++;
-            if (isset($certsByStudent[$sid])) $statsPerClass[$csid]['cert']++;
 
-            // Group students by class
             $studentsByClass[$csid][] = $row;
         }
 
         // ── Determine default active tab ───────────────────────────────────────
         $firstCsid = !empty($allEnrolled) ? (int)$allEnrolled[0]['class_section_id'] : null;
+
         if ($selectedCsid === null && $firstCsid !== null) {
             $selectedCsid = (string)$firstCsid;
         }
@@ -181,9 +210,12 @@ class CertificateController extends BaseController
                 ->where('cr.verification_token', $certNumber)
                 ->orWhere('cr.certificate_number', strtoupper($certNumber))
             ->groupEnd()
-            ->get()->getRowArray();
+            ->get()
+            ->getRowArray();
 
-        return view('certificates/verify', ['record' => $record ?: null]);
+        return view('certificates/verify', [
+            'record' => $record ?: null,
+        ]);
     }
 
     // ─── Reprint an existing certificate ──────────────────────────────────────
@@ -193,7 +225,8 @@ class CertificateController extends BaseController
         $record = \Config\Database::connect()
             ->table('certificate_records')
             ->where('certificate_number', strtoupper($certNumber))
-            ->get()->getRowArray();
+            ->get()
+            ->getRowArray();
 
         if (!$record) {
             return redirect()->to('administrator/certificates/log')
@@ -201,23 +234,26 @@ class CertificateController extends BaseController
         }
 
         $certDateFormatted = '';
+
         if (!empty($record['cert_date'])) {
             $ts = strtotime((string)$record['cert_date']);
+
             if ($ts) {
                 $certDateFormatted = date('m/d/Y', $ts);
             }
         }
 
         $student = [
-            'firstname'   => (string)($record['student_name'] ?? ''),
+            'firstname'    => (string)($record['student_name'] ?? ''),
-            'lastname'    => '',
+            'lastname'     => '',
-            'grade'       => (string)($record['grade'] ?? ''),
+            'grade'        => (string)($record['grade'] ?? ''),
-            'cert_number' => (string)($record['certificate_number'] ?? ''),
+            'cert_number'  => (string)($record['certificate_number'] ?? ''),
             'verify_token' => $this->ensureVerificationTokenForRecord($record),
         ];
 
-        // student_name is stored as "Firstname Lastname" — split for the PDF builder
+        // student_name is stored as "Firstname Lastname" — split for the PDF builder.
         $parts = explode(' ', trim($student['firstname']), 2);
+
         if (count($parts) === 2) {
             $student['firstname'] = $parts[0];
             $student['lastname']  = $parts[1];
@@ -246,13 +282,15 @@ class CertificateController extends BaseController
                 return $this->response
                     ->setStatusCode(422)
                     ->setJSON([
-                        'ok' => false,
+                        'ok'         => false,
-                        'error' => 'Please select at least one student.',
+                        'error'      => 'Please select at least one student.',
                         'csrf_token' => csrf_token(),
-                        'csrf_hash' => csrf_hash(),
+                        'csrf_hash'  => csrf_hash(),
                     ]);
             }
-            return redirect()->to('administrator/certificates')->with('error', 'Please select at least one student.');
+
+            return redirect()->to('administrator/certificates')
+                ->with('error', 'Please select at least one student.');
         }
 
         $studentIds = array_filter(array_map('intval', $studentIds));
@@ -263,13 +301,15 @@ class CertificateController extends BaseController
                 return $this->response
                     ->setStatusCode(422)
                     ->setJSON([
-                        'ok' => false,
+                        'ok'         => false,
-                        'error' => 'Invalid student selection.',
+                        'error'      => 'Invalid student selection.',
                         'csrf_token' => csrf_token(),
-                        'csrf_hash' => csrf_hash(),
+                        'csrf_hash'  => csrf_hash(),
                     ]);
             }
-            return redirect()->to('administrator/certificates')->with('error', 'Invalid student selection.');
+
+            return redirect()->to('administrator/certificates')
+                ->with('error', 'Invalid student selection.');
         }
 
         $db       = \Config\Database::connect();
@@ -277,21 +317,26 @@ class CertificateController extends BaseController
 
         foreach ($studentIds as $id) {
             $row = null;
+
             if ($classSectionId) {
                 $row = $db->table('student_class sc')
                     ->select('s.id, s.firstname, s.lastname, cs.class_section_name AS grade')
                     ->join('students s', 's.id = sc.student_id')
                     ->join('classSection cs', 'cs.class_section_id = sc.class_section_id')
-                    ->where('sc.class_section_id', (int) $classSectionId)
+                    ->where('sc.class_section_id', (int)$classSectionId)
+                    ->where('sc.school_year', $schoolYear)
                     ->where('s.id', $id)
-                    ->get()->getRowArray();
+                    ->get()
+                    ->getRowArray();
             }
 
             if (!$row) {
                 $s = $db->table('students')
                     ->select('id, firstname, lastname, registration_grade AS grade')
                     ->where('id', $id)
-                    ->get()->getRowArray();
+                    ->get()
+                    ->getRowArray();
+
                 $row = $s ?: null;
             }
 
@@ -305,57 +350,62 @@ class CertificateController extends BaseController
                 return $this->response
                     ->setStatusCode(422)
                     ->setJSON([
-                        'ok' => false,
+                        'ok'         => false,
-                        'error' => 'No valid students found.',
+                        'error'      => 'No valid students found.',
                         'csrf_token' => csrf_token(),
-                        'csrf_hash' => csrf_hash(),
+                        'csrf_hash'  => csrf_hash(),
                     ]);
             }
-            return redirect()->to('administrator/certificates')->with('error', 'No valid students found.');
+
+            return redirect()->to('administrator/certificates')
+                ->with('error', 'No valid students found.');
         }
 
-        $issuedBy   = session()->get('user_id');
+        $issuedAt = date('Y-m-d H:i:s');
-        $issuedAt   = date('Y-m-d H:i:s');
-        $certDateDb = $this->parseCertDate($certDate);
 
-        // Load any existing certificates for these students this school year
+        // Load existing certificates for these students this school year.
-        $db         = \Config\Database::connect();
         $existingCerts = $db->table('certificate_records')
             ->select('id, student_id, certificate_number, verification_token')
             ->whereIn('student_id', array_column($students, 'id'))
             ->where('school_year', $schoolYear)
-            ->get()->getResultArray();
+            ->get()
+            ->getResultArray();
+
         $existingCertMap = [];
+
         foreach ($existingCerts as $ec) {
             $existingCertMap[(int)$ec['student_id']] = $ec;
         }
 
         foreach ($students as &$student) {
             $sid = (int)$student['id'];
+
             if (isset($existingCertMap[$sid])) {
-                // Reuse existing certificate number — do not create a new record
+                // Reuse existing certificate number — do not create a new record.
                 $existing = $existingCertMap[$sid];
-                $student['cert_number'] = (string)($existing['certificate_number'] ?? '');
+
+                $student['cert_number']  = (string)($existing['certificate_number'] ?? '');
                 $student['verify_token'] = $this->ensureVerificationTokenForRecord($existing);
             } else {
-                $certNumber = $this->certRecordModel->nextNumber($schoolYear);
+                $certNumber  = $this->certRecordModel->nextNumber($schoolYear);
                 $verifyToken = $this->certRecordModel->generateVerificationToken();
+
                 $this->certRecordModel->insert([
                     'certificate_number' => $certNumber,
                     'verification_token' => $verifyToken,
                     'student_id'         => $sid,
                     'student_name'       => $student['firstname'] . ' ' . $student['lastname'],
                     'grade'              => $this->formatGrade($student['grade'] ?? ''),
-                    //'cert_date'          => $certDateDb,
                     'school_year'        => $schoolYear,
                     'class_section_id'   => $classSectionId ?: null,
-                    //'issued_by'          => $issuedBy,
                     'issued_at'          => $issuedAt,
                 ]);
-                $student['cert_number'] = $certNumber;
+
+                $student['cert_number']  = $certNumber;
                 $student['verify_token'] = $verifyToken;
             }
         }
+
         unset($student);
 
         $pdfData  = $this->buildPdf($students, $certDate);
@@ -376,9 +426,11 @@ class CertificateController extends BaseController
         if (preg_match('#^(\d{2})/(\d{2})/(\d{4})$#', $certDate, $m)) {
             return $m[3] . '-' . $m[1] . '-' . $m[2];
         }
+
         if (preg_match('#^\d{4}-\d{2}-\d{2}$#', $certDate)) {
             return $certDate;
         }
+
         return null;
     }
 
@@ -387,17 +439,20 @@ class CertificateController extends BaseController
         $clean = trim($raw);
         $lower = strtolower($clean);
 
-        // Strip section suffix: "Grade 1-A" → "Grade 1", "Grade 2-B" → "Grade 2"
+        // Strip section suffix: "Grade 1-A" → "Grade 1", "Grade 2-B" → "Grade 2".
         if (preg_match('/^grade\s*(\d+)/i', $clean, $m)) {
             return 'Grade ' . (int)$m[1];
         }
+
         if ($lower === 'youth') {
             return 'Youth';
         }
+
         if ($lower === 'kg' || $lower === 'kindergarten') {
             return 'Kindergarten';
         }
-        // Raw number or number-section (e.g. "1", "1-A", "2-B") → keep number only
+
+        // Raw number or number-section: "1", "1-A", "2-B" → keep number only.
         if (preg_match('/^(\d+)([- ][A-Za-z0-9]+)?$/', $clean, $m)) {
             return 'Grade ' . (int)$m[1];
         }
@@ -408,17 +463,22 @@ class CertificateController extends BaseController
     private function ensureVerificationTokenForRecord(array $record): string
     {
         $token = trim((string)($record['verification_token'] ?? ''));
+
         if ($token !== '') {
             return $token;
         }
 
         $recordId = (int)($record['id'] ?? 0);
+
         if ($recordId <= 0) {
             return '';
         }
 
         $token = $this->certRecordModel->generateVerificationToken();
-        $this->certRecordModel->update($recordId, ['verification_token' => $token]);
+
+        $this->certRecordModel->update($recordId, [
+            'verification_token' => $token,
+        ]);
 
         return $token;
     }
@@ -431,10 +491,11 @@ class CertificateController extends BaseController
         $imgDir  = FCPATH . 'assets' . DIRECTORY_SEPARATOR . 'certificates' . DIRECTORY_SEPARATOR . 'images' . DIRECTORY_SEPARATOR;
 
         $edwardianFont = \TCPDF_FONTS::addTTFfont($fontDir . 'Edwardian Script ITC Regular.ttf', 'TrueTypeUnicode', '', 32);
-        $garamondBold  = \TCPDF_FONTS::addTTFfont($fontDir . 'Garamond Bold.ttf',                'TrueTypeUnicode', '', 32);
+        $garamondBold  = \TCPDF_FONTS::addTTFfont($fontDir . 'Garamond Bold.ttf', 'TrueTypeUnicode', '', 32);
-        $ebGaramond    = \TCPDF_FONTS::addTTFfont($fontDir . 'EBGaramond-Regular.ttf',           'TrueTypeUnicode', '', 32);
+        $ebGaramond    = \TCPDF_FONTS::addTTFfont($fontDir . 'EBGaramond-Regular.ttf', 'TrueTypeUnicode', '', 32);
 
         $pdf = new \TCPDF('L', 'pt', 'A4', true, 'UTF-8', false);
+
         $pdf->SetCreator('Al Rahma Sunday School');
         $pdf->SetTitle('Student Certificates');
         $pdf->SetMargins(0, 0, 0, true);
@@ -450,9 +511,9 @@ class CertificateController extends BaseController
         foreach ($students as $student) {
             $pdf->AddPage();
 
-            $name       = $student['firstname'] . ' ' . $student['lastname'];
+            $name        = trim((string)($student['firstname'] ?? '') . ' ' . (string)($student['lastname'] ?? ''));
-            $grade      = $this->formatGrade($student['grade'] ?? '');
+            $grade       = $this->formatGrade($student['grade'] ?? '');
-            $certNumber = $student['cert_number'] ?? '';
+            $certNumber  = $student['cert_number'] ?? '';
             $verifyToken = $student['verify_token'] ?? '';
 
             $verifyUrl = $verifyToken !== ''
@@ -460,10 +521,18 @@ class CertificateController extends BaseController
                 : null;
 
             $this->drawCertificate(
-                $pdf, $W, $H,
+                $pdf,
-                $name, $grade, $certDate, $certNumber,
+                $W,
+                $H,
+                $name,
+                $grade,
+                $certDate,
+                $certNumber,
                 $verifyUrl,
-                $imgDir, $edwardianFont, $garamondBold, $ebGaramond
+                $imgDir,
+                $edwardianFont,
+                $garamondBold,
+                $ebGaramond
             );
         }
 
@@ -476,8 +545,8 @@ class CertificateController extends BaseController
      */
     private function drawCertificate(
         \TCPDF $pdf,
-        float  $W,
+        float $W,
-        float  $H,
+        float $H,
         string $name,
         string $grade,
         string $certDate,
@@ -489,9 +558,9 @@ class CertificateController extends BaseController
         string $ebGaramond
     ): void {
         // ── Images
-        $pdf->Image($imgDir . 'title.png',      126, 0,   600);
+        $pdf->Image($imgDir . 'title.png', 126, 0, 600);
         $pdf->Image($imgDir . 'background.png', 280, 176, 291, 340);
-        $pdf->Image($imgDir . 'signature.png',  140, 410,  90,  80);
+        $pdf->Image($imgDir . 'signature.png', 140, 410, 90, 80);
 
         // ── "Presented to:"
         $pdf->SetFont('times', 'B', 24);
@@ -499,25 +568,28 @@ class CertificateController extends BaseController
         $pdf->SetXY(0, 151);
         $pdf->Cell($W, 24, 'Presented to:', 0, 0, 'C');
 
-        // ── QR code — left-aligned with "Presented to:", vertically centred on that line
+        // ── QR code — left-aligned with "Presented to:", vertically centred on that line.
-        $qrSize = 42; // pt
+        $qrSize = 42;
+
         if (!empty($verifyUrl)) {
-            $qrX = 120;                       // ~1 cm from left edge
+            $qrX = 120;
-            $qrY = 171 + (24 - $qrSize) / 2; // vertically centred on "Presented to:" row
+            $qrY = 171 + (24 - $qrSize) / 2;
+
             $style = [
-                'border' => false,
+                'border'  => false,
                 'padding' => 0,
                 'fgcolor' => [0, 0, 0],
                 'bgcolor' => false,
             ];
+
             $pdf->write2DBarcode($verifyUrl, 'QRCODE,L', $qrX, $qrY, $qrSize, $qrSize, $style, 'N');
         }
 
-        // ── Student name — center based on actual string width
+        // ── Student name — center based on actual string width.
         $pdf->SetFont($edwardianFont, '', 38);
         $nameX = ($W - $pdf->GetStringWidth($name)) / 2;
         $this->drawGradientText($pdf, $edwardianFont, 38, $name, $nameX, 221.5);
-  
+
         // ── Line under name
         $pdf->SetFont('times', '', 20);
         $pdf->SetXY(0, 236);
@@ -543,7 +615,7 @@ class CertificateController extends BaseController
         $pdf->SetXY(0, 375);
         $pdf->Cell($W, 20, 'Al Rahma Sunday School', 0, 0, 'C');
 
-        // ── Date (gradient script)
+        // ── Date
         $this->drawGradientText($pdf, $edwardianFont, 26, $certDate, 586, 456);
 
         // ── Date underline + label
@@ -559,7 +631,7 @@ class CertificateController extends BaseController
         $pdf->SetXY(106, 492);
         $pdf->Cell(168, 20, 'Signature', 0, 0, 'C');
 
-        // ── Certificate number — 1.4 cm from bottom, 2.5 cm from left
+        // ── Certificate number — 1.4 cm from bottom, 2.5 cm from left.
         if ($certNumber !== '') {
             $pdf->SetFont('helvetica', '', 8);
             $pdf->SetTextColor(150, 150, 150);
@@ -569,8 +641,14 @@ class CertificateController extends BaseController
         }
     }
 
-    private function drawGradientText(\TCPDF $pdf, string $fontName, float $fontSize, string $text, float $x, float $y): void
+    private function drawGradientText(
-    {
+        \TCPDF $pdf,
+        string $fontName,
+        float $fontSize,
+        string $text,
+        float $x,
+        float $y
+    ): void {
         $pdf->SetFont($fontName, '', $fontSize);
 
         for ($i = 0; $i < 6; $i++) {
@@ -583,4 +661,4 @@ class CertificateController extends BaseController
         $pdf->SetTextColor(0, 0, 0);
         $pdf->Text($x, $y, $text);
     }
-}
+}

app/Controllers/View/ReportCardsController.php

@@ -317,18 +317,27 @@ class ReportCardsController extends PrintablesBaseController
         $examCommentTypes = $isSecond
             ? ['final']
             : ($isFirst ? ['midterm'] : ['midterm', 'final']);
-        $commentTypes = array_merge($examCommentTypes, ['ptap', 'attendance', 'attendance_comment']);
+        $wantedCommentTypes = array_merge($examCommentTypes, ['ptap', 'attendance']);
         $hasCommentReview = $this->db->fieldExists('comment_review', 'score_comments');
-        $commentSelect = $hasCommentReview
+        $hasCommentSemester = $this->db->fieldExists('semester', 'score_comments');
-            ? 'student_id, score_type, comment, comment_review'
+        $hasCommentUpdatedAt = $this->db->fieldExists('updated_at', 'score_comments');
-            : 'student_id, score_type, comment';
+        $commentSelectParts = ['student_id', 'score_type', 'comment'];
+        if ($hasCommentReview) {
+            $commentSelectParts[] = 'comment_review';
+        }
+        if ($hasCommentSemester) {
+            $commentSelectParts[] = 'semester';
+        }
+
+        // Do NOT filter comments by score_type or semester here.
+        // A single bad row like "Attendance Comment", "attendance-comment", or a blank semester
+        // should not make one student look incomplete while the PDF can still print the comment.
         $commentBuilder = $this->scoreCommentModel
-            ->select($commentSelect)
+            ->select(implode(', ', $commentSelectParts))
             ->whereIn('student_id', $studentIds)
-            ->where('school_year', $year)
+            ->where('school_year', $year);
-            ->whereIn('score_type', $commentTypes);
+        if ($hasCommentUpdatedAt) {
-        if ($sem !== '') {
+            $commentBuilder->orderBy('updated_at', 'DESC');
-            $this->applySemesterFilter($commentBuilder, $sem, 'semester');
         }
         $commentRows = [];
         try {
@@ -336,25 +345,92 @@ class ReportCardsController extends PrintablesBaseController
         } catch (\Throwable $e) {
         }
 
+        $cleanComment = static function ($value): string {
+            $text = (string)($value ?? '');
+            $text = str_replace("\xc2\xa0", ' ', $text); // normalize non-breaking spaces
+            return trim($text);
+        };
+
+        $normalizeCommentType = static function ($value) use ($cleanComment): string {
+            $type = strtolower($cleanComment($value));
+            $type = preg_replace('/[^a-z0-9]+/', '_', $type);
+            $type = trim((string)$type, '_');
+
+            $map = [
+                'attendance_comment' => 'attendance',
+                'attendance_comments' => 'attendance',
+                'attendance' => 'attendance',
+                'attendence' => 'attendance',
+                'attendence_comment' => 'attendance',
+                'attendence_comments' => 'attendance',
+                'ptap_comment' => 'ptap',
+                'ptap_comments' => 'ptap',
+                'ptap' => 'ptap',
+                'midterm_comment' => 'midterm',
+                'midterm_comments' => 'midterm',
+                'midterm' => 'midterm',
+                'final_comment' => 'final',
+                'final_comments' => 'final',
+                'final' => 'final',
+            ];
+
+            return $map[$type] ?? $type;
+        };
+
+        $normalizeSemester = static function ($value) use ($cleanComment): string {
+            $v = strtolower($cleanComment($value));
+            $v = preg_replace('/[^a-z0-9]+/', ' ', $v);
+            $v = trim((string)$v);
+            if (in_array($v, ['fall', 'first', 'first semester', 'semester 1', '1'], true)) {
+                return 'fall';
+            }
+            if (in_array($v, ['spring', 'second', 'second semester', 'semester 2', '2'], true)) {
+                return 'spring';
+            }
+            return $v;
+        };
+        $wantedSemester = $normalizeSemester($sem);
+
         $commentsByStudent = [];
+        $commentPriorityByStudent = [];
         foreach ($commentRows as $row) {
             $sid = (int)($row['student_id'] ?? 0);
             if ($sid <= 0) {
                 continue;
             }
-            $typeRaw = strtolower(trim((string)($row['score_type'] ?? '')));
+
-            if ($typeRaw === '') {
+            $typeRaw = $normalizeCommentType($row['score_type'] ?? '');
+            if ($typeRaw === '' || !in_array($typeRaw, $wantedCommentTypes, true)) {
                 continue;
             }
-            if ($typeRaw === 'attendance_comment') {
+
-                $typeRaw = 'attendance';
+            $rawVal = $cleanComment($row['comment'] ?? '');
+            if ($typeRaw === 'attendance') {
+                // Attendance comments are stored in score_comments.comment.
+                // Do not use comment_review here, because it can be blank while the PDF comment exists.
+                $commentVal = $rawVal;
+            } else {
+                // For non-attendance comments, prefer reviewed text but fall back to the saved comment.
+                $reviewVal = $hasCommentReview ? $cleanComment($row['comment_review'] ?? '') : '';
+                $commentVal = $reviewVal !== '' ? $reviewVal : $rawVal;
             }
-            $reviewVal = $hasCommentReview ? trim((string)($row['comment_review'] ?? '')) : '';
-            $commentVal = $hasCommentReview ? $reviewVal : trim((string)($row['comment'] ?? ''));
             if ($commentVal === '') {
                 continue;
             }
-            $commentsByStudent[$sid][$typeRaw] = $commentVal;
+
+            $rowSemester = $hasCommentSemester ? $normalizeSemester($row['semester'] ?? '') : '';
+            $priority = 1;
+            if ($wantedSemester !== '' && $rowSemester === $wantedSemester) {
+                $priority = 3;
+            } elseif ($rowSemester === '') {
+                $priority = 2;
+            }
+
+            $existingPriority = $commentPriorityByStudent[$sid][$typeRaw] ?? 0;
+            if (!isset($commentsByStudent[$sid][$typeRaw]) || $priority >= $existingPriority) {
+                $commentsByStudent[$sid][$typeRaw] = $commentVal;
+                $commentPriorityByStudent[$sid][$typeRaw] = $priority;
+            }
         }
 
         $isNumeric = static fn($v) => $v !== null && $v !== '' && is_numeric($v);
@@ -467,6 +543,20 @@ class ReportCardsController extends PrintablesBaseController
             if (trim((string)($commentSet['ptap'] ?? '')) === '') {
                 $missing[] = 'PTAP comment';
             }
+
+            // Keep completeness consistent with the PDF report generation:
+            // the report can auto-generate an attendance comment from attendance_score.
+            if (trim((string)($commentSet['attendance'] ?? '')) === '' && $isNumeric($attendanceScore)) {
+                $autoAttendance = attendance_comment_from_score(
+                    (float)$attendanceScore,
+                    trim((string)($student['firstname'] ?? ''))
+                );
+                if ($autoAttendance !== null && trim((string)$autoAttendance) !== '') {
+                    $commentSet['attendance'] = $autoAttendance;
+                    $warnings[] = 'Attendance comment computed from attendance score';
+                }
+            }
+
             if (trim((string)($commentSet['attendance'] ?? '')) === '') {
                 $missing[] = 'Attendance comment';
             }
@@ -1014,9 +1104,9 @@ $drawRankCell = static function (
     $pdf->Rect($x, $y, $w, $h);
     $pdf->SetXY($x + $pad, $y + 3);
     $pdf->SetFont('Helvetica', 'B', 11);
-    $pdf->Write(5, ' Ranking: ');
+    $pdf->Write(5, ' Class Rank: ');
 
-    $labelWidth = $pdf->GetStringWidth(' Ranking: ');
+    $labelWidth = $pdf->GetStringWidth(' Class Rank: ');
     $pdf->SetFont('Helvetica', '', 12);
     $pdf->SetXY($x + 2 + $labelWidth, $y + 3);
     $pdf->Write(5, $rankValue);
@@ -1531,10 +1621,16 @@ $scoresEndY = $pdf->GetY();
                 if ($typeRaw === 'attendance_comment') {
                     $typeRaw = 'attendance';
                 }
-                $reviewVal  = trim((string)($row['comment_review'] ?? ''));
+                $rawComment = trim((string)($row['comment'] ?? ''));
-                $commentVal = $this->db->fieldExists('comment_review', 'score_comments')
+                if ($typeRaw === 'attendance') {
-                    ? $reviewVal
+                    // Attendance comments must come from score_comments.comment.
-                    : trim((string)($row['comment'] ?? ''));
+                    $commentVal = $rawComment;
+                } else {
+                    $reviewVal  = trim((string)($row['comment_review'] ?? ''));
+                    $commentVal = $this->db->fieldExists('comment_review', 'score_comments') && $reviewVal !== ''
+                        ? $reviewVal
+                        : $rawComment;
+                }
                 if ($commentVal === '') {
                     continue;
                 }
@@ -1732,161 +1828,200 @@ $scoresEndY = $pdf->GetY();
         ];
     }
 
-    private function calculateTermRanking(
+private function calculateTermRanking(
-        int $studentId,
+    int $studentId,
-        int $sectionCode,
+    int $sectionCode,
-        ?int $sectionId,
+    ?int $sectionId,
-        string $schoolYear,
+    string $schoolYear,
-        ?string $semester,
+    ?string $semester,
-        ?float $studentScore
+    ?float $studentScore
-    ): ?array {
+): ?array {
-        if ($studentId <= 0 || $schoolYear === '' || $studentScore === null) {
+    if ($studentId <= 0 || $schoolYear === '' || $studentScore === null) {
-            return null;
+        return null;
+    }
+
+    $sectionIds = array_values(array_unique(array_filter([
+        $sectionCode > 0 ? $sectionCode : null,
+        $sectionId && $sectionId > 0 ? $sectionId : null,
+    ])));
+
+    if (empty($sectionIds)) {
+        return null;
+    }
+
+    $semesterForRank  = trim((string)$semester);
+    $rankByFinalScore = $this->normalizeSemester($semesterForRank) === 'spring';
+
+    $builder = $this->db->table('semester_scores ss')
+        ->select('ss.student_id, ss.semester, ss.semester_score, ss.updated_at, ss.id, s.firstname, s.lastname')
+        ->join('students s', 's.id = ss.student_id', 'inner')
+        ->where('s.is_active', 1)
+        ->where('ss.school_year', $schoolYear)
+        ->whereIn('ss.class_section_id', $sectionIds)
+        ->orderBy('ss.updated_at', 'DESC')
+        ->orderBy('ss.id', 'DESC');
+
+    if ($semesterForRank !== '') {
+        $this->applySemesterFilter($builder, $semesterForRank, 'ss.semester');
+    }
+
+    $rows = $builder->get()->getResultArray();
+
+    if (empty($rows)) {
+        return null;
+    }
+
+    $scoresByStudent = [];
+
+    foreach ($rows as $row) {
+        $sid = (int)($row['student_id'] ?? 0);
+
+        if ($sid <= 0 || isset($scoresByStudent[$sid])) {
+            continue;
         }
 
-        $sectionIds = array_values(array_unique(array_filter([
+        $scoreVal = $row['semester_score'] ?? null;
-            $sectionCode > 0 ? $sectionCode : null,
-            $sectionId && $sectionId > 0 ? $sectionId : null,
-        ])));
 
-        if (empty($sectionIds)) {
+        if (!is_numeric($scoreVal)) {
-            return null;
+            continue;
         }
 
-        $semesterForRank = trim((string)$semester);
+        $rawScore = (float)$scoreVal;
-        $rankByFinalScore = $this->normalizeSemester($semesterForRank) === 'spring';
 
-        $builder = $this->db->table('semester_scores ss')
+        $scoresByStudent[$sid] = [
-            ->select('ss.student_id, ss.semester, ss.semester_score, ss.updated_at, ss.id, s.firstname, s.lastname')
+            'student_id' => $sid,
-            ->join('students s', 's.id = ss.student_id', 'inner')
+            'score'      => $rawScore,
-            ->where('s.is_active', 1)
+            'rank_score' => round($rawScore, 1),
+            'firstname'  => trim((string)($row['firstname'] ?? '')),
+            'lastname'   => trim((string)($row['lastname'] ?? '')),
+        ];
+    }
+
+    if (empty($scoresByStudent) || !isset($scoresByStudent[$studentId])) {
+        return null;
+    }
+
+    // For Spring, rank by final year average:
+    // (first semester score + second semester score) / 2.
+    if ($rankByFinalScore) {
+        $studentIds = array_keys($scoresByStudent);
+
+        $firstRowsBuilder = $this->db->table('semester_scores ss')
+            ->select('ss.student_id, ss.semester, ss.semester_score, ss.updated_at, ss.id')
             ->where('ss.school_year', $schoolYear)
+            ->whereIn('ss.student_id', $studentIds)
             ->whereIn('ss.class_section_id', $sectionIds)
             ->orderBy('ss.updated_at', 'DESC')
             ->orderBy('ss.id', 'DESC');
 
         if ($semesterForRank !== '') {
-            $this->applySemesterFilter($builder, $semesterForRank, 'ss.semester');
+            $this->applySemesterExclusion($firstRowsBuilder, $semesterForRank, 'ss.semester');
         }
 
-        $rows = $builder->get()->getResultArray();
+        $firstRows = $firstRowsBuilder->get()->getResultArray();
-        if (empty($rows)) {
-            return null;
-        }
 
-        $scoresByStudent = [];
+        $firstScoresByStudent = [];
-        $studentIds = [];
+
-        foreach ($rows as $row) {
+        foreach ($firstRows as $row) {
             $sid = (int)($row['student_id'] ?? 0);
-            if ($sid <= 0 || isset($scoresByStudent[$sid])) {
+
+            if ($sid <= 0 || isset($firstScoresByStudent[$sid])) {
                 continue;
             }
 
             $scoreVal = $row['semester_score'] ?? null;
+
             if (!is_numeric($scoreVal)) {
                 continue;
             }
 
-            $scoresByStudent[$sid] = [
+            $firstScoresByStudent[$sid] = (float)$scoreVal;
-                'student_id' => $sid,
-                'score'      => round((float)$scoreVal, 4),
-                'firstname'  => trim((string)($row['firstname'] ?? '')),
-                'lastname'   => trim((string)($row['lastname'] ?? '')),
-            ];
-            $studentIds[] = $sid;
         }
 
+        foreach ($scoresByStudent as $sid => &$rankRow) {
+            if (!isset($firstScoresByStudent[$sid])) {
+                unset($scoresByStudent[$sid]);
+                continue;
+            }
+
+            $avg = ((float)$firstScoresByStudent[$sid] + (float)$rankRow['score']) / 2;
+
+            $rankRow['score']      = $avg;
+            $rankRow['rank_score'] = round($avg, 1);
+        }
+
+        unset($rankRow);
+
         if (empty($scoresByStudent) || !isset($scoresByStudent[$studentId])) {
             return null;
         }
 
-        if ($rankByFinalScore) {
+        // Force the selected student to use the exact score already computed for the report.
-            $firstRowsBuilder = $this->db->table('semester_scores ss')
+        // Then rank using the displayed precision: one decimal.
-                ->select('ss.student_id, ss.semester_score, ss.updated_at, ss.id')
+        $scoresByStudent[$studentId]['score']      = (float)$studentScore;
-                ->where('ss.school_year', $schoolYear)
+        $scoresByStudent[$studentId]['rank_score'] = round((float)$studentScore, 1);
-                ->whereIn('ss.class_section_id', $sectionIds)
+    } else {
-                ->whereIn('ss.student_id', $studentIds)
+        // Fall: also force selected student to match the report-card computed score.
-                ->orderBy('ss.updated_at', 'DESC')
+        $scoresByStudent[$studentId]['score']      = (float)$studentScore;
-                ->orderBy('ss.id', 'DESC');
+        $scoresByStudent[$studentId]['rank_score'] = round((float)$studentScore, 1);
-
-            if ($semesterForRank !== '') {
-                $this->applySemesterExclusion($firstRowsBuilder, $semesterForRank, 'ss.semester');
-            }
-
-            $firstRows = $firstRowsBuilder->get()->getResultArray();
-            $firstScoresByStudent = [];
-            foreach ($firstRows as $row) {
-                $sid = (int)($row['student_id'] ?? 0);
-                if ($sid <= 0 || isset($firstScoresByStudent[$sid])) {
-                    continue;
-                }
-
-                $scoreVal = $row['semester_score'] ?? null;
-                if (!is_numeric($scoreVal)) {
-                    continue;
-                }
-
-                $firstScoresByStudent[$sid] = (float)$scoreVal;
-            }
-
-            foreach ($scoresByStudent as $sid => &$rankRow) {
-                if (!isset($firstScoresByStudent[$sid])) {
-                    unset($scoresByStudent[$sid]);
-                    continue;
-                }
-
-                $rankRow['score'] = round(((float)$firstScoresByStudent[$sid] + (float)$rankRow['score']) / 2, 4);
-            }
-            unset($rankRow);
-
-            if (empty($scoresByStudent) || !isset($scoresByStudent[$studentId])) {
-                return null;
-            }
-
-            $scoresByStudent[$studentId]['score'] = round((float)$studentScore, 4);
-        }
-
-        $rankable = array_values($scoresByStudent);
-        usort($rankable, static function (array $a, array $b): int {
-            $scoreCmp = $b['score'] <=> $a['score'];
-            if ($scoreCmp !== 0) {
-                return $scoreCmp;
-            }
-
-            $lastCmp = strcasecmp($a['lastname'], $b['lastname']);
-            if ($lastCmp !== 0) {
-                return $lastCmp;
-            }
-
-            $firstCmp = strcasecmp($a['firstname'], $b['firstname']);
-            if ($firstCmp !== 0) {
-                return $firstCmp;
-            }
-
-            return $a['student_id'] <=> $b['student_id'];
-        });
-
-        $position = null;
-        $previousScore = null;
-        foreach ($rankable as $index => $row) {
-            if ($previousScore === null || abs($row['score'] - $previousScore) > 0.0001) {
-                $position = $index + 1;
-                $previousScore = $row['score'];
-            }
-
-            if ((int)$row['student_id'] === $studentId) {
-                $total = count($rankable);
-                return [
-                    'position' => $position,
-                    'total'    => $total,
-                    'display'  => $this->formatOrdinal($position) . ' of ' . $total,
-                ];
-            }
-        }
-
-        return null;
     }
 
+    $rankable = array_values($scoresByStudent);
+
+    usort($rankable, static function (array $a, array $b): int {
+        // Highest displayed/rank score first.
+        $scoreCmp = $b['rank_score'] <=> $a['rank_score'];
+
+        if ($scoreCmp !== 0) {
+            return $scoreCmp;
+        }
+
+        // Tie-breakers only control display order.
+        // They do NOT change rank.
+        $lastCmp = strcasecmp($a['lastname'], $b['lastname']);
+
+        if ($lastCmp !== 0) {
+            return $lastCmp;
+        }
+
+        $firstCmp = strcasecmp($a['firstname'], $b['firstname']);
+
+        if ($firstCmp !== 0) {
+            return $firstCmp;
+        }
+
+        return $a['student_id'] <=> $b['student_id'];
+    });
+
+    $position      = null;
+    $previousScore = null;
+
+    foreach ($rankable as $index => $row) {
+        $currentScore = (float)$row['rank_score'];
+
+        // Competition ranking:
+        // 1, 1, 3, 4...
+        // Same score = same rank.
+        if ($previousScore === null || abs($currentScore - $previousScore) > 0.0001) {
+            $position = $index + 1;
+            $previousScore = $currentScore;
+        }
+
+        if ((int)$row['student_id'] === $studentId) {
+            $total = count($rankable);
+
+            return [
+                'position' => $position,
+                'total'    => $total,
+                'score'    => $currentScore,
+                'display'  => $this->formatOrdinal($position) . ' out of ' . $total,
+            ];
+        }
+    }
+
+    return null;
+}
+
     private function formatOrdinal(?int $value): string
     {
         $n = (int)$value;

app/Listeners/DecisionEmailListener.php

@@ -65,7 +65,6 @@ class DecisionEmailListener
             'parent_name'        => $parentName !== '' ? $parentName : 'Parent/Guardian',
             'student_name'       => $studentName !== '' ? $studentName : 'your student',
             'class_section_name' => $classSection,
-            'semester'           => $semester,
             'school_year'        => $schoolYear,
             'decision'           => $decision,
             'notes'              => $notes,

app/Models/StudentDecisionModel.php

@@ -11,10 +11,9 @@ class StudentDecisionModel extends Model
 
     protected $allowedFields = [
         'student_id',
-        'semester',
         'school_year',
         'class_section_name',
-        'semester_score',
+        'year_score',
         'decision',
         'source',
         'notes',

app/Views/admin/certificates/index.php

@@ -38,11 +38,14 @@ foreach ($statsPerClass as $csid => $cs) {
     if (!isset($gradeGroups[$sortKey])) {
         $gradeGroups[$sortKey] = ['label' => $label, 'slug' => $slug, 'csids' => []];
     }
+
     $gradeGroups[$sortKey]['csids'][] = $csid;
 }
+
 ksort($gradeGroups);
-$gradeKeys   = array_keys($gradeGroups);
+
-$defaultKey  = $gradeKeys[0] ?? null;
+$gradeKeys  = array_keys($gradeGroups);
+$defaultKey = $gradeKeys[0] ?? null;
 
 $decisionBadge = [
     'Pass'                 => 'success',
@@ -54,14 +57,20 @@ $decisionBadge = [
 ];
 ?>
 
-<h2 class="text-center mt-4 mb-3"><i class="bi bi-award me-2"></i>Generate Certificates</h2>
+<h2 class="text-center mt-4 mb-3">
+    <i class="bi bi-award me-2"></i>Generate Certificates
+</h2>
 
 <!-- School year filter -->
 <div class="d-flex justify-content-end mb-3">
     <form method="get" action="<?= site_url('administrator/certificates') ?>" class="d-flex gap-2 align-items-center">
         <label class="form-label mb-0 me-1 text-muted small">School Year</label>
-        <input type="text" name="school_year" class="form-control form-control-sm" style="width:130px;"
+        <input type="text"
-               value="<?= esc($schoolYear) ?>" placeholder="e.g. 2024-2025">
+               name="school_year"
+               class="form-control form-control-sm"
+               style="width:130px;"
+               value="<?= esc($schoolYear) ?>"
+               placeholder="e.g. 2024-2025">
         <button type="submit" class="btn btn-sm btn-outline-primary">
             <i class="bi bi-arrow-repeat me-1"></i>Reload
         </button>
@@ -82,167 +91,228 @@ $decisionBadge = [
 <!-- Grade tabs -->
 <ul class="nav nav-tabs justify-content-center" id="certTabs" role="tablist" style="flex-wrap:wrap;row-gap:.25rem;">
     <?php foreach ($gradeGroups as $key => $group): ?>
-    <?php
+        <?php
-    $slug     = $group['slug'];
+        $slug      = $group['slug'];
-    $label    = $group['label'];
+        $label     = $group['label'];
-    $total     = array_sum(array_map(fn($id) => $statsPerClass[$id]['total'] ?? 0, $group['csids']));
+        $total     = array_sum(array_map(fn($id) => $statsPerClass[$id]['total'] ?? 0, $group['csids']));
-    $grpPass   = array_sum(array_map(fn($id) => $statsPerClass[$id]['pass'] ?? 0, $group['csids']));
+        $grpPass   = array_sum(array_map(fn($id) => $statsPerClass[$id]['pass'] ?? 0, $group['csids']));
-    $grpCert   = array_sum(array_map(fn($id) => $statsPerClass[$id]['cert'] ?? 0, $group['csids']));
+        $grpCert   = array_sum(array_map(fn($id) => $statsPerClass[$id]['cert'] ?? 0, $group['csids']));
-    $fullyDone = $grpPass > 0 && $grpCert >= $grpPass;
+        $fullyDone = $grpPass > 0 && $grpCert >= $grpPass;
-    $hasPass   = $grpPass > 0;
+        $hasPass   = $grpPass > 0;
-    $isActive  = ($key === $defaultKey);
+        $isActive  = ($key === $defaultKey);
-    $statusTitle = $hasPass
+
-        ? ($fullyDone ? 'Fully generated (' . $grpCert . '/' . $grpPass . ')' : 'Not fully generated (' . $grpCert . '/' . $grpPass . ')')
+        $statusTitle = $hasPass
-        : 'No eligible students';
+            ? ($fullyDone ? 'Fully generated (' . $grpCert . '/' . $grpPass . ')' : 'Not fully generated (' . $grpCert . '/' . $grpPass . ')')
-    ?>
+            : 'No eligible students';
-    <li class="nav-item" role="presentation">
+        ?>
-        <a class="nav-link <?= $isActive ? 'active' : '' ?>"
+
-           id="cert-<?= esc($slug) ?>-tab"
+        <li class="nav-item" role="presentation">
-           data-bs-toggle="tab"
+            <a class="nav-link <?= $isActive ? 'active' : '' ?>"
-           href="#cert-<?= esc($slug) ?>"
+               id="cert-<?= esc($slug) ?>-tab"
-           role="tab"
+               data-bs-toggle="tab"
-           aria-controls="cert-<?= esc($slug) ?>"
+               href="#cert-<?= esc($slug) ?>"
-           aria-selected="<?= $isActive ? 'true' : 'false' ?>">
+               role="tab"
-            <span class="cert-status-dot <?= !$hasPass ? 'no-eligible' : ($fullyDone ? 'done' : 'pending') ?>"
+               aria-controls="cert-<?= esc($slug) ?>"
-                  title="<?= esc($statusTitle) ?>"></span>
+               aria-selected="<?= $isActive ? 'true' : 'false' ?>">
-            <?= esc($label) ?>
+                <span class="cert-status-dot <?= !$hasPass ? 'no-eligible' : ($fullyDone ? 'done' : 'pending') ?>"
-            <span class="badge bg-secondary ms-1"><?= $total ?></span>
+                      title="<?= esc($statusTitle) ?>"></span>
-        </a>
+                <?= esc($label) ?>
-    </li>
+                <span class="badge bg-secondary ms-1"><?= $total ?></span>
+            </a>
+        </li>
     <?php endforeach; ?>
 </ul>
 
 <!-- Tab content -->
 <div class="tab-content mt-3" id="certTabContent">
     <?php foreach ($gradeGroups as $key => $group): ?>
-    <?php $isActive = ($key === $defaultKey); ?>
+        <?php $isActive = ($key === $defaultKey); ?>
-    <div class="tab-pane fade <?= $isActive ? 'show active' : '' ?>"
-         id="cert-<?= esc($group['slug']) ?>"
-         role="tabpanel"
-         aria-labelledby="cert-<?= esc($group['slug']) ?>-tab">
 
-        <?php foreach ($group['csids'] as $csid): ?>
+        <div class="tab-pane fade <?= $isActive ? 'show active' : '' ?>"
-        <?php
+             id="cert-<?= esc($group['slug']) ?>"
-        $cs       = $statsPerClass[$csid];
+             role="tabpanel"
-        $students = $studentsByClass[$csid] ?? [];
+             aria-labelledby="cert-<?= esc($group['slug']) ?>-tab">
-        $csPass   = $cs['pass'];
-        $csCert   = $cs['cert'];
-        $csRemain = max(0, $csPass - $csCert);
-        $formId   = 'certForm-' . $csid;
-        $tableId  = 'studentsTable-' . $csid;
-        ?>
 
-        <h4 class="mt-4 mb-2 text-center"><?= esc($cs['name']) ?></h4>
+            <?php foreach ($group['csids'] as $csid): ?>
+                <?php
+                $cs       = $statsPerClass[$csid];
+                $students = $studentsByClass[$csid] ?? [];
+                $csPass   = $cs['pass'];
+                $csCert   = $cs['cert'];
+                $csRemain = max(0, $csPass - $csCert);
+                $formId   = 'certForm-' . $csid;
+                $tableId  = 'studentsTable-' . $csid;
+                ?>
 
-        <?php if (empty($students)): ?>
+                <h4 class="mt-4 mb-2 text-center"><?= esc($cs['name']) ?></h4>
-            <p class="text-muted text-center">No active students.</p>
-        <?php else: ?>
 
-        <form method="post" action="<?= site_url('administrator/certificates/generate') ?>"
+                <?php if (empty($students)): ?>
-              id="<?= esc($formId) ?>" class="cert-form mb-5">
+                    <p class="text-muted text-center">No active students.</p>
-            <?= csrf_field() ?>
+                <?php else: ?>
-            <input type="hidden" name="class_section_id" value="<?= (int)$csid ?>">
-            <input type="hidden" name="school_year"      value="<?= esc($schoolYear) ?>">
 
-            <!-- Stats + date picker -->
+                    <form method="post"
-            <div class="d-flex justify-content-between align-items-center flex-wrap gap-2 mb-2">
+                          action="<?= site_url('administrator/certificates/generate') ?>"
-                <div class="d-flex align-items-center gap-4 flex-wrap">
+                          id="<?= esc($formId) ?>"
-                    <span class="fw-semibold">
+                          class="cert-form mb-5">
-                        Students <span class="badge bg-secondary ms-1"><?= count($students) ?></span>
+                        <?= csrf_field() ?>
-                    </span>
-                    <span class="text-muted small">
-                        <strong class="text-success"><?= $csPass ?></strong> Pass
-                    </span>
-                    <span class="text-muted small">
-                        <strong class="text-primary"><?= $csCert ?></strong> Generated
-                    </span>
-                    <span class="text-muted small">
-                        <strong class="<?= $csRemain > 0 ? 'text-warning' : 'text-muted' ?>"><?= $csRemain ?></strong> Remaining
-                    </span>
-                </div>
-                <div class="input-group input-group-sm" style="width:200px;">
-                    <span class="input-group-text"><i class="bi bi-calendar3"></i></span>
-                    <input type="date" class="form-control cert-date-picker"
-                           value="<?= date('Y-m-d') ?>" title="Certificate date">
-                    <input type="hidden" name="cert_date" class="cert-date-hidden" value="<?= esc($certDate) ?>">
-                </div>
-            </div>
 
-            <!-- Student table -->
+                        <input type="hidden" name="class_section_id" value="<?= (int)$csid ?>">
-            <div class="table-responsive">
+                        <input type="hidden" name="school_year" value="<?= esc($schoolYear) ?>">
-                <table class="table table-hover table-striped align-middle mb-0 cert-students-table"
+
-                       id="<?= esc($tableId) ?>">
+                        <!-- Stats + date picker -->
-                    <thead class="table-light">
+                        <div class="d-flex justify-content-between align-items-center flex-wrap gap-2 mb-2">
-                        <tr>
+                            <div class="d-flex align-items-center gap-4 flex-wrap">
-                            <th style="width:40px;" class="text-center">
+                                <span class="fw-semibold">
-                                <input class="form-check-input cert-select-all" type="checkbox">
+                                    Students <span class="badge bg-secondary ms-1"><?= count($students) ?></span>
-                            </th>
+                                </span>
-                            <th>First Name</th>
+                                <span class="text-muted small">
-                            <th>Last Name</th>
+                                    <strong class="text-success"><?= $csPass ?></strong> Pass
-                            <th>Decision</th>
+                                </span>
-                            <th>Certificate No.</th>
+                                <span class="text-muted small">
-                        </tr>
+                                    <strong class="text-primary"><?= $csCert ?></strong> Generated
-                    </thead>
+                                </span>
-                    <tbody>
+                                <span class="text-muted small">
-                        <?php foreach ($students as $s): ?>
+                                    <strong class="<?= $csRemain > 0 ? 'text-warning' : 'text-muted' ?>">
-                        <?php
+                                        <?= $csRemain ?>
-                        $sid         = (int)$s['student_id'];
+                                    </strong>
-                        $stuDec      = $decisionsByStudent[$sid] ?? [];
+                                    Remaining
-                        $certNo      = $certsByStudent[$sid] ?? null;
+                                </span>
-                        $allDecs     = array_map(fn($d) => $d['decision'], $stuDec);
+                            </div>
-                        $hasPending  = in_array('', $allDecs, true);
+
-                        $unique      = array_values(array_unique(array_filter($allDecs, fn($d) => $d !== '')));
+                            <div class="input-group input-group-sm" style="width:200px;">
-                        $isPass      = !empty($stuDec) && !$hasPending && $unique === ['Pass'];
+                                <span class="input-group-text"><i class="bi bi-calendar3"></i></span>
-                        $displayDecs = $isPass ? ['Pass'] : array_values(array_filter($unique, fn($d) => $d !== 'Pass'));
+                                <input type="date"
-                        ?>
+                                       class="form-control cert-date-picker"
-                        <tr>
+                                       value="<?= date('Y-m-d') ?>"
-                            <td class="text-center">
+                                       title="Certificate date">
-                                <input class="form-check-input cert-student-check" type="checkbox"
+                                <input type="hidden"
-                                       name="student_ids[]" value="<?= $sid ?>"
+                                       name="cert_date"
-                                       <?= $isPass ? '' : 'disabled' ?>>
+                                       class="cert-date-hidden"
-                            </td>
+                                       value="<?= esc($certDate) ?>">
-                            <td><?= esc($s['firstname']) ?></td>
+                            </div>
-                            <td><?= esc($s['lastname']) ?></td>
+                        </div>
-                            <td>
+
-                                <?php if (empty($stuDec)): ?>
+                        <!-- Student table -->
-                                    <span class="text-muted small">—</span>
+                        <div class="table-responsive">
-                                <?php elseif ($hasPending || empty($unique) || empty($displayDecs)): ?>
+                            <table class="table table-hover table-striped align-middle mb-0 cert-students-table"
-                                    <span class="badge bg-warning text-dark">Pending</span>
+                                   id="<?= esc($tableId) ?>">
-                                <?php else: ?>
+                                <thead class="table-light">
-                                    <?php foreach ($displayDecs as $dec): $color = $decisionBadge[$dec] ?? 'secondary'; ?>
+                                    <tr>
-                                        <span class="badge bg-<?= esc($color) ?> me-1"><?= esc($dec) ?></span>
+                                        <th style="width:40px;" class="text-center">
+                                            <input class="form-check-input cert-select-all" type="checkbox">
+                                        </th>
+                                        <th>First Name</th>
+                                        <th>Last Name</th>
+                                        <th class="text-center">Year Score</th>
+                                        <th>Decision</th>
+                                        <th>Certificate No.</th>
+                                    </tr>
+                                </thead>
+
+                                <tbody>
+                                    <?php foreach ($students as $s): ?>
+                                        <?php
+                                        $sid    = (int)$s['student_id'];
+                                        $stuDec = $decisionsByStudent[$sid] ?? [];
+                                        $certNo = $certsByStudent[$sid] ?? null;
+
+                                        $allDecs = array_map(
+                                            fn($d) => trim((string)($d['decision'] ?? '')),
+                                            $stuDec
+                                        );
+
+                                        $hasPending = in_array('', $allDecs, true);
+
+                                        $unique = array_values(array_unique(array_filter(
+                                            $allDecs,
+                                            fn($d) => $d !== ''
+                                        )));
+
+                                        $isPass = !empty($stuDec) && !$hasPending && $unique === ['Pass'];
+
+                                        $displayDecs = $isPass
+                                            ? ['Pass']
+                                            : array_values(array_filter($unique, fn($d) => $d !== 'Pass'));
+
+                                        $yearScore = null;
+
+                                        foreach ($stuDec as $d) {
+                                            if (
+                                                array_key_exists('year_score', $d)
+                                                && $d['year_score'] !== null
+                                                && $d['year_score'] !== ''
+                                            ) {
+                                                $yearScore = $d['year_score'];
+                                                break;
+                                            }
+                                        }
+
+                                        $fmtYearScore = is_numeric($yearScore)
+                                            ? number_format((float)$yearScore, 2)
+                                            : '—';
+                                        ?>
+
+                                        <tr>
+                                            <td class="text-center">
+                                                <input class="form-check-input cert-student-check"
+                                                       type="checkbox"
+                                                       name="student_ids[]"
+                                                       value="<?= $sid ?>"
+                                                       <?= $isPass ? '' : 'disabled' ?>>
+                                            </td>
+
+                                            <td><?= esc($s['firstname']) ?></td>
+                                            <td><?= esc($s['lastname']) ?></td>
+
+                                            <td class="text-center fw-semibold">
+                                                <?= esc($fmtYearScore) ?>
+                                            </td>
+
+                                            <td>
+                                                <?php if (empty($stuDec)): ?>
+                                                    <span class="text-muted small">—</span>
+                                                <?php elseif ($hasPending || empty($unique) || empty($displayDecs)): ?>
+                                                    <span class="badge bg-warning text-dark">Pending</span>
+                                                <?php else: ?>
+                                                    <?php foreach ($displayDecs as $dec): ?>
+                                                        <?php $color = $decisionBadge[$dec] ?? 'secondary'; ?>
+                                                        <span class="badge bg-<?= esc($color) ?> me-1">
+                                                            <?= esc($dec) ?>
+                                                        </span>
+                                                    <?php endforeach; ?>
+                                                <?php endif; ?>
+                                            </td>
+
+                                            <td>
+                                                <?php if ($certNo): ?>
+                                                    <a href="<?= site_url('administrator/certificates/reprint/' . rawurlencode($certNo)) ?>"
+                                                       target="_blank"
+                                                       class="font-monospace small">
+                                                        <?= esc($certNo) ?>
+                                                    </a>
+                                                <?php else: ?>
+                                                    <span class="text-muted">—</span>
+                                                <?php endif; ?>
+                                            </td>
+                                        </tr>
                                     <?php endforeach; ?>
-                                <?php endif; ?>
+                                </tbody>
-                            </td>
+                            </table>
-                            <td>
+                        </div>
-                                <?php if ($certNo): ?>
-                                    <a href="<?= site_url('administrator/certificates/reprint/' . rawurlencode($certNo)) ?>"
-                                       target="_blank" class="font-monospace small">
-                                        <?= esc($certNo) ?>
-                                    </a>
-                                <?php else: ?>
-                                    <span class="text-muted">—</span>
-                                <?php endif; ?>
-                            </td>
-                        </tr>
-                        <?php endforeach; ?>
-                    </tbody>
-                </table>
-            </div>
 
-            <!-- Footer -->
+                        <!-- Footer -->
-            <div class="d-flex justify-content-between align-items-center mt-2">
+                        <div class="d-flex justify-content-between align-items-center mt-2">
-                <span class="text-muted small cert-selected-count">0 students selected</span>
+                            <span class="text-muted small cert-selected-count">0 students selected</span>
-                <button type="submit" class="btn btn-success cert-generate-btn" disabled>
+                            <button type="submit" class="btn btn-success cert-generate-btn" disabled>
-                    <i class="bi bi-printer me-1"></i>Generate &amp; Print Certificates
+                                <i class="bi bi-printer me-1"></i>Generate &amp; Print Certificates
-                </button>
+                            </button>
-            </div>
+                        </div>
-        </form>
+                    </form>
 
-        <?php endif; ?>
+                <?php endif; ?>
-        <?php endforeach; ?>
+            <?php endforeach; ?>
 
-    </div>
+        </div>
     <?php endforeach; ?>
 </div>
 
@@ -253,44 +323,102 @@ $decisionBadge = [
 
 <script>
 (function () {
-    const csrfRefreshUrl      = <?= json_encode(site_url('administrator/certificates/csrf-token'), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+    const csrfRefreshUrl = <?= json_encode(site_url('administrator/certificates/csrf-token'), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
-    let   currentCsrfTokenName = <?= json_encode(csrf_token(), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
+    let currentCsrfTokenName = <?= json_encode(csrf_token(), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT) ?>;
 
     function cssEscape(v) {
         return window.CSS?.escape ? window.CSS.escape(v) : String(v).replace(/(["\\.#:[\],= ])/g, '\\$1');
     }
+
     function updateCsrfInForm(form, name, hash) {
         if (!form || !name || !hash) return;
+
         form.querySelectorAll('input[type="hidden"]').forEach(inp => {
-            if (inp.name === name || inp.name === currentCsrfTokenName) { inp.name = name; inp.value = hash; }
+            if (inp.name === name || inp.name === currentCsrfTokenName) {
+                inp.name = name;
+                inp.value = hash;
+            }
         });
+
         let inp = form.querySelector(`input[name="${cssEscape(name)}"]`);
-        if (!inp) { inp = document.createElement('input'); inp.type = 'hidden'; inp.name = name; form.appendChild(inp); }
+
+        if (!inp) {
+            inp = document.createElement('input');
+            inp.type = 'hidden';
+            inp.name = name;
+            form.appendChild(inp);
+        }
+
         inp.value = hash;
         currentCsrfTokenName = name;
     }
+
     async function refreshCsrf(form) {
-        const r = await fetch(csrfRefreshUrl, { method: 'GET', credentials: 'same-origin', headers: { 'X-Requested-With': 'XMLHttpRequest', 'Cache-Control': 'no-store' } });
+        const r = await fetch(csrfRefreshUrl, {
+            method: 'GET',
+            credentials: 'same-origin',
+            headers: {
+                'X-Requested-With': 'XMLHttpRequest',
+                'Cache-Control': 'no-store'
+            }
+        });
+
         if (!r.ok) throw new Error('CSRF refresh failed');
+
         const d = await r.json();
-        if (d?.csrf_token && d?.csrf_hash) updateCsrfInForm(form, d.csrf_token, d.csrf_hash);
+
+        if (d?.csrf_token && d?.csrf_hash) {
+            updateCsrfInForm(form, d.csrf_token, d.csrf_hash);
+        }
     }
 
-    let pdfWindow = null, activePdfBlobUrl = null;
+    let pdfWindow = null;
+    let activePdfBlobUrl = null;
+
     function openPdfWindow() {
-        if (!pdfWindow || pdfWindow.closed) pdfWindow = window.open('', 'certificatePdfWindow');
+        if (!pdfWindow || pdfWindow.closed) {
+            pdfWindow = window.open('', 'certificatePdfWindow');
+        }
+
         if (!pdfWindow) return null;
+
         pdfWindow.document.open();
         pdfWindow.document.write(`<!DOCTYPE html><html><head><meta charset="utf-8"><title>Certificate PDF</title>
-<style>html,body{margin:0;height:100%;background:#f3f4f6;font-family:Arial,sans-serif}.viewer-shell{display:flex;flex-direction:column;height:100%}.viewer-status{padding:12px 16px;background:#111827;color:#fff;font-size:14px}.viewer-frame{flex:1;width:100%;border:0;background:#cbd5e1}</style></head>
+<style>
-<body><div class="viewer-shell"><div class="viewer-status" id="viewerStatus">Preparing certificate PDF...</div><iframe class="viewer-frame" id="pdfFrame"></iframe></div>
+html,body{margin:0;height:100%;background:#f3f4f6;font-family:Arial,sans-serif}
-<script>window.showCertificatePdf=function(url,fn){const f=document.getElementById('pdfFrame'),s=document.getElementById('viewerStatus');if(s)s.textContent=fn?'Showing '+fn:'Certificate PDF ready';if(f)f.src=url;document.title=fn||'Certificate PDF'};<\/script></body></html>`);
+.viewer-shell{display:flex;flex-direction:column;height:100%}
+.viewer-status{padding:12px 16px;background:#111827;color:#fff;font-size:14px}
+.viewer-frame{flex:1;width:100%;border:0;background:#cbd5e1}
+</style></head>
+<body>
+<div class="viewer-shell">
+    <div class="viewer-status" id="viewerStatus">Preparing certificate PDF...</div>
+    <iframe class="viewer-frame" id="pdfFrame"></iframe>
+</div>
+<script>
+window.showCertificatePdf=function(url,fn){
+    const f=document.getElementById('pdfFrame'),s=document.getElementById('viewerStatus');
+    if(s)s.textContent=fn?'Showing '+fn:'Certificate PDF ready';
+    if(f)f.src=url;
+    document.title=fn||'Certificate PDF';
+};
+<\/script>
+</body></html>`);
+
         pdfWindow.document.close();
+
         return pdfWindow;
     }
+
     function showError(pane, msg) {
         let el = pane.querySelector('.cert-inline-error');
-        if (!el) { el = document.createElement('div'); el.className = 'alert alert-danger alert-dismissible fade show cert-inline-error'; pane.prepend(el); }
+
+        if (!el) {
+            el = document.createElement('div');
+            el.className = 'alert alert-danger alert-dismissible fade show cert-inline-error';
+            pane.prepend(el);
+        }
+
         el.innerHTML = `${msg}<button type="button" class="btn-close" data-bs-dismiss="alert"></button>`;
     }
 
@@ -307,62 +435,134 @@ $decisionBadge = [
         if (datePicker && dateHidden) {
             datePicker.addEventListener('change', function () {
                 const d = new Date(this.value + 'T00:00:00');
-                if (!isNaN(d)) dateHidden.value = String(d.getMonth()+1).padStart(2,'0')+'/'+String(d.getDate()).padStart(2,'0')+'/'+d.getFullYear();
+
+                if (!isNaN(d)) {
+                    dateHidden.value =
+                        String(d.getMonth() + 1).padStart(2, '0') + '/' +
+                        String(d.getDate()).padStart(2, '0') + '/' +
+                        d.getFullYear();
+                }
             });
         }
 
         function updateState() {
             const chosen = form.querySelectorAll('.cert-student-check:checked').length;
-            if (selectedCount) selectedCount.textContent = chosen + ' student' + (chosen !== 1 ? 's' : '') + ' selected';
+
-            if (generateBtn)   generateBtn.disabled = chosen === 0;
+            if (selectedCount) {
+                selectedCount.textContent = chosen + ' student' + (chosen !== 1 ? 's' : '') + ' selected';
+            }
+
+            if (generateBtn) {
+                generateBtn.disabled = chosen === 0;
+            }
+
             if (selectAll) {
                 const ec = eligibleChecks.length;
-                selectAll.checked       = ec > 0 && chosen === ec;
+                selectAll.checked = ec > 0 && chosen === ec;
                 selectAll.indeterminate = chosen > 0 && chosen < ec;
             }
         }
+
         if (selectAll) {
             selectAll.addEventListener('change', function () {
-                eligibleChecks.forEach(c => { c.checked = this.checked; });
+                eligibleChecks.forEach(c => {
+                    c.checked = this.checked;
+                });
+
                 updateState();
             });
         }
+
         checks.forEach(c => c.addEventListener('change', updateState));
         updateState();
 
         form.addEventListener('submit', async function (e) {
             e.preventDefault();
+
             const chosen = form.querySelectorAll('.cert-student-check:checked').length;
-            if (chosen === 0) { showError(pane, 'Please select at least one student.'); return; }
+
+            if (chosen === 0) {
+                showError(pane, 'Please select at least one student.');
+                return;
+            }
+
             const win = openPdfWindow();
-            if (!win) { showError(pane, 'Unable to open PDF tab — please allow pop-ups.'); return; }
+
+            if (!win) {
+                showError(pane, 'Unable to open PDF tab — please allow pop-ups.');
+                return;
+            }
+
             const origHtml = generateBtn.innerHTML;
             generateBtn.disabled = true;
             generateBtn.innerHTML = '<span class="spinner-border spinner-border-sm me-1"></span>Generating...';
+
             try {
                 await refreshCsrf(form);
+
                 const csrfVal = form.querySelector(`input[name="${cssEscape(currentCsrfTokenName)}"]`)?.value || '';
                 const fd = new FormData(form);
-                if (csrfVal) fd.set(currentCsrfTokenName, csrfVal);
+
-                const resp = await fetch(form.action, { method: 'POST', body: fd, credentials: 'same-origin', headers: { 'X-Requested-With': 'XMLHttpRequest' } });
+                if (csrfVal) {
-                const nn = resp.headers.get('X-CSRF-TOKEN-NAME'), nh = resp.headers.get('X-CSRF-TOKEN');
+                    fd.set(currentCsrfTokenName, csrfVal);
-                if (nn && nh) updateCsrfInForm(form, nn, nh);
+                }
+
+                const resp = await fetch(form.action, {
+                    method: 'POST',
+                    body: fd,
+                    credentials: 'same-origin',
+                    headers: {
+                        'X-Requested-With': 'XMLHttpRequest'
+                    }
+                });
+
+                const nn = resp.headers.get('X-CSRF-TOKEN-NAME');
+                const nh = resp.headers.get('X-CSRF-TOKEN');
+
+                if (nn && nh) {
+                    updateCsrfInForm(form, nn, nh);
+                }
+
                 const ct = resp.headers.get('Content-Type') || '';
+
                 if (!resp.ok || !ct.toLowerCase().includes('application/pdf')) {
                     let msg = 'Certificate generation failed.';
-                    try { const d = await resp.json(); if (d?.error) msg = d.error; } catch (_) { try { msg = await resp.text() || msg; } catch (_2) {} }
+
-                    win.close(); showError(pane, msg);
+                    try {
+                        const d = await resp.json();
+
+                        if (d?.error) {
+                            msg = d.error;
+                        }
+                    } catch (_) {
+                        try {
+                            msg = await resp.text() || msg;
+                        } catch (_2) {}
+                    }
+
+                    win.close();
+                    showError(pane, msg);
                     await refreshCsrf(form).catch(() => {});
                     return;
                 }
+
                 const disp = resp.headers.get('Content-Disposition') || '';
-                const fn   = (disp.match(/filename="?([^"]+)"?/i) || [])[1] || 'Certificates.pdf';
+                const fn = (disp.match(/filename="?([^"]+)"?/i) || [])[1] || 'Certificates.pdf';
-                const url  = URL.createObjectURL(await resp.blob());
+                const url = URL.createObjectURL(await resp.blob());
-                if (activePdfBlobUrl) URL.revokeObjectURL(activePdfBlobUrl);
+
+                if (activePdfBlobUrl) {
+                    URL.revokeObjectURL(activePdfBlobUrl);
+                }
+
                 activePdfBlobUrl = url;
                 win.showCertificatePdf(url, fn);
+
                 const activePane = form.closest('.tab-pane');
-                if (activePane?.id) window.location.hash = activePane.id;
+
+                if (activePane?.id) {
+                    window.location.hash = activePane.id;
+                }
+
                 window.location.reload();
             } catch (_) {
                 showError(pane, 'Certificate generation failed. Please try again.');
@@ -376,49 +576,68 @@ $decisionBadge = [
 })();
 </script>
 
+<?= $this->endSection() ?>
+
 <?= $this->section('scripts') ?>
 <script>
 // Restore tab from hash — runs after Bootstrap is loaded
 (function () {
     const hash = window.location.hash;
+
     if (!hash) return;
+
     const tabLink = document.querySelector('a[href="' + hash + '"][data-bs-toggle="tab"]');
+
     if (tabLink && window.bootstrap?.Tab) {
         new bootstrap.Tab(tabLink).show();
     }
 })();
 </script>
+
 <style>
 .cert-status-dot {
     display: inline-block;
-    width: 8px; height: 8px;
+    width: 8px;
+    height: 8px;
     border-radius: 50%;
     margin-right: 5px;
     vertical-align: middle;
     flex-shrink: 0;
 }
-.cert-status-dot.done        { background-color: #198754; }
+
-.cert-status-dot.pending     { background-color: #dc3545; }
+.cert-status-dot.done {
-.cert-status-dot.no-eligible { background-color: #adb5bd; }
+    background-color: #198754;
+}
+
+.cert-status-dot.pending {
+    background-color: #dc3545;
+}
+
+.cert-status-dot.no-eligible {
+    background-color: #adb5bd;
+}
 </style>
+
 <script>
 (function () {
     if (!window.$ || !$.fn?.DataTable) return;
+
     $(function () {
         document.querySelectorAll('.cert-students-table').forEach(function (tbl) {
             if ($.fn.DataTable.isDataTable(tbl)) return;
+
             try {
                 $(tbl).DataTable({
                     order: [[1, 'asc'], [2, 'asc']],
                     pageLength: 100,
                     lengthMenu: [25, 50, 100, 200],
-                    columnDefs: [{ orderable: false, targets: [0, 3, 4] }]
+                    columnDefs: [
+                        { orderable: false, targets: [0, 4, 5] }
+                    ]
                 });
             } catch (_) {}
         });
     });
 })();
 </script>
-<?= $this->endSection() ?>
+<?= $this->endSection() ?>
-
-<?= $this->endSection() ?>

app/Views/administrator/trophy_final.php

@@ -14,6 +14,47 @@ $totalConfirmed = array_sum(array_column($classResults, 'confirmed'));
 $totalSurprise  = array_sum(array_column($classResults, 'surprises'));
 $totalMissed    = array_sum(array_column($classResults, 'missed'));
 $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted * 100) : ($totalActual === 0 ? 100 : 0);
+
+// Winner gender breakdown.
+// "Winner" means actual year-end trophy winner.
+$totalWinnerBoys  = 0;
+$totalWinnerGirls = 0;
+$totalWinnerOther = 0;
+
+// Sticker names.
+// 2 columns x 10 rows = 20 stickers per page.
+// Stickers print NAME ONLY.
+$winnerStickerNames = [];
+
+foreach ($classResults as $cls) {
+    foreach (($cls['students'] ?? []) as $s) {
+        if (empty($s['actual'])) {
+            continue;
+        }
+
+        $gender = strtolower(trim((string)($s['gender'] ?? '')));
+
+        if (in_array($gender, ['male', 'm', 'boy', 'boys'], true)) {
+            $totalWinnerBoys++;
+        } elseif (in_array($gender, ['female', 'f', 'girl', 'girls'], true)) {
+            $totalWinnerGirls++;
+        } else {
+            $totalWinnerOther++;
+        }
+
+        $name = trim((string)($s['name'] ?? ''));
+
+        if ($name !== '') {
+            $winnerStickerNames[] = $name;
+        }
+    }
+}
+
+$totalWinners = $totalWinnerBoys + $totalWinnerGirls + $totalWinnerOther;
+
+$winnerBoysPct  = $totalWinners > 0 ? round(($totalWinnerBoys / $totalWinners) * 100, 1) : 0;
+$winnerGirlsPct = $totalWinners > 0 ? round(($totalWinnerGirls / $totalWinners) * 100, 1) : 0;
+$winnerOtherPct = $totalWinners > 0 ? round(($totalWinnerOther / $totalWinners) * 100, 1) : 0;
 ?>
 
 <style>
@@ -23,23 +64,141 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
 .status-none      { color:#adb5bd; }
 
 .print-only { display: none !important; }
+.winner-sticker-print-area { display: none; }
+
+@page {
+    size: Letter portrait;
+    margin: 0.35in;
+}
 
 @media print {
     .no-print    { display: none !important; }
     .print-only  { display: block !important; }
     .screen-only { display: none !important; }
+
     body { font-size: 11px; }
     .container-fluid { padding: 0 !important; }
     h2, h3 { font-size: 13px; margin-bottom: .3rem; }
     .print-page-break { break-before: page; }
+
     table { width: 100%; border-collapse: collapse; font-size: 10px; }
     table th, table td { border: 1px solid #bbb; padding: 3px 5px; }
-    table thead { background: #333 !important; color: #fff !important;
+    table thead {
-                  -webkit-print-color-adjust: exact; print-color-adjust: exact; }
+        background: #333 !important;
-    table thead th { position: static !important; top: auto !important; box-shadow: none !important; }
+        color: #fff !important;
+        -webkit-print-color-adjust: exact;
+        print-color-adjust: exact;
+    }
+    table thead th {
+        position: static !important;
+        top: auto !important;
+        box-shadow: none !important;
+    }
+
     .s-confirmed { color: #198754; font-weight: bold; }
     .s-surprise  { color: #0d6efd; font-weight: bold; }
     .s-missed    { color: #fd7e14; font-weight: bold; }
+
+    body.print-stickers-mode {
+        margin: 0 !important;
+        padding: 0 !important;
+        background: #fff !important;
+    }
+
+    body.print-stickers-mode * {
+        box-shadow: none !important;
+    }
+
+    body.print-stickers-mode header,
+    body.print-stickers-mode nav,
+    body.print-stickers-mode aside,
+    body.print-stickers-mode footer,
+    body.print-stickers-mode .navbar,
+    body.print-stickers-mode .sidebar,
+    body.print-stickers-mode .topbar,
+    body.print-stickers-mode .app-header,
+    body.print-stickers-mode .main-header,
+    body.print-stickers-mode .layout-header,
+    body.print-stickers-mode .management-header,
+    body.print-stickers-mode .page-header,
+    body.print-stickers-mode .breadcrumb,
+    body.print-stickers-mode .brand,
+    body.print-stickers-mode .logo,
+    body.print-stickers-mode .header,
+    body.print-stickers-mode .no-print,
+    body.print-stickers-mode .screen-only,
+    body.print-stickers-mode .print-only {
+        display: none !important;
+    }
+
+    body.print-stickers-mode .container-fluid > *:not(#winnerStickerPrintArea) {
+        display: none !important;
+    }
+
+    body.print-stickers-mode .container-fluid {
+        display: block !important;
+        visibility: visible !important;
+        padding: 0 !important;
+        margin: 0 !important;
+        width: 100% !important;
+        max-width: none !important;
+    }
+
+    body.print-stickers-mode #winnerStickerPrintArea {
+        display: block !important;
+        visibility: visible !important;
+        position: static !important;
+        width: 100% !important;
+        height: auto !important;
+        margin: 0 !important;
+        padding: 0 !important;
+        overflow: visible !important;
+    }
+
+    body.print-stickers-mode #winnerStickerPrintArea,
+    body.print-stickers-mode #winnerStickerPrintArea * {
+        visibility: visible !important;
+    }
+
+    body.print-stickers-mode .sticker-page {
+        display: grid !important;
+        grid-template-columns: repeat(2, 1fr);
+        grid-template-rows: repeat(10, 1fr);
+        gap: 0;
+        width: 100%;
+        height: 10.3in;
+        page-break-after: always;
+        break-after: page;
+    }
+
+    body.print-stickers-mode .sticker-page:last-child {
+        page-break-after: auto;
+        break-after: auto;
+    }
+
+    body.print-stickers-mode .sticker-cell {
+        display: flex !important;
+        justify-content: center;
+        align-items: center;
+        text-align: center;
+        border: none;
+        padding: 0;
+        overflow: hidden;
+        min-height: 0;
+        box-sizing: border-box;
+    }
+
+    body.print-stickers-mode .sticker-name {
+        display: block !important;
+        font-size: 20pt;
+        font-weight: 700;
+        line-height: 1.1;
+        color: #000 !important;
+    }
+
+    body.print-stickers-mode .sticker-empty {
+        visibility: hidden !important;
+    }
 }
 </style>
 
@@ -57,10 +216,16 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                 with the <strong>year-end result</strong> based on the average of Fall &amp; Spring scores.
             </p>
         </div>
-        <div class="d-flex gap-2">
+
+        <div class="d-flex gap-2 flex-wrap">
             <button onclick="printWithCharts()" class="btn btn-outline-secondary btn-sm">
                 <i class="bi bi-printer-fill me-1"></i>Print
             </button>
+
+            <button onclick="printWinnerStickers()" class="btn btn-warning btn-sm">
+                <i class="bi bi-tags-fill me-1"></i>Print Winner Stickers
+            </button>
+
             <a href="<?= site_url('administrator/trophy?' . http_build_query(['school_year' => $selectedYear, 'percentile' => $selectedPercentile])) ?>"
                class="btn btn-outline-secondary btn-sm">
                 <i class="bi bi-arrow-left me-1"></i>Back
@@ -75,18 +240,27 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
             <label class="form-label mb-1 small fw-semibold">School Year</label>
             <select name="school_year" class="form-select form-select-sm" style="min-width:130px;">
                 <?php foreach ($years as $yr): ?>
-                    <option value="<?= esc($yr) ?>" <?= $yr === $selectedYear ? 'selected' : '' ?>><?= esc($yr) ?></option>
+                    <option value="<?= esc($yr) ?>" <?= $yr === $selectedYear ? 'selected' : '' ?>>
+                        <?= esc($yr) ?>
+                    </option>
                 <?php endforeach; ?>
             </select>
         </div>
+
         <div class="col-auto">
             <label class="form-label mb-1 small fw-semibold">Percentile</label>
             <div class="input-group input-group-sm" style="width:110px;">
-                <input type="number" name="percentile" class="form-control"
+                <input type="number"
-                       min="1" max="99" step="1" value="<?= (int)$selectedPercentile ?>">
+                       name="percentile"
+                       class="form-control"
+                       min="1"
+                       max="99"
+                       step="1"
+                       value="<?= (int)$selectedPercentile ?>">
                 <span class="input-group-text">%</span>
             </div>
         </div>
+
         <div class="col-auto">
             <button type="submit" class="btn btn-primary btn-sm">Apply</button>
         </div>
@@ -111,7 +285,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
         <div class="row g-3 mb-4">
             <?php foreach ([
                 [$totalPredicted, 'Predicted (Fall)',  'primary',  null,   $totalStudents > 0 ? round($totalPredicted / $totalStudents * 100) . '% of students' : '—'],
-                [$totalActual,    'Actual (Year)',      'warning',  'dark', $totalStudents > 0 ? round($totalActual    / $totalStudents * 100) . '% of students' : '—'],
+                [$totalActual,    'Actual (Year)',      'warning',  'dark', $totalStudents > 0 ? round($totalActual / $totalStudents * 100) . '% of students' : '—'],
                 [$totalConfirmed, 'Confirmed',          'success',  null,   $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted * 100) . '% of predicted' : '—'],
                 [$totalSurprise,  'Surprises',          'info',     'dark', 'Not in prediction'],
                 [$totalMissed,    'Missed',             'orange',   null,   'Were predicted'],
@@ -147,19 +321,29 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                     <span class="fw-bold fs-6"><?= esc($cls['section_name']) ?></span>
                     <span class="badge bg-primary"><?= $cls['predicted_count'] ?> predicted</span>
                     <span class="badge bg-warning text-dark"><?= $cls['actual_count'] ?> actual</span>
+
                     <?php if ($cls['confirmed'] > 0): ?>
                         <span class="badge bg-success"><?= $cls['confirmed'] ?> confirmed</span>
                     <?php endif; ?>
+
                     <?php if ($cls['surprises'] > 0): ?>
                         <span class="badge bg-info text-dark"><?= $cls['surprises'] ?> surprise<?= $cls['surprises'] > 1 ? 's' : '' ?></span>
                     <?php endif; ?>
+
                     <?php if ($cls['missed'] > 0): ?>
                         <span class="badge" style="background:#fd7e14"><?= $cls['missed'] ?> missed</span>
                     <?php endif; ?>
                 </div>
+
                 <div class="d-flex gap-3 small align-items-center">
-                    <span class="text-muted">Fall &#8805; <strong><?= $cls['fall_threshold'] !== null ? number_format((float)$cls['fall_threshold'], 1) : '—' ?></strong></span>
+                    <span class="text-muted">
-                    <span class="text-muted">Year &#8805; <strong><?= $cls['year_threshold'] !== null ? number_format((float)$cls['year_threshold'], 1) : '—' ?></strong></span>
+                        Fall &#8805;
+                        <strong><?= $cls['fall_threshold'] !== null ? number_format((float)$cls['fall_threshold'], 1) : '—' ?></strong>
+                    </span>
+                    <span class="text-muted">
+                        Year &#8805;
+                        <strong><?= $cls['year_threshold'] !== null ? number_format((float)$cls['year_threshold'], 1) : '—' ?></strong>
+                    </span>
                     <span class="fw-semibold">
                         Accuracy:
                         <span class="<?= $cls['accuracy'] >= 80 ? 'text-success' : ($cls['accuracy'] >= 50 ? 'text-warning' : 'text-danger') ?>">
@@ -168,6 +352,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                     </span>
                 </div>
             </div>
+
             <div class="card-body p-0">
                 <table class="table table-sm table-hover mb-0 align-middle" data-no-mgmt-sticky>
                     <thead class="table-light">
@@ -183,12 +368,16 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                             <th class="text-center">Status</th>
                         </tr>
                     </thead>
+
                     <tbody>
                         <?php $rank = 0; foreach ($cls['students'] as $s):
                             if ($s['status'] === 'none') continue;
                             $rank++;
-                            $isMale = strtolower($s['gender'] ?? '') === 'male';
+
-                            $rowBg  = match ($s['status']) {
+                            $genderNorm = strtolower(trim((string)($s['gender'] ?? '')));
+                            $isMale = in_array($genderNorm, ['male', 'm', 'boy'], true);
+
+                            $rowBg = match ($s['status']) {
                                 'confirmed' => 'table-success',
                                 'surprise'  => 'table-primary',
                                 'missed'    => 'table-warning',
@@ -203,9 +392,9 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                                     <?= $isMale ? 'M' : 'F' ?>
                                 </span>
                             </td>
-                            <td class="text-end small"><?= $s['fall_score']   !== null ? number_format($s['fall_score'],   1) : '<span class="text-muted">—</span>' ?></td>
+                            <td class="text-end small"><?= $s['fall_score'] !== null ? number_format($s['fall_score'], 1) : '<span class="text-muted">—</span>' ?></td>
                             <td class="text-end small"><?= $s['spring_score'] !== null ? number_format($s['spring_score'], 1) : '<span class="text-muted">—</span>' ?></td>
-                            <td class="text-end small fw-semibold"><?= $s['year_score']  !== null ? number_format($s['year_score'],  1) : '<span class="text-muted">—</span>' ?></td>
+                            <td class="text-end small fw-semibold"><?= $s['year_score'] !== null ? number_format($s['year_score'], 1) : '<span class="text-muted">—</span>' ?></td>
                             <td class="text-center small">
                                 <?= $s['predicted']
                                     ? '<span class="badge bg-primary">Yes</span>'
@@ -237,6 +426,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
             <div class="card-header bg-dark text-white fw-semibold py-2">
                 <i class="bi bi-bar-chart-fill me-2"></i>Prediction Accuracy Summary — <?= esc($selectedYear) ?>
             </div>
+
             <div class="card-body p-0">
                 <table class="table table-sm table-bordered mb-0 align-middle" data-no-mgmt-sticky>
                     <thead class="table-dark">
@@ -253,6 +443,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                             <th class="text-end pe-2">Year ≥</th>
                         </tr>
                     </thead>
+
                     <tbody>
                         <?php foreach ($classResults as $cls): ?>
                         <tr>
@@ -271,6 +462,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                         </tr>
                         <?php endforeach; ?>
                     </tbody>
+
                     <tfoot class="table-secondary fw-semibold">
                         <tr>
                             <td class="ps-2">Total</td>
@@ -292,7 +484,6 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
 
         <!-- Charts -->
         <div class="row g-4 mt-1 mb-4">
-            <!-- Grouped bar: predicted / actual / confirmed / surprises / missed per class -->
             <div class="col-12 col-lg-8">
                 <div class="border rounded p-3 h-100">
                     <div class="small fw-semibold text-muted mb-2">
@@ -301,7 +492,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                     <canvas id="chart-counts" style="max-height:280px;"></canvas>
                 </div>
             </div>
-            <!-- Bar: accuracy % per class + doughnut overall breakdown -->
+
             <div class="col-12 col-lg-4">
                 <div class="row g-3 h-100">
                     <div class="col-12">
@@ -312,6 +503,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                             <canvas id="chart-accuracy" style="max-height:130px;"></canvas>
                         </div>
                     </div>
+
                     <div class="col-12">
                         <div class="border rounded p-3">
                             <div class="small fw-semibold text-muted mb-2">
@@ -324,6 +516,93 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
             </div>
         </div>
 
+        <!-- Winner gender summary -->
+        <div class="card shadow-sm mt-2 mb-4">
+            <div class="card-header bg-dark text-white fw-semibold py-2">
+                <i class="bi bi-gender-ambiguous me-2"></i>Winner Gender Breakdown
+            </div>
+
+            <div class="card-body">
+                <div class="row g-3 align-items-stretch">
+                    <div class="col-12 col-lg-3">
+                        <div class="border rounded p-3 text-center h-100">
+                            <div class="text-muted small mb-1">Total Winners</div>
+                            <div class="display-6 fw-bold"><?= (int)$totalWinners ?></div>
+                            <div class="small text-muted">Actual year-end trophy winners</div>
+                        </div>
+                    </div>
+
+                    <div class="col-6 col-lg-3">
+                        <div class="border rounded p-3 text-center h-100">
+                            <div class="text-muted small mb-1">Boys</div>
+                            <div class="display-6 fw-bold text-primary"><?= (int)$totalWinnerBoys ?></div>
+                            <div class="fw-semibold"><?= number_format($winnerBoysPct, 1) ?>%</div>
+                        </div>
+                    </div>
+
+                    <div class="col-6 col-lg-3">
+                        <div class="border rounded p-3 text-center h-100">
+                            <div class="text-muted small mb-1">Girls</div>
+                            <div class="display-6 fw-bold" style="color:#E47AB0;"><?= (int)$totalWinnerGirls ?></div>
+                            <div class="fw-semibold"><?= number_format($winnerGirlsPct, 1) ?>%</div>
+                        </div>
+                    </div>
+
+                    <div class="col-12 col-lg-3">
+                        <div class="border rounded p-3 text-center h-100">
+                            <div class="text-muted small mb-1">Unknown / Other</div>
+                            <div class="display-6 fw-bold text-secondary"><?= (int)$totalWinnerOther ?></div>
+                            <div class="fw-semibold"><?= number_format($winnerOtherPct, 1) ?>%</div>
+                        </div>
+                    </div>
+                </div>
+
+                <div class="mt-3">
+                    <div class="progress" style="height:26px;">
+                        <?php if ($totalWinners > 0): ?>
+                            <div class="progress-bar bg-primary"
+                                 role="progressbar"
+                                 style="width: <?= $winnerBoysPct ?>%;"
+                                 aria-valuenow="<?= $winnerBoysPct ?>"
+                                 aria-valuemin="0"
+                                 aria-valuemax="100">
+                                Boys <?= number_format($winnerBoysPct, 1) ?>%
+                            </div>
+
+                            <div class="progress-bar"
+                                 role="progressbar"
+                                 style="width: <?= $winnerGirlsPct ?>%; background:#E47AB0;"
+                                 aria-valuenow="<?= $winnerGirlsPct ?>"
+                                 aria-valuemin="0"
+                                 aria-valuemax="100">
+                                Girls <?= number_format($winnerGirlsPct, 1) ?>%
+                            </div>
+
+                            <?php if ($totalWinnerOther > 0): ?>
+                                <div class="progress-bar bg-secondary"
+                                     role="progressbar"
+                                     style="width: <?= $winnerOtherPct ?>%;"
+                                     aria-valuenow="<?= $winnerOtherPct ?>"
+                                     aria-valuemin="0"
+                                     aria-valuemax="100">
+                                    Other <?= number_format($winnerOtherPct, 1) ?>%
+                                </div>
+                            <?php endif; ?>
+                        <?php else: ?>
+                            <div class="progress-bar bg-secondary"
+                                 role="progressbar"
+                                 style="width: 100%;"
+                                 aria-valuenow="0"
+                                 aria-valuemin="0"
+                                 aria-valuemax="100">
+                                No winners
+                            </div>
+                        <?php endif; ?>
+                    </div>
+                </div>
+            </div>
+        </div>
+
     </div><!-- /screen-only -->
 
     <!-- ══ PRINT VIEW ════════════════════════════════════════════════════════ -->
@@ -340,7 +619,6 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
             </p>
         </div>
 
-        <!-- All students flat table -->
         <h3 style="margin-bottom:4px;">Student Detail</h3>
         <table data-no-mgmt-sticky>
             <thead>
@@ -357,6 +635,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                     <th style="text-align:center;">Status</th>
                 </tr>
             </thead>
+
             <tbody>
                 <?php
                 $rank = 0;
@@ -364,7 +643,8 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                     foreach ($cls['students'] as $s):
                         if (!in_array($s['status'], ['confirmed', 'surprise'], true)) continue;
                         $rank++;
-                        $isMale = strtolower($s['gender'] ?? '') === 'male';
+                        $genderNorm = strtolower(trim((string)($s['gender'] ?? '')));
+                        $isMale = in_array($genderNorm, ['male', 'm', 'boy'], true);
                         $statusLabel = match ($s['status']) {
                             'confirmed' => '✓ Confirmed',
                             'surprise'  => '↑ Surprise',
@@ -378,18 +658,17 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                     <td><?= esc($cls['section_name']) ?></td>
                     <td><strong><?= esc($s['name']) ?></strong></td>
                     <td style="text-align:center;"><?= $isMale ? 'M' : 'F' ?></td>
-                    <td style="text-align:right;"><?= $s['fall_score']   !== null ? number_format($s['fall_score'],   1) : '—' ?></td>
+                    <td style="text-align:right;"><?= $s['fall_score'] !== null ? number_format($s['fall_score'], 1) : '—' ?></td>
                     <td style="text-align:right;"><?= $s['spring_score'] !== null ? number_format($s['spring_score'], 1) : '—' ?></td>
-                    <td style="text-align:right;font-weight:bold;"><?= $s['year_score']  !== null ? number_format($s['year_score'],  1) : '—' ?></td>
+                    <td style="text-align:right;font-weight:bold;"><?= $s['year_score'] !== null ? number_format($s['year_score'], 1) : '—' ?></td>
                     <td style="text-align:center;"><?= $s['predicted'] ? 'Yes' : 'No' ?></td>
-                    <td style="text-align:center;"><?= $s['actual']    ? 'Yes' : 'No' ?></td>
+                    <td style="text-align:center;"><?= $s['actual'] ? 'Yes' : 'No' ?></td>
                     <td style="text-align:center;" class="<?= $statusClass ?>"><?= $statusLabel ?></td>
                 </tr>
                 <?php endforeach; endforeach; ?>
             </tbody>
         </table>
 
-        <!-- Accuracy summary (new page) -->
         <div class="print-page-break"></div>
         <h3 style="margin-bottom:4px;">Prediction Accuracy Summary</h3>
         <table data-no-mgmt-sticky style="margin-bottom:14px;">
@@ -407,6 +686,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                     <th style="text-align:right;">Year ≥</th>
                 </tr>
             </thead>
+
             <tbody>
                 <?php foreach ($classResults as $cls): ?>
                 <tr>
@@ -423,6 +703,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
                 </tr>
                 <?php endforeach; ?>
             </tbody>
+
             <tfoot>
                 <tr>
                     <td style="font-weight:bold;">Total</td>
@@ -438,17 +719,18 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
             </tfoot>
         </table>
 
-        <!-- Charts as images (populated by JS before printing) -->
         <div class="print-page-break"></div>
         <h3 style="margin-bottom:6px;">Charts</h3>
+
         <div style="margin-bottom:14px;">
             <p style="font-size:10px;font-weight:bold;margin:0 0 4px;">Trophy Counts per Class</p>
-            <img id="print-chart-counts"   style="width:100%;max-height:220px;object-fit:contain;" src="" alt="">
+            <img id="print-chart-counts" style="width:100%;max-height:220px;object-fit:contain;" src="" alt="">
         </div>
+
         <div style="display:flex;gap:16px;margin-bottom:14px;">
             <div style="flex:1;">
                 <p style="font-size:10px;font-weight:bold;margin:0 0 4px;">Prediction Accuracy per Class</p>
-                <img id="print-chart-accuracy"  style="width:100%;max-height:160px;object-fit:contain;" src="" alt="">
+                <img id="print-chart-accuracy" style="width:100%;max-height:160px;object-fit:contain;" src="" alt="">
             </div>
             <div style="flex:1;">
                 <p style="font-size:10px;font-weight:bold;margin:0 0 4px;">Overall Outcome Breakdown</p>
@@ -456,8 +738,69 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
             </div>
         </div>
 
+        <div style="margin-top:10px;border:1px solid #bbb;padding:8px;">
+            <p style="font-size:11px;font-weight:bold;margin:0 0 6px;">Winner Gender Breakdown</p>
+
+            <table data-no-mgmt-sticky>
+                <thead>
+                    <tr>
+                        <th>Group</th>
+                        <th style="text-align:center;">Winners</th>
+                        <th style="text-align:center;">Percentage</th>
+                    </tr>
+                </thead>
+
+                <tbody>
+                    <tr>
+                        <td>Boys</td>
+                        <td style="text-align:center;"><?= (int)$totalWinnerBoys ?></td>
+                        <td style="text-align:center;"><?= number_format($winnerBoysPct, 1) ?>%</td>
+                    </tr>
+                    <tr>
+                        <td>Girls</td>
+                        <td style="text-align:center;"><?= (int)$totalWinnerGirls ?></td>
+                        <td style="text-align:center;"><?= number_format($winnerGirlsPct, 1) ?>%</td>
+                    </tr>
+                    <?php if ($totalWinnerOther > 0): ?>
+                    <tr>
+                        <td>Unknown / Other</td>
+                        <td style="text-align:center;"><?= (int)$totalWinnerOther ?></td>
+                        <td style="text-align:center;"><?= number_format($winnerOtherPct, 1) ?>%</td>
+                    </tr>
+                    <?php endif; ?>
+                </tbody>
+
+                <tfoot>
+                    <tr>
+                        <td style="font-weight:bold;">Total Winners</td>
+                        <td style="text-align:center;font-weight:bold;"><?= (int)$totalWinners ?></td>
+                        <td style="text-align:center;font-weight:bold;"><?= $totalWinners > 0 ? '100.0%' : '0.0%' ?></td>
+                    </tr>
+                </tfoot>
+            </table>
+        </div>
+
     </div><!-- /print-only -->
 
+    <!-- Winner sticker print area: names only, no header, no score, no class -->
+    <div id="winnerStickerPrintArea" class="winner-sticker-print-area">
+        <?php if (!empty($winnerStickerNames)): ?>
+            <?php foreach (array_chunk($winnerStickerNames, 20) as $chunk): ?>
+                <div class="sticker-page">
+                    <?php foreach ($chunk as $winnerName): ?>
+                        <div class="sticker-cell">
+                            <div class="sticker-name"><?= esc($winnerName) ?></div>
+                        </div>
+                    <?php endforeach; ?>
+
+                    <?php for ($i = 0, $remaining = 20 - count($chunk); $i < $remaining; $i++): ?>
+                        <div class="sticker-cell sticker-empty"></div>
+                    <?php endfor; ?>
+                </div>
+            <?php endforeach; ?>
+        <?php endif; ?>
+    </div>
+
     <?php endif; ?>
 </div>
 
@@ -473,6 +816,7 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
     $cSurprise  = [];
     $cMissed    = [];
     $cAccuracy  = [];
+
     foreach ($classResults as $cls) {
         $cLabels[]    = $cls['section_name'];
         $cPredicted[] = $cls['predicted_count'];
@@ -483,94 +827,105 @@ $overallAccuracy = $totalPredicted > 0 ? round($totalConfirmed / $totalPredicted
         $cAccuracy[]  = $cls['accuracy'];
     }
     ?>
-    var labels    = <?= json_encode($cLabels)    ?>;
+
+    var labels    = <?= json_encode($cLabels) ?>;
     var predicted = <?= json_encode($cPredicted) ?>;
-    var actual    = <?= json_encode($cActual)    ?>;
+    var actual    = <?= json_encode($cActual) ?>;
     var confirmed = <?= json_encode($cConfirmed) ?>;
-    var surprises = <?= json_encode($cSurprise)  ?>;
+    var surprises = <?= json_encode($cSurprise) ?>;
-    var missed    = <?= json_encode($cMissed)    ?>;
+    var missed    = <?= json_encode($cMissed) ?>;
-    var accuracy  = <?= json_encode($cAccuracy)  ?>;
+    var accuracy  = <?= json_encode($cAccuracy) ?>;
 
-    /* ── Chart 1: grouped bar – counts per class ── */
+    if (document.getElementById('chart-counts')) {
-    new Chart(document.getElementById('chart-counts'), {
+        new Chart(document.getElementById('chart-counts'), {
-        type: 'bar',
+            type: 'bar',
-        data: {
+            data: {
-            labels: labels,
+                labels: labels,
-            datasets: [
+                datasets: [
-                { label: 'Predicted', data: predicted, backgroundColor: '#4A90E2', borderRadius: 3 },
+                    { label: 'Predicted', data: predicted, backgroundColor: '#4A90E2', borderRadius: 3 },
-                { label: 'Actual',    data: actual,    backgroundColor: '#f0a500', borderRadius: 3 },
+                    { label: 'Actual',    data: actual,    backgroundColor: '#f0a500', borderRadius: 3 },
-                { label: 'Confirmed', data: confirmed, backgroundColor: '#28a745', borderRadius: 3 },
+                    { label: 'Confirmed', data: confirmed, backgroundColor: '#28a745', borderRadius: 3 },
-                { label: 'Surprises', data: surprises, backgroundColor: '#17a2b8', borderRadius: 3 },
+                    { label: 'Surprises', data: surprises, backgroundColor: '#17a2b8', borderRadius: 3 },
-                { label: 'Missed',    data: missed,    backgroundColor: '#fd7e14', borderRadius: 3 },
+                    { label: 'Missed',    data: missed,    backgroundColor: '#fd7e14', borderRadius: 3 },
-            ]
+                ]
-        },
+            },
-        options: {
+            options: {
-            responsive: true,
+                responsive: true,
-            maintainAspectRatio: true,
+                maintainAspectRatio: true,
-            plugins: { legend: { position: 'bottom' } },
+                plugins: { legend: { position: 'bottom' } },
-            scales: {
+                scales: {
-                x: { grid: { color: '#f0f0f0' } },
+                    x: { grid: { color: '#f0f0f0' } },
-                y: { beginAtZero: true, ticks: { stepSize: 1 }, grid: { color: '#f0f0f0' } }
+                    y: { beginAtZero: true, ticks: { stepSize: 1 }, grid: { color: '#f0f0f0' } }
-            }
-        }
-    });
-
-    /* ── Chart 2: accuracy % per class ── */
-    new Chart(document.getElementById('chart-accuracy'), {
-        type: 'bar',
-        data: {
-            labels: labels,
-            datasets: [{
-                label: 'Accuracy %',
-                data: accuracy,
-                backgroundColor: accuracy.map(function(a) {
-                    return a >= 80 ? '#28a745' : a >= 50 ? '#f0a500' : '#dc3545';
-                }),
-                borderRadius: 3
-            }]
-        },
-        options: {
-            responsive: true,
-            maintainAspectRatio: true,
-            plugins: { legend: { display: false } },
-            scales: {
-                y: {
-                    beginAtZero: true, max: 100,
-                    ticks: { callback: function(v) { return v + '%'; } },
-                    grid: { color: '#f0f0f0' }
                 }
             }
-        }
+        });
-    });
+    }
 
-    /* ── Chart 3: doughnut overall outcome breakdown ── */
+    if (document.getElementById('chart-accuracy')) {
-    new Chart(document.getElementById('chart-breakdown'), {
+        new Chart(document.getElementById('chart-accuracy'), {
-        type: 'doughnut',
+            type: 'bar',
-        data: {
+            data: {
-            labels: ['Confirmed', 'Surprises', 'Missed'],
+                labels: labels,
-            datasets: [{
+                datasets: [{
-                data: [<?= $totalConfirmed ?>, <?= $totalSurprise ?>, <?= $totalMissed ?>],
+                    label: 'Accuracy %',
-                backgroundColor: ['#28a745', '#17a2b8', '#fd7e14'],
+                    data: accuracy,
-                borderWidth: 2
+                    backgroundColor: accuracy.map(function(a) {
-            }]
+                        return a >= 80 ? '#28a745' : a >= 50 ? '#f0a500' : '#dc3545';
-        },
+                    }),
-        options: {
+                    borderRadius: 3
-            responsive: true,
+                }]
-            maintainAspectRatio: true,
+            },
-            plugins: {
+            options: {
-                legend: { position: 'bottom' },
+                responsive: true,
-                tooltip: {
+                maintainAspectRatio: true,
-                    callbacks: {
+                plugins: { legend: { display: false } },
-                        label: function(ctx) {
+                scales: {
-                            var total = ctx.dataset.data.reduce(function(a, b) { return a + b; }, 0);
+                    y: {
-                            var pct   = total > 0 ? Math.round(ctx.parsed / total * 100) : 0;
+                        beginAtZero: true,
-                            return ctx.label + ': ' + ctx.parsed + ' (' + pct + '%)';
+                        max: 100,
+                        ticks: {
+                            callback: function(v) {
+                                return v + '%';
+                            }
+                        },
+                        grid: { color: '#f0f0f0' }
+                    }
+                }
+            }
+        });
+    }
+
+    if (document.getElementById('chart-breakdown')) {
+        new Chart(document.getElementById('chart-breakdown'), {
+            type: 'doughnut',
+            data: {
+                labels: ['Confirmed', 'Surprises', 'Missed'],
+                datasets: [{
+                    data: [<?= $totalConfirmed ?>, <?= $totalSurprise ?>, <?= $totalMissed ?>],
+                    backgroundColor: ['#28a745', '#17a2b8', '#fd7e14'],
+                    borderWidth: 2
+                }]
+            },
+            options: {
+                responsive: true,
+                maintainAspectRatio: true,
+                plugins: {
+                    legend: { position: 'bottom' },
+                    tooltip: {
+                        callbacks: {
+                            label: function(ctx) {
+                                var total = ctx.dataset.data.reduce(function(a, b) {
+                                    return a + b;
+                                }, 0);
+                                var pct = total > 0 ? Math.round(ctx.parsed / total * 100) : 0;
+                                return ctx.label + ': ' + ctx.parsed + ' (' + pct + '%)';
+                            }
                         }
                     }
                 }
             }
-        }
+        });
-    });
+    }
 })();
 
 function captureCharts() {
@@ -579,20 +934,41 @@ function captureCharts() {
         'chart-accuracy':  'print-chart-accuracy',
         'chart-breakdown': 'print-chart-breakdown',
     };
+
     Object.keys(map).forEach(function(canvasId) {
         var canvas = document.getElementById(canvasId);
         var img    = document.getElementById(map[canvasId]);
-        if (canvas && img) img.src = canvas.toDataURL('image/png');
+
+        if (canvas && img) {
+            img.src = canvas.toDataURL('image/png');
+        }
     });
 }
 
 function printWithCharts() {
+    document.body.classList.remove('print-stickers-mode');
     captureCharts();
     window.print();
 }
 
-window.addEventListener('beforeprint', captureCharts);
+function printWinnerStickers() {
+    document.body.classList.add('print-stickers-mode');
+
+    setTimeout(function () {
+        window.print();
+
+        setTimeout(function () {
+            document.body.classList.remove('print-stickers-mode');
+        }, 1000);
+    }, 250);
+}
+
+window.addEventListener('beforeprint', function () {
+    if (!document.body.classList.contains('print-stickers-mode')) {
+        captureCharts();
+    }
+});
 </script>
 <?= $this->endSection() ?>
 
-<?= $this->endSection() ?>
+<?= $this->endSection() ?>

app/Views/grading/below_sixty.php

@@ -1,23 +1,71 @@
 <?= $this->extend('layout/management_layout') ?>
 <?= $this->section('content') ?>
 
+<?php
+// This page is Fall semester only.
+// Do not expose Whole Year mode here.
+$semester = 'fall';
+$actionSemester = 'fall';
+
+$schoolYear  = $schoolYear ?? '';
+$schoolYears = $schoolYears ?? [];
+
+if (empty($schoolYears) && $schoolYear !== '') {
+    $schoolYears = [$schoolYear];
+}
+?>
+
 <div class="container-fluid">
     <div class="wrapper below-sixty-wrapper">
         <h2 class="text-center mt-4 mb-4 below-sixty-title">Below 60 Summary</h2>
 
-        <?= $this->include('partials/academic_filter') ?>
+        <!-- School year filter only -->
+        <div class="card shadow-sm mb-3">
+            <div class="card-body py-3">
+                <form method="get"
+                      action="<?= site_url('grading/below-60') ?>"
+                      class="row g-2 align-items-end justify-content-center">
+
+                    <input type="hidden" name="semester" value="fall">
+
+                    <div class="col-12 col-sm-auto">
+                        <label class="form-label mb-1 small fw-semibold">School Year</label>
+
+                        <?php if (!empty($schoolYears)): ?>
+                            <select name="school_year"
+                                    class="form-select form-select-sm"
+                                    style="min-width:160px;">
+                                <?php foreach ($schoolYears as $yr): ?>
+                                    <option value="<?= esc($yr) ?>" <?= (string)$yr === (string)$schoolYear ? 'selected' : '' ?>>
+                                        <?= esc($yr) ?>
+                                    </option>
+                                <?php endforeach; ?>
+                            </select>
+                        <?php else: ?>
+                            <input type="text"
+                                   name="school_year"
+                                   class="form-control form-control-sm"
+                                   style="min-width:160px;"
+                                   value="<?= esc($schoolYear) ?>"
+                                   placeholder="2025-2026">
+                        <?php endif; ?>
+                    </div>
+
+                    <div class="col-12 col-sm-auto">
+                        <button type="submit" class="btn btn-sm btn-primary">
+                            <i class="bi bi-funnel me-1"></i>Apply
+                        </button>
+                    </div>
+                </form>
+            </div>
+        </div>
 
         <div class="d-flex justify-content-between align-items-center mb-3 flex-wrap gap-2">
             <div class="text-muted">
-                <?= !empty($isYearMode) ? 'Whole Year' : esc(ucfirst($semester ?? '')) ?> • <?= esc($schoolYear ?? '') ?>
+                Fall • <?= esc($schoolYear) ?>
             </div>
+
             <div class="d-flex gap-2">
-                <?php if (empty($isYearMode)): ?>
-                <a class="btn btn-outline-primary btn-sm"
-                   href="<?= site_url('grading/below-60/decisions?' . http_build_query(['semester' => $semester ?? '', 'school_year' => $schoolYear ?? ''])) ?>">
-                    Decisions
-                </a>
-                <?php endif; ?>
                 <?php if (!empty($canViewGrading)): ?>
                     <a class="btn btn-outline-secondary btn-sm" href="<?= base_url('grading') ?>">
                         Back to Grading
@@ -31,46 +79,44 @@
             if ($value === null || $value === '') {
                 return '—';
             }
+
             if (is_numeric($value)) {
                 return esc(number_format((float)$value, 2, '.', ''));
             }
+
             return esc($value);
         };
         ?>
 
         <?php if (empty($rows)): ?>
             <div class="alert alert-success text-center d-inline-block">
-                No students below 60 for this selection.
+                No students below 60 for Fall in <?= esc($schoolYear) ?>.
             </div>
         <?php else: ?>
             <div class="table-responsive below-sixty-table">
-                <table class="table table-bordered table-striped align-middle w-100 no-mgmt-sticky below-sixty-dt" data-no-mgmt-sticky data-no-dt-fixedheader>
+                <table class="table table-bordered table-striped align-middle w-100 no-mgmt-sticky below-sixty-dt"
+                       data-no-mgmt-sticky
+                       data-no-dt-fixedheader>
                     <thead class="table-light">
                         <tr>
                             <th>Student Name</th>
                             <th>Section</th>
-                            <?php if (!empty($isYearMode)): ?><th>Semester</th><?php endif; ?>
+                            <th class="text-center">Fall Score</th>
-                            <th>Hwk Avg</th>
-                            <th>Project Avg</th>
-                            <th>Participation</th>
-                            <th>Test Avg</th>
-                            <th>PTAP Score</th>
-                            <th>Attendance</th>
-                            <th>Midterm Score</th>
-                            <th><?= !empty($isYearMode) ? 'Semester Score' : (strcasecmp($semester ?? '', 'fall') === 0 ? '1st Semester Score' : 'Semester Score') ?></th>
-                            <?php if (empty($isYearMode)): ?>
                             <th>Status</th>
                             <th>Email Parent</th>
                             <th>Schedule Meeting</th>
-                            <?php endif; ?>
                         </tr>
                     </thead>
+
                     <tbody>
                         <?php foreach ($rows as $row): ?>
                             <?php
+                            // Fall-only page: use semester_score.
                             $scoreRaw = $row['semester_score'] ?? null;
                             $scoreVal = is_numeric($scoreRaw) ? (float)$scoreRaw : null;
+
                             $scoreClass = '';
+
                             if ($scoreVal !== null) {
                                 if ($scoreVal < 50) {
                                     $scoreClass = 'grade-red';
@@ -78,60 +124,104 @@
                                     $scoreClass = 'grade-orange';
                                 }
                             }
-                            $studentName = trim((string)($row['firstname'] ?? '') . ' ' . (string)($row['lastname'] ?? ''));
+
+                            $studentName  = trim((string)($row['firstname'] ?? '') . ' ' . (string)($row['lastname'] ?? ''));
+                            $studentLabel = $studentName !== '' ? $studentName : 'N/A';
+                            $isClosed     = ($row['status'] ?? 'Open') === 'Closed';
                             ?>
-                            <?php $isClosed = ($row['status'] ?? 'Open') === 'Closed'; ?>
+
-                            <?php $rowSemester = ucfirst(strtolower(trim((string)($row['semester'] ?? ($semester ?? ''))))); ?>
                             <tr class="<?= esc($scoreClass) ?>">
-                                <td><?= esc($studentName !== '' ? $studentName : 'N/A') ?></td>
+                                <td><?= esc($studentLabel) ?></td>
                                 <td><?= esc($row['class_section_name'] ?? '—') ?></td>
-                                <?php if (!empty($isYearMode)): ?>
+
-                                <td class="text-center"><?= esc($rowSemester) ?></td>
-                                <?php endif; ?>
-                                <td class="text-center"><?= $displayScore($row['homework_avg'] ?? null) ?></td>
-                                <td class="text-center"><?= $displayScore($row['project_avg'] ?? null) ?></td>
-                                <td class="text-center"><?= $displayScore($row['participation_score'] ?? null) ?></td>
-                                <td class="text-center"><?= $displayScore($row['test_avg'] ?? null) ?></td>
-                                <td class="text-center"><?= $displayScore($row['ptap_score'] ?? null) ?></td>
-                                <td class="text-center"><?= $displayScore($row['attendance_score'] ?? null) ?></td>
-                                <td class="text-center"><?= $displayScore($row['midterm_exam_score'] ?? null) ?></td>
-                                <td class="text-center"><?= $displayScore($row['semester_score'] ?? null) ?></td>
-                                <?php if (empty($isYearMode)): ?>
                                 <td class="text-center">
-                                    <form method="post" action="<?= site_url('grading/below-60/status') ?>" class="d-flex align-items-center gap-2 justify-content-center">
+                                    <div class="fw-semibold"><?= $displayScore($scoreRaw) ?></div>
+
+                                    <button type="button"
+                                            class="btn btn-outline-secondary btn-xs mt-1 btn-show-details"
+                                            style="font-size:0.72rem;padding:1px 7px;"
+                                            data-student-id="<?= (int)($row['student_id'] ?? 0) ?>"
+                                            data-student-name="<?= esc($studentLabel) ?>"
+                                            data-school-year="<?= esc((string)$schoolYear) ?>">
+                                        Details
+                                    </button>
+                                </td>
+
+                                <td class="text-center">
+                                    <form method="post"
+                                          action="<?= site_url('grading/below-60/status') ?>"
+                                          class="d-flex align-items-center gap-2 justify-content-center">
                                         <?= csrf_field() ?>
-                                        <input type="hidden" name="student_id" value="<?= esc((string)($row['student_id'] ?? '')) ?>">
+
-                                        <input type="hidden" name="semester" value="<?= esc((string)($semester ?? '')) ?>">
+                                        <input type="hidden"
-                                        <input type="hidden" name="school_year" value="<?= esc((string)($schoolYear ?? '')) ?>">
+                                               name="student_id"
-                                        <select name="status" class="form-select form-select-sm" style="width: 110px;">
+                                               value="<?= esc((string)($row['student_id'] ?? '')) ?>">
-                                            <option value="Open" <?= ($row['status'] ?? 'Open') === 'Open' ? 'selected' : '' ?>>Open</option>
+
-                                            <option value="Closed" <?= ($row['status'] ?? '') === 'Closed' ? 'selected' : '' ?>>Closed</option>
+                                        <input type="hidden"
+                                               name="semester"
+                                               value="fall">
+
+                                        <input type="hidden"
+                                               name="school_year"
+                                               value="<?= esc((string)$schoolYear) ?>">
+
+                                        <select name="status"
+                                                class="form-select form-select-sm"
+                                                style="width:110px;">
+                                            <option value="Open" <?= ($row['status'] ?? 'Open') === 'Open' ? 'selected' : '' ?>>
+                                                Open
+                                            </option>
+                                            <option value="Closed" <?= ($row['status'] ?? '') === 'Closed' ? 'selected' : '' ?>>
+                                                Closed
+                                            </option>
                                         </select>
-                                        <input type="text" name="note" class="form-control form-control-sm" style="width: 140px;" placeholder="Note (optional)" value="<?= esc((string)($row['note'] ?? '')) ?>">
+
-                                        <button type="submit" class="btn btn-sm btn-outline-secondary">Update</button>
+                                        <input type="text"
+                                               name="note"
+                                               class="form-control form-control-sm"
+                                               style="width:140px;"
+                                               placeholder="Note (optional)"
+                                               value="<?= esc((string)($row['note'] ?? '')) ?>">
+
+                                        <button type="submit" class="btn btn-sm btn-outline-secondary">
+                                            Update
+                                        </button>
                                     </form>
                                 </td>
+
                                 <td class="text-center">
                                     <?php if ($isClosed): ?>
-                                        <button type="button" class="btn btn-sm btn-secondary" disabled>Send Email</button>
+                                        <button type="button" class="btn btn-sm btn-secondary" disabled>
+                                            Send Email
+                                        </button>
                                     <?php else: ?>
                                         <a class="btn btn-sm btn-outline-primary"
-                                           href="<?= site_url('grading/below-60/email/edit?student_id=' . (int)($row['student_id'] ?? 0) . '&semester=' . rawurlencode((string)($semester ?? '')) . '&school_year=' . rawurlencode((string)($schoolYear ?? ''))) ?>">
+                                           href="<?= site_url('grading/below-60/email/edit?' . http_build_query([
+                                               'student_id'  => (int)($row['student_id'] ?? 0),
+                                               'semester'    => 'fall',
+                                               'school_year' => (string)$schoolYear,
+                                           ])) ?>">
                                             Send Email
                                         </a>
                                     <?php endif; ?>
                                 </td>
+
                                 <td class="text-center">
                                     <?php if ($isClosed): ?>
-                                        <button class="btn btn-sm btn-secondary" disabled>Schedule</button>
+                                        <button class="btn btn-sm btn-secondary" disabled>
+                                            Schedule
+                                        </button>
                                     <?php else: ?>
                                         <a class="btn btn-sm btn-outline-secondary"
-                                           href="<?= site_url('grading/below-60/schedule?student_id=' . (int)($row['student_id'] ?? 0) . '&semester=' . rawurlencode((string)($semester ?? '')) . '&school_year=' . rawurlencode((string)($schoolYear ?? ''))) ?>">
+                                           href="<?= site_url('grading/below-60/schedule?' . http_build_query([
+                                               'student_id'  => (int)($row['student_id'] ?? 0),
+                                               'semester'    => 'fall',
+                                               'school_year' => (string)$schoolYear,
+                                           ])) ?>">
                                             Schedule
                                         </a>
                                     <?php endif; ?>
                                 </td>
-                                <?php endif; ?>
                             </tr>
                         <?php endforeach; ?>
                     </tbody>
@@ -141,31 +231,251 @@
     </div>
 </div>
 
+<!-- Score details modal -->
+<div class="modal fade" id="detailsModal" tabindex="-1" aria-labelledby="detailsModalLabel" aria-hidden="true">
+    <div class="modal-dialog modal-lg modal-dialog-scrollable">
+        <div class="modal-content">
+            <div class="modal-header">
+                <h5 class="modal-title" id="detailsModalLabel">Score Details</h5>
+                <button type="button" class="btn-close" data-bs-dismiss="modal"></button>
+            </div>
+
+            <div class="modal-body" id="detailsModalBody">
+                <div class="text-center py-4">
+                    <div class="spinner-border text-primary" role="status"></div>
+                </div>
+            </div>
+
+            <div class="modal-footer">
+                <button type="button"
+                        class="btn btn-outline-secondary"
+                        data-bs-dismiss="modal">
+                    Close
+                </button>
+            </div>
+        </div>
+    </div>
+</div>
+
 <?= $this->endSection() ?>
 
 <?= $this->section('scripts') ?>
 <style>
-    .grade-orange td { background: #fff3cd !important; color: #8a5d00; font-weight: 600; }
+    .grade-orange td {
-    .grade-red td { background: #f8d7da !important; color: #842029; font-weight: 600; }
+        background: #fff3cd !important;
-    .below-sixty-wrapper { padding-top: 0.75rem; }
+        color: #8a5d00;
-    .below-sixty-title { position: relative; z-index: 1; margin-bottom: 1.25rem; }
+        font-weight: 600;
-    .below-sixty-table { margin-top: 0.75rem; }
+    }
+
+    .grade-red td {
+        background: #f8d7da !important;
+        color: #842029;
+        font-weight: 600;
+    }
+
+    .below-sixty-wrapper {
+        padding-top: 0.75rem;
+    }
+
+    .below-sixty-title {
+        position: relative;
+        z-index: 1;
+        margin-bottom: 1.25rem;
+    }
+
+    .below-sixty-table {
+        margin-top: 0.75rem;
+    }
+
+    .below-sixty-dt td {
+        vertical-align: top;
+    }
 </style>
+
 <script>
-    (function() {
+(function () {
-        if (!window.$ || !$.fn || !$.fn.DataTable) return;
+    if (window.$ && $.fn && $.fn.DataTable) {
-        $(function() {
+        $(function () {
             const table = $('.below-sixty-dt');
+
             if (!table.length) return;
+
             try {
-                const semesterOffset = <?= !empty($isYearMode) ? '1' : '0' ?>;
                 table.DataTable({
-                    order: [[8 + semesterOffset, 'asc']],
+                    order: [[2, 'asc']],
                     pageLength: 100,
-                    lengthMenu: [10, 25, 50, 100, 200]
+                    lengthMenu: [10, 25, 50, 100, 200],
+                    columnDefs: [
+                        { orderable: false, targets: [3, 4, 5] }
+                    ]
                 });
             } catch (_) {}
         });
-    })();
+    }
+
+    const detailsModal      = document.getElementById('detailsModal');
+    const detailsModalBody  = document.getElementById('detailsModalBody');
+    const detailsModalTitle = document.getElementById('detailsModalLabel');
+
+    const SCORE_LABELS = {
+        homework_avg:        'Homework Avg',
+        project_avg:         'Project Avg',
+        participation_score: 'Participation',
+        test_avg:            'Test Avg',
+        ptap_score:          'PTAP Score',
+        attendance_score:    'Attendance',
+        midterm_exam_score:  'Midterm Score',
+        final_exam_score:    'Final Exam',
+        semester_score:      'Semester Score'
+    };
+
+    const COMMENT_TYPE_LABELS = {
+        general:             'General',
+        attendance:          'Attendance',
+        attendance_comment:  'Attendance',
+        midterm:             'Midterm',
+        final:               'Final Exam',
+        ptap:                'PTAP'
+    };
+
+    function fmtScore(value) {
+        if (value === null || value === '' || value === undefined) return '—';
+
+        const parsed = parseFloat(value);
+
+        return isNaN(parsed) ? value : parsed.toFixed(2);
+    }
+
+    function esc(str) {
+        return String(str)
+            .replace(/&/g, '&amp;')
+            .replace(/</g, '&lt;')
+            .replace(/>/g, '&gt;')
+            .replace(/"/g, '&quot;');
+    }
+
+    function buildDetailsHtml(semesters) {
+        if (!semesters || semesters.length === 0) {
+            return '<div class="alert alert-warning mb-0">No score data found.</div>';
+        }
+
+        let html = '';
+
+        semesters.forEach(function (sem) {
+            html += '<h6 class="fw-bold mt-3 mb-2">' + esc(sem.semester || '') + ' Semester';
+
+            if (sem.class_section_name) {
+                html += ' <span class="text-muted fw-normal fs-6">— ' + esc(sem.class_section_name) + '</span>';
+            }
+
+            html += '</h6>';
+
+            html += '<table class="table table-sm table-bordered mb-2">';
+            html += '<thead class="table-light"><tr><th>Item</th><th class="text-center">Score</th></tr></thead><tbody>';
+
+            let hasScoreRow = false;
+
+            Object.entries(SCORE_LABELS).forEach(function (entry) {
+                const key   = entry[0];
+                const label = entry[1];
+                const value = sem[key];
+
+                if (value === null || value === '' || value === undefined) return;
+
+                const bold = key === 'semester_score' ? ' fw-bold' : '';
+
+                html += '<tr><td>' + esc(label) + '</td><td class="text-center' + bold + '">' + esc(fmtScore(value)) + '</td></tr>';
+                hasScoreRow = true;
+            });
+
+            if (!hasScoreRow) {
+                html += '<tr><td colspan="2" class="text-muted">No scores recorded.</td></tr>';
+            }
+
+            html += '</tbody></table>';
+
+            const comments = sem.comments || {};
+
+            const commentEntries = Object.entries(comments).filter(function (entry) {
+                return entry[1] && String(entry[1]).trim();
+            });
+
+            const seen    = {};
+            const deduped = [];
+
+            commentEntries.forEach(function (entry) {
+                const type  = entry[0];
+                const text  = String(entry[1]);
+                const label = COMMENT_TYPE_LABELS[type] || type;
+                const key   = label + '|' + text.trim();
+
+                if (!seen[key]) {
+                    seen[key] = true;
+                    deduped.push([label, text]);
+                }
+            });
+
+            if (deduped.length > 0) {
+                html += '<div class="mb-3">';
+                html += '<p class="fw-semibold mb-1" style="font-size:0.9rem;">Comments</p>';
+
+                deduped.forEach(function (entry) {
+                    const label = entry[0];
+                    const text  = entry[1];
+
+                    html += '<div class="mb-2 p-2 bg-light rounded border-start border-3 border-secondary">';
+                    html += '<span class="badge bg-secondary me-1" style="font-size:0.7rem;">' + esc(label) + '</span>';
+                    html += '<span class="text-dark" style="font-size:0.9rem;">' + esc(text).replace(/\n/g, '<br>') + '</span>';
+                    html += '</div>';
+                });
+
+                html += '</div>';
+            }
+        });
+
+        return html;
+    }
+
+    if (detailsModal) {
+        document.addEventListener('click', function (e) {
+            const btn = e.target.closest('.btn-show-details');
+
+            if (!btn) return;
+
+            const studentId   = btn.dataset.studentId;
+            const studentName = btn.dataset.studentName;
+            const schoolYear  = btn.dataset.schoolYear;
+
+            detailsModalTitle.textContent = studentName + ' — Score Details';
+            detailsModalBody.innerHTML = '<div class="text-center py-4"><div class="spinner-border text-primary" role="status"></div></div>';
+
+            bootstrap.Modal.getOrCreateInstance(detailsModal).show();
+
+            const url = '<?= site_url('grading/below-60/decisions/student-details') ?>'
+                + '?student_id=' + encodeURIComponent(studentId)
+                + '&school_year=' + encodeURIComponent(schoolYear);
+
+            fetch(url, {
+                headers: {
+                    'X-Requested-With': 'XMLHttpRequest'
+                }
+            })
+                .then(function (response) {
+                    return response.json();
+                })
+                .then(function (data) {
+                    if (data.error) {
+                        detailsModalBody.innerHTML = '<div class="alert alert-danger">' + esc(data.error) + '</div>';
+                        return;
+                    }
+
+                    detailsModalBody.innerHTML = buildDetailsHtml(data.semesters);
+                })
+                .catch(function () {
+                    detailsModalBody.innerHTML = '<div class="alert alert-danger">Failed to load details.</div>';
+                });
+        });
+    }
+})();
 </script>
-<?= $this->endSection() ?>
+<?= $this->endSection() ?>

app/Views/grading/below_sixty_decisions.php

@@ -1,25 +1,85 @@
 <?= $this->extend('layout/management_layout') ?>
 <?= $this->section('content') ?>
 
+<?php
+// This page is Whole Year only.
+// Do not expose semester filter here.
+$semester = 'year';
+
+$schoolYear  = $schoolYear ?? '';
+$schoolYears = $schoolYears ?? [];
+
+if (empty($schoolYears) && $schoolYear !== '') {
+    $schoolYears = [$schoolYear];
+}
+?>
+
 <div class="container-fluid">
     <div class="wrapper below-sixty-decisions-wrapper">
-        <h2 class="text-center mt-4 mb-4">Below 60 — Decisions</h2>
+        <h2 class="text-center mt-4 mb-4">School Year Decisions</h2>
 
-        <?= $this->include('partials/academic_filter') ?>
+        <!-- School year filter -->
+        <div class="card shadow-sm mb-3">
+            <div class="card-body py-3">
+                <form method="get"
+                      action="<?= site_url('grading/below-60/decisions') ?>"
+                      class="row g-2 align-items-end justify-content-center">
+
+                    <input type="hidden" name="semester" value="year">
+
+                    <div class="col-12 col-sm-auto">
+                        <label class="form-label mb-1 small fw-semibold">School Year</label>
+
+                        <?php if (!empty($schoolYears)): ?>
+                            <select name="school_year"
+                                    class="form-select form-select-sm"
+                                    style="min-width: 160px;">
+                                <?php foreach ($schoolYears as $yr): ?>
+                                    <option value="<?= esc($yr) ?>" <?= (string)$yr === (string)$schoolYear ? 'selected' : '' ?>>
+                                        <?= esc($yr) ?>
+                                    </option>
+                                <?php endforeach; ?>
+                            </select>
+                        <?php else: ?>
+                            <input type="text"
+                                   name="school_year"
+                                   class="form-control form-control-sm"
+                                   style="min-width: 160px;"
+                                   value="<?= esc($schoolYear) ?>"
+                                   placeholder="2025-2026">
+                        <?php endif; ?>
+                    </div>
+
+                    <div class="col-12 col-sm-auto">
+                        <button type="submit" class="btn btn-sm btn-primary">
+                            <i class="bi bi-funnel me-1"></i>Apply
+                        </button>
+                    </div>
+                </form>
+            </div>
+        </div>
 
         <div class="d-flex justify-content-between align-items-center mb-3 flex-wrap gap-2">
             <div class="text-muted">
-                <?= esc(ucfirst($semester ?? '')) ?> • <?= esc($schoolYear ?? '') ?>
+                Whole Year • <?= esc($schoolYear) ?>
             </div>
-            <div class="d-flex gap-2">
+
+            <div class="d-flex gap-2 flex-wrap">
                 <a class="btn btn-outline-secondary btn-sm"
-                   href="<?= site_url('grading/below-60?' . http_build_query(['semester' => $semester, 'school_year' => $schoolYear])) ?>">
+                   href="<?= site_url('grading/below-60?' . http_build_query([
+                       'semester'    => 'year',
+                       'school_year' => $schoolYear,
+                   ])) ?>">
                     ← Back to Below 60
                 </a>
+
                 <a class="btn btn-outline-primary btn-sm"
-                   href="<?= site_url('grading/decisions?' . http_build_query(['semester' => $semester, 'school_year' => $schoolYear])) ?>">
+                   href="<?= site_url('grading/decisions?' . http_build_query([
+                       'school_year' => $schoolYear,
+                   ])) ?>">
                     All Decisions
                 </a>
+
                 <?php if (!empty($canViewGrading)): ?>
                     <a class="btn btn-outline-secondary btn-sm" href="<?= base_url('grading') ?>">
                         Grading
@@ -34,6 +94,7 @@
                 <button type="button" class="btn-close" data-bs-dismiss="alert"></button>
             </div>
         <?php endif; ?>
+
         <?php if (!empty(session()->getFlashdata('error'))): ?>
             <div class="alert alert-danger alert-dismissible fade show" role="alert">
                 <?= esc(session()->getFlashdata('error')) ?>
@@ -47,7 +108,7 @@
             'Pass'                 => 'Pass',
             'Repeat Class'         => 'Repeat Class',
             'Make-up exam in fall' => 'Make-up exam in fall',
-            'Deferred decision'     => 'Deferred decision',
+            'Deferred decision'    => 'Deferred decision',
             'Expel'                => 'Expel',
             'Withdrawn'            => 'Withdrawn',
         ];
@@ -56,128 +117,136 @@
             'Pass'                 => 'success',
             'Repeat Class'         => 'danger',
             'Make-up exam in fall' => 'info',
-            'Deferred decision'     => 'info',
+            'Deferred decision'    => 'info',
             'Expel'                => 'danger',
             'Withdrawn'            => 'secondary',
         ];
 
         $displayScore = function ($value) {
-            if ($value === null || $value === '') return '—';
+            if ($value === null || $value === '') {
-            if (is_numeric($value)) return esc(number_format((float)$value, 2, '.', ''));
+                return '—';
+            }
+
+            if (is_numeric($value)) {
+                return esc(number_format((float)$value, 2, '.', ''));
+            }
+
             return esc($value);
         };
         ?>
 
         <?php if (empty($rows)): ?>
             <div class="alert alert-success text-center d-inline-block">
-                No students below 60 for this selection.
+                No students below 60 for the whole year in <?= esc($schoolYear) ?>.
             </div>
         <?php else: ?>
             <div class="table-responsive">
-                <table class="table table-bordered table-striped align-middle w-100 decisions-dt" data-no-mgmt-sticky data-no-dt-fixedheader>
+                <table class="table table-bordered table-striped align-middle w-100 decisions-dt"
+                       data-no-mgmt-sticky
+                       data-no-dt-fixedheader>
                     <thead class="table-light">
                         <tr>
                             <th style="min-width:160px">Student Name</th>
-                            <th>Section</th>
+                            <th>Class-Section</th>
-                            <th class="text-center">Score</th>
+                            <th class="text-center">Year Score</th>
                             <th style="min-width:300px">Comments / Rationale &amp; Decision</th>
                             <th style="min-width:150px" class="text-center">Below-60 Decision</th>
-                            <th style="min-width:130px" class="text-center">Final Decision</th>
-                            <th style="min-width:130px" class="text-center">Certificate</th>
                         </tr>
                     </thead>
+
                     <tbody>
                         <?php foreach ($rows as $row): ?>
                             <?php
-                            $scoreRaw        = $row['semester_score'] ?? null;
+                            // Whole-year page: use year_score only.
-                            $scoreVal        = is_numeric($scoreRaw) ? (float)$scoreRaw : null;
+                            // Do not fall back to semester_score, because this page should not display semester results.
-                            $rowClass        = '';
+                            $scoreRaw = $row['year_score'] ?? null;
+                            $scoreVal = is_numeric($scoreRaw) ? (float)$scoreRaw : null;
+
+                            $rowClass = '';
                             if ($scoreVal !== null) {
                                 $rowClass = $scoreVal < 50 ? 'grade-red' : 'grade-orange';
                             }
+
                             $studentName     = trim((string)($row['firstname'] ?? '') . ' ' . (string)($row['lastname'] ?? ''));
+                            $studentLabel    = $studentName !== '' ? $studentName : 'N/A';
                             $currentDecision = (string)($row['decision'] ?? '');
                             $currentNotes    = (string)($row['decision_notes'] ?? '');
                             $badge           = $decisionBadge[$currentDecision] ?? null;
                             ?>
+
                             <tr class="<?= esc($rowClass) ?>">
-                                <td><?= esc($studentName !== '' ? $studentName : 'N/A') ?></td>
+                                <td><?= esc($studentLabel) ?></td>
+
                                 <td><?= esc($row['class_section_name'] ?? '—') ?></td>
+
                                 <td class="text-center">
                                     <div class="fw-semibold"><?= $displayScore($scoreRaw) ?></div>
+
                                     <button type="button"
                                             class="btn btn-outline-secondary btn-xs mt-1 btn-show-details"
                                             style="font-size:0.72rem;padding:1px 7px;"
                                             data-student-id="<?= (int)($row['student_id'] ?? 0) ?>"
-                                            data-student-name="<?= esc($studentName !== '' ? $studentName : 'N/A') ?>"
+                                            data-student-name="<?= esc($studentLabel) ?>"
-                                            data-school-year="<?= esc((string)($schoolYear ?? '')) ?>">
+                                            data-school-year="<?= esc((string)$schoolYear) ?>">
                                         Details
                                     </button>
                                 </td>
+
                                 <td>
                                     <form method="post" action="<?= site_url('grading/below-60/decisions/save') ?>">
                                         <?= csrf_field() ?>
-                                        <input type="hidden" name="student_id"  value="<?= esc((string)($row['student_id'] ?? '')) ?>">
+
-                                        <input type="hidden" name="semester"    value="<?= esc((string)($semester ?? '')) ?>">
+                                        <input type="hidden"
-                                        <input type="hidden" name="school_year" value="<?= esc((string)($schoolYear ?? '')) ?>">
+                                               name="student_id"
+                                               value="<?= esc((string)($row['student_id'] ?? '')) ?>">
+
+                                        <input type="hidden"
+                                               name="semester"
+                                               value="year">
+
+                                        <input type="hidden"
+                                               name="school_year"
+                                               value="<?= esc((string)$schoolYear) ?>">
+
                                         <textarea name="notes"
                                                   class="form-control form-control-sm decision-notes"
                                                   rows="3"
                                                   placeholder="Add comments or rationale…"><?= esc($currentNotes) ?></textarea>
+
                                         <div class="d-flex gap-2 mt-2 align-items-center">
-                                            <select name="decision" class="form-select form-select-sm decision-select flex-grow-1">
+                                            <select name="decision"
+                                                    class="form-select form-select-sm decision-select flex-grow-1">
                                                 <?php foreach ($decisionOptions as $val => $label): ?>
                                                     <option value="<?= esc($val) ?>" <?= $currentDecision === $val ? 'selected' : '' ?>>
                                                         <?= esc($label) ?>
                                                     </option>
                                                 <?php endforeach; ?>
                                             </select>
-                                            <button type="submit" class="btn btn-sm btn-primary">Save</button>
+
+                                            <button type="submit" class="btn btn-sm btn-primary">
+                                                Save
+                                            </button>
                                         </div>
                                     </form>
                                 </td>
+
                                 <td class="text-center align-middle">
                                     <?php if ($currentDecision !== '' && $badge): ?>
-                                        <span class="badge bg-<?= esc($badge) ?> fs-6 px-3 py-2 d-block mb-2"><?= esc($currentDecision) ?></span>
+                                        <span class="badge bg-<?= esc($badge) ?> fs-6 px-3 py-2 d-block mb-2">
+                                            <?= esc($currentDecision) ?>
+                                        </span>
+
                                         <button type="button"
                                                 class="btn btn-sm btn-outline-primary btn-send-email"
                                                 data-student-id="<?= (int)($row['student_id'] ?? 0) ?>"
-                                                data-semester="<?= esc((string)($semester ?? '')) ?>"
+                                                data-semester="year"
-                                                data-school-year="<?= esc((string)($schoolYear ?? '')) ?>">
+                                                data-school-year="<?= esc((string)$schoolYear) ?>">
                                             Send Email
                                         </button>
                                     <?php else: ?>
                                         <span class="text-muted small">Pending</span>
                                     <?php endif; ?>
                                 </td>
-
-                                <?php
-                                // Final (consolidated) decision from student_decisions table
-                                $finalDecision = $row['consolidated_decision'] ?? null;
-                                $finalBadge    = $finalDecision !== null ? ($decisionBadge[$finalDecision] ?? 'secondary') : null;
-                                ?>
-                                <td class="text-center align-middle">
-                                    <?php if ($finalDecision !== null && $finalDecision !== ''): ?>
-                                        <span class="badge bg-<?= esc($finalBadge) ?> px-2 py-1"><?= esc($finalDecision) ?></span>
-                                    <?php else: ?>
-                                        <a href="<?= site_url('grading/decisions?' . http_build_query(['semester' => $semester ?? '', 'school_year' => $schoolYear ?? ''])) ?>"
-                                           class="text-muted small">Generate</a>
-                                    <?php endif; ?>
-                                </td>
-
-                                <?php $certNumber = (string)($row['certificate_number'] ?? ''); ?>
-                                <td class="text-center align-middle">
-                                    <?php if ($certNumber !== ''): ?>
-                                        <a href="<?= site_url('administrator/certificates/reprint/' . rawurlencode($certNumber)) ?>"
-                                           target="_blank"
-                                           class="font-monospace text-decoration-none fw-semibold"
-                                           title="Click to reprint certificate">
-                                            <?= esc($certNumber) ?>
-                                        </a>
-                                    <?php else: ?>
-                                        <span class="text-muted small">—</span>
-                                    <?php endif; ?>
-                                </td>
                             </tr>
                         <?php endforeach; ?>
                     </tbody>
@@ -186,9 +255,14 @@
 
             <div class="mt-3 mb-4 d-flex gap-2 flex-wrap">
                 <?php foreach ($decisionBadge as $label => $color): ?>
-                    <span class="badge bg-<?= esc($color) ?> px-3 py-2"><?= esc($label) ?></span>
+                    <span class="badge bg-<?= esc($color) ?> px-3 py-2">
+                        <?= esc($label) ?>
+                    </span>
                 <?php endforeach; ?>
-                <span class="text-muted small align-self-center ms-1">— decision colour key</span>
+
+                <span class="text-muted small align-self-center ms-1">
+                    — decision colour key
+                </span>
             </div>
         <?php endif; ?>
     </div>
@@ -202,13 +276,19 @@
                 <h5 class="modal-title" id="detailsModalLabel">Score Details</h5>
                 <button type="button" class="btn-close" data-bs-dismiss="modal"></button>
             </div>
+
             <div class="modal-body" id="detailsModalBody">
                 <div class="text-center py-4">
                     <div class="spinner-border text-primary" role="status"></div>
                 </div>
             </div>
+
             <div class="modal-footer">
-                <button type="button" class="btn btn-outline-secondary" data-bs-dismiss="modal">Close</button>
+                <button type="button"
+                        class="btn btn-outline-secondary"
+                        data-bs-dismiss="modal">
+                    Close
+                </button>
             </div>
         </div>
     </div>
@@ -232,29 +312,46 @@
                 <div class="alert alert-danger mb-0" id="emailModalErrorMsg"></div>
             </div>
 
-            <form id="decisionEmailForm" method="post"
+            <form id="decisionEmailForm"
+                  method="post"
                   action="<?= site_url('grading/below-60/decisions/email') ?>"
                   style="display:none;">
                 <?= csrf_field() ?>
-                <input type="hidden" name="student_id"  id="emailStudentId">
+
-                <input type="hidden" name="semester"    id="emailSemester">
+                <input type="hidden" name="student_id" id="emailStudentId">
+                <input type="hidden" name="semester" id="emailSemester" value="year">
                 <input type="hidden" name="school_year" id="emailSchoolYear">
-                <input type="hidden" name="html"        id="emailHtmlHidden">
+                <input type="hidden" name="html" id="emailHtmlHidden">
 
                 <div class="modal-body">
                     <div class="mb-3">
                         <label class="form-label fw-semibold" for="emailSubjectInput">Subject</label>
-                        <input type="text" class="form-control" id="emailSubjectInput" name="subject" required>
+                        <input type="text"
+                               class="form-control"
+                               id="emailSubjectInput"
+                               name="subject"
+                               required>
                     </div>
+
                     <div class="mb-1">
                         <label class="form-label fw-semibold">Body</label>
                     </div>
+
                     <textarea id="decisionEmailEditor" rows="18"></textarea>
                 </div>
 
                 <div class="modal-footer">
-                    <button type="button" class="btn btn-outline-secondary" data-bs-dismiss="modal">Cancel</button>
+                    <button type="button"
-                    <button type="submit" class="btn btn-primary" id="emailSendBtn">Send Email</button>
+                            class="btn btn-outline-secondary"
+                            data-bs-dismiss="modal">
+                        Cancel
+                    </button>
+
+                    <button type="submit"
+                            class="btn btn-primary"
+                            id="emailSendBtn">
+                        Send Email
+                    </button>
                 </div>
             </form>
         </div>
@@ -265,34 +362,55 @@
 
 <?= $this->section('scripts') ?>
 <style>
-    .grade-orange td { background: #fff3cd !important; color: #8a5d00; }
+    .grade-orange td {
-    .grade-red td    { background: #f8d7da !important; color: #842029; }
+        background: #fff3cd !important;
-    .decision-notes  { font-size: 0.85rem; resize: vertical; min-height: 60px; }
+        color: #8a5d00;
-    .decisions-dt td { vertical-align: top; }
+    }
+
+    .grade-red td {
+        background: #f8d7da !important;
+        color: #842029;
+    }
+
+    .decision-notes {
+        font-size: 0.85rem;
+        resize: vertical;
+        min-height: 60px;
+    }
+
+    .decisions-dt td {
+        vertical-align: top;
+    }
 </style>
+
 <script src="<?= base_url('assets/tinymce/tinymce.min.js') ?>"></script>
+
 <script>
 (function () {
     // DataTable
     if (window.$ && $.fn && $.fn.DataTable) {
         $(function () {
             const tbl = $('.decisions-dt');
+
             if (!tbl.length) return;
+
             try {
                 tbl.DataTable({
                     order: [[2, 'asc']],
                     pageLength: 100,
                     lengthMenu: [10, 25, 50, 100, 200],
-                    columnDefs: [{ orderable: false, targets: [3] }]
+                    columnDefs: [
+                        { orderable: false, targets: [3] }
+                    ]
                 });
             } catch (_) {}
         });
     }
 
     // ── Details modal ────────────────────────────────────────────
-    const detailsModal     = document.getElementById('detailsModal');
+    const detailsModal      = document.getElementById('detailsModal');
-    const detailsModalBody = document.getElementById('detailsModalBody');
+    const detailsModalBody  = document.getElementById('detailsModalBody');
-    const detailsModalTitle= document.getElementById('detailsModalLabel');
+    const detailsModalTitle = document.getElementById('detailsModalLabel');
 
     const SCORE_LABELS = {
         homework_avg:        'Homework Avg',
@@ -303,7 +421,7 @@
         attendance_score:    'Attendance',
         midterm_exam_score:  'Midterm Score',
         final_exam_score:    'Final Exam',
-        semester_score:      'Semester Score',
+        semester_score:      'Semester Score'
     };
 
     const COMMENT_TYPE_LABELS = {
@@ -312,12 +430,14 @@
         attendance_comment:  'Attendance',
         midterm:             'Midterm',
         final:               'Final Exam',
-        ptap:                'PTAP',
+        ptap:                'PTAP'
     };
 
     function fmtScore(v) {
         if (v === null || v === '' || v === undefined) return '—';
+
         const n = parseFloat(v);
+
         return isNaN(n) ? v : n.toFixed(2);
     }
 
@@ -333,78 +453,110 @@
         if (!semesters || semesters.length === 0) {
             return '<div class="alert alert-warning mb-0">No score data found.</div>';
         }
+
         let html = '';
+
         semesters.forEach(function (sem) {
             html += '<h6 class="fw-bold mt-3 mb-2">' + esc(sem.semester || '') + ' Semester';
-            if (sem.class_section_name) html += ' <span class="text-muted fw-normal fs-6">— ' + esc(sem.class_section_name) + '</span>';
+
+            if (sem.class_section_name) {
+                html += ' <span class="text-muted fw-normal fs-6">— ' + esc(sem.class_section_name) + '</span>';
+            }
+
             html += '</h6>';
 
-            // Scores table
             html += '<table class="table table-sm table-bordered mb-2">';
             html += '<thead class="table-light"><tr><th>Item</th><th class="text-center">Score</th></tr></thead><tbody>';
+
             let hasRow = false;
+
             Object.entries(SCORE_LABELS).forEach(function ([key, label]) {
                 const v = sem[key];
+
                 if (v === null || v === '' || v === undefined) return;
+
                 const bold = key === 'semester_score' ? ' fw-bold' : '';
-                html += '<tr><td>' + label + '</td><td class="text-center' + bold + '">' + fmtScore(v) + '</td></tr>';
+
+                html += '<tr><td>' + esc(label) + '</td><td class="text-center' + bold + '">' + esc(fmtScore(v)) + '</td></tr>';
                 hasRow = true;
             });
-            if (!hasRow) html += '<tr><td colspan="2" class="text-muted">No scores recorded.</td></tr>';
+
+            if (!hasRow) {
+                html += '<tr><td colspan="2" class="text-muted">No scores recorded.</td></tr>';
+            }
+
             html += '</tbody></table>';
 
-            // Comments section
             const comments = sem.comments || {};
-            const commentEntries = Object.entries(comments).filter(function ([, v]) { return v && v.trim(); });
 
-            // Deduplicate attendance + attendance_comment (show once)
+            const commentEntries = Object.entries(comments).filter(function ([, v]) {
+                return v && String(v).trim();
+            });
+
             const seen = {};
             const deduped = [];
+
             commentEntries.forEach(function ([type, text]) {
                 const label = COMMENT_TYPE_LABELS[type] || type;
-                const key = label + '|' + text.trim();
+                const key = label + '|' + String(text).trim();
-                if (!seen[key]) { seen[key] = true; deduped.push([label, text]); }
+
+                if (!seen[key]) {
+                    seen[key] = true;
+                    deduped.push([label, String(text)]);
+                }
             });
 
             if (deduped.length > 0) {
                 html += '<div class="mb-3">';
                 html += '<p class="fw-semibold mb-1" style="font-size:0.9rem;">Comments</p>';
+
                 deduped.forEach(function ([label, text]) {
                     html += '<div class="mb-2 p-2 bg-light rounded border-start border-3 border-secondary">';
                     html += '<span class="badge bg-secondary me-1" style="font-size:0.7rem;">' + esc(label) + '</span>';
                     html += '<span class="text-dark" style="font-size:0.9rem;">' + esc(text).replace(/\n/g, '<br>') + '</span>';
                     html += '</div>';
                 });
+
                 html += '</div>';
             }
         });
+
         return html;
     }
 
     if (detailsModal) {
         document.addEventListener('click', function (e) {
             const btn = e.target.closest('.btn-show-details');
+
             if (!btn) return;
 
-            const studentId  = btn.dataset.studentId;
+            const studentId   = btn.dataset.studentId;
-            const studentName= btn.dataset.studentName;
+            const studentName = btn.dataset.studentName;
-            const schoolYear = btn.dataset.schoolYear;
+            const schoolYear  = btn.dataset.schoolYear;
 
             detailsModalTitle.textContent = studentName + ' — Score Details';
             detailsModalBody.innerHTML = '<div class="text-center py-4"><div class="spinner-border text-primary" role="status"></div></div>';
+
             bootstrap.Modal.getOrCreateInstance(detailsModal).show();
 
             const url = '<?= site_url('grading/below-60/decisions/student-details') ?>'
                 + '?student_id=' + encodeURIComponent(studentId)
                 + '&school_year=' + encodeURIComponent(schoolYear);
 
-            fetch(url, { headers: { 'X-Requested-With': 'XMLHttpRequest' } })
+            fetch(url, {
-                .then(function (r) { return r.json(); })
+                headers: {
+                    'X-Requested-With': 'XMLHttpRequest'
+                }
+            })
+                .then(function (r) {
+                    return r.json();
+                })
                 .then(function (data) {
                     if (data.error) {
-                        detailsModalBody.innerHTML = '<div class="alert alert-danger">' + data.error + '</div>';
+                        detailsModalBody.innerHTML = '<div class="alert alert-danger">' + esc(data.error) + '</div>';
                         return;
                     }
+
                     detailsModalBody.innerHTML = buildDetailsHtml(data.semesters);
                 })
                 .catch(function () {
@@ -429,8 +581,8 @@
 
     function showPane(which) {
         loadingPane.style.display = which === 'loading' ? '' : 'none';
-        errorPane.style.display   = which === 'error'   ? '' : 'none';
+        errorPane.style.display   = which === 'error' ? '' : 'none';
-        form.style.display        = which === 'form'    ? '' : 'none';
+        form.style.display        = which === 'form' ? '' : 'none';
     }
 
     function destroyEditor() {
@@ -441,6 +593,7 @@
 
     function initEditor(html) {
         if (!window.tinymce) return;
+
         tinymce.init({
             selector: '#decisionEmailEditor',
             base_url: '<?= base_url('assets/tinymce') ?>',
@@ -462,65 +615,76 @@
             setup(editor) {
                 editor.on('init', function () {
                     editor.setContent(html);
-                    if (htmlHidden) htmlHidden.value = html;
+
+                    if (htmlHidden) {
+                        htmlHidden.value = html;
+                    }
                 });
+
                 editor.on('keyup change undo redo SetContent', function () {
-                    if (htmlHidden) htmlHidden.value = editor.getContent({ format: 'html' });
+                    if (htmlHidden) {
+                        htmlHidden.value = editor.getContent({ format: 'html' });
+                    }
                 });
             }
         });
     }
 
-    // Sync hidden field before form submit
     form.addEventListener('submit', function () {
         if (window.tinymce) {
             const ed = tinymce.get('decisionEmailEditor');
-            if (ed && htmlHidden) htmlHidden.value = ed.getContent({ format: 'html' });
+
+            if (ed && htmlHidden) {
+                htmlHidden.value = ed.getContent({ format: 'html' });
+            }
         }
     });
 
-    // Destroy editor when modal closes
     modal.addEventListener('hidden.bs.modal', function () {
         destroyEditor();
         showPane('loading');
     });
 
-    // Open modal when "Send Email" is clicked
     document.addEventListener('click', function (e) {
         const btn = e.target.closest('.btn-send-email');
+
         if (!btn) return;
 
         const studentId  = btn.dataset.studentId;
-        const semester   = btn.dataset.semester;
         const schoolYear = btn.dataset.schoolYear;
 
         studentIdIn.value  = studentId;
-        semesterIn.value   = semester;
+        semesterIn.value   = 'year';
         schoolYearIn.value = schoolYear;
 
         showPane('loading');
 
-        // Open the modal immediately (shows spinner)
         const bsModal = bootstrap.Modal.getOrCreateInstance(modal);
         bsModal.show();
 
-        // Fetch the pre-rendered email
         const url = '<?= site_url('grading/below-60/decisions/email/preview') ?>'
             + '?student_id=' + encodeURIComponent(studentId)
-            + '&semester='   + encodeURIComponent(semester)
+            + '&semester=year'
-            + '&school_year='+ encodeURIComponent(schoolYear);
+            + '&school_year=' + encodeURIComponent(schoolYear);
 
-        fetch(url, { headers: { 'X-Requested-With': 'XMLHttpRequest' } })
+        fetch(url, {
-            .then(function (res) { return res.json(); })
+            headers: {
+                'X-Requested-With': 'XMLHttpRequest'
+            }
+        })
+            .then(function (res) {
+                return res.json();
+            })
             .then(function (data) {
                 if (data.error) {
                     errorMsg.textContent = data.error;
                     showPane('error');
                     return;
                 }
+
                 subjectInput.value = data.subject || '';
-                // Reset textarea content before init
                 document.getElementById('decisionEmailEditor').value = data.html || '';
+
                 showPane('form');
                 initEditor(data.html || '');
             })
@@ -531,4 +695,4 @@
     });
 })();
 </script>
-<?= $this->endSection() ?>
+<?= $this->endSection() ?>