fix db delete data issue

.gitea/workflows/tests.yml

@@ -0,0 +1,171 @@
+name: Tests
+
+on:
+  push:
+    branches:
+      - develop
+  pull_request:
+  workflow_dispatch:
+
+jobs:
+  phpunit:
+    name: PHPUnit
+    runs-on: ubuntu-latest
+
+    services:
+      mysql:
+        image: mariadb:10.11
+        env:
+          MYSQL_ROOT_PASSWORD: root
+          MYSQL_DATABASE: alrahma_test
+          MYSQL_USER: alrahma
+          MYSQL_PASSWORD: alrahma
+        ports:
+          - 3306:3306
+        options: >-
+          --health-cmd="mariadb-admin ping -h 127.0.0.1 -uroot -proot"
+          --health-interval=10s
+          --health-timeout=5s
+          --health-retries=10
+
+    env:
+      GIT_SSL_NO_VERIFY: 'true'
+      CI_ENVIRONMENT: testing
+      TEST_DB_HOST: mysql
+      TEST_DB_PORT: 3306
+      TEST_DB_NAME: alrahma_test
+      TEST_DB_USER: alrahma
+      TEST_DB_PASSWORD: alrahma
+      database.tests.hostname: mysql
+      database.tests.database: alrahma_test
+      database.tests.username: alrahma
+      database.tests.password: alrahma
+      database.tests.DBDriver: MySQLi
+      database.tests.DBPrefix: ''
+      database.tests.port: 3306
+      JWT_SECRET: test-secret-for-ci
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.2'
+          extensions: dom, gd, intl, mbstring, mysqli, zip
+          coverage: xdebug
+
+      - name: Validate Composer config
+        run: composer validate --no-check-publish --strict
+
+      - name: Install dependencies
+        run: composer install --no-interaction --prefer-dist --no-progress
+
+      - name: Prepare writable directories
+        run: |
+          mkdir -p \
+            writable/cache/testing \
+            writable/logs \
+            writable/session \
+            writable/uploads \
+            writable/debugbar
+          chown -R "$(id -u):$(id -g)" writable || true
+          chmod -R u+rwX,g+rwX writable
+
+      - name: Verify writable cache
+        run: |
+          test -d writable/cache/testing
+          test -w writable/cache/testing
+          touch writable/cache/testing/.ci-write-test
+          rm writable/cache/testing/.ci-write-test
+
+      - name: Verify database configuration
+        run: |
+          php -d variables_order=EGPCS -r '
+            define("FCPATH", __DIR__ . "/public/");
+            define("ENVIRONMENT", getenv("CI_ENVIRONMENT") ?: "production");
+            require "app/Config/Paths.php";
+            $paths = new Config\Paths();
+            require $paths->systemDirectory . "/Boot.php";
+            CodeIgniter\Boot::bootConsole($paths);
+
+            $database = config("Database");
+            $dbConfig = $database->{$database->defaultGroup};
+
+            echo "Environment: " . ENVIRONMENT . PHP_EOL;
+            echo "DB group: " . $database->defaultGroup . PHP_EOL;
+            echo "DB host: " . $dbConfig["hostname"] . PHP_EOL;
+            echo "DB port: " . $dbConfig["port"] . PHP_EOL;
+            echo "DB name: " . $dbConfig["database"] . PHP_EOL;
+          '
+
+      - name: Wait for MariaDB
+        run: |
+          for attempt in $(seq 1 30); do
+            php -r '
+              mysqli_report(MYSQLI_REPORT_OFF);
+              $db = @new mysqli(
+                  getenv("TEST_DB_HOST"),
+                  getenv("TEST_DB_USER"),
+                  getenv("TEST_DB_PASSWORD"),
+                  getenv("TEST_DB_NAME"),
+                  (int) getenv("TEST_DB_PORT")
+              );
+
+              exit($db->connect_errno === 0 ? 0 : 1);
+            ' && break
+
+            echo "Waiting for MariaDB, attempt ${attempt}/30"
+            sleep 2
+          done
+
+          php -r '
+            mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
+
+            new mysqli(
+                getenv("TEST_DB_HOST"),
+                getenv("TEST_DB_USER"),
+                getenv("TEST_DB_PASSWORD"),
+                getenv("TEST_DB_NAME"),
+                (int) getenv("TEST_DB_PORT")
+            );
+
+            echo "MariaDB connection successful\n";
+          '
+
+      - name: Import baseline schema
+        run: php tests/_support/import_baseline.php Database_Dump/u280815660_school_12072025
+
+      - name: Run database migrations
+        run: |
+          php -d variables_order=EGPCS spark migrate --all --no-interaction
+          php -r '
+            mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
+
+            $db = new mysqli(
+                getenv("TEST_DB_HOST"),
+                getenv("TEST_DB_USER"),
+                getenv("TEST_DB_PASSWORD"),
+                getenv("TEST_DB_NAME"),
+                (int) getenv("TEST_DB_PORT")
+            );
+
+            $tables = [];
+            $result = $db->query("SHOW TABLES");
+            while ($row = $result->fetch_array(MYSQLI_NUM)) {
+                $tables[$row[0]] = true;
+            }
+
+            foreach (["users", "students", "attendance_data"] as $table) {
+                if (!isset($tables[$table])) {
+                    fwrite(STDERR, "Migration schema missing required table: {$table}\n");
+                    exit(1);
+                }
+            }
+
+            echo "Migration schema verified\n";
+          '
+
+      - name: Run tests
+        run: composer test

app/Config/Cache.php

@@ -168,4 +168,12 @@ class Cache extends BaseConfig
         'redis'     => RedisHandler::class,
         'wincache'  => WincacheHandler::class,
     ];
+
+    public function __construct()
+    {
+        if (ENVIRONMENT === 'testing') {
+            $this->storePath         = WRITEPATH . 'cache/testing/';
+            $this->file['storePath'] = WRITEPATH . 'cache/testing/';
+        }
+    }
 }

app/Config/Constants.php

@@ -25,6 +25,19 @@ defined('APP_NAMESPACE') || define('APP_NAMESPACE', 'App');
  */
 defined('COMPOSER_PATH') || define('COMPOSER_PATH', ROOTPATH . 'vendor/autoload.php');
 
+/*
+ | --------------------------------------------------------------------------
+ | Test Support Path
+ | --------------------------------------------------------------------------
+ |
+ | Spark may be run with CI_ENVIRONMENT=testing before CodeIgniter's PHPUnit
+ | bootstrap has had a chance to define SUPPORTPATH. The framework autoloader
+ | expects this constant when that environment flag is present.
+ */
+if (! defined('SUPPORTPATH') && defined('TESTPATH') && is_dir(TESTPATH . '_support')) {
+    define('SUPPORTPATH', realpath(TESTPATH . '_support') . DIRECTORY_SEPARATOR);
+}
+
 /*
  |--------------------------------------------------------------------------
  | Timing Constants

app/Config/Database.php

@@ -16,6 +16,10 @@ class Database extends Config
     {
         parent::__construct();
 
+        if (ENVIRONMENT === 'testing') {
+            $this->defaultGroup = 'tests';
+        }
+
         $this->default = [
             'DSN'      => '',
             'hostname' => env('database.default.hostname'),
@@ -38,22 +42,22 @@ class Database extends Config
 
         $this->tests = [
             'DSN'      => '',
-            'hostname' => env('database.tests.hostname', env('database.default.hostname')),
+            'hostname' => env('TEST_DB_HOST', env('database.tests.hostname', env('database.default.hostname'))),
-            'username' => env('database.tests.username', env('database.default.username')),
+            'username' => env('TEST_DB_USER', env('database.tests.username', env('database.default.username'))),
-            'password' => env('database.tests.password', env('database.default.password')),
+            'password' => env('TEST_DB_PASSWORD', env('database.tests.password', env('database.default.password'))),
-            'database' => env('database.tests.database', env('database.default.database')),
+            'database' => env('TEST_DB_NAME', env('database.tests.database', 'alrahma_test')),
             'DBDriver' => env('database.tests.DBDriver', env('database.default.DBDriver', 'MySQLi')),
-            'DBPrefix' => env('database.tests.DBPrefix', 'db_'),
+            'DBPrefix' => env('database.tests.DBPrefix', ''),
             'pConnect' => false,
             'DBDebug'  => true,
-            'charset'  => 'utf8',
+            'charset'  => 'utf8mb4',
-            'DBCollat' => 'utf8_general_ci',
+            'DBCollat' => 'utf8mb4_general_ci',
             'swapPre'  => '',
             'encrypt'  => false,
             'compress' => false,
             'strictOn' => false,
             'failover' => [],
-            'port'     => (int) env('database.tests.port', env('database.default.port', 3306)),
+            'port'     => (int) env('TEST_DB_PORT', env('database.tests.port', env('database.default.port', 3306))),
         ];
     }
 }

app/Config/Filters.php

@@ -31,6 +31,7 @@ class Filters extends BaseConfig
         'cleanupScheduler'  => \App\Filters\CleanupScheduler::class,
         'permission'        => \App\Filters\PermissionFilter::class, 
         'timezone'          => \App\Filters\TimezoneFilter::class,
+        'schoolYear'        => \App\Filters\RequireSchoolYearFilter::class,
     ];
 
 

app/Config/Routes.php

@@ -743,11 +743,6 @@ $routes->get('payments/getByParent/(:num)', 'View\PaymentController::getByParent
 $routes->get('payments/create', 'View\PaymentController::create');
 $routes->post('payments/updateBalance/(:num)', 'View\PaymentController::updateBalance/$1');
 
-// Web View Routes for Payment Transactions
-$routes->get('payment_transactions/getByPayment/(:num)', 'View\PaymentTransactionController::getByPayment/$1');
-$routes->get('payment_transactions/create', 'View\PaymentTransactionController::create');
-$routes->post('payment_transactions/updateStatus/(:num)', 'View\PaymentTransactionController::updateStatus/$1');
-
 // Routes for payment pages
 $routes->get('payments/view/(:num)', 'View\PaymentController::viewPayment/$1');
 $routes->get('payment/get_enrolled_students/(:num)', 'View\PaymentController::getEnrolledStudents/$1', ['filter' => 'auth:view_invoice|view_payment|view_financial_reports|administrator|administrative staff|principal']);
@@ -1037,6 +1032,7 @@ $routes->match(['get', 'post'], 'families/import-legacy', 'FamilyController::imp
 
 // Admin page (protect with your auth/permission)
 $routes->group('family',  static function ($routes) {
+    $routes->get('', 'View\FamilyAdminController::index');
     $routes->get('index', 'View\FamilyAdminController::index');
     $routes->get('search', 'View\FamilyAdminController::search');
     $routes->get('card', 'View\FamilyAdminController::card');
@@ -1137,6 +1133,22 @@ $routes->post('/configuration/addConfig', 'View\ConfigurationController::addConf
 $routes->match(['get', 'post'], '/configuration/editConfig/(:num)', 'View\ConfigurationController::editConfig/$1');
 $routes->post('/configuration/deleteConfig/(:num)', 'View\ConfigurationController::deleteConfig/$1');
 
+// School-year management
+$routes->group('administrator/school-years', ['filter' => 'auth:admin'], static function ($routes) {
+    $routes->get('', 'Administrator\SchoolYearController::index');
+    $routes->post('store', 'Administrator\SchoolYearController::store');
+    $routes->post('(:num)/update', 'Administrator\SchoolYearController::update/$1');
+    $routes->post('(:num)/activate', 'Administrator\SchoolYearController::activate/$1');
+    $routes->post('(:num)/delete-draft', 'Administrator\SchoolYearController::deleteDraft/$1');
+    $routes->post('(:num)/archive', 'Administrator\SchoolYearController::archive/$1');
+    $routes->post('(:num)/reopen', 'Administrator\SchoolYearController::reopen/$1');
+    $routes->get('(:num)/closing/preview', 'Administrator\SchoolYearClosingController::preview/$1');
+    $routes->post('(:num)/closing/start', 'Administrator\SchoolYearClosingController::start/$1');
+    $routes->post('(:num)/closing/execute', 'Administrator\SchoolYearClosingController::execute/$1');
+    $routes->post('(:num)/closing/complete', 'Administrator\SchoolYearClosingController::complete/$1');
+    $routes->post('(:num)/closing/cancel', 'Administrator\SchoolYearClosingController::cancel/$1');
+});
+
 // Route for Attendance Comment Templates
 $routes->get('/administrator/attendance-templates', 'View\AttendanceCommentTemplateController::index');
 $routes->get('/api/attendance-templates', 'View\AttendanceCommentTemplateController::listData');

app/Config/Services.php

@@ -180,4 +180,66 @@ class Services extends BaseService
         $http = static::curlrequest($options);
         return new \App\Services\ApiClient($http, $apiConfig);
     }
+
+    public static function schoolYearContext(bool $getShared = true): \App\Services\SchoolYearContextService
+    {
+        if ($getShared) {
+            return static::getSharedInstance('schoolYearContext');
+        }
+
+        return new \App\Services\SchoolYearContextService(
+            model(\App\Models\SchoolYearModel::class)
+        );
+    }
+
+    public static function schoolYearWriteGuard(bool $getShared = true): \App\Services\SchoolYearWriteGuard
+    {
+        if ($getShared) {
+            return static::getSharedInstance('schoolYearWriteGuard');
+        }
+
+        return new \App\Services\SchoolYearWriteGuard();
+    }
+
+    public static function schoolYearValidation(bool $getShared = true): \App\Services\SchoolYearValidationService
+    {
+        if ($getShared) {
+            return static::getSharedInstance('schoolYearValidation');
+        }
+
+        return new \App\Services\SchoolYearValidationService(
+            model(\App\Models\SchoolYearModel::class)
+        );
+    }
+
+    public static function schoolYearManagement(bool $getShared = true): \App\Services\SchoolYearManagementService
+    {
+        if ($getShared) {
+            return static::getSharedInstance('schoolYearManagement');
+        }
+
+        return new \App\Services\SchoolYearManagementService(
+            model(\App\Models\SchoolYearModel::class),
+            model(\App\Models\ConfigurationModel::class),
+            model(\App\Models\SchoolYearTransitionLogModel::class),
+            model(\App\Models\SchoolYearClosingBatchModel::class),
+            static::schoolYearValidation(),
+            \Config\Database::connect()
+        );
+    }
+
+    public static function schoolYearClosing(bool $getShared = true): \App\Services\SchoolYearClosingService
+    {
+        if ($getShared) {
+            return static::getSharedInstance('schoolYearClosing');
+        }
+
+        return new \App\Services\SchoolYearClosingService(
+            model(\App\Models\SchoolYearModel::class),
+            model(\App\Models\SchoolYearClosingBatchModel::class),
+            model(\App\Models\SchoolYearClosingItemModel::class),
+            static::schoolYearManagement(),
+            \Config\Database::connect()
+        );
+    }
 }

app/Controllers/Administrator/SchoolYearClosingController.php

@@ -0,0 +1,82 @@
+<?php
+
+namespace App\Controllers\Administrator;
+
+use App\Controllers\BaseController;
+use App\Models\SchoolYearModel;
+use Throwable;
+
+class SchoolYearClosingController extends BaseController
+{
+    public function preview(int $id)
+    {
+        try {
+            $targetId = $this->normalizeInt($this->request->getGet('target_school_year_id'));
+
+            return view('school_years/closing_preview', [
+                'preview' => service('schoolYearClosing')->preview($id, $targetId),
+                'latestBatch' => service('schoolYearClosing')->latestBatch($id),
+                'schoolYears' => (new SchoolYearModel())->orderBy('name', 'DESC')->findAll(),
+            ]);
+        } catch (Throwable $e) {
+            return redirect()->to('/administrator/school-years')->with('error', $e->getMessage());
+        }
+    }
+
+    public function start(int $id)
+    {
+        try {
+            $targetId = $this->normalizeInt($this->request->getPost('target_school_year_id'));
+            if ($targetId === null) {
+                return redirect()->back()->with('error', 'Select a target school year.');
+            }
+
+            service('schoolYearClosing')->start($id, $targetId, $this->userId());
+            return redirect()->to('/administrator/school-years/' . $id . '/closing/preview')->with('success', 'Closing started.');
+        } catch (Throwable $e) {
+            return redirect()->back()->with('error', $e->getMessage());
+        }
+    }
+
+    public function execute(int $id)
+    {
+        try {
+            service('schoolYearClosing')->execute($id, $this->userId());
+            return redirect()->to('/administrator/school-years/' . $id . '/closing/preview')->with('success', 'Carry-forward batch executed.');
+        } catch (Throwable $e) {
+            return redirect()->back()->with('error', $e->getMessage());
+        }
+    }
+
+    public function complete(int $id)
+    {
+        try {
+            service('schoolYearClosing')->complete($id, $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'School year closed.');
+        } catch (Throwable $e) {
+            return redirect()->back()->with('error', $e->getMessage());
+        }
+    }
+
+    public function cancel(int $id)
+    {
+        try {
+            service('schoolYearClosing')->cancel($id, $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'Closing cancelled.');
+        } catch (Throwable $e) {
+            return redirect()->back()->with('error', $e->getMessage());
+        }
+    }
+
+    private function normalizeInt(mixed $value): ?int
+    {
+        return is_numeric($value) && (int) $value > 0 ? (int) $value : null;
+    }
+
+    private function userId(): ?int
+    {
+        $id = session('user_id') ?? session('id');
+
+        return is_numeric($id) ? (int) $id : null;
+    }
+}

app/Controllers/Administrator/SchoolYearController.php

@@ -0,0 +1,127 @@
+<?php
+
+namespace App\Controllers\Administrator;
+
+use App\Controllers\BaseController;
+use App\Models\SchoolYearModel;
+use App\Support\SchoolYear\SchoolYearStatus;
+use Throwable;
+
+class SchoolYearController extends BaseController
+{
+    private SchoolYearModel $schoolYearModel;
+
+    public function __construct()
+    {
+        $this->schoolYearModel = new SchoolYearModel();
+    }
+
+    public function index()
+    {
+        $schoolYears = $this->schoolYearModel
+            ->orderBy('name', 'DESC')
+            ->findAll();
+
+        $activeYear = null;
+        $nextDraftYear = null;
+        $closingYear = null;
+        $archivedCount = 0;
+
+        foreach ($schoolYears as $year) {
+            $status = (string) ($year['status'] ?? '');
+            if ($status === SchoolYearStatus::ACTIVE && $activeYear === null) {
+                $activeYear = $year;
+            }
+            if ($status === SchoolYearStatus::DRAFT && $nextDraftYear === null) {
+                $nextDraftYear = $year;
+            }
+            if ($status === SchoolYearStatus::CLOSING && $closingYear === null) {
+                $closingYear = $year;
+            }
+            if ($status === SchoolYearStatus::ARCHIVED) {
+                $archivedCount++;
+            }
+        }
+
+        return view('school_years/index', [
+            'schoolYears' => $schoolYears,
+            'statuses' => SchoolYearStatus::ALL,
+            'activeYear' => $activeYear,
+            'nextDraftYear' => $nextDraftYear,
+            'closingYear' => $closingYear,
+            'archivedCount' => $archivedCount,
+            'latestTransitions' => service('schoolYearManagement')->latestTransitionByYear(),
+        ]);
+    }
+
+    public function store()
+    {
+        try {
+            service('schoolYearManagement')->createDraft($this->request->getPost(), $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'Draft school year created.');
+        } catch (Throwable $e) {
+            return redirect()->back()->withInput()->with('error', $e->getMessage());
+        }
+    }
+
+    public function update(int $id)
+    {
+        try {
+            service('schoolYearManagement')->updateMetadata($id, $this->request->getPost(), $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'School year metadata updated.');
+        } catch (Throwable $e) {
+            return redirect()->back()->withInput()->with('error', $e->getMessage());
+        }
+    }
+
+    public function activate(int $id)
+    {
+        try {
+            if ($this->request->getPost('confirm_activation') !== '1') {
+                return redirect()->to('/administrator/school-years')->with('error', 'Activation confirmation is required.');
+            }
+
+            service('schoolYearManagement')->activate($id, $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'School year activated. Any previous active year is now in closing.');
+        } catch (Throwable $e) {
+            return redirect()->to('/administrator/school-years')->with('error', $e->getMessage());
+        }
+    }
+
+    public function deleteDraft(int $id)
+    {
+        try {
+            service('schoolYearManagement')->deleteDraft($id, $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'Draft school year deleted.');
+        } catch (Throwable $e) {
+            return redirect()->to('/administrator/school-years')->with('error', $e->getMessage());
+        }
+    }
+
+    public function archive(int $id)
+    {
+        try {
+            service('schoolYearManagement')->archive($id, $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'School year archived.');
+        } catch (Throwable $e) {
+            return redirect()->to('/administrator/school-years')->with('error', $e->getMessage());
+        }
+    }
+
+    public function reopen(int $id)
+    {
+        try {
+            service('schoolYearManagement')->reopen($id, (string) $this->request->getPost('reason'), $this->userId());
+            return redirect()->to('/administrator/school-years')->with('success', 'School year reopened.');
+        } catch (Throwable $e) {
+            return redirect()->to('/administrator/school-years')->with('error', $e->getMessage());
+        }
+    }
+
+    private function userId(): ?int
+    {
+        $id = session('user_id') ?? session('id');
+
+        return is_numeric($id) ? (int) $id : null;
+    }
+}

app/Controllers/AuthController.php

@@ -211,13 +211,7 @@ class AuthController extends BaseController
         $this->logLoginAttempt($user['id'], $user['email'], $ip, $this->request->getUserAgent());
 
         // Fetch roles
-        $userRoleModel = new UserRoleModel();
+        $roleNames = $this->getUserRoleNames((int) $user['id']);
-        $rolesRows = $userRoleModel->select('roles.name')
-            ->join('roles', 'roles.id = user_roles.role_id')
-            ->where('user_roles.user_id', $user['id'])
-            ->get()
-            ->getResultArray();
-        $roleNames = array_column($rolesRows, 'name');
 
         // Build roles map (object with keys per example)
         $rolesMap = [];
@@ -454,6 +448,18 @@ class AuthController extends BaseController
         $this->userModel->update($userId, ['failed_attempts' => 0, 'last_failed_at' => null]);
     }
 
+    protected function getUserRoleNames(int $userId): array
+    {
+        $userRoleModel = new UserRoleModel();
+        $rolesRows = $userRoleModel->select('roles.name')
+            ->join('roles', 'roles.id = user_roles.role_id')
+            ->where('user_roles.user_id', $userId)
+            ->get()
+            ->getResultArray();
+
+        return array_column($rolesRows, 'name');
+    }
+
     public function logout()
     {
         // Get user ID and email from session

app/Controllers/BaseController.php

@@ -9,6 +9,8 @@ use CodeIgniter\HTTP\RequestInterface;
 use CodeIgniter\HTTP\ResponseInterface;
 use Psr\Log\LoggerInterface;
 use App\Services\ApiClient;
+use App\Support\SchoolYear\SchoolYearContext;
+use Throwable;
 
 /**
  * Class BaseController
@@ -88,6 +90,27 @@ abstract class BaseController extends Controller
 
         return $payload;
     }
+
+    protected function resolveSchoolYearContext(?int $routeSchoolYearId = null): SchoolYearContext
+    {
+        try {
+            return service('schoolYearContext')->resolve($this->request, $routeSchoolYearId);
+        } catch (Throwable $e) {
+            log_message('warning', 'School-year resolution failed: {message}', [
+                'message' => $e->getMessage(),
+            ]);
+
+            throw $e;
+        }
+    }
+
+    protected function assertSchoolYearWritable(
+        SchoolYearContext $context,
+        bool $allowDraftForAdmin = false,
+        bool $isAdmin = false
+    ): void {
+        service('schoolYearWriteGuard')->assertWritable($context, $allowDraftForAdmin, $isAdmin);
+    }
 }
 
 ?>

app/Controllers/InitializeRolesPermissions.php

@@ -115,9 +115,10 @@ class InitializeRolesPermissions extends Controller
 
         // Associating permissions with roles
         $rolePermissions = [
-            'administrator' => ['manage_users', 'view_reports', 'manage_settings', 'access_student_records', 'communicate_parents'],
+            'administrator' => ['manage_users', 'view_reports', 'manage_settings', 'access_student_records', 'communicate_parents', 'update_curriculum'],
-            'principal' => ['manage_users', 'view_reports', 'access_student_records', 'communicate_parents'],
+            'principal' => ['manage_users', 'view_reports', 'access_student_records', 'communicate_parents', 'update_curriculum'],
-            'vice principal' => ['view_reports', 'access_student_records', 'communicate_parents'],
+            'vice principal' => ['view_reports', 'access_student_records', 'communicate_parents', 'update_curriculum'],
+            'admin' => ['manage_users', 'view_reports', 'manage_settings', 'access_student_records', 'communicate_parents', 'update_curriculum'],
             'teacher' => ['access_student_records', 'communicate_parents', 'enter_grades', 'access_assignments'],
             'student' => ['view_own_records', 'access_assignments'],
             'parent' => ['view_own_records', 'communicate_parents'],

app/Controllers/ParentReportCardController.php

@@ -45,9 +45,6 @@ class ParentReportCardController extends BaseController
         if ($schoolYear !== '') {
             $builder->where('sc.school_year', $schoolYear);
         }
-        if ($semester !== '') {
-            $builder->where('sc.semester', $semester);
-        }
 
         $students = $builder->get()->getResultArray();
         $studentIds = array_values(array_filter(array_map(static fn ($s) => (int) ($s['id'] ?? 0), $students)));

app/Controllers/View/AdministratorController.php

@@ -570,21 +570,21 @@ class AdministratorController extends BaseController
         // USERS (phone col: cellphone)
         $uCols = ['firstname', 'lastname', 'email', 'cellphone', 'school_id', 'city', 'state'];
         $uQB = $db->table('users')
-            ->select('id, firstname, lastname, email, cellphone, school_id, city, state, school_year, semester');
+            ->select('id, firstname, lastname, email, cellphone, school_id, city, state, semester');
         $applyMultiTokenLike($uQB, $uCols, $tokens, ['cellphone']);
         $users = $uQB->limit(150)->get()->getResultArray();
 
         // STUDENTS (no phone column to search)
         $sCols = ['firstname', 'lastname', 'school_id', 'rfid_tag', 'dob', 'gender'];
         $sQB = $db->table('students')
-            ->select('id, parent_id, school_id, firstname, lastname, dob, gender, school_year, semester, rfid_tag');
+            ->select('id, parent_id, school_id, firstname, lastname, dob, gender, semester, rfid_tag');
         $applyMultiTokenLike($sQB, $sCols, $tokens, []);
         $students = $sQB->limit(150)->get()->getResultArray();
 
         // PARENTS (phone col: secondparent_phone)
         $pCols = ['secondparent_firstname', 'secondparent_lastname', 'secondparent_email', 'secondparent_phone'];
         $pQB = $db->table('parents')
-            ->select('id, firstparent_id, secondparent_firstname, secondparent_lastname, secondparent_email, secondparent_phone, school_year, semester');
+            ->select('id, firstparent_id, secondparent_firstname, secondparent_lastname, secondparent_email, secondparent_phone');
         $applyMultiTokenLike($pQB, $pCols, $tokens, ['secondparent_phone']);
         foreach ($tokens as $t) {
             if (ctype_digit($t)) {
@@ -596,14 +596,14 @@ class AdministratorController extends BaseController
         // STAFF (phone col: phone)
         $stCols = ['firstname', 'lastname', 'email', 'role_name', 'phone'];
         $stQB = $db->table('staff')
-            ->select('id, user_id, firstname, lastname, email, phone, role_name, school_year, active_role');
+            ->select('id, user_id, firstname, lastname, email, phone, role_name, active_role');
         $applyMultiTokenLike($stQB, $stCols, $tokens, ['phone']);
         $staff = $stQB->limit(150)->get()->getResultArray();
 
         // EMERGENCY CONTACTS (phone col: cellphone)
         $ecCols = ['emergency_contact_name', 'relation', 'email', 'cellphone'];
         $ecQB = $db->table('emergency_contacts')
-            ->select('id, parent_id, emergency_contact_name, relation, cellphone, email, school_year, semester');
+            ->select('id, parent_id, emergency_contact_name, relation, cellphone, email');
         $applyMultiTokenLike($ecQB, $ecCols, $tokens, ['cellphone']);
         $emergency = $ecQB->limit(150)->get()->getResultArray();
 
@@ -730,18 +730,15 @@ class AdministratorController extends BaseController
             ->join('users u', 'u.id = tc.teacher_id', 'left')
             ->orderBy('cs.class_section_name', 'ASC');
 
-        $filteredQuery = clone $assignmentQuery;
+        // teacher_class assignments are scoped by school year only.
+        // The table has no semester column; semester filtering belongs on
+        // semester-specific records such as scores, comments, attendance,
+        // homework, and exam drafts.
         if ($schoolYear !== '') {
-            $filteredQuery = $filteredQuery->where('tc.school_year', $schoolYear);
+            $assignmentQuery->where('tc.school_year', $schoolYear);
-        }
-        if (!empty($semesterCandidates)) {
-            $filteredQuery = $filteredQuery->whereIn('tc.semester', $semesterCandidates);
         }
 
-        $assignmentRows = $filteredQuery->get()->getResultArray();
-        if (empty($assignmentRows) && ($schoolYear !== '' || $semester !== '')) {
         $assignmentRows = $assignmentQuery->get()->getResultArray();
-        }
 
         $studentCounts = $this->studentClassModel->getStudentCountsBySection($schoolYear !== '' ? $schoolYear : null);
         $sectionRows = $this->classSectionModel
@@ -873,14 +870,12 @@ class AdministratorController extends BaseController
                 continue;
             }
 
-            $studentQuery = $this->studentClassModel
+            $studentEntries = $this->db->table('student_class')
                 ->select('student_id')
                 ->where('class_section_id', $classSectionId)
-                ->where('school_year', $schoolYear);
+                ->where('school_year', $schoolYear)
-            if (!empty($semesterCandidates)) {
+                ->get()
-                $studentQuery->whereIn('semester', $semesterCandidates);
+                ->getResultArray();
-            }
-            $studentEntries = $studentQuery->findAll();
             if (empty($studentEntries)) {
                 $studentEntries = $this->studentClassModel
                     ->select('student_id')
@@ -2207,7 +2202,6 @@ class AdministratorController extends BaseController
                     'cellphone' => $user['cellphone'],
                     'gender' => $user['gender'],
                     'created_at' => $user['created_at'],
-                    'school_year' => $user['school_year'],
                     'paid_amount' => $paidAmount,
                     'balance' => $balance,
                     'students' => $students,

app/Controllers/View/ClassPreparationController.php

@@ -21,8 +21,10 @@ class ClassPreparationController extends BaseController
     protected $schoolYear;
     protected $semester;
     protected $adjustmentModel;
-    /** cache for roster presence per term */
+    /** Cache for roster presence per school year. */
     private array $rosterPresenceCache = [];
+    /** Cache table-column checks to avoid repeated schema queries. */
+    private array $columnExistsCache = [];
     // Inside ClassPreparationController (class scope, not inside a method)
     private array $allowedPrepCategories = [
         'Grade Box',
@@ -57,7 +59,7 @@ class ClassPreparationController extends BaseController
         $semParam   = $this->request->getGet('semester');
         $semester   = (is_string($semParam) && $semParam !== '') ? (string)$semParam : (string)$this->semester;
         $allowed    = $this->allowedPrepCategories;
-        $limitToSemester = $this->hasRosterForSemester($schoolYear, $semester);
+        $hasRoster  = $this->hasRosterForSchoolYear($schoolYear);
 
         // 1) Get student count per class-section (distinct students, correct semester)
         $scQ = $this->db->table('student_class sc')
@@ -65,13 +67,12 @@ class ClassPreparationController extends BaseController
             ->join('students s', 's.id = sc.student_id', 'inner')
             ->where('s.is_active', 1)
             ->where('sc.school_year', $schoolYear);
-        if ($limitToSemester && $semester !== '') {
+        $classSections = $hasRoster
-            $scQ->where('sc.semester', $semester);
+            ? $scQ->groupBy('sc.class_section_id')->get()->getResultArray()
-        }
+            : [];
-        $classSections = $scQ->groupBy('sc.class_section_id')->get()->getResultArray();
 
         // 2) Inventory availability — prefer good_qty when present, else condition='good' quantity.
-        $inventoryMap = $this->buildInventoryAvailability($schoolYear, $semester, $limitToSemester, $allowed);
+        $inventoryMap = $this->buildInventoryAvailability($schoolYear, $semester, $allowed);
 
         // Seed totals with allowed categories for stable ordering
         $requiredTotals = array_fill_keys($allowed, 0);
@@ -85,14 +86,10 @@ class ClassPreparationController extends BaseController
             $className      = $this->classSectionModel->getClassSectionNameBySectionId($classSectionId);
 
             // Calculate required items (whitelist inside)
-            $baseItems = $this->calculatePrepItems($studentCount, $classLevel, $classSectionId);
+            $baseItems = $this->calculatePrepItems($studentCount, $classLevel, $classSectionId, $schoolYear);
 
             // --- Apply adjustments (only Small/Large Table), clamp >= 0 ---
-            $rawAdjustments = $this->adjustmentModel
+            $rawAdjustments = $this->getAdjustments((string) $classSectionId, (string) $schoolYear);
-                ->where('class_section_id', $classSectionId)
-                ->where('school_year', $schoolYear)
-                ->where('adjustable', 1)
-                ->findAll();
 
             $adjMap = ['Large Table' => 0, 'Small Table' => 0];
 
@@ -117,11 +114,7 @@ class ClassPreparationController extends BaseController
             }
 
             // 5) Compare with last snapshot; do not save here — only on print or explicit API
-            $oldSnap = $this->prepLogModel
+            $oldSnap = $this->getLatestPrepSnapshot((string) $classSectionId, (string) $schoolYear);
-                ->where('class_section_id', $classSectionId)
-                ->where('school_year', $schoolYear)
-                ->orderBy('created_at', 'DESC')
-                ->first();
 
             $oldPrep    = $oldSnap ? json_decode($oldSnap['prep_data'], true) : [];
             $hasChanged = $this->hasPrepChanged($baseItems, $oldPrep);
@@ -175,7 +168,7 @@ class ClassPreparationController extends BaseController
         return (int)($row['cnt'] ?? 0);
     }
 
-    private function calculatePrepItems(int $students, int $classLevel, string $classSectionId): array
+    private function calculatePrepItems(int $students, int $classLevel, string $classSectionId, ?string $schoolYear = null): array
     {
         // 1) Stable keys: your whitelist, all zero to start
         $allowed = $this->allowedPrepCategories;
@@ -192,7 +185,7 @@ class ClassPreparationController extends BaseController
 
         // 3) Rules
         $isLowerGrades = in_array($classLevel, [1, 2], true);
-        $teacherCount  = $this->getTeacherCountForSection($classSectionId, $this->schoolYear);
+        $teacherCount  = $this->getTeacherCountForSection($classSectionId, $schoolYear ?? $this->schoolYear);
 
         foreach ($categories as $cat) {
             $name     = $cat['name'];      // e.g., 'Small Table'
@@ -253,24 +246,35 @@ class ClassPreparationController extends BaseController
         $adjustments = $data['adjustments'] ?? [];
 
         foreach ($adjustments as $itemName => $adjustment) {
-            $row = $this->adjustmentModel
+            $adjustmentQuery = $this->adjustmentModel
                 ->where('class_section_id', $sectionId)
-                ->where('school_year', $schoolYear)
+                ->where('item_name', $itemName);
-                ->where('item_name', $itemName)
+
-                ->first();
+            $adjustmentTable = $this->adjustmentModel->getTable();
+
+            if ($schoolYear !== '' && $this->tableHasColumn($adjustmentTable, 'school_year')) {
+                $adjustmentQuery->where('school_year', $schoolYear);
+            }
+
+            $row = $adjustmentQuery->first();
 
             if ($row) {
                 // Update
                 $this->adjustmentModel->update($row['id'], ['adjustment' => (int)$adjustment, 'adjustable' => 1]);
             } else {
                 // Insert
-                $this->adjustmentModel->insert([
+                $insertData = [
                     'class_section_id' => $sectionId,
                     'item_name'        => $itemName,
-                    'adjustment'       => (int)$adjustment,
+                    'adjustment'       => (int) $adjustment,
-                    'school_year'      => $schoolYear,
                     'adjustable'       => 1,
-                ]);
+                ];
+
+                if ($schoolYear !== '' && $this->tableHasColumn($adjustmentTable, 'school_year')) {
+                    $insertData['school_year'] = $schoolYear;
+                }
+
+                $this->adjustmentModel->insert($insertData);
             }
         }
 
@@ -292,10 +296,6 @@ class ClassPreparationController extends BaseController
 
     public function print($classSectionId, $schoolYear)
     {
-        $semParam = $this->request->getGet('semester');
-        $semester = (is_string($semParam) && $semParam !== '') ? (string) $semParam : (string) $this->semester;
-        $limitToSemester = $this->hasRosterForSemester((string) $schoolYear, $semester);
-
         // Friendly label (if you have this helper)
         $className = $this->classSectionModel->getClassSectionNameBySectionId($classSectionId) ?? $classSectionId;
 
@@ -306,15 +306,12 @@ class ClassPreparationController extends BaseController
             ->where('s.is_active', 1)
             ->where('sc.class_section_id', $classSectionId)
             ->where('sc.school_year', $schoolYear);
-        if ($limitToSemester && $semester !== '') {
-            $studentQ->where('sc.semester', $semester);
-        }
         $studentRow = $studentQ->get()->getRowArray();
         $studentCount = (int)($studentRow['cnt'] ?? 0);
 
         // Live calc + adjustments
         $classLevel = $this->getClassLevelBySection((string)$classSectionId);
-        $items      = $this->calculatePrepItems($studentCount, $classLevel, (string)$classSectionId);
+        $items      = $this->calculatePrepItems($studentCount, $classLevel, (string)$classSectionId, (string)$schoolYear);
 
         $rawAdjustments = $this->adjustmentModel
             ->where('class_section_id', $classSectionId)
@@ -328,13 +325,15 @@ class ClassPreparationController extends BaseController
         }
 
         // Record a snapshot at print time as the new baseline
-        $this->prepLogModel->insert([
+        $this->prepLogModel->insert(
-            'class_section_id' => (string)$classSectionId,
+            $this->buildPrepLogData(
-            'class_section'    => $className,
+                (string) $classSectionId,
-            'school_year'      => $schoolYear,
+                (string) $className,
-            'prep_data'        => json_encode($items),
+                (string) $schoolYear,
-            'created_at'       => utc_now(),
+                $items,
-        ]);
+                utc_now()
+            )
+        );
 
         // Return PRINT view (HTML) that auto-opens the print dialog
         return view('class_prep/print', [
@@ -356,7 +355,7 @@ class ClassPreparationController extends BaseController
         $semParam   = $this->request->getGet('semester');
         $semester   = (is_string($semParam) && $semParam !== '') ? (string)$semParam : (string)$this->semester;
         $allowed    = $this->allowedPrepCategories;
-        $limitToSemester = $this->hasRosterForSemester($schoolYear, $semester);
+        $hasRoster  = $this->hasRosterForSchoolYear($schoolYear);
 
         // Student counts
         $scQ = $this->db->table('student_class sc')
@@ -364,13 +363,12 @@ class ClassPreparationController extends BaseController
             ->join('students s', 's.id = sc.student_id', 'inner')
             ->where('s.is_active', 1)
             ->where('sc.school_year', $schoolYear);
-        if ($limitToSemester && $semester !== '') {
+        $classSections = $hasRoster
-            $scQ->where('sc.semester', $semester);
+            ? $scQ->groupBy('sc.class_section_id')->get()->getResultArray()
-        }
+            : [];
-        $classSections = $scQ->groupBy('sc.class_section_id')->get()->getResultArray();
 
         // Build inventory availability maps
-        $inventoryMap = $this->buildInventoryAvailability($schoolYear, $semester, $limitToSemester, $allowed);
+        $inventoryMap = $this->buildInventoryAvailability($schoolYear, $semester, $allowed);
 
         $requiredTotals = array_fill_keys($allowed, 0);
         $results = [];
@@ -381,12 +379,8 @@ class ClassPreparationController extends BaseController
             $classLevel     = $this->getClassLevelBySection($classSectionId);
             $className      = $this->classSectionModel->getClassSectionNameBySectionId($classSectionId);
 
-            $baseItems = $this->calculatePrepItems($studentCount, $classLevel, $classSectionId);
+            $baseItems = $this->calculatePrepItems($studentCount, $classLevel, $classSectionId, $schoolYear);
-            $rawAdjustments = $this->adjustmentModel
+            $rawAdjustments = $this->getAdjustments((string) $classSectionId, (string) $schoolYear);
-                ->where('class_section_id', $classSectionId)
-                ->where('school_year', $schoolYear)
-                ->where('adjustable', 1)
-                ->findAll();
             $adjMap = ['Large Table' => 0, 'Small Table' => 0];
             foreach ($rawAdjustments as $a) {
                 $item  = $a['item_name'];
@@ -399,11 +393,7 @@ class ClassPreparationController extends BaseController
                 $requiredTotals[$cat] += (int)($baseItems[$cat] ?? 0);
             }
 
-            $oldSnap = $this->prepLogModel
+            $oldSnap = $this->getLatestPrepSnapshot((string) $classSectionId, (string) $schoolYear);
-                ->where('class_section_id', $classSectionId)
-                ->where('school_year', $schoolYear)
-                ->orderBy('created_at', 'DESC')
-                ->first();
             $oldPrep    = $oldSnap ? json_decode($oldSnap['prep_data'], true) : [];
             $hasChanged = $this->hasPrepChanged($baseItems, $oldPrep);
 
@@ -446,7 +436,6 @@ class ClassPreparationController extends BaseController
         $schoolYear = (string)($this->request->getPost('school_year') ?? $this->schoolYear);
         $semParam   = $this->request->getPost('semester');
         $semester   = (is_string($semParam) && $semParam !== '') ? (string)$semParam : (string)$this->semester;
-        $limitToSemester = $this->hasRosterForSemester($schoolYear, $semester);
         $ids = $this->request->getPost('class_section_ids') ?? [];
         if (!is_array($ids)) $ids = $ids ? [$ids] : [];
         $ids = array_values(array_unique(array_filter(array_map('strval', $ids))));
@@ -460,21 +449,14 @@ class ClassPreparationController extends BaseController
                 ->where('s.is_active', 1)
                 ->where('sc.class_section_id', $classSectionId)
                 ->where('sc.school_year', $schoolYear);
-            if ($limitToSemester && $semester !== '') {
-                $studentQ->where('sc.semester', $semester);
-            }
             $studentRow = $studentQ->get()->getRowArray();
             $studentCount = (int)($studentRow['cnt'] ?? 0);
             $classLevel   = $this->getClassLevelBySection((string)$classSectionId);
             $className    = $this->classSectionModel->getClassSectionNameBySectionId($classSectionId) ?? $classSectionId;
 
-            $items = $this->calculatePrepItems($studentCount, $classLevel, (string)$classSectionId);
+            $items = $this->calculatePrepItems($studentCount, $classLevel, (string)$classSectionId, (string)$schoolYear);
 
-            $rawAdjustments = $this->adjustmentModel
+            $rawAdjustments = $this->getAdjustments((string) $classSectionId, (string) $schoolYear);
-                ->where('class_section_id', $classSectionId)
-                ->where('school_year', $schoolYear)
-                ->where('adjustable', 1)
-                ->findAll();
             foreach ($rawAdjustments as $a) {
                 $item  = $a['item_name'];
                 $delta = (int)$a['adjustment'];
@@ -482,13 +464,15 @@ class ClassPreparationController extends BaseController
             }
 
             try {
-                $this->prepLogModel->insert([
+                $this->prepLogModel->insert(
-                    'class_section_id' => (string)$classSectionId,
+                    $this->buildPrepLogData(
-                    'class_section'    => $className,
+                        (string) $classSectionId,
-                    'school_year'      => $schoolYear,
+                        (string) $className,
-                    'prep_data'        => json_encode($items),
+                        (string) $schoolYear,
-                    'created_at'       => $now,
+                        $items,
-                ]);
+                        $now
+                    )
+                );
                 $count++;
             } catch (\Throwable $e) {
                 // ignore and continue
@@ -528,12 +512,12 @@ class ClassPreparationController extends BaseController
     }
 
     /**
-     * Returns true if student_class has any rows for the given school year.
+     * Returns true when student_class contains at least one assignment
-     * The semester argument is ignored because assignments are tracked per year.
+     * for the selected school year.
      */
-    private function hasRosterForTerm(string $schoolYear, string $semester): bool
+    private function hasRosterForSchoolYear(string $schoolYear): bool
     {
-        $year = trim((string)$schoolYear);
+        $year = trim($schoolYear);
         if ($year === '') {
             return false;
         }
@@ -542,72 +526,163 @@ class ClassPreparationController extends BaseController
             return $this->rosterPresenceCache[$year];
         }
 
-        $cnt = $this->db->table('student_class')
+        $exists = $this->db->table('student_class')
             ->where('school_year', $year)
-            ->countAllResults();
+            ->limit(1)
+            ->countAllResults() > 0;
 
-        $this->rosterPresenceCache[$year] = $cnt > 0;
+        $this->rosterPresenceCache[$year] = $exists;
-        return $this->rosterPresenceCache[$year];
+
+        return $exists;
     }
 
-    private function hasRosterForSemester(string $schoolYear, string $semester): bool
+    private function tableHasColumn(string $table, string $column): bool
     {
-        $year = trim((string)$schoolYear);
+        $key = $table . '.' . $column;
-        $sem  = trim((string)$semester);
+
-        if ($year === '' || $sem === '') {
+        if (!array_key_exists($key, $this->columnExistsCache)) {
-            return false;
+            $this->columnExistsCache[$key] = $this->db->fieldExists($column, $table);
         }
 
-        $key = sprintf('sem:%s:%s', $year, $sem);
+        return $this->columnExistsCache[$key];
-        if (array_key_exists($key, $this->rosterPresenceCache)) {
-            return $this->rosterPresenceCache[$key];
     }
 
-        $cnt = $this->db->table('student_class')
+    private function getAdjustments(string $classSectionId, string $schoolYear): array
-            ->where('school_year', $year)
+    {
-            ->where('semester', $sem)
+        $query = $this->adjustmentModel
-            ->countAllResults();
+            ->where('class_section_id', $classSectionId)
+            ->where('adjustable', 1);
 
-        $this->rosterPresenceCache[$key] = $cnt > 0;
+        $table = $this->adjustmentModel->getTable();
-        return $this->rosterPresenceCache[$key];
+
+        if ($schoolYear !== '' && $this->tableHasColumn($table, 'school_year')) {
+            $query->where('school_year', $schoolYear);
         }
 
-    private function buildInventoryAvailability(string $schoolYear, string $semester, bool $limitToSemester, array $allowed): array
+        return $query->findAll();
+    }
+
+    private function getLatestPrepSnapshot(string $classSectionId, string $schoolYear): ?array
+    {
+        $query = $this->prepLogModel
+            ->where('class_section_id', $classSectionId);
+
+        $table = $this->prepLogModel->getTable();
+
+        if ($schoolYear !== '' && $this->tableHasColumn($table, 'school_year')) {
+            $query->where('school_year', $schoolYear);
+        }
+
+        return $query
+            ->orderBy('created_at', 'DESC')
+            ->first();
+    }
+
+    private function buildPrepLogData(
+        string $classSectionId,
+        string $className,
+        string $schoolYear,
+        array $items,
+        string $createdAt
+    ): array {
+        $data = [
+            'class_section_id' => $classSectionId,
+            'class_section'    => $className,
+            'prep_data'        => json_encode($items),
+            'created_at'       => $createdAt,
+        ];
+
+        $table = $this->prepLogModel->getTable();
+
+        if ($schoolYear !== '' && $this->tableHasColumn($table, 'school_year')) {
+            $data['school_year'] = $schoolYear;
+        }
+
+        return $data;
+    }
+
+    private function buildInventoryAvailability(string $schoolYear, string $semester, array $allowed): array
     {
         $inventoryMap = array_fill_keys($allowed, 0);
 
+        $hasName       = $this->tableHasColumn('inventory_items', 'name');
+
         $joinRows = $this->db->table('inventory_items ii')
-            ->select('ic.name AS item_name, COALESCE(SUM(CASE WHEN ii.good_qty IS NOT NULL THEN ii.good_qty WHEN ii.`condition`="good" THEN ii.quantity ELSE 0 END),0) AS available')
+            ->select('ic.name AS item_name, COALESCE(SUM(CASE WHEN ii.good_qty IS NOT NULL THEN ii.good_qty WHEN ii.`condition` = "good" THEN ii.quantity ELSE 0 END), 0) AS available', false)
             ->join('inventory_categories ic', 'ic.id = ii.category_id', 'left')
             ->where('ii.type', 'classroom')
-            ->where('ii.school_year', $schoolYear)
             ->whereIn('ic.name', $allowed);
-        if ($limitToSemester && $semester !== '') {
-            $joinRows->where('ii.semester', $semester);
-        }
-        $joinRows = $joinRows->groupBy('ic.name')->get()->getResultArray();
 
-        foreach ($joinRows as $r) {
+        $periodClauses = ['im.item_id = ii.id'];
-            $name = (string)($r['item_name'] ?? '');
+        if ($schoolYear !== '') {
-            if ($name !== '' && isset($inventoryMap[$name])) {
+            $periodClauses[] = 'im.school_year = ' . $this->db->escape($schoolYear);
-                $inventoryMap[$name] = max($inventoryMap[$name], (int)($r['available'] ?? 0));
+        }
+        if ($semester !== '') {
+            $periodClauses[] = 'im.semester = ' . $this->db->escape($semester);
+        }
+        if (count($periodClauses) > 1) {
+            $joinRows->where(
+                'EXISTS (SELECT 1 FROM inventory_movements im WHERE ' . implode(' AND ', $periodClauses) . ')',
+                null,
+                false
+            );
+        }
+
+        $joinRows = $joinRows
+            ->groupBy('ic.name')
+            ->get()
+            ->getResultArray();
+
+        foreach ($joinRows as $row) {
+            $name = (string) ($row['item_name'] ?? '');
+
+            if ($name !== '' && array_key_exists($name, $inventoryMap)) {
+                $inventoryMap[$name] = max(
+                    $inventoryMap[$name],
+                    (int) ($row['available'] ?? 0)
+                );
             }
         }
 
-        $nameRows = $this->db->table('inventory_items')
+        /*
-            ->select('name AS item_name, COALESCE(SUM(CASE WHEN good_qty IS NOT NULL THEN good_qty WHEN `condition`="good" THEN quantity ELSE 0 END),0) AS available')
+         * Some older schemas store the item name directly on inventory_items.
-            ->where('type', 'classroom')
+         * Only run this fallback when that column actually exists.
-            ->where('school_year', $schoolYear)
+         */
-            ->whereIn('name', $allowed);
+        if ($hasName) {
-        if ($limitToSemester && $semester !== '') {
+            $nameRows = $this->db->table('inventory_items ii_name')
-            $nameRows->where('semester', $semester);
+                ->select('ii_name.name AS item_name, COALESCE(SUM(CASE WHEN ii_name.good_qty IS NOT NULL THEN ii_name.good_qty WHEN ii_name.`condition` = "good" THEN ii_name.quantity ELSE 0 END), 0) AS available', false)
-        }
+                ->where('ii_name.type', 'classroom')
-        $nameRows = $nameRows->groupBy('name')->get()->getResultArray();
+                ->whereIn('ii_name.name', $allowed);
 
-        foreach ($nameRows as $r) {
+            $fallbackPeriodClauses = ['im.item_id = ii_name.id'];
-            $name = (string)($r['item_name'] ?? '');
+            if ($schoolYear !== '') {
-            if ($name !== '' && isset($inventoryMap[$name])) {
+                $fallbackPeriodClauses[] = 'im.school_year = ' . $this->db->escape($schoolYear);
-                $inventoryMap[$name] = max($inventoryMap[$name], (int)($r['available'] ?? 0));
+            }
+            if ($semester !== '') {
+                $fallbackPeriodClauses[] = 'im.semester = ' . $this->db->escape($semester);
+            }
+            if (count($fallbackPeriodClauses) > 1) {
+                $nameRows->where(
+                    'EXISTS (SELECT 1 FROM inventory_movements im WHERE ' . implode(' AND ', $fallbackPeriodClauses) . ')',
+                    null,
+                    false
+                );
+            }
+
+            $nameRows = $nameRows
+                ->groupBy('ii_name.name')
+                ->get()
+                ->getResultArray();
+
+            foreach ($nameRows as $row) {
+                $name = (string) ($row['item_name'] ?? '');
+
+                if ($name !== '' && array_key_exists($name, $inventoryMap)) {
+                    $inventoryMap[$name] = max(
+                        $inventoryMap[$name],
+                        (int) ($row['available'] ?? 0)
+                    );
+                }
             }
         }
 

app/Controllers/View/FamilyAdminController.php

@@ -150,10 +150,12 @@ class FamilyAdminController extends BaseController
                     }
 
                     // Recent payments (limit 10)
-                    $payRows = $db->table('payments')
+                    $payRows = $db->table('payments p')
-                        ->select('id, parent_id, invoice_id, paid_amount, balance, payment_method, payment_date, status')
+                        ->select('p.id, p.parent_id, p.invoice_id, p.paid_amount, p.payment_method, p.payment_date, p.status AS payment_status, p.installment_seq, p.number_of_installments, i.invoice_number, i.balance AS invoice_current_balance, i.status AS invoice_status, i.school_year')
-                        ->whereIn('parent_id', $parentIds)
+                        ->join('invoices i', 'i.id = p.invoice_id', 'inner')
-                        ->orderBy('payment_date', 'DESC')
+                        ->whereIn('p.parent_id', $parentIds)
+                        ->orderBy('p.payment_date', 'DESC')
+                        ->orderBy('p.id', 'DESC')
                         ->limit(10)
                         ->get()->getResultArray();
                     $fam['payments'] = $payRows;
@@ -359,7 +361,7 @@ class FamilyAdminController extends BaseController
                 if ($gid > 0) $gmap[$gid] = trim(($g['firstname'] ?? '') . ' ' . ($g['lastname'] ?? ''));
             }
             $ecRows = $db->table('emergency_contacts')
-                ->select('id, parent_id, emergency_contact_name, relation, cellphone, email, school_year, semester, created_at, updated_at')
+                ->select('id, parent_id, emergency_contact_name, relation, cellphone, email, created_at, updated_at')
                 ->whereIn('parent_id', $parentIds)
                 ->orderBy('updated_at', 'DESC')
                 ->get()->getResultArray();
@@ -395,10 +397,12 @@ class FamilyAdminController extends BaseController
             }
 
             // Payments
-            $payRows = $db->table('payments')
+            $payRows = $db->table('payments p')
-                ->select('id, parent_id, invoice_id, paid_amount, balance, payment_method, payment_date, status')
+                ->select('p.id, p.parent_id, p.invoice_id, p.paid_amount, p.payment_method, p.payment_date, p.status AS payment_status, p.installment_seq, p.number_of_installments, i.invoice_number, i.balance AS invoice_current_balance, i.status AS invoice_status, i.school_year')
-                ->whereIn('parent_id', $parentIds)
+                ->join('invoices i', 'i.id = p.invoice_id', 'inner')
-                ->orderBy('payment_date', 'DESC')
+                ->whereIn('p.parent_id', $parentIds)
+                ->orderBy('p.payment_date', 'DESC')
+                ->orderBy('p.id', 'DESC')
                 ->limit(10)
                 ->get()->getResultArray();
             $family['payments'] = $payRows;

app/Controllers/View/FinancialController.php

@@ -658,10 +658,10 @@ public function financialReport()
         }, $discountDetailsBuilder->orderBy('du.id', 'ASC')->get()->getResultArray());
 
         $paymentDetailsBuilder = $db->table('payments p')
-            ->select("p.id, p.invoice_id, p.parent_id, p.paid_amount, p.payment_method, p.payment_date, p.status, p.transaction_id, p.check_number, i.invoice_number, CONCAT(COALESCE(u.firstname, ''), ' ', COALESCE(u.lastname, '')) AS parent_name")
+            ->select("p.id, p.invoice_id, p.parent_id, p.paid_amount, p.payment_method, p.payment_date, p.status AS payment_status, p.transaction_id, p.check_number, i.invoice_number, i.balance AS invoice_current_balance, i.status AS invoice_status, i.school_year, CONCAT(COALESCE(u.firstname, ''), ' ', COALESCE(u.lastname, '')) AS parent_name")
-            ->join('invoices i', 'i.id = p.invoice_id', 'left')
+            ->join('invoices i', 'i.id = p.invoice_id', 'inner')
             ->join('users u', 'u.id = p.parent_id', 'left')
-            ->where('p.school_year', $schoolYear)
+            ->where('i.school_year', $schoolYear)
             ->groupStart()
                 ->whereNotIn('p.status', $paymentExclude)
                 ->orWhere('p.status IS NULL', null, false)
@@ -677,9 +677,12 @@ public function financialReport()
                 'Parent' => trim((string)($row['parent_name'] ?? '')),
                 'Invoice #' => (string)($row['invoice_number'] ?? ''),
                 'Paid Amount' => (float)($row['paid_amount'] ?? 0),
+                'Invoice Balance' => (float)($row['invoice_current_balance'] ?? 0),
                 'Payment Method' => (string)($row['payment_method'] ?? ''),
                 'Payment Date' => (string)($row['payment_date'] ?? ''),
-                'Status' => (string)($row['status'] ?? ''),
+                'Payment Status' => (string)($row['payment_status'] ?? ''),
+                'Invoice Status' => (string)($row['invoice_status'] ?? ''),
+                'School Year' => (string)($row['school_year'] ?? ''),
                 'Transaction ID' => (string)($row['transaction_id'] ?? ''),
                 'Check Number' => (string)($row['check_number'] ?? ''),
             ];

app/Controllers/View/GradingController.php

@@ -706,7 +706,6 @@ class GradingController extends Controller
 
         $existing = $this->placementLevelModel
             ->where('student_id', $studentId)
-            ->where('school_year', $schoolYear)
             ->first();
 
         if ($level === null) {
@@ -718,7 +717,6 @@ class GradingController extends Controller
 
         $payload = [
             'student_id' => $studentId,
-            'school_year' => $schoolYear,
             'level' => $level,
             'updated_by' => session()->get('user_id'),
         ];
@@ -768,7 +766,6 @@ class GradingController extends Controller
         if (!empty($studentIds)) {
             $rows = $this->placementLevelModel
                 ->whereIn('student_id', $studentIds)
-                ->where('school_year', $schoolYear)
                 ->findAll();
             foreach ($rows as $row) {
                 $levels[(int) $row['student_id']] = $row['level'] ?? null;
@@ -806,7 +803,6 @@ class GradingController extends Controller
         if (!empty($validIds)) {
             $rows = $this->placementLevelModel
                 ->whereIn('student_id', $validIds)
-                ->where('school_year', $schoolYear)
                 ->findAll();
             foreach ($rows as $row) {
                 $existingRows[(int) $row['student_id']] = $row;
@@ -834,7 +830,6 @@ class GradingController extends Controller
 
             $payload = [
                 'student_id' => $studentId,
-                'school_year' => $schoolYear,
                 'level' => $level,
                 'updated_by' => $userId,
             ];
@@ -1648,7 +1643,7 @@ public function belowSixty()
             ->join('students s', 's.id = sc.student_id', 'inner')
             ->join(
                 'placement_levels pl',
-                "pl.student_id = s.id AND pl.school_year = {$yrEsc}",
+                'pl.student_id = s.id',
                 'left'
             )
 

app/Controllers/View/InventoryController.php

@@ -75,12 +75,7 @@ class InventoryController extends BaseController
         $selectedSem    = $selectedSemRaw === null ? (string) $this->semester : trim((string) $selectedSemRaw);
 
         $builder = $this->itemModel->where('type', $type);
-        if (strtolower($selectedYear) !== 'all') {
+        $this->applyInventoryMovementPeriodFilter($builder, 'inventory_items', $selectedYear, $selectedSem);
-            $builder->where('school_year', $selectedYear);
-        }
-        if ($selectedSem !== '') {
-            $builder->where('semester', $selectedSem);
-        }
 
         $items = $builder->orderBy('name', 'ASC')->findAll();
         $categories = $this->catModel->optionsForType($type);
@@ -474,17 +469,26 @@ class InventoryController extends BaseController
     // Distinct school years for a given inventory type (newest first).
     private function getSchoolYearsForType(string $type): array
     {
-        // Pull distinct non-null school_year values for this type
+        $rows = $this->db->table('inventory_movements m')
-        $years = $this->itemModel
+            ->distinct()
-            ->select('school_year')
+            ->select('m.school_year')
-            ->where('type', $type)
+            ->join('inventory_items i', 'i.id = m.item_id', 'inner')
-            ->where('school_year IS NOT NULL', null, false)
+            ->where('m.school_year IS NOT NULL', null, false)
-            ->groupBy('school_year')
+            ->where("TRIM(m.school_year) <>", '')
-            ->orderBy('school_year', 'DESC')
+            ->orderBy('m.school_year', 'DESC');
-            ->findColumn('school_year') ?? [];
+
+        if (strtolower($type) !== 'all') {
+            $rows->where('i.type', $type);
+        }
+
+        $rows = $rows->get()
+            ->getResultArray();
 
         // Normalize & de-dup
-        $years = array_values(array_unique(array_filter(array_map('trim', $years))));
+        $years = array_values(array_unique(array_filter(array_map(
+            static fn(array $row): string => trim((string)($row['school_year'] ?? '')),
+            $rows
+        ))));
 
         // Ensure the current year (from config) appears as an option, even if no rows yet
         $currentYear = $this->schoolYear;
@@ -496,6 +500,31 @@ class InventoryController extends BaseController
         return $years;
     }
 
+    private function applyInventoryMovementPeriodFilter($builder, string $itemAlias, ?string $schoolYear, ?string $semester = null): void
+    {
+        $schoolYear = trim((string) $schoolYear);
+        $semester = trim((string) $semester);
+        $clauses = ["m.item_id = {$itemAlias}.id"];
+
+        if ($schoolYear !== '' && strtolower($schoolYear) !== 'all') {
+            $clauses[] = 'm.school_year = ' . $this->db->escape($schoolYear);
+        }
+
+        if ($semester !== '') {
+            $clauses[] = 'm.semester = ' . $this->db->escape($semester);
+        }
+
+        if (count($clauses) <= 1) {
+            return;
+        }
+
+        $builder->where(
+            'EXISTS (SELECT 1 FROM inventory_movements m WHERE ' . implode(' AND ', $clauses) . ')',
+            null,
+            false
+        );
+    }
+
     private function recordMovement(
         int $itemId,
         int $qtyChange,
@@ -1095,7 +1124,7 @@ class InventoryController extends BaseController
 
         // Filter by school year unless "all"
         if (!$isAllYears) {
-            $qbItems->where('i.school_year', $selectedYear);
+            $this->applyInventoryMovementPeriodFilter($qbItems, 'i', $selectedYear);
         }
 
         // Optional: filter by type if provided
@@ -1123,7 +1152,7 @@ class InventoryController extends BaseController
             return view('inventory/summary', [
                 'selectedYear' => $selectedYear,
                 'currentYear'  => $this->schoolYear,
-                'schoolYears'  => $this->getSchoolYearsForType('book') ?: [$this->schoolYear, 'All'],
+                'schoolYears'  => $this->getSchoolYearsForType($selectedType) ?: [$this->schoolYear, 'All'],
                 'rows'         => $rows,
                 'totals'       => $totals,
                 // optionally pass the selected type if your view supports it
@@ -1202,7 +1231,7 @@ class InventoryController extends BaseController
         }
 
         // -------- Year dropdown options --------
-        $schoolYears = $this->getSchoolYearsForType('book') ?: [$this->schoolYear, 'All'];
+        $schoolYears = $this->getSchoolYearsForType($selectedType) ?: [$this->schoolYear, 'All'];
 
         return view('inventory/summary', [
             'selectedYear' => $selectedYear,

app/Controllers/View/ParentController.php

@@ -1487,8 +1487,6 @@ $existing = $this->studentModel
                 'cellphone'              => $phone,
                 'email'                  => $email,
                 'relation'               => $relation,
-                'semester'               => $semester,
-                'school_year'            => $schoolYear,
                 'updated_at'             => utc_now(),
             ];
 
@@ -1568,8 +1566,6 @@ $existing = $this->studentModel
                     'cellphone'              => $phone,
                     'email'                  => $email,
                     'relation'               => $relation,
-                    'semester'               => $semester,
-                    'school_year'            => $schoolYear,
                 ]);
             }
         }

app/Controllers/View/PaymentController.php

@@ -85,7 +85,6 @@ class PaymentController extends ResourceController
             'payment_date' => $this->request->getPost('payment_date'),
             'payment_type' => $this->request->getPost('payment_type'),
             'status' => 'Pending',
-            'semester' => $this->request->getPost('semester'),
             'school_year' => $this->request->getPost('school_year'),
         ];
 
@@ -99,7 +98,8 @@ class PaymentController extends ResourceController
     // API: Get payments by parent ID
     public function getByParentAPI($parentId)
     {
-        $payments = $this->paymentModel->getPaymentsByParentId($parentId);
+        $selectedYear = $this->getSelectedPaymentHistoryYear();
+        $payments = $this->paymentModel->getPaymentsByParentId((int) $parentId, $selectedYear);
         if ($payments) {
             return $this->respond($payments);
         } else {
@@ -110,13 +110,8 @@ class PaymentController extends ResourceController
     // View: Get payments by parent ID (for web views)
     public function getByParent($parentId)
     {
-        // Fetch payments with invoice number (if available)
+        $selectedYear = $this->getSelectedPaymentHistoryYear();
-        $payments = $this->paymentModel
+        $payments = $this->paymentModel->getPaymentsByParentId((int) $parentId, $selectedYear);
-            ->select('payments.*, invoices.invoice_number')
-            ->join('invoices', 'invoices.id = payments.invoice_id', 'left')
-            ->where('payments.parent_id', (int)$parentId)
-            ->orderBy('payments.payment_date', 'DESC')
-            ->findAll();
 
         // Parent display name
         $parent = $this->userModel->select('firstname, lastname')->find((int)$parentId) ?: [];
@@ -140,6 +135,13 @@ class PaymentController extends ResourceController
         ]);
     }
 
+    private function getSelectedPaymentHistoryYear(): ?string
+    {
+        $selectedYear = $this->request->getGet('school_year') ?? $this->schoolYear;
+
+        return $selectedYear !== '' ? $selectedYear : null;
+    }
+
     // View: Create a new payment plan
     public function create()
     {
@@ -346,12 +348,12 @@ class PaymentController extends ResourceController
                     ->where('parent_id', $parentId)
                     ->findAll();
 
-                // Payments (paginated)
+                // Payments (paginated). Join invoices so history is filtered by invoice term
-                $payments = $this->paymentModel
+                // and displays current invoice state instead of stale payment snapshots.
-                    ->where('parent_id', $parentId)
+                $selectedYear = $this->getSelectedPaymentHistoryYear();
-                    ->orderBy('payment_date', 'DESC')
+                $paymentHistory = $this->paymentModel->parentPaymentHistoryQuery($parentId, $selectedYear);
-                    ->paginate(10);
+                $payments = $paymentHistory->paginate(10);
-                $pager = $this->paymentModel->pager;
+                $pager = $paymentHistory->pager;
 
                 // Invoices
                 $rawInvoices = $this->invoiceModel
@@ -972,8 +974,7 @@ class PaymentController extends ResourceController
                 $checkFile,
                 $transactionId,
                 $paymentDate,
-                $this->schoolYear,
+                $invYear,
-                $this->semester,
                 $checkNumber,
                 $installmentSeq,
                 (array) $this->invoiceModel->find($invoiceId),
@@ -1015,7 +1016,7 @@ class PaymentController extends ResourceController
             $rowParentDisc = $this->db->table('discount_usages')
                 ->selectSum('discount_amount', 'sum_disc')
                 ->where('parent_id', $parentId)
-                ->where('school_year', $this->schoolYear)
+                ->where('school_year', $invYear)
                 ->get()->getRowArray();
             if ($rowParentDisc && isset($rowParentDisc['sum_disc'])) {
                 $parentYearDiscountTotal = (float)$rowParentDisc['sum_disc'];
@@ -1270,7 +1271,6 @@ class PaymentController extends ResourceController
         $transactionId = null,
         $paymentDate = null,
         $schoolYear = null,
-        $semester = null,
         $checkNumber = null,
         ?int $installmentSeq = null,
         ?array $invoice = null,
@@ -1315,8 +1315,7 @@ class PaymentController extends ResourceController
             'check_file' => $checkFile,
             'check_number' => (strtolower($paymentMethod) === 'check') ? $checkNumber : null,
             'updated_by' => session()->get('user_id'),
-            'school_year' => $schoolYear ?? $this->schoolYear,
+            'school_year' => $schoolYear ?? ($invoice['school_year'] ?? $this->schoolYear),
-            'semester' => $semester ?? $this->semester,
         ];
 
         if (!$this->paymentModel->insert($paymentData)) {

app/Controllers/View/SchoolYearController.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace App\Controllers\View;
+
+/**
+ * Compatibility wrapper for older references.
+ *
+ * School-year lifecycle behavior lives in App\Controllers\Administrator,
+ * where status changes are exposed only as dedicated actions.
+ */
+class SchoolYearController extends \App\Controllers\Administrator\SchoolYearController
+{
+}

app/Controllers/View/WhatsappController.php

@@ -106,14 +106,12 @@ class WhatsappController extends BaseController
         $existing = $this->linkModel->where([
             'class_section_id' => $sectionId,
             'school_year'      => $this->schoolYear,
-            'semester'         => $this->semester,
         ])->first();
 
         $payload = [
             'class_section_id'   => $sectionId,
             'class_section_name' => $sectionName,
             'school_year'        => $this->schoolYear,
-            'semester'           => $this->semester,
             'invite_link'        => trim($inviteLink),
             'active'             => $active,
         ];
@@ -434,8 +432,6 @@ class WhatsappController extends BaseController
         sp.secondparent_lastname            AS sp_lastname,
         sp.secondparent_email               AS sp_email,
         sp.secondparent_phone               AS sp_phone,
-        sp.school_year                      AS sp_school_year,
-        sp.semester                         AS sp_semester,
 
         u.id                                AS u_id,
         u.firstname                         AS u_firstname,
@@ -448,14 +444,6 @@ class WhatsappController extends BaseController
 
         $b->join('users u', 'u.id = sp.firstparent_id', 'left');
 
-        // Scope by term using parents table (authoritative for the pairing)
-        if ($schoolYear !== '') {
-            $b->where('sp.school_year', $schoolYear);
-        }
-        if ($semester !== '') {
-            $b->where('sp.semester', $semester);
-        }
-
         $rows = $b->get()->getResultArray();
 
         // Flatten into a single contacts list (primary + second parent as separate rows)
@@ -606,11 +594,10 @@ class WhatsappController extends BaseController
         $pb->join('students s', 's.id = sc.student_id', 'inner');          // adjust if your table is named `student` (singular)
         $pb->join('users u',    'u.id = s.parent_id',   'inner');          // primary parent lives on students.parent_id
 
-        // Second parent row for the same term (allow NULL/'' semester if you sometimes omit it)
+        // Second parent row for the primary parent.
         $pb->join(
             'parents sp',
-            "sp.firstparent_id = u.id
+            'sp.firstparent_id = u.id',
-         AND sp.school_year = sc.school_year",
             'left'
         );
 
@@ -1202,7 +1189,7 @@ class WhatsappController extends BaseController
 
     /**
      * Class mode:
-     * - Given classSectionId, find students (student_class) for the term
+     * - Given classSectionId, find students (student_class) for the year
      * - Map to primaries (users) and second-parents (parents)
      * - Return one bundle **per primary parent** (so each parent gets their own email)
      */
@@ -1215,7 +1202,6 @@ class WhatsappController extends BaseController
         $stuRows = $this->db->table('student_class')
             ->select('student_id')
             ->where('school_year', $this->schoolYear)
-            ->where('semester', $this->semester)
             ->where('class_section_id', $classSectionId)
             ->get()->getResultArray();
         if (empty($stuRows)) return [];
@@ -1302,7 +1288,7 @@ class WhatsappController extends BaseController
 
     /**
      * All mode:
-     * - Iterate all distinct class_section_id in student_class (for the term)
+     * - Iterate all distinct class_section_id in student_class (for the year)
      * - Reuse class bundles per section and flatten
      */
     private function bundlesForAllParentsAllClasses(array $linkBySection): array
@@ -1310,7 +1296,6 @@ class WhatsappController extends BaseController
         $secRows = $this->db->table('student_class')
             ->select('DISTINCT class_section_id', false)
             ->where('school_year', $this->schoolYear)
-            ->where('semester', $this->semester)
             ->orderBy('class_section_id', 'ASC')
             ->get()->getResultArray();
 

app/Database/Migrations/2026-01-05-215851_FixPrintRequestsForeignKey.php

@@ -11,6 +11,10 @@ class FixPrintRequestsForeignKey extends Migration
         // Drop the old foreign key if it exists
         $this->forge->dropForeignKey('print_requests', 'print_requests_class_id_foreign');
 
+        if (! $this->db->tableExists('class_sections')) {
+            return;
+        }
+
         // Add the new foreign key
         $this->forge->addForeignKey('class_id', 'class_sections', 'id', 'CASCADE', 'CASCADE');
         

app/Database/Migrations/2026-01-06-195508_RevertPrintRequestsForeignKey.php

@@ -9,7 +9,14 @@ class RevertPrintRequestsForeignKey extends Migration
     public function up()
     {
         // Drop the old foreign key if it exists
+        $db = \Config\Database::connect();
+        $keys = $db->getForeignKeyData('print_requests');
+        foreach ($keys as $key) {
+            if ($key->constraint_name === 'print_requests_class_id_foreign') {
                 $this->forge->dropForeignKey('print_requests', 'print_requests_class_id_foreign');
+                break;
+            }
+        }
 
         // Add the new foreign key
         $this->forge->addForeignKey('class_id', 'classes', 'id', 'CASCADE', 'CASCADE');

app/Database/Migrations/2026-01-06-205621_FixPrintRequestsForeignKeyAgain.php

@@ -12,8 +12,7 @@ class FixPrintRequestsForeignKeyAgain extends Migration
         $keys = $db->getForeignKeyData('print_requests');
         foreach ($keys as $key) {
             if ($key->constraint_name === 'print_requests_class_id_foreign') {
-                $this->forge->dropForeignKey('print_requests', 'print_requests_class_id_foreign');
+                return;
-                break;
             }
         }
 

app/Database/Migrations/2026-01-25-000000_RemoveClassAssignmentSemester.php

@@ -8,12 +8,8 @@ class RemoveClassAssignmentSemester extends Migration
 {
     public function up()
     {
-        if ($this->db->tableExists('teacher_class')) {
+        // Keep the legacy semester columns in place. They may be ignored by newer
-            $this->forge->dropColumn('teacher_class', 'semester');
+        // code, but dropping them during a post-restore migrate destroys data.
-        }
-        if ($this->db->tableExists('student_class')) {
-            $this->forge->dropColumn('student_class', 'semester');
-        }
     }
 
     public function down()
@@ -39,4 +35,13 @@ class RemoveClassAssignmentSemester extends Migration
             ]);
         }
     }
+
+    private function indexExists(string $table, string $index): bool
+    {
+        $result = $this->db->query(
+            'SHOW INDEX FROM `' . str_replace('`', '``', $table) . '` WHERE Key_name = ' . $this->db->escape($index)
+        );
+
+        return $result->getNumRows() > 0;
+    }
 }

app/Database/Migrations/2026-01-26-000100_CreateTeacherSubmissionNotificationHistory.php

@@ -3,6 +3,7 @@
 namespace App\Database\Migrations;
 
 use CodeIgniter\Database\Migration;
+use CodeIgniter\Database\RawSql;
 
 class CreateTeacherSubmissionNotificationHistory extends Migration
 {
@@ -56,7 +57,7 @@ class CreateTeacherSubmissionNotificationHistory extends Migration
             'sent_at' => [
                 'type' => 'DATETIME',
                 'null' => false,
-                'default' => 'CURRENT_TIMESTAMP',
+                'default' => new RawSql('CURRENT_TIMESTAMP'),
             ],
         ]);
 

app/Database/Migrations/2026-01-27-130500_CreateExamDraftSubmissions.php

@@ -3,6 +3,7 @@
 namespace App\Database\Migrations;
 
 use CodeIgniter\Database\Migration;
+use CodeIgniter\Database\RawSql;
 
 class CreateExamDraftSubmissions extends Migration
 {
@@ -89,7 +90,7 @@ class CreateExamDraftSubmissions extends Migration
             'created_at' => [
                 'type'     => 'DATETIME',
                 'null'     => false,
-                'default'  => 'CURRENT_TIMESTAMP',
+                'default'  => new RawSql('CURRENT_TIMESTAMP'),
             ],
             'updated_at' => [
                 'type' => 'DATETIME',

app/Database/Migrations/2026-02-04-000400_CreatePlacementLevels.php

@@ -24,7 +24,7 @@ class CreatePlacementLevels extends Migration
         ]);
 
         $this->forge->addKey('id', true);
-        $this->forge->addKey(['student_id', 'school_year'], true);
+        $this->forge->addUniqueKey(['student_id', 'school_year'], 'unique_student_school_year');
         $this->forge->addKey('school_year');
         $this->forge->createTable('placement_levels');
     }

app/Database/Migrations/2026-02-04-000430_CreatePlacementScores.php

@@ -24,7 +24,7 @@ class CreatePlacementScores extends Migration
         ]);
 
         $this->forge->addKey('id', true);
-        $this->forge->addKey(['batch_id', 'student_id'], true);
+        $this->forge->addUniqueKey(['batch_id', 'student_id'], 'unique_batch_student');
         $this->forge->addKey('batch_id');
         $this->forge->createTable('placement_scores');
     }

app/Database/Migrations/2026-02-05-000200_CreateMissingScoreOverrides.php

@@ -65,7 +65,10 @@ class CreateMissingScoreOverrides extends Migration
 
         $this->forge->addKey('id', true);
         $this->forge->addKey(['class_section_id', 'semester', 'school_year', 'item_type'], false, 'missing_score_overrides_scope');
-        $this->forge->addKey(['student_id', 'class_section_id', 'semester', 'school_year', 'item_type', 'item_index'], true, 'missing_score_overrides_unique');
+        $this->forge->addUniqueKey(
+            ['student_id', 'class_section_id', 'semester', 'school_year', 'item_type', 'item_index'],
+            'missing_score_overrides_unique'
+        );
         $this->forge->createTable('missing_score_overrides', true);
     }
 

app/Database/Migrations/2026-02-10-000100_AddStyleToPreferences.php

@@ -10,13 +10,18 @@ class AddStyleToPreferences extends Migration
     {
         if ($this->db->tableExists('user_preferences')) {
             if (! $this->db->fieldExists('style_color', 'user_preferences')) {
-                $this->forge->addColumn('user_preferences', [
+                $styleColor = [
-                    'style_color' => [
                     'type'       => 'VARCHAR',
                     'constraint' => 32,
                     'null'       => true,
-                        'after'      => 'timezone',
+                ];
-                    ],
+
+                if ($this->db->fieldExists('timezone', 'user_preferences')) {
+                    $styleColor['after'] = 'timezone';
+                }
+
+                $this->forge->addColumn('user_preferences', [
+                    'style_color' => $styleColor,
                 ]);
             }
             if (! $this->db->fieldExists('menu_color', 'user_preferences')) {

app/Database/Migrations/2026-05-30-000001_FinancialSystemLedgerCleanup.php

@@ -9,7 +9,11 @@ class FinancialSystemLedgerCleanup extends Migration
     public function up()
     {
         $this->addInstallmentSequenceColumn();
+        $this->backfillInstallmentSequence();
+        $this->ensurePaymentDateHasTime();
         $this->ensureIndexes();
+        $this->repairPaymentInvoiceTerms();
+        $this->normalizePaymentStatuses();
         $this->ensureConfigurationDefaults();
         $this->archivePaypalTables();
         $this->refreshFinancialNavItems();
@@ -24,6 +28,7 @@ class FinancialSystemLedgerCleanup extends Migration
         }
 
         $this->dropIndexIfExists('payments', 'idx_payments_invoice_id');
+        $this->dropIndexIfExists('payments', 'idx_payments_parent_year');
         $this->dropIndexIfExists('payments', 'idx_payments_parent_year_semester');
         $this->dropIndexIfExists('payments', 'uniq_payments_transaction_id');
         $this->dropIndexIfExists('invoices', 'idx_invoices_parent_year_semester');
@@ -68,10 +73,33 @@ class FinancialSystemLedgerCleanup extends Migration
         }
     }
 
+    protected function backfillInstallmentSequence(): void
+    {
+        if (!$this->db->tableExists('payments') || !$this->db->fieldExists('installment_seq', 'payments')) {
+            return;
+        }
+
+        $this->db->query(
+            'UPDATE `payments`
+                SET `installment_seq` = `number_of_installments`
+              WHERE `installment_seq` IS NULL'
+        );
+    }
+
+    protected function ensurePaymentDateHasTime(): void
+    {
+        if (!$this->db->tableExists('payments') || !$this->db->fieldExists('payment_date', 'payments')) {
+            return;
+        }
+
+        $this->db->query('ALTER TABLE `payments` MODIFY `payment_date` DATETIME NOT NULL');
+    }
+
     protected function ensureIndexes(): void
     {
         $this->addIndexIfMissing('payments', 'idx_payments_invoice_id', ['invoice_id']);
-        $this->addIndexIfMissing('payments', 'idx_payments_parent_year_semester', ['parent_id', 'school_year', 'semester']);
+        $this->dropIndexIfExists('payments', 'idx_payments_parent_year_semester');
+        $this->addIndexIfMissing('payments', 'idx_payments_parent_year', ['parent_id', 'school_year']);
         $this->addIndexIfMissing('invoices', 'idx_invoices_parent_year_semester', ['parent_id', 'school_year', 'semester']);
         $this->addIndexIfMissing('discount_usages', 'idx_discount_usages_invoice_id', ['invoice_id']);
         $this->addIndexIfMissing('discount_usages', 'idx_discount_usages_parent_year_semester', ['parent_id', 'school_year', 'semester']);
@@ -86,6 +114,33 @@ class FinancialSystemLedgerCleanup extends Migration
         }
     }
 
+    protected function repairPaymentInvoiceTerms(): void
+    {
+        if (!$this->db->tableExists('payments') || !$this->db->tableExists('invoices')) {
+            return;
+        }
+
+        $this->db->query(
+            'UPDATE `payments` p
+                JOIN `invoices` i ON i.`id` = p.`invoice_id`
+                 SET p.`school_year` = i.`school_year`
+               WHERE COALESCE(p.`school_year`, \'\') <> COALESCE(i.`school_year`, \'\')'
+        );
+    }
+
+    protected function normalizePaymentStatuses(): void
+    {
+        if (!$this->db->tableExists('payments') || !$this->db->fieldExists('status', 'payments')) {
+            return;
+        }
+
+        $this->db->query(
+            "UPDATE `payments`
+                SET `status` = 'recorded'
+              WHERE LOWER(TRIM(`status`)) IN ('paid', 'partially paid', 'payment recorded', 'full', 'completed')"
+        );
+    }
+
     protected function ensureConfigurationDefaults(): void
     {
         if (!$this->db->tableExists('configuration')) {
@@ -128,8 +183,8 @@ class FinancialSystemLedgerCleanup extends Migration
 
     protected function archivePaypalTables(): void
     {
-        $this->renameTableIfPresent('paypal_payments', 'archived_paypal_payments');
+        // Keep legacy PayPal tables under their original names. Renaming them
-        $this->renameTableIfPresent('paypal_transactions', 'archived_paypal_transactions');
+        // during migrate makes restored data appear to disappear from the app.
     }
 
     protected function restorePaypalTables(): void

app/Database/Migrations/2026-07-12-040600_FixSchoolYearColumns.php

@@ -0,0 +1,363 @@
+<?php
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+
+class FixSchoolYearColumns extends Migration
+{
+    /**
+     * Tables that directly own an academic year.
+     *
+     * school_year stays as text by design. Valid values use YYYY-YYYY and
+     * the second year must be exactly the first year + 1.
+     */
+    private array $yearOwnerTables = [
+        'additional_charges',
+        'archived_paypal_transactions',
+        'attendance_data',
+        'attendance_day',
+        'attendance_record',
+        'attendance_tracking',
+        'badge_print_logs',
+        'below_sixty_decisions',
+        'calendar_events',
+        'certificate_records',
+        'classSection',
+        'class_progress_reports',
+        'competitions',
+        'current_flag',
+        'discount_vouchers',
+        'early_dismissal_signatures',
+        'enrollments',
+        'events',
+        'exams',
+        'exam_drafts',
+        'expenses',
+        'final_exam',
+        'final_score',
+        'flag',
+        'grading_locks',
+        'homework',
+        'inventory_movements',
+        'invoices',
+        'late_slip_logs',
+        'manual_payments',
+        'midterm_exam',
+        'missing_score_overrides',
+        'parent_attendance_reports',
+        'parent_meeting_schedules',
+        'parent_notifications',
+        'participation',
+        'payments',
+        'placement_batches',
+        'print_requests',
+        'project',
+        'quiz',
+        'refunds',
+        'reimbursements',
+        'reimbursement_batches',
+        'report_card_acknowledgements',
+        'scan_log',
+        'score_comments',
+        'semester_scores',
+        'staff_attendance',
+        'student_class',
+        'student_decisions',
+        'teacher_attendance_data',
+        'teacher_class',
+        'teacher_submission_notification_history',
+        'whatsapp_group_links',
+        'whatsapp_group_memberships',
+    ];
+
+    /** Tables where school_year is redundant or conceptually incorrect. */
+    private array $tablesWithoutDirectYear = [
+        'chapters',
+        'classes',
+        'class_preparation_log',
+        'class_prep_adjustments',
+        'contactus',
+        'emergency_contacts',
+        'inventory_items',
+        'invoice_students_list',
+        'ip_attempts',
+        'login_activity',
+        'messages',
+        'notifications',
+        'notification_recipients',
+        'parents',
+        'paypal_transactions',
+        'placement_levels',
+        'preferences',
+        'staff',
+        'students',
+        'support_requests',
+        'users',
+        'user_notifications',
+    ];
+
+    /** Tables where semester is redundant or conceptually incorrect. */
+    private array $tablesWithoutDirectSemester = [
+        'badge_print_logs',
+        'classes',
+        'contactus',
+        'emergency_contacts',
+        'inventory_items',
+        'invoice_students_list',
+        'ip_attempts',
+        'notification_recipients',
+        'notifications',
+        'parents',
+        'support_requests',
+        'user_notifications',
+        'whatsapp_group_links',
+    ];
+
+    public function up(): void
+    {
+        if ($this->db->DBDriver !== 'MySQLi') {
+            throw new \RuntimeException(
+                'This migration targets MySQL 8 because it uses enforced CHECK constraints.'
+            );
+        }
+
+        $this->db->resetDataCache();
+
+        // These annual entities lacked a school_year column in the audited schema.
+        foreach (['class_progress_reports', 'exams', 'print_requests'] as $table) {
+            if ($this->db->tableExists($table) && ! $this->db->fieldExists('school_year', $table)) {
+                // Nullable avoids inventing a year for existing rows.
+                // Backfill it, then make it NOT NULL in a later migration.
+                $this->forge->addColumn($table, [
+                    'school_year' => [
+                        'type'       => 'VARCHAR',
+                        'constraint' => 9,
+                        'null'       => true,
+                    ],
+                ]);
+            }
+        }
+
+        $this->db->resetDataCache();
+
+        // Validate and normalize every direct owner to VARCHAR(9).
+        foreach ($this->yearOwnerTables as $table) {
+            if (! $this->db->tableExists($table) || ! $this->db->fieldExists('school_year', $table)) {
+                continue;
+            }
+
+            $this->assertValidSchoolYears($table, 'school_year');
+            $this->normalizeYearColumn($table, 'school_year');
+            $this->ensureYearCheckConstraint($table, 'school_year');
+            $this->ensureYearIndex($table, 'school_year');
+        }
+
+        // Promotion is a transition and legitimately owns both source and target years.
+        if ($this->db->tableExists('promotion_queue')) {
+            foreach (['school_year_from', 'school_year_to'] as $column) {
+                if (! $this->db->fieldExists($column, 'promotion_queue')) {
+                    continue;
+                }
+
+                $this->assertValidSchoolYears('promotion_queue', $column);
+                $this->normalizeYearColumn('promotion_queue', $column);
+                $this->ensureYearCheckConstraint('promotion_queue', $column);
+                $this->ensureYearIndex('promotion_queue', $column);
+            }
+        }
+
+        $this->db->resetDataCache();
+
+        // Do not drop redundant school_year/semester columns here. Restored
+        // production dumps may still contain meaningful historical data in
+        // those columns, and running migrate must not destroy it.
+
+        $this->db->resetDataCache();
+
+        foreach ($this->yearOwnerTables as $table) {
+            if (! $this->db->tableExists($table) || ! $this->db->fieldExists('school_year', $table)) {
+                continue;
+            }
+
+            $this->ensureYearIndex($table, 'school_year');
+        }
+    }
+
+    public function down(): void
+    {
+        throw new \RuntimeException(
+            'This migration is intentionally irreversible because dropping school_year columns destroys data. Restore from backup instead of pretending rollback can resurrect it.'
+        );
+    }
+
+    private function assertValidSchoolYears(string $table, string $column): void
+    {
+        $tableName = $this->quoteIdentifier($table);
+        $columnName = $this->quoteIdentifier($column);
+
+        $invalid = $this->db->query(
+            "SELECT COUNT(*) AS aggregate
+             FROM {$tableName}
+             WHERE {$columnName} IS NOT NULL
+               AND (
+                    {$columnName} NOT REGEXP '^[0-9]{4}-[0-9]{4}$'
+                    OR CAST(RIGHT({$columnName}, 4) AS UNSIGNED)
+                       <> CAST(LEFT({$columnName}, 4) AS UNSIGNED) + 1
+               )"
+        )->getRow();
+
+        if ((int) ($invalid->aggregate ?? 0) > 0) {
+            throw new \RuntimeException(
+                "Cannot migrate {$table}.{$column}: found {$invalid->aggregate} invalid school-year value(s). Expected YYYY-YYYY with consecutive years, for example 2025-2026."
+            );
+        }
+    }
+
+    private function normalizeYearColumn(string $table, string $column): void
+    {
+        $metadata = $this->db->query(
+            'SELECT IS_NULLABLE, COLLATION_NAME
+             FROM information_schema.COLUMNS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?',
+            [$table, $column]
+        )->getRow();
+
+        if ($metadata === null) {
+            return;
+        }
+
+        $nullable = $metadata->IS_NULLABLE === 'YES' ? 'NULL' : 'NOT NULL';
+        $collation = $metadata->COLLATION_NAME && preg_match('/^[A-Za-z0-9_]+$/', $metadata->COLLATION_NAME)
+            ? ' COLLATE ' . $metadata->COLLATION_NAME
+            : '';
+
+        $this->db->query(sprintf(
+            'ALTER TABLE %s MODIFY COLUMN %s VARCHAR(9)%s %s',
+            $this->quoteIdentifier($table),
+            $this->quoteIdentifier($column),
+            $collation,
+            $nullable
+        ));
+    }
+
+    private function ensureYearCheckConstraint(string $table, string $column): void
+    {
+        $constraint = $this->objectName('chk_sy', $table, $column);
+
+        $exists = $this->db->query(
+            'SELECT COUNT(*) AS aggregate
+             FROM information_schema.TABLE_CONSTRAINTS
+             WHERE CONSTRAINT_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND CONSTRAINT_NAME = ?
+               AND CONSTRAINT_TYPE = \'CHECK\'',
+            [$table, $constraint]
+        )->getRow();
+
+        if ((int) ($exists->aggregate ?? 0) > 0) {
+            return;
+        }
+
+        $columnName = $this->quoteIdentifier($column);
+
+        $this->db->query(sprintf(
+            "ALTER TABLE %s ADD CONSTRAINT %s CHECK (
+                %s IS NULL OR (
+                    %s REGEXP '^[0-9]{4}-[0-9]{4}$'
+                    AND CAST(RIGHT(%s, 4) AS UNSIGNED)
+                        = CAST(LEFT(%s, 4) AS UNSIGNED) + 1
+                )
+            )",
+            $this->quoteIdentifier($table),
+            $this->quoteIdentifier($constraint),
+            $columnName,
+            $columnName,
+            $columnName,
+            $columnName
+        ));
+    }
+
+    private function ensureYearIndex(string $table, string $column): void
+    {
+        $indexed = $this->db->query(
+            'SELECT COUNT(*) AS aggregate
+             FROM information_schema.STATISTICS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?',
+            [$table, $column]
+        )->getRow();
+
+        if ((int) ($indexed->aggregate ?? 0) > 0) {
+            return;
+        }
+
+        $index = $this->objectName('idx_sy', $table, $column);
+
+        $this->db->query(sprintf(
+            'ALTER TABLE %s ADD INDEX %s (%s)',
+            $this->quoteIdentifier($table),
+            $this->quoteIdentifier($index),
+            $this->quoteIdentifier($column)
+        ));
+    }
+
+    private function dropIndexesContainingColumn(string $table, string $column): void
+    {
+        $indexes = $this->db->query(
+            'SELECT DISTINCT INDEX_NAME
+             FROM information_schema.STATISTICS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?
+               AND INDEX_NAME <> \'PRIMARY\'',
+            [$table, $column]
+        )->getResult();
+
+        foreach ($indexes as $index) {
+            $this->db->query(sprintf(
+                'ALTER TABLE %s DROP INDEX %s',
+                $this->quoteIdentifier($table),
+                $this->quoteIdentifier($index->INDEX_NAME)
+            ));
+        }
+    }
+
+    private function dropChecksContainingColumn(string $table, string $column): void
+    {
+        $checks = $this->db->query(
+            'SELECT tc.CONSTRAINT_NAME
+             FROM information_schema.TABLE_CONSTRAINTS tc
+             INNER JOIN information_schema.CHECK_CONSTRAINTS cc
+                ON cc.CONSTRAINT_SCHEMA = tc.CONSTRAINT_SCHEMA
+               AND cc.CONSTRAINT_NAME = tc.CONSTRAINT_NAME
+             WHERE tc.CONSTRAINT_SCHEMA = DATABASE()
+               AND tc.TABLE_NAME = ?
+               AND tc.CONSTRAINT_TYPE = \'CHECK\'
+               AND cc.CHECK_CLAUSE LIKE ?',
+            [$table, '%' . $column . '%']
+        )->getResult();
+
+        foreach ($checks as $check) {
+            $this->db->query(sprintf(
+                'ALTER TABLE %s DROP CHECK %s',
+                $this->quoteIdentifier($table),
+                $this->quoteIdentifier($check->CONSTRAINT_NAME)
+            ));
+        }
+    }
+
+    private function objectName(string $prefix, string $table, string $column): string
+    {
+        // MySQL identifiers are limited to 64 characters.
+        return substr($prefix . '_' . $table . '_' . $column . '_' . substr(sha1($table . '.' . $column), 0, 8), 0, 64);
+    }
+
+    private function quoteIdentifier(string $identifier): string
+    {
+        return '`' . str_replace('`', '``', $identifier) . '`';
+    }
+}

app/Database/Migrations/2026-07-12-041500_EnsureSchoolYearOnFinancialTables.php

@@ -0,0 +1,141 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+use RuntimeException;
+
+final class EnsureSchoolYearOnFinancialTables extends Migration
+{
+    /** @var array<string, bool> table => nullable */
+    private array $tables = [
+        'discount_usages'            => false,
+        'event_charges'              => true,
+        'invoice_event'              => true,
+        'payment_error'              => true,
+        'payment_notification_logs'  => true,
+        'payment_transactions'       => true,
+        'reimbursement_batch_items'  => true,
+    ];
+
+    public function up(): void
+    {
+        if ($this->db->DBDriver !== 'MySQLi') {
+            throw new RuntimeException('This migration requires MySQL/MariaDB through the MySQLi driver.');
+        }
+
+        foreach ($this->tables as $table => $nullable) {
+            if (! $this->db->tableExists($table)) {
+                continue;
+            }
+
+            if (! $this->db->fieldExists('school_year', $table)) {
+                $this->forge->addColumn($table, [
+                    'school_year' => [
+                        'type'       => 'VARCHAR',
+                        'constraint' => 9,
+                        'null'       => $nullable,
+                        'after'      => $this->afterColumn($table),
+                    ],
+                ]);
+            }
+
+            $this->backfillSchoolYear($table);
+            $this->assertValidYears($table);
+
+            $nullSql = $nullable ? 'NULL' : 'NOT NULL';
+            $this->db->query(
+                sprintf(
+                    'ALTER TABLE `%s` MODIFY `school_year` VARCHAR(9) %s',
+                    str_replace('`', '``', $table),
+                    $nullSql
+                )
+            );
+
+            $this->ensureIndex($table);
+        }
+    }
+
+    public function down(): void
+    {
+        // These columns are part of the business model and must not be dropped on rollback.
+    }
+
+    private function assertValidYears(string $table): void
+    {
+        $sql = sprintf(
+            "SELECT COUNT(*) AS invalid_count
+             FROM `%s`
+             WHERE `school_year` IS NOT NULL
+               AND (
+                    `school_year` NOT REGEXP '^[0-9]{4}-[0-9]{4}$'
+                    OR CAST(RIGHT(`school_year`, 4) AS UNSIGNED)
+                       <> CAST(LEFT(`school_year`, 4) AS UNSIGNED) + 1
+               )",
+            str_replace('`', '``', $table)
+        );
+
+        $row = $this->db->query($sql)->getRowArray();
+        $invalid = (int) ($row['invalid_count'] ?? 0);
+
+        if ($invalid > 0) {
+            throw new RuntimeException(
+                "Cannot normalize {$table}.school_year: {$invalid} invalid value(s). Expected YYYY-YYYY, for example 2025-2026."
+            );
+        }
+    }
+
+    private function backfillSchoolYear(string $table): void
+    {
+        if ($table === 'discount_usages' && $this->db->tableExists('invoices')) {
+            $this->db->query(
+                "UPDATE `discount_usages` du
+                 INNER JOIN `invoices` i ON i.`id` = du.`invoice_id`
+                 SET du.`school_year` = i.`school_year`
+                 WHERE (du.`school_year` IS NULL OR TRIM(du.`school_year`) = '')
+                   AND i.`school_year` REGEXP '^[0-9]{4}-[0-9]{4}$'
+                   AND CAST(RIGHT(i.`school_year`, 4) AS UNSIGNED)
+                       = CAST(LEFT(i.`school_year`, 4) AS UNSIGNED) + 1"
+            );
+        }
+    }
+
+    private function ensureIndex(string $table): void
+    {
+        $indexName = 'idx_' . $table . '_school_year';
+        if (strlen($indexName) > 64) {
+            $indexName = 'idx_' . substr(hash('sha256', $table . '_school_year'), 0, 24);
+        }
+
+        $row = $this->db->query(
+            'SELECT COUNT(*) AS index_count
+             FROM information_schema.STATISTICS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?',
+            [$table, 'school_year']
+        )->getRowArray();
+
+        if ((int) ($row['index_count'] ?? 0) === 0) {
+            $this->db->query(sprintf(
+                'CREATE INDEX `%s` ON `%s` (`school_year`)',
+                str_replace('`', '``', $indexName),
+                str_replace('`', '``', $table)
+            ));
+        }
+    }
+
+    private function afterColumn(string $table): string
+    {
+        $preferred = ['semester', 'invoice_id', 'payment_id', 'batch_id'];
+        foreach ($preferred as $column) {
+            if ($this->db->fieldExists($column, $table)) {
+                return $column;
+            }
+        }
+
+        return 'id';
+    }
+}

app/Database/Migrations/2026-07-12-044500_ApplySchoolYearSemesterAuditCorrections.php

@@ -0,0 +1,249 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+use RuntimeException;
+
+final class ApplySchoolYearSemesterAuditCorrections extends Migration
+{
+    private array $schoolYearIndexTables = [
+        'archived_paypal_transactions',
+        'class_progress_reports',
+        'exams',
+        'missing_score_overrides',
+        'payments',
+        'placement_batches',
+        'print_requests',
+        'report_card_acknowledgements',
+        'semester_scores',
+        'staff_attendance',
+        'teacher_attendance_data',
+        'whatsapp_group_links',
+    ];
+
+    private array $semesterDropTables = [
+        'badge_print_logs',
+        'classes',
+        'contactus',
+        'emergency_contacts',
+        'inventory_items',
+        'invoice_students_list',
+        'ip_attempts',
+        'notification_recipients',
+        'notifications',
+        'parents',
+        'support_requests',
+        'user_notifications',
+        'whatsapp_group_links',
+    ];
+
+    public function up(): void
+    {
+        if ($this->db->DBDriver !== 'MySQLi') {
+            throw new RuntimeException('This migration requires MySQL/MariaDB through the MySQLi driver.');
+        }
+
+        if ($this->db->tableExists('archived_paypal_transactions')
+            && $this->db->fieldExists('school_year', 'archived_paypal_transactions')) {
+            $this->assertValidSchoolYears('archived_paypal_transactions', 'school_year');
+            $this->normalizeYearColumn('archived_paypal_transactions', 'school_year');
+            $this->ensureYearCheckConstraint('archived_paypal_transactions', 'school_year');
+        }
+
+        foreach ($this->schoolYearIndexTables as $table) {
+            if (! $this->db->tableExists($table) || ! $this->db->fieldExists('school_year', $table)) {
+                continue;
+            }
+
+            $this->ensureIndex($table, 'school_year');
+        }
+
+        // Preserve legacy semester columns. They may be redundant for newer
+        // code paths, but migration must not delete restored production data.
+
+        foreach ($this->schoolYearIndexTables as $table) {
+            if (! $this->db->tableExists($table) || ! $this->db->fieldExists('school_year', $table)) {
+                continue;
+            }
+
+            $this->ensureIndex($table, 'school_year');
+        }
+    }
+
+    public function down(): void
+    {
+        throw new RuntimeException(
+            'This migration is intentionally irreversible because dropping semester columns destroys data.'
+        );
+    }
+
+    private function assertValidSchoolYears(string $table, string $column): void
+    {
+        $tableName = $this->quoteIdentifier($table);
+        $columnName = $this->quoteIdentifier($column);
+
+        $invalid = $this->db->query(
+            "SELECT COUNT(*) AS aggregate
+             FROM {$tableName}
+             WHERE {$columnName} IS NOT NULL
+               AND (
+                    {$columnName} NOT REGEXP '^[0-9]{4}-[0-9]{4}$'
+                    OR CAST(RIGHT({$columnName}, 4) AS UNSIGNED)
+                       <> CAST(LEFT({$columnName}, 4) AS UNSIGNED) + 1
+               )"
+        )->getRow();
+
+        if ((int) ($invalid->aggregate ?? 0) > 0) {
+            throw new RuntimeException(
+                "Cannot migrate {$table}.{$column}: found {$invalid->aggregate} invalid school-year value(s)."
+            );
+        }
+    }
+
+    private function normalizeYearColumn(string $table, string $column): void
+    {
+        $metadata = $this->db->query(
+            'SELECT IS_NULLABLE, COLLATION_NAME
+             FROM information_schema.COLUMNS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?',
+            [$table, $column]
+        )->getRow();
+
+        if ($metadata === null) {
+            return;
+        }
+
+        $nullable = $metadata->IS_NULLABLE === 'YES' ? 'NULL' : 'NOT NULL';
+        $collation = $metadata->COLLATION_NAME && preg_match('/^[A-Za-z0-9_]+$/', $metadata->COLLATION_NAME)
+            ? ' COLLATE ' . $metadata->COLLATION_NAME
+            : '';
+
+        $this->db->query(sprintf(
+            'ALTER TABLE %s MODIFY COLUMN %s VARCHAR(9)%s %s',
+            $this->quoteIdentifier($table),
+            $this->quoteIdentifier($column),
+            $collation,
+            $nullable
+        ));
+    }
+
+    private function ensureYearCheckConstraint(string $table, string $column): void
+    {
+        $constraint = $this->objectName('chk_sy', $table, $column);
+
+        $exists = $this->db->query(
+            'SELECT COUNT(*) AS aggregate
+             FROM information_schema.TABLE_CONSTRAINTS
+             WHERE CONSTRAINT_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND CONSTRAINT_NAME = ?
+               AND CONSTRAINT_TYPE = \'CHECK\'',
+            [$table, $constraint]
+        )->getRow();
+
+        if ((int) ($exists->aggregate ?? 0) > 0) {
+            return;
+        }
+
+        $columnName = $this->quoteIdentifier($column);
+
+        $this->db->query(sprintf(
+            "ALTER TABLE %s ADD CONSTRAINT %s CHECK (
+                %s IS NULL OR (
+                    %s REGEXP '^[0-9]{4}-[0-9]{4}$'
+                    AND CAST(RIGHT(%s, 4) AS UNSIGNED)
+                        = CAST(LEFT(%s, 4) AS UNSIGNED) + 1
+                )
+            )",
+            $this->quoteIdentifier($table),
+            $this->quoteIdentifier($constraint),
+            $columnName,
+            $columnName,
+            $columnName,
+            $columnName
+        ));
+    }
+
+    private function ensureIndex(string $table, string $column): void
+    {
+        $indexed = $this->db->query(
+            'SELECT COUNT(*) AS aggregate
+             FROM information_schema.STATISTICS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?',
+            [$table, $column]
+        )->getRow();
+
+        if ((int) ($indexed->aggregate ?? 0) > 0) {
+            return;
+        }
+
+        $this->db->query(sprintf(
+            'ALTER TABLE %s ADD INDEX %s (%s)',
+            $this->quoteIdentifier($table),
+            $this->quoteIdentifier($this->objectName('idx_sy', $table, $column)),
+            $this->quoteIdentifier($column)
+        ));
+    }
+
+    private function dropIndexesContainingColumn(string $table, string $column): void
+    {
+        $indexes = $this->db->query(
+            'SELECT DISTINCT INDEX_NAME
+             FROM information_schema.STATISTICS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?
+               AND INDEX_NAME <> \'PRIMARY\'',
+            [$table, $column]
+        )->getResult();
+
+        foreach ($indexes as $index) {
+            $this->db->query(sprintf(
+                'ALTER TABLE %s DROP INDEX %s',
+                $this->quoteIdentifier($table),
+                $this->quoteIdentifier($index->INDEX_NAME)
+            ));
+        }
+    }
+
+    private function dropChecksContainingColumn(string $table, string $column): void
+    {
+        $checks = $this->db->query(
+            'SELECT tc.CONSTRAINT_NAME
+             FROM information_schema.TABLE_CONSTRAINTS tc
+             INNER JOIN information_schema.CHECK_CONSTRAINTS cc
+                ON cc.CONSTRAINT_SCHEMA = tc.CONSTRAINT_SCHEMA
+               AND cc.CONSTRAINT_NAME = tc.CONSTRAINT_NAME
+             WHERE tc.CONSTRAINT_SCHEMA = DATABASE()
+               AND tc.TABLE_NAME = ?
+               AND tc.CONSTRAINT_TYPE = \'CHECK\'
+               AND cc.CHECK_CLAUSE LIKE ?',
+            [$table, '%' . $column . '%']
+        )->getResult();
+
+        foreach ($checks as $check) {
+            $this->db->query(sprintf(
+                'ALTER TABLE %s DROP CHECK %s',
+                $this->quoteIdentifier($table),
+                $this->quoteIdentifier($check->CONSTRAINT_NAME)
+            ));
+        }
+    }
+
+    private function objectName(string $prefix, string $table, string $column): string
+    {
+        return substr($prefix . '_' . $table . '_' . $column . '_' . substr(sha1($table . '.' . $column), 0, 8), 0, 64);
+    }
+
+    private function quoteIdentifier(string $identifier): string
+    {
+        return '`' . str_replace('`', '``', $identifier) . '`';
+    }
+}

app/Database/Migrations/2026-07-12-044700_EnsureWhatsappGroupLinksSchoolYearIndex.php

@@ -0,0 +1,47 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+use RuntimeException;
+
+final class EnsureWhatsappGroupLinksSchoolYearIndex extends Migration
+{
+    public function up(): void
+    {
+        if ($this->db->DBDriver !== 'MySQLi') {
+            throw new RuntimeException('This migration requires MySQL/MariaDB through the MySQLi driver.');
+        }
+
+        if (! $this->db->tableExists('whatsapp_group_links')
+            || ! $this->db->fieldExists('school_year', 'whatsapp_group_links')) {
+            return;
+        }
+
+        $indexed = $this->db->query(
+            'SELECT COUNT(*) AS aggregate
+             FROM information_schema.STATISTICS
+             WHERE TABLE_SCHEMA = DATABASE()
+               AND TABLE_NAME = ?
+               AND COLUMN_NAME = ?',
+            ['whatsapp_group_links', 'school_year']
+        )->getRow();
+
+        if ((int) ($indexed->aggregate ?? 0) > 0) {
+            return;
+        }
+
+        $this->db->query(
+            'ALTER TABLE `whatsapp_group_links` ADD INDEX `idx_sy_whatsapp_group_links_school_year` (`school_year`)'
+        );
+    }
+
+    public function down(): void
+    {
+        if ($this->db->tableExists('whatsapp_group_links')) {
+            $this->db->query('ALTER TABLE `whatsapp_group_links` DROP INDEX `idx_sy_whatsapp_group_links_school_year`');
+        }
+    }
+}

app/Database/Migrations/2026-07-12-050000_CreateSchoolYears.php

@@ -0,0 +1,258 @@
+<?php
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+
+class CreateSchoolYears extends Migration
+{
+    public function up(): void
+    {
+        if (! $this->db->tableExists('school_years')) {
+            $this->forge->addField([
+                'id' => [
+                    'type' => 'INT',
+                    'constraint' => 11,
+                    'unsigned' => true,
+                    'auto_increment' => true,
+                ],
+                'name' => [
+                    'type' => 'VARCHAR',
+                    'constraint' => 9,
+                    'null' => false,
+                ],
+                'status' => [
+                    'type' => 'VARCHAR',
+                    'constraint' => 20,
+                    'null' => false,
+                    'default' => 'draft',
+                ],
+                'starts_on' => [
+                    'type' => 'DATE',
+                    'null' => true,
+                ],
+                'ends_on' => [
+                    'type' => 'DATE',
+                    'null' => true,
+                ],
+                'description' => [
+                    'type' => 'TEXT',
+                    'null' => true,
+                ],
+                'registration_starts_on' => [
+                    'type' => 'DATE',
+                    'null' => true,
+                ],
+                'registration_ends_on' => [
+                    'type' => 'DATE',
+                    'null' => true,
+                ],
+                'previous_school_year_id' => [
+                    'type' => 'INT',
+                    'constraint' => 11,
+                    'unsigned' => true,
+                    'null' => true,
+                ],
+                'next_school_year_id' => [
+                    'type' => 'INT',
+                    'constraint' => 11,
+                    'unsigned' => true,
+                    'null' => true,
+                ],
+                'activated_at' => [
+                    'type' => 'DATETIME',
+                    'null' => true,
+                ],
+                'closing_started_at' => [
+                    'type' => 'DATETIME',
+                    'null' => true,
+                ],
+                'closed_at' => [
+                    'type' => 'DATETIME',
+                    'null' => true,
+                ],
+                'archived_at' => [
+                    'type' => 'DATETIME',
+                    'null' => true,
+                ],
+                'created_by' => [
+                    'type' => 'INT',
+                    'constraint' => 11,
+                    'unsigned' => true,
+                    'null' => true,
+                ],
+                'updated_by' => [
+                    'type' => 'INT',
+                    'constraint' => 11,
+                    'unsigned' => true,
+                    'null' => true,
+                ],
+                'created_at' => [
+                    'type' => 'DATETIME',
+                    'null' => true,
+                ],
+                'updated_at' => [
+                    'type' => 'DATETIME',
+                    'null' => true,
+                ],
+            ]);
+
+            $this->forge->addKey('id', true);
+            $this->forge->addKey('name', false, true);
+            $this->forge->addKey('status');
+            $this->forge->createTable('school_years');
+        } else {
+            $this->ensureSchoolYearColumns();
+        }
+
+        $this->createClosingTables();
+
+        $configuredYear = $this->configuredSchoolYear();
+
+        if ($configuredYear !== null && $this->isValidYearName($configuredYear)) {
+            $existing = $this->db->table('school_years')
+                ->where('name', $configuredYear)
+                ->get()
+                ->getRowArray();
+
+            if ($existing === null) {
+                $this->db->table('school_years')->insert([
+                    'name' => $configuredYear,
+                    'status' => 'active',
+                    'activated_at' => date('Y-m-d H:i:s'),
+                    'created_at' => date('Y-m-d H:i:s'),
+                    'updated_at' => date('Y-m-d H:i:s'),
+                ]);
+            }
+        }
+    }
+
+    public function down(): void
+    {
+        $this->forge->dropTable('school_year_transition_logs', true);
+        $this->forge->dropTable('school_year_closing_items', true);
+        $this->forge->dropTable('school_year_closing_batches', true);
+        $this->forge->dropTable('school_years', true);
+    }
+
+    private function ensureSchoolYearColumns(): void
+    {
+        $fields = $this->db->getFieldNames('school_years');
+        $add = [];
+
+        $definitions = [
+            'description' => ['type' => 'TEXT', 'null' => true],
+            'registration_starts_on' => ['type' => 'DATE', 'null' => true],
+            'registration_ends_on' => ['type' => 'DATE', 'null' => true],
+            'previous_school_year_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+            'next_school_year_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+            'activated_at' => ['type' => 'DATETIME', 'null' => true],
+            'closing_started_at' => ['type' => 'DATETIME', 'null' => true],
+            'closed_at' => ['type' => 'DATETIME', 'null' => true],
+            'archived_at' => ['type' => 'DATETIME', 'null' => true],
+            'created_by' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+            'updated_by' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+        ];
+
+        foreach ($definitions as $field => $definition) {
+            if (! in_array($field, $fields, true)) {
+                $add[$field] = $definition;
+            }
+        }
+
+        if ($add !== []) {
+            $this->forge->addColumn('school_years', $add);
+        }
+    }
+
+    private function createClosingTables(): void
+    {
+        if (! $this->db->tableExists('school_year_closing_batches')) {
+            $this->forge->addField([
+                'id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'auto_increment' => true],
+                'source_school_year_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true],
+                'target_school_year_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+                'status' => ['type' => 'VARCHAR', 'constraint' => 30, 'null' => false, 'default' => 'preview'],
+                'preview_hash' => ['type' => 'CHAR', 'constraint' => 64, 'null' => true],
+                'total_families' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => false, 'default' => 0],
+                'total_positive_balance' => ['type' => 'DECIMAL', 'constraint' => '12,2', 'null' => false, 'default' => '0.00'],
+                'total_credit_balance' => ['type' => 'DECIMAL', 'constraint' => '12,2', 'null' => false, 'default' => '0.00'],
+                'started_by' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+                'completed_by' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+                'started_at' => ['type' => 'DATETIME', 'null' => true],
+                'completed_at' => ['type' => 'DATETIME', 'null' => true],
+                'failed_at' => ['type' => 'DATETIME', 'null' => true],
+                'failure_message' => ['type' => 'TEXT', 'null' => true],
+                'created_at' => ['type' => 'DATETIME', 'null' => true],
+                'updated_at' => ['type' => 'DATETIME', 'null' => true],
+            ]);
+            $this->forge->addKey('id', true);
+            $this->forge->addKey(['source_school_year_id', 'status']);
+            $this->forge->createTable('school_year_closing_batches');
+        }
+
+        if (! $this->db->tableExists('school_year_closing_items')) {
+            $this->forge->addField([
+                'id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'auto_increment' => true],
+                'closing_batch_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true],
+                'family_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true],
+                'source_balance' => ['type' => 'DECIMAL', 'constraint' => '12,2', 'null' => false, 'default' => '0.00'],
+                'credit_amount' => ['type' => 'DECIMAL', 'constraint' => '12,2', 'null' => false, 'default' => '0.00'],
+                'adjustment_amount' => ['type' => 'DECIMAL', 'constraint' => '12,2', 'null' => false, 'default' => '0.00'],
+                'carry_forward_amount' => ['type' => 'DECIMAL', 'constraint' => '12,2', 'null' => false, 'default' => '0.00'],
+                'target_invoice_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+                'target_adjustment_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+                'status' => ['type' => 'VARCHAR', 'constraint' => 30, 'null' => false, 'default' => 'pending'],
+                'error_message' => ['type' => 'TEXT', 'null' => true],
+                'created_at' => ['type' => 'DATETIME', 'null' => true],
+                'updated_at' => ['type' => 'DATETIME', 'null' => true],
+            ]);
+            $this->forge->addKey('id', true);
+            $this->forge->addKey(['closing_batch_id', 'family_id'], false, true);
+            $this->forge->createTable('school_year_closing_items');
+        }
+
+        if (! $this->db->tableExists('school_year_transition_logs')) {
+            $this->forge->addField([
+                'id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'auto_increment' => true],
+                'school_year_id' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true],
+                'from_status' => ['type' => 'VARCHAR', 'constraint' => 20, 'null' => true],
+                'to_status' => ['type' => 'VARCHAR', 'constraint' => 20, 'null' => true],
+                'action' => ['type' => 'VARCHAR', 'constraint' => 80, 'null' => false],
+                'performed_by' => ['type' => 'INT', 'constraint' => 11, 'unsigned' => true, 'null' => true],
+                'metadata_json' => ['type' => 'TEXT', 'null' => true],
+                'created_at' => ['type' => 'DATETIME', 'null' => false],
+            ]);
+            $this->forge->addKey('id', true);
+            $this->forge->addKey(['school_year_id', 'created_at']);
+            $this->forge->createTable('school_year_transition_logs');
+        }
+    }
+
+    private function configuredSchoolYear(): ?string
+    {
+        if (! $this->db->tableExists('configuration')) {
+            return null;
+        }
+
+        $row = $this->db->table('configuration')
+            ->select('config_value')
+            ->where('config_key', 'school_year')
+            ->orderBy('id', 'DESC')
+            ->get(1)
+            ->getRowArray();
+
+        $value = trim((string) ($row['config_value'] ?? ''));
+
+        return $value !== '' ? $value : null;
+    }
+
+    private function isValidYearName(string $value): bool
+    {
+        if (! preg_match('/^(\d{4})-(\d{4})$/', $value, $matches)) {
+            return false;
+        }
+
+        return (int) $matches[2] === (int) $matches[1] + 1;
+    }
+}

app/Database/Migrations/2026-07-12-051000_AddSchoolYearsNavItem.php

@@ -0,0 +1,111 @@
+<?php
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+
+class AddSchoolYearsNavItem extends Migration
+{
+    private string $url = 'administrator/school-years';
+
+    public function up(): void
+    {
+        if (! $this->db->tableExists('nav_items')) {
+            return;
+        }
+
+        $parentColumn = $this->parentColumn();
+        $existingQuery = $this->db->table('nav_items')
+            ->where('url', $this->url)
+            ->get();
+
+        $existing = $existingQuery !== false ? $existingQuery->getRowArray() : null;
+
+        if ($existing !== null) {
+            return;
+        }
+
+        $parentBuilder = $this->db->table('nav_items')
+            ->where('label', 'Configuration');
+
+        if ($parentColumn !== null) {
+            $parentBuilder->where($parentColumn, null);
+        }
+
+        $parentQuery = $parentBuilder->get();
+        $parent = $parentQuery !== false ? $parentQuery->getRowArray() : null;
+
+        $insert = [
+            'label' => 'School Years',
+            'url' => $this->url,
+            'sort_order' => 2,
+            'is_enabled' => 1,
+            'created_at' => date('Y-m-d H:i:s'),
+        ];
+
+        if ($parentColumn !== null) {
+            $insert[$parentColumn] = $parent['id'] ?? null;
+        }
+
+        $this->db->table('nav_items')->insert($insert);
+
+        $navItemId = (int) $this->db->insertID();
+
+        if (
+            $navItemId <= 0
+            || ! $this->db->tableExists('role_nav_items')
+            || ! $this->db->fieldExists('role', 'role_nav_items')
+            || ! $this->db->fieldExists('nav_item_id', 'role_nav_items')
+        ) {
+            return;
+        }
+
+        foreach (['administrator', 'principal', 'vice_principal'] as $role) {
+            $this->db->table('role_nav_items')->insert([
+                'role' => $role,
+                'nav_item_id' => $navItemId,
+                'created_at' => date('Y-m-d H:i:s'),
+            ]);
+        }
+    }
+
+    public function down(): void
+    {
+        if (! $this->db->tableExists('nav_items')) {
+            return;
+        }
+
+        $query = $this->db->table('nav_items')
+            ->where('url', $this->url)
+            ->get();
+
+        $row = $query !== false ? $query->getRowArray() : null;
+
+        if ($row === null) {
+            return;
+        }
+
+        if ($this->db->tableExists('role_nav_items')) {
+            $this->db->table('role_nav_items')
+                ->where('nav_item_id', (int) $row['id'])
+                ->delete();
+        }
+
+        $this->db->table('nav_items')
+            ->where('id', (int) $row['id'])
+            ->delete();
+    }
+
+    private function parentColumn(): ?string
+    {
+        if ($this->db->fieldExists('parent_id', 'nav_items')) {
+            return 'parent_id';
+        }
+
+        if ($this->db->fieldExists('menu_parent_id', 'nav_items')) {
+            return 'menu_parent_id';
+        }
+
+        return null;
+    }
+}

app/Database/Migrations/2026-07-12-052000_GrantSubjectCurriculumPermission.php

@@ -0,0 +1,145 @@
+<?php
+
+namespace App\Database\Migrations;
+
+use CodeIgniter\Database\Migration;
+
+class GrantSubjectCurriculumPermission extends Migration
+{
+    private string $permissionName = 'update_curriculum';
+
+    public function up(): void
+    {
+        if (
+            ! $this->db->tableExists('roles')
+            || ! $this->db->tableExists('permissions')
+            || ! $this->db->tableExists('role_permissions')
+        ) {
+            return;
+        }
+
+        $now = date('Y-m-d H:i:s');
+        $permission = $this->db->table('permissions')
+            ->where('name', $this->permissionName)
+            ->get()
+            ->getRowArray();
+
+        if ($permission === null) {
+            $insert = [
+                'name' => $this->permissionName,
+                'created_at' => $now,
+                'updated_at' => $now,
+            ];
+
+            if ($this->db->fieldExists('description', 'permissions')) {
+                $insert['description'] = 'Manage subject curriculum entries';
+            }
+
+            $this->db->table('permissions')->insert($insert);
+            $permissionId = (int) $this->db->insertID();
+        } else {
+            $permissionId = (int) $permission['id'];
+        }
+
+        if ($permissionId <= 0) {
+            return;
+        }
+
+        $roleRows = $this->db->table('roles')
+            ->select('id, name')
+            ->whereIn('name', [
+                'administrator',
+                'admin',
+                'principal',
+                'vice principal',
+                'vice_principal',
+                'administrative staff',
+            ])
+            ->get()
+            ->getResultArray();
+
+        foreach ($roleRows as $role) {
+            $roleId = (int) ($role['id'] ?? 0);
+            if ($roleId <= 0) {
+                continue;
+            }
+
+            $existing = $this->db->table('role_permissions')
+                ->where('role_id', $roleId)
+                ->where('permission_id', $permissionId)
+                ->get()
+                ->getRowArray();
+
+            $permissionData = [
+                'can_create' => 1,
+                'can_read' => 1,
+                'can_update' => 1,
+                'can_delete' => 1,
+                'updated_at' => $now,
+            ];
+
+            if ($this->db->fieldExists('can_manage', 'role_permissions')) {
+                $permissionData['can_manage'] = 1;
+            }
+
+            if ($existing === null) {
+                $permissionData['role_id'] = $roleId;
+                $permissionData['permission_id'] = $permissionId;
+                $permissionData['created_at'] = $now;
+
+                $this->db->table('role_permissions')->insert($permissionData);
+                continue;
+            }
+
+            $this->db->table('role_permissions')
+                ->where('id', (int) $existing['id'])
+                ->update($permissionData);
+        }
+    }
+
+    public function down(): void
+    {
+        if (
+            ! $this->db->tableExists('roles')
+            || ! $this->db->tableExists('permissions')
+            || ! $this->db->tableExists('role_permissions')
+        ) {
+            return;
+        }
+
+        $permission = $this->db->table('permissions')
+            ->select('id')
+            ->where('name', $this->permissionName)
+            ->get()
+            ->getRowArray();
+
+        if ($permission === null) {
+            return;
+        }
+
+        $roleRows = $this->db->table('roles')
+            ->select('id')
+            ->whereIn('name', [
+                'administrator',
+                'admin',
+                'principal',
+                'vice principal',
+                'vice_principal',
+                'administrative staff',
+            ])
+            ->get()
+            ->getResultArray();
+
+        $roleIds = array_map(static fn(array $role): int => (int) ($role['id'] ?? 0), $roleRows);
+        $roleIds = array_values(array_filter($roleIds, static fn(int $roleId): bool => $roleId > 0));
+
+        if ($roleIds === []) {
+            return;
+        }
+
+        $this->db->table('role_permissions')
+            ->where('permission_id', (int) $permission['id'])
+            ->whereIn('role_id', $roleIds)
+            ->delete();
+    }
+}

app/Database/Seeds/NavSeeder.php

@@ -52,6 +52,7 @@ class NavSeeder extends Seeder
 
             // Configuration
             ['parent'=>'Configuration','label'=>'Add/Edit Configuration','url'=>'configuration/configuration_view','sort_order'=>1],
+            ['parent'=>'Configuration','label'=>'School Years','url'=>'administrator/school-years','sort_order'=>2],
 
             // Staffing
             ['parent'=>'Staffing','label'=>'Staff Profile','url'=>'staff/index','sort_order'=>1],

app/Filters/RequireSchoolYearFilter.php

@@ -0,0 +1,41 @@
+<?php
+
+namespace App\Filters;
+
+use CodeIgniter\Filters\FilterInterface;
+use CodeIgniter\HTTP\IncomingRequest;
+use CodeIgniter\HTTP\RequestInterface;
+use CodeIgniter\HTTP\ResponseInterface;
+use Throwable;
+
+final class RequireSchoolYearFilter implements FilterInterface
+{
+    public function before(RequestInterface $request, $arguments = null)
+    {
+        if (! $request instanceof IncomingRequest) {
+            return null;
+        }
+
+        try {
+            service('schoolYearContext')->resolve($request);
+            return null;
+        } catch (Throwable $e) {
+            log_message('warning', 'School-year request rejected: {message}', [
+                'message' => $e->getMessage(),
+            ]);
+
+            return service('response')
+                ->setStatusCode(400)
+                ->setJSON([
+                    'status' => 400,
+                    'error' => 'Invalid school year',
+                    'message' => $e->getMessage(),
+                ]);
+        }
+    }
+
+    public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
+    {
+        return null;
+    }
+}

app/Libraries/FinancialStatus.php

@@ -7,6 +7,8 @@ final class FinancialStatus
     public const INVOICE_UNPAID = 'unpaid';
     public const INVOICE_PARTIALLY_PAID = 'partially_paid';
     public const INVOICE_PAID = 'paid';
+    public const INVOICE_OVERPAID = 'overpaid';
+    public const INVOICE_CANCELLED = 'cancelled';
 
     public const PAYMENT_RECORDED = 'recorded';
     public const PAYMENT_VOIDED = 'voided';
@@ -18,6 +20,23 @@ final class FinancialStatus
     public const PAYMENT_CANCELED = 'canceled';
     public const PAYMENT_CANCELLED = 'cancelled';
 
+    public const PAYMENT_TRANSACTION_STATUSES = [
+        self::PAYMENT_RECORDED,
+        self::PAYMENT_VOIDED,
+        self::PAYMENT_REFUNDED,
+        self::PAYMENT_FAILED,
+        self::PAYMENT_REVERSED,
+        self::PAYMENT_CHARGEBACK,
+    ];
+
+    public const INVOICE_STATUSES = [
+        self::INVOICE_UNPAID,
+        self::INVOICE_PARTIALLY_PAID,
+        self::INVOICE_PAID,
+        self::INVOICE_OVERPAID,
+        self::INVOICE_CANCELLED,
+    ];
+
     public const REFUND_PENDING = 'pending';
     public const REFUND_APPROVED = 'approved';
     public const REFUND_REJECTED = 'rejected';
@@ -55,6 +74,8 @@ final class FinancialStatus
         return match (self::normalize($status)) {
             'paid', 'full' => self::INVOICE_PAID,
             'partially paid', 'partially_paid', 'partial' => self::INVOICE_PARTIALLY_PAID,
+            'overpaid' => self::INVOICE_OVERPAID,
+            'cancelled', 'canceled', 'cancelled invoice', 'canceled invoice' => self::INVOICE_CANCELLED,
             default => self::INVOICE_UNPAID,
         };
     }

app/Models/AttendanceDataModel.php

@@ -18,7 +18,6 @@ class AttendanceDataModel extends Model
         'date',
         'status',
         'reason',
-        'reported',
         'is_reported',          
         'is_notified',      
         'semester',

app/Models/ClassPrepAdjustmentModel.php

@@ -7,6 +7,6 @@ class ClassPrepAdjustmentModel extends Model
 {
     protected $table = 'class_prep_adjustments';
     protected $allowedFields = [
-        'class_section_id', 'item_name', 'adjustment', 'school_year', 'created_at'
+        'class_section_id', 'item_name', 'adjustment', 'adjustable', 'created_at'
     ];
 }

app/Models/ClassPreparationLogModel.php

@@ -11,7 +11,6 @@ class ClassPreparationLogModel extends Model
     protected $allowedFields = [
         'class_section_id',
         'class_section',
-        'school_year',
         'prep_data',
         'created_at',
     ];

app/Models/Concerns/SchoolYearScopedModelTrait.php

@@ -0,0 +1,31 @@
+<?php
+
+namespace App\Models\Concerns;
+
+use App\Support\SchoolYear\SchoolYearContext;
+use CodeIgniter\Model;
+
+trait SchoolYearScopedModelTrait
+{
+    public function forSchoolYear(SchoolYearContext|string|int $schoolYear): Model
+    {
+        if ($schoolYear instanceof SchoolYearContext) {
+            if ($this->fieldExists('school_year_id')) {
+                return $this->where($this->table . '.school_year_id', $schoolYear->id());
+            }
+
+            return $this->where($this->table . '.school_year', $schoolYear->yearName());
+        }
+
+        if (is_int($schoolYear)) {
+            return $this->where($this->table . '.school_year_id', $schoolYear);
+        }
+
+        return $this->where($this->table . '.school_year', $schoolYear);
+    }
+
+    private function fieldExists(string $field): bool
+    {
+        return in_array($field, $this->db->getFieldNames($this->table), true);
+    }
+}

app/Models/ContactUsModel.php

@@ -19,8 +19,6 @@ class ContactUsModel extends Model
         'reciever_id',
         'subject',
         'message',
-        'semester',
-        'school_year',
         'created_at',
         'updated_at'
     ];
@@ -35,8 +33,6 @@ class ContactUsModel extends Model
         'reciever_id'  => 'required|integer',
         'subject'      => 'required|string|max_length[255]',
         'message'      => 'required|string',
-        'semester'     => 'required|string|max_length[255]',
-        'school_year'  => 'permit_empty|string|max_length[9]'
     ];
 
     protected $validationMessages = [];
@@ -51,10 +47,7 @@ class ContactUsModel extends Model
      */
     public function getMessagesBySemesterAndYear($semester, $school_year)
     {
-        return $this->where([
+        return $this->findAll();
-            'semester' => $semester,
-            'school_year' => $school_year
-        ])->findAll();
     }
 
     /**

app/Models/EmailTemplateModel.php

@@ -6,12 +6,19 @@ use CodeIgniter\Model;
 class EmailTemplateModel extends Model {
     protected $table = 'email_templates';
     protected $primaryKey = 'id';
-    protected $allowedFields = ['template_key','name','subject','body','is_active'];
+    protected $allowedFields = ['code', 'variant', 'subject', 'body_html', 'is_active', 'updated_by', 'updated_at'];
 
     public function getActiveTemplates(): array {
-        return $this->where('is_active', 1)->orderBy('template_key','asc')->findAll();
+        return $this->select('email_templates.*, code AS template_key, code AS name, body_html AS body')
+            ->where('is_active', 1)
+            ->orderBy('code', 'asc')
+            ->findAll();
     }
+
     public function findByKey(string $key): ?array {
-        return $this->where('template_key', $key)->where('is_active', 1)->first();
+        return $this->select('email_templates.*, code AS template_key, code AS name, body_html AS body')
+            ->where('code', $key)
+            ->where('is_active', 1)
+            ->first();
     }
 }

app/Models/EmergencyContactModel.php

@@ -16,8 +16,6 @@ class EmergencyContactModel extends Model
         'cellphone',
         'email',
         'relation',
-        'semester',
-        'school_year',
         'created_at',
         'updated_at'
     ];

app/Models/EnrollmentModel.php

@@ -86,7 +86,7 @@ class EnrollmentModel extends Model
     /**
      * Get all enrolled students (full student info) for a given parent.
      */
-    public function getEnrolledStudents(int $parentId, string $schoolYear = null, string $semester = null): array
+    public function getEnrolledStudents(int $parentId, ?string $schoolYear = null, ?string $semester = null): array
     {
         $builder = $this->select('students.*')
                         ->join('students', 'students.id = enrollments.student_id')
@@ -106,7 +106,7 @@ class EnrollmentModel extends Model
     /**
      * Get student basic info (ID, name, grade) for a given parent where status is enrolled or Payment pending.
      */
-    public function getenrolledStudentDetails(int $parentId, string $schoolYear = null): array
+    public function getenrolledStudentDetails(int $parentId, ?string $schoolYear = null): array
     {
         $builder = $this->db->table('enrollments')
             ->select('students.id, students.firstname, students.lastname, students.grade_level')

app/Models/EventChargesModel.php

@@ -28,7 +28,6 @@ class EventChargesModel extends Model
         'external_parent_email',
         'semester',
         'school_year',
-        'created_by',
         'updated_by',
         'created_at',
         'updated_at'

app/Models/ExamDraftModel.php

@@ -32,7 +32,6 @@ class ExamDraftModel extends Model
     ];
 
     protected $allowedFields = [
-        'teacher_id',
         'author_id',
         'class_section_id',
         'semester',
@@ -40,20 +39,14 @@ class ExamDraftModel extends Model
         'exam_type',
         'draft_title',
         'author_comment',
-        'description',
-        'teacher_file',
-        'teacher_filename',
         'author_file',
         'author_filename',
         'status',
         'acceptance_type',
         'review_revision',
         'reviewer_id',
-        'admin_id',
         'is_legacy',
-        'reviewer_comment',
         'reviewer_comments',
-        'admin_comments',
         'reviewed_at',
         'final_file',
         'final_filename',
@@ -87,11 +80,9 @@ class ExamDraftModel extends Model
      * @var array<string, string>
      */
     protected array $casts = [
-        'teacher_id' => 'int',
         'author_id' => 'int',
         'class_section_id' => 'int',
         'reviewer_id' => '?int',
-        'admin_id' => '?int',
         'review_revision' => 'int',
         'version' => 'int',
         'previous_draft_id' => '?int',

app/Models/ExamModel.php

@@ -12,9 +12,10 @@ class ExamModel extends Model
 
     protected $allowedFields = [
         'student_id',
-        'school_id',
+        'student_school_id',
         'class_section_id',
         'exam_name',
+        'school_year',
         'created_at'
     ];
 

app/Models/FinalExamModel.php

@@ -18,7 +18,6 @@ class FinalExamModel extends Model
         'class_section_id', 
         'updated_by', 
         'score', 
-        'comment', 
         'semester', 
         'school_year', 
         'created_at', 

app/Models/FinalScoreModel.php

@@ -14,7 +14,7 @@ class FinalScoreModel extends Model
         'student_id',
         'school_id',
         'class_section_id',
-        'teacher_id',
+        'updated_by',
         'score',
         'score_letter',
         'comment',

app/Models/InventoryItemModel.php

@@ -30,10 +30,6 @@ protected $table      = 'inventory_items';
         'sku',
         'notes',
 
-        // academic tags
-        'semester',
-        'school_year',
-
         // audit
         'updated_by',
 
@@ -50,7 +46,5 @@ protected $table      = 'inventory_items';
         'name'        => 'required|min_length[2]',
         'quantity'    => 'permit_empty|integer',
         'unit_price'  => 'permit_empty|decimal',
-        'semester'    => 'permit_empty|in_list[Spring,Fall]',
-        'school_year' => 'permit_empty|max_length[16]', // e.g. 2025-2026
     ];
 }

app/Models/InvoiceStudentListModel.php

@@ -20,8 +20,6 @@ class InvoiceStudentListModel extends Model
         'student_lastname',
         'school_id',
         'enrolled',
-        'school_year',
-        'semester',
         'created_at',
         'updated_at'
     ];

app/Models/IpAttemptModel.php

@@ -13,8 +13,6 @@ class IpAttemptModel extends Model
         'ip_address',
         'attempts',
         'last_attempt_at',
-        'semester',
-        'school_year',
         'blocked_until'
     ];
 

app/Models/LoginActivityModel.php

@@ -15,7 +15,6 @@ class LoginActivityModel extends Model
         'logout_time', 
         'ip_address', 
         'user_agent',
-        'school_year',
         'semester',
         'created_at',
         'updated_at'

app/Models/MessageModel.php

@@ -21,8 +21,7 @@ class MessageModel extends Model
         'priority',
         'attachment',
         'status',
-        'semester',      // Added field
+        'semester'
-        'school_year'    // Added field
     ];
 
     protected $useTimestamps = false; // Since you're manually handling date fields
@@ -150,10 +149,8 @@ class MessageModel extends Model
      */
     public function getMessagesBySemesterAndYear($semester, $school_year = null)
     {
-        $builder = $this->where('semester', $semester);
+        return $this->where('semester', $semester)
-        if ($school_year) {
+            ->orderBy('sent_datetime', 'DESC')
-            $builder->where('school_year', $school_year);
+            ->findAll();
-        }
-        return $builder->orderBy('sent_datetime', 'DESC')->findAll();
     }
 }

app/Models/NotificationModel.php

@@ -17,9 +17,10 @@ class NotificationModel extends Model
         'status', 
         'action_url', 
         'attachment_path', 
-        'semester',
+        'expires_at',
-        'school_year',
+        'sent_at',
-        'scheduled_at'
+        'scheduled_at',
+        'deleted_at'
     ];
     protected $useTimestamps = true;
     protected $createdField = 'created_at';

app/Models/ParentModel.php

@@ -15,7 +15,5 @@ class ParentModel extends Model
         'secondparent_email',
         'secondparent_phone',
         'firstparent_id',
-        'semester',
-        'school_year',
     ];
 }

app/Models/PaymentModel.php

@@ -2,7 +2,9 @@
 
 namespace App\Models;
 
+use CodeIgniter\Database\ConnectionInterface;
 use CodeIgniter\Model;
+use CodeIgniter\Validation\ValidationInterface;
 
 class PaymentModel extends Model
 {
@@ -10,12 +12,22 @@ class PaymentModel extends Model
     protected $primaryKey = 'id';
     protected $returnType = 'array';
 
+    /**
+     * Keep this list broad, then filter it in __construct() against the real DB schema.
+     * This prevents CI from trying to insert/update columns that do not exist, such as `balance`.
+     */
     protected $allowedFields = [
         'parent_id',
         'invoice_id',
+
+        // Legacy invoice snapshot fields. Do not use these as source of truth.
         'total_amount',
+
+        // Actual payment fields.
         'paid_amount',
         'balance',
+        'balance_amount',
+        'balance_after_payment',
         'number_of_installments',
         'installment_seq',
         'transaction_id',
@@ -24,26 +36,44 @@ class PaymentModel extends Model
         'payment_method',
         'payment_date',
         'school_year',
-        'semester',
         'status',
-        'updated_by'
+        'updated_by',
     ];
 
     protected $useTimestamps = false;
-    // ❗ Your DB handles created_at / updated_at with default CURRENT_TIMESTAMP
+
-    // Remove CI4 auto timestamp management unless you modify your DB schema
+    /**
+     * Let DB defaults handle created_at / updated_at.
+     * Keep validation strict for actual payment data, but do not require legacy snapshot columns.
+     */
+    protected $cleanValidationRules = true;
 
     protected $validationRules = [
         'parent_id' => 'required|integer',
         'invoice_id' => 'required|integer',
-        'total_amount' => 'required|decimal',
+
-        'paid_amount' => 'required|decimal',
+        // Legacy snapshot. Optional because invoice total belongs to invoices table.
-        'balance' => 'required|decimal',
+        'total_amount' => 'permit_empty|decimal',
-        'number_of_installments' => 'required|integer',
+
-        'payment_method' => 'required|max_length[50]', // ✅ Removed 'string'
+        'paid_amount' => 'required|decimal|greater_than[0]',
-        'payment_date' => 'required|valid_date[Y-m-d H:i:s]',
+
-        'status' => 'required|max_length[50]', // ✅ Removed 'string'
+        // These are optional because not every live schema has a balance snapshot column.
+        'balance' => 'permit_empty|decimal|greater_than_equal_to[0]',
+        'balance_amount' => 'permit_empty|decimal|greater_than_equal_to[0]',
+        'balance_after_payment' => 'permit_empty|decimal|greater_than_equal_to[0]',
+
+        // Legacy/current sequence columns.
+        'number_of_installments' => 'permit_empty|integer|greater_than[0]',
+        'installment_seq' => 'permit_empty|integer|greater_than[0]',
+
+        'transaction_id' => 'permit_empty|max_length[100]',
+        'payment_method' => 'required|in_list[cash,check,card]',
+        'payment_date' => 'required|valid_date',
+        'school_year' => 'permit_empty|max_length[20]',
+        'status' => 'required|max_length[50]',
+        'check_file' => 'permit_empty|max_length[255]',
         'check_number' => 'permit_empty|max_length[100]',
+        'updated_by' => 'permit_empty|integer',
     ];
 
     protected $validationMessages = [
@@ -56,112 +86,225 @@ class PaymentModel extends Model
             'integer' => 'Invoice ID must be an integer.',
         ],
         'total_amount' => [
-            'required' => 'Total amount is required.',
             'decimal' => 'Total amount must be a valid decimal.',
         ],
         'paid_amount' => [
             'required' => 'Paid amount is required.',
             'decimal' => 'Paid amount must be a valid decimal.',
+            'greater_than' => 'Paid amount must be greater than zero.',
         ],
         'balance' => [
-            'required' => 'Balance is required.',
             'decimal' => 'Balance must be a valid decimal.',
+            'greater_than_equal_to' => 'Balance must not be negative.',
+        ],
+        'balance_amount' => [
+            'decimal' => 'Balance amount must be a valid decimal.',
+            'greater_than_equal_to' => 'Balance amount must not be negative.',
+        ],
+        'balance_after_payment' => [
+            'decimal' => 'Balance after payment must be a valid decimal.',
+            'greater_than_equal_to' => 'Balance after payment must not be negative.',
         ],
         'payment_date' => [
             'required' => 'Payment date is required.',
-            'valid_date' => 'Payment date must be a valid date (Y-m-d H:i:s).',
+            'valid_date' => 'Payment date must be a valid date.',
         ],
         'payment_method' => [
             'required' => 'Payment method is required.',
-            'max_length' => 'Payment method must not exceed 50 characters.',
+            'in_list' => 'Payment method must be cash, check, or card.',
         ],
         'status' => [
             'required' => 'Status is required.',
             'max_length' => 'Status must not exceed 50 characters.',
         ],
         'number_of_installments' => [
-            'required' => 'Number of installments is required.',
             'integer' => 'Number of installments must be an integer.',
+            'greater_than' => 'Number of installments must be greater than zero.',
+        ],
+        'installment_seq' => [
+            'integer' => 'Installment sequence must be an integer.',
+            'greater_than' => 'Installment sequence must be greater than zero.',
         ],
         'check_number' => [
             'max_length' => 'Check number must not exceed 100 characters.',
-        ]
+        ],
     ];
 
-    /**
+    private array $paymentColumns = [];
-     * Get payments by parent ID.
+
-     *
+    private array $excludedPaymentStatuses = [
-     * @param int $parentId
+        'void',
-     * @return array
+        'voided',
-     */
+        'refunded',
-    public function getPaymentsByParentId(int $parentId): array
+        'failed',
+        'chargeback',
+        'declined',
+        'reversed',
+        'canceled',
+        'cancelled',
+    ];
+
+    public function __construct(?ConnectionInterface $db = null, ?ValidationInterface $validation = null)
     {
-        return $this->where('parent_id', $parentId)
+        parent::__construct($db, $validation);
-            ->orderBy('payment_date', 'DESC')
+
-            ->findAll();
+        $this->paymentColumns = $this->getTableColumns($this->table);
+
+        // Remove fields that do not exist in the live DB.
+        // This prevents errors like "Unknown column balance".
+        $this->allowedFields = array_values(array_filter(
+            $this->allowedFields,
+            fn (string $field): bool => in_array($field, $this->paymentColumns, true)
+        ));
+
+        // Remove validation rules for columns that do not exist.
+        foreach (array_keys($this->validationRules) as $field) {
+            if (!in_array($field, $this->allowedFields, true)) {
+                unset($this->validationRules[$field]);
+                unset($this->validationMessages[$field]);
+            }
+        }
     }
 
     /**
-     * Calculate the total paid amount for a specific parent.
+     * Get payments by parent ID with invoice context.
      *
-     * @param int $parentId
+     * The source of truth for invoice total/status/balance is invoices, not payments.
-     * @return float
+     */
+    public function getPaymentsByParentId(int $parentId, ?string $schoolYear = null): array
+    {
+        return $this->parentPaymentHistoryQuery($parentId, $schoolYear)->findAll();
+    }
+
+    /**
+     * Build parent payment history query with invoice context.
+     *
+     * Kept public so controllers can paginate without duplicating select/join logic.
+     */
+    public function parentPaymentHistoryQuery(int $parentId, ?string $schoolYear = null): self
+    {
+        $paymentBalanceSelect = $this->getPaymentBalanceSelectExpression('payments');
+
+        $invoiceBalanceSelect = $this->columnExists('invoices', 'balance')
+            ? 'invoices.balance AS invoice_current_balance'
+            : '0.00 AS invoice_current_balance';
+
+        $select = [
+            'payments.id',
+            'payments.invoice_id',
+            'invoices.invoice_number',
+            'payments.transaction_id',
+            'payments.paid_amount',
+            'payments.payment_method',
+            'payments.check_number',
+            'payments.check_file',
+            'payments.payment_date',
+            'payments.installment_seq',
+            'payments.number_of_installments',
+            $paymentBalanceSelect,
+            'payments.status AS payment_status',
+            'invoices.total_amount AS invoice_total',
+            $invoiceBalanceSelect,
+            'invoices.status AS invoice_status',
+            'invoices.school_year',
+        ];
+
+        $model = new self($this->db);
+        $builder = $model->select(implode(', ', $select), false)
+            ->join('invoices', 'invoices.id = payments.invoice_id', 'inner')
+            ->where('payments.parent_id', $parentId);
+
+        if ($schoolYear !== null && $schoolYear !== '') {
+            $builder->where('invoices.school_year', $schoolYear);
+        }
+
+        return $builder
+            ->orderBy('payments.payment_date', 'DESC')
+            ->orderBy('payments.id', 'DESC');
+    }
+
+    /**
+     * Calculate the total paid amount for a parent in a school year.
+     *
+     * Uses invoices.school_year so old poisoned payment term data does not corrupt totals.
      */
     public function getTotalPaidByParentId(int $parentId, string $schoolYear): float
     {
-        $db = \Config\Database::connect();
+        $result = $this->db->table('payments p')
-
+            ->selectSum('p.paid_amount', 'total_paid')
-        $result = $db->table('payments')
+            ->join('invoices i', 'i.id = p.invoice_id', 'inner')
-            ->selectSum('paid_amount', 'total_paid')
+            ->where('p.parent_id', $parentId)
-            ->where('parent_id', $parentId)
+            ->where('i.school_year', $schoolYear)
-            ->where('school_year', $schoolYear)
+            ->where($this->successfulPaymentWhereSql('p.status'), null, false)
-            ->whereNotIn('status', ['void', 'voided', 'refunded', 'failed', 'chargeback', 'declined', 'reversed', 'canceled', 'cancelled'])
             ->get()
             ->getRowArray();
 
-        return $result && isset($result['total_paid']) ? (float) $result['total_paid'] : 0.00;
+        return $result && isset($result['total_paid'])
+            ? (float) $result['total_paid']
+            : 0.00;
     }
 
-
     /**
-     * Update balance for a specific payment.
+     * Legacy method.
      *
-     * @param int $paymentId
+     * This should not be used for new payment recording.
-     * @param float $amountPaid
+     * New payments should be inserted as immutable payment rows and invoices recalculated by InvoiceLedgerService.
-     * @return bool
      */
     public function updateBalance(int $paymentId, float $amountPaid): bool
     {
+        if ($amountPaid <= 0) {
+            return false;
+        }
+
         $payment = $this->find($paymentId);
         if (!$payment) {
             return false;
         }
 
-        $newBalance = $payment['balance'] - $amountPaid;
+        $updateData = [
+            'paid_amount' => round((float) ($payment['paid_amount'] ?? 0) + $amountPaid, 2),
+        ];
 
-        return $this->update($paymentId, [
+        $balanceField = $this->getPaymentBalanceField();
-            'paid_amount' => $payment['paid_amount'] + $amountPaid,
+        if ($balanceField !== null && array_key_exists($balanceField, $payment)) {
-            'balance' => $newBalance
+            $updateData[$balanceField] = max(
-        ]);
+                0.00,
+                round((float) ($payment[$balanceField] ?? 0) - $amountPaid, 2)
+            );
+        }
+
+        return $this->update($paymentId, $updateData);
     }
 
     /**
      * Get payments by school year.
      *
-     * @param string $schoolYear
+     * Uses invoice school year when possible because payment.school_year may have old bad data.
-     * @return array
      */
     public function getPaymentsByYear(string $schoolYear): array
     {
-        return $this->where('school_year', $schoolYear)
+        $paymentBalanceSelect = $this->getPaymentBalanceSelectExpression('payments');
-            ->orderBy('payment_date', 'DESC')
+
+        $select = [
+            'payments.*',
+            'invoices.invoice_number',
+            'invoices.total_amount AS invoice_total',
+            'invoices.status AS invoice_status',
+            $paymentBalanceSelect,
+        ];
+
+        return $this->select(implode(', ', $select), false)
+            ->join('invoices', 'invoices.id = payments.invoice_id', 'inner')
+            ->where('invoices.school_year', $schoolYear)
+            ->orderBy('payments.payment_date', 'DESC')
+            ->orderBy('payments.id', 'DESC')
             ->findAll();
     }
 
     /**
-     * Generate a new transaction ID.
+     * Generate a transaction ID.
      *
-     * @return string
+     * This is okay for legacy/manual use, but a DB UNIQUE KEY on transaction_id is still required.
      */
     public function generateNewTransactionId(): string
     {
@@ -173,41 +316,115 @@ class PaymentModel extends Model
             ->first();
 
         if ($latest && isset($latest['transaction_id'])) {
-            $lastNumber = (int) str_replace($prefix, '', $latest['transaction_id']);
+            $lastNumber = (int) str_replace($prefix, '', (string) $latest['transaction_id']);
             $newNumber = $lastNumber + 1;
         } else {
             $newNumber = 1;
         }
 
-        return $prefix . str_pad($newNumber, 6, '0', STR_PAD_LEFT);
+        return $prefix . str_pad((string) $newNumber, 6, '0', STR_PAD_LEFT);
     }
 
+    /**
+     * Get latest successful payment per invoice.
+     *
+     * Uses latest payment ID instead of latest date only.
+     * Date-only matching can return duplicates when several payments happen on the same day.
+     */
     public function getPaymentsByInvoice($invoiceIds): array
     {
-        // Normalize input
         if (!is_array($invoiceIds)) {
             $invoiceIds = [$invoiceIds];
         }
+
+        $invoiceIds = array_values(array_filter(array_map('intval', $invoiceIds)));
+
         if (empty($invoiceIds)) {
             return [];
         }
 
-        // Subquery: latest payment_date per invoice (Full/Partial only)
         $latestSub = $this->db->table('payments')
-            ->select('invoice_id, MAX(payment_date) AS max_date')
+            ->select('invoice_id, MAX(id) AS max_id')
-            ->whereIn('status', ['Full', 'Paid', 'Partially Paid'])
+            ->whereIn('invoice_id', $invoiceIds)
+            ->where('paid_amount >', 0)
+            ->where($this->successfulPaymentWhereSql('status'), null, false)
             ->groupBy('invoice_id')
             ->getCompiledSelect();
 
-        // Main query: join payments to the latest per invoice
+        return $this->db->table('payments p')
-        $rows = $this->db->table('payments p')
+            ->select([
-            ->select('p.invoice_id, p.paid_amount AS last_paid_amount, p.payment_date AS last_payment_date, p.status AS last_payment_status')
+                'p.invoice_id',
-            ->join("($latestSub) latest", 'latest.invoice_id = p.invoice_id AND latest.max_date = p.payment_date', 'inner', false) // <-- escape=false
+                'p.id AS payment_id',
+                'p.paid_amount AS last_paid_amount',
+                'p.payment_date AS last_payment_date',
+                'p.status AS last_payment_status',
+                'p.transaction_id',
+                'p.payment_method',
+                'p.check_number',
+            ])
+            ->join(
+                "($latestSub) latest",
+                'latest.invoice_id = p.invoice_id AND latest.max_id = p.id',
+                'inner',
+                false
+            )
             ->whereIn('p.invoice_id', $invoiceIds)
-            ->whereIn('p.status', ['Full', 'Paid', 'Partially Paid'])
             ->get()
             ->getResultArray();
+    }
 
-        return $rows;
+    private function getPaymentBalanceField(): ?string
+    {
+        foreach (['balance_after_payment', 'balance', 'balance_amount'] as $field) {
+            if (in_array($field, $this->paymentColumns, true)) {
+                return $field;
+            }
+        }
+
+        return null;
+    }
+
+    private function getPaymentBalanceSelectExpression(string $alias = 'payments'): string
+    {
+        $field = $this->getPaymentBalanceField();
+
+        if ($field === null) {
+            return '0.00 AS balance_after_payment';
+        }
+
+        return "{$alias}.{$field} AS balance_after_payment";
+    }
+
+    private function successfulPaymentWhereSql(string $column): string
+    {
+        $escapedStatuses = array_map(
+            fn (string $status): string => $this->db->escape($status),
+            $this->excludedPaymentStatuses
+        );
+
+        return 'LOWER(COALESCE(' . $column . ', \'\')) NOT IN (' . implode(', ', $escapedStatuses) . ')';
+    }
+
+    private function getTableColumns(string $table): array
+    {
+        try {
+            return $this->db->getFieldNames($table);
+        } catch (\Throwable $e) {
+            log_message('error', '[PaymentModel] Could not read table columns for {table}: {error}', [
+                'table' => $table,
+                'error' => $e->getMessage(),
+            ]);
+
+            return [];
+        }
+    }
+
+    private function columnExists(string $table, string $column): bool
+    {
+        try {
+            return $this->db->fieldExists($column, $table);
+        } catch (\Throwable $e) {
+            return false;
+        }
     }
 }

app/Models/PaymentTransactionModel.php

@@ -18,18 +18,11 @@ class PaymentTransactionModel extends Model
         'payment_method',
         'payment_status',
         'transaction_fee',
-        'payment_reference',
         'semester',
-        'school_year',
+        'school_year'
-        'is_full_payment',          // Flag to track full payment status
-        'created_at',
-        'updated_at'
     ];
 
-    // Set automatic timestamps
+    protected $useTimestamps = false;
-    protected $useTimestamps = true;
-    protected $createdField  = 'created_at';
-    protected $updatedField  = 'updated_at';
 
     // Validation rules
     protected $validationRules = [
@@ -38,8 +31,8 @@ class PaymentTransactionModel extends Model
         'amount' => 'required|decimal',
         'payment_method' => 'required|string|max_length[50]',
         'payment_status' => 'required|string|max_length[50]',
-        'payment_reference' => 'permit_empty|string|max_length[255]',
+        'semester' => 'permit_empty|string|max_length[30]',
-        'is_full_payment' => 'required|in_list[0,1]', // 0 for installment, 1 for full payment
+        'school_year' => 'permit_empty|string|max_length[9]',
     ];
 
     // Custom error messages
@@ -64,13 +57,6 @@ class PaymentTransactionModel extends Model
             'required' => 'The payment status is required.',
             'max_length' => 'The payment status must not exceed 50 characters.',
         ],
-        'payment_reference' => [
-            'max_length' => 'The payment reference must not exceed 255 characters.',
-        ],
-        'is_full_payment' => [
-            'required' => 'The full payment flag is required.',
-            'in_list' => 'The full payment flag must be either 0 (installment) or 1 (full payment).',
-        ],
     ];
 
     /**
@@ -106,7 +92,9 @@ class PaymentTransactionModel extends Model
      */
     public function updateTransactionStatus($transactionId, $status)
     {
-        return $this->update($transactionId, ['payment_status' => $status]);
+        return $this->where('transaction_id', $transactionId)
+            ->set(['payment_status' => $status])
+            ->update();
     }
 
     /**
@@ -130,7 +118,9 @@ class PaymentTransactionModel extends Model
      */
     public function updateTransactionFee($transactionId, $transactionFee)
     {
-        return $this->update($transactionId, ['transaction_fee' => $transactionFee]);
+        return $this->where('transaction_id', $transactionId)
+            ->set(['transaction_fee' => $transactionFee])
+            ->update();
     }
 
     /**

app/Models/PlacementLevelModel.php

@@ -12,7 +12,6 @@ class PlacementLevelModel extends Model
     protected $allowedFields = [
         'student_id',
         'level',
-        'school_year',
         'created_by',
         'updated_by',
         'created_at',

app/Models/PreferencesModel.php

@@ -12,8 +12,8 @@ class PreferencesModel extends Model
     // Allowed fields to enable mass assignment
     protected $allowedFields = [
         'user_id',                      // Foreign key linking preferences to a user
-        'receive_email_notifications',  // Email notifications preference
+        'notification_email',           // Email notifications preference
-        'receive_sms_notifications',    // SMS notifications preference
+        'notification_sms',             // SMS notifications preference
         'theme',                        // Theme preference (e.g., light or dark)
         'language',                     // Language preference
         'timezone',                     // Timezone preference
@@ -22,11 +22,6 @@ class PreferencesModel extends Model
         'menu_custom_bg',               // Custom menu background color
         'menu_custom_text',             // Custom menu text color
         'menu_custom_mode',             // Custom menu mode: light|dark
-        'receive_push_notifications',   // Push notifications preference
-        'daily_summary_email',          // Daily summary email preference
-        'privacy_mode',                 // Privacy mode setting
-        'marketing_emails',             // Marketing emails preference
-        'account_activity_alerts',      // Account activity alerts preference
         'created_at',                   // Timestamp of when the record was created
         'updated_at'                    // Timestamp of when the record was last updated
     ];

app/Models/SchoolYearClosingBatchModel.php

@@ -0,0 +1,29 @@
+<?php
+
+namespace App\Models;
+
+use CodeIgniter\Model;
+
+class SchoolYearClosingBatchModel extends Model
+{
+    protected $table = 'school_year_closing_batches';
+    protected $primaryKey = 'id';
+    protected $returnType = 'array';
+    protected $useTimestamps = true;
+
+    protected $allowedFields = [
+        'source_school_year_id',
+        'target_school_year_id',
+        'status',
+        'preview_hash',
+        'total_families',
+        'total_positive_balance',
+        'total_credit_balance',
+        'started_by',
+        'completed_by',
+        'started_at',
+        'completed_at',
+        'failed_at',
+        'failure_message',
+    ];
+}

app/Models/SchoolYearClosingItemModel.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace App\Models;
+
+use CodeIgniter\Model;
+
+class SchoolYearClosingItemModel extends Model
+{
+    protected $table = 'school_year_closing_items';
+    protected $primaryKey = 'id';
+    protected $returnType = 'array';
+    protected $useTimestamps = true;
+
+    protected $allowedFields = [
+        'closing_batch_id',
+        'family_id',
+        'source_balance',
+        'credit_amount',
+        'adjustment_amount',
+        'carry_forward_amount',
+        'target_invoice_id',
+        'target_adjustment_id',
+        'status',
+        'error_message',
+    ];
+}

app/Models/SchoolYearModel.php

@@ -0,0 +1,47 @@
+<?php
+
+namespace App\Models;
+
+use CodeIgniter\Model;
+
+class SchoolYearModel extends Model
+{
+    protected $table = 'school_years';
+    protected $primaryKey = 'id';
+    protected $returnType = 'array';
+    protected $useTimestamps = true;
+
+    protected $allowedFields = [
+        'name',
+        'status',
+        'starts_on',
+        'ends_on',
+        'description',
+        'registration_starts_on',
+        'registration_ends_on',
+        'previous_school_year_id',
+        'next_school_year_id',
+        'activated_at',
+        'closing_started_at',
+        'closed_at',
+        'archived_at',
+        'created_by',
+        'updated_by',
+    ];
+
+    protected $validationRules = [
+        'name' => 'required|regex_match[/^\d{4}-\d{4}$/]|max_length[9]',
+        'status' => 'required|in_list[draft,active,closing,closed,archived]',
+        'starts_on' => 'permit_empty|valid_date[Y-m-d]',
+        'ends_on' => 'permit_empty|valid_date[Y-m-d]',
+        'registration_starts_on' => 'permit_empty|valid_date[Y-m-d]',
+        'registration_ends_on' => 'permit_empty|valid_date[Y-m-d]',
+    ];
+
+    public function active(): ?array
+    {
+        return $this->where('status', 'active')
+            ->orderBy('id', 'DESC')
+            ->first();
+    }
+}

app/Models/SchoolYearTransitionLogModel.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace App\Models;
+
+use CodeIgniter\Model;
+
+class SchoolYearTransitionLogModel extends Model
+{
+    protected $table = 'school_year_transition_logs';
+    protected $primaryKey = 'id';
+    protected $returnType = 'array';
+    protected $useTimestamps = false;
+
+    protected $allowedFields = [
+        'school_year_id',
+        'from_status',
+        'to_status',
+        'action',
+        'performed_by',
+        'metadata_json',
+        'created_at',
+    ];
+}

app/Models/SectionModel.php

@@ -12,12 +12,11 @@ class SectionModel extends Model
     protected $allowedFields = [
         'section_name',
         'description',
-        'created_at',
         'updated_at',
         'updated_by'
     ];
 
-    protected $useTimestamps = true;
+    protected $useTimestamps = false;
     protected $createdField  = 'created_at';
     protected $updatedField  = 'updated_at';
 

app/Models/StaffModel.php

@@ -21,8 +21,6 @@ class StaffModel extends Model
         'phone',
         'role_name',
         'active_role',
-        'status',
-        'school_year',
         'created_at',
         'updated_at'
     ];
@@ -44,7 +42,7 @@ class StaffModel extends Model
     }
 
     /**
-     * Insert or update a staff row keyed by (user_id, school_year).
+     * Insert or update a staff row keyed by user_id.
      * Ensures created_at is only set on insert and updated_at can be provided by caller.
      */
     public function upsert(array $data): bool
@@ -53,27 +51,9 @@ class StaffModel extends Model
             return false;
         }
 
-        // If school_year not provided, try to keep existing or fall back to current year string
+        unset($data['school_year'], $data['status']);
-        $schoolYear = $data['school_year'] ?? null;
 
-        if ($schoolYear === null) {
-            // Try to find any existing row by user_id to reuse its school_year
-            $existingAny = $this->where('user_id', $data['user_id'])->orderBy('id', 'DESC')->first();
-            if ($existingAny && isset($existingAny['school_year'])) {
-                $schoolYear = $existingAny['school_year'];
-                $data['school_year'] = $schoolYear;
-            }
-        }
-
-        $existing = null;
-        if ($schoolYear !== null) {
-            $existing = $this->where('user_id', $data['user_id'])
-                             ->where('school_year', $schoolYear)
-                             ->first();
-        } else {
-            // If no school_year, treat (user_id) as key
         $existing = $this->where('user_id', $data['user_id'])->first();
-        }
 
         $now = utc_now();
 
@@ -100,4 +80,3 @@ class StaffModel extends Model
         return $this->insert($data) !== false;
     }
 }
-

app/Models/StatsModel.php

@@ -1,22 +0,0 @@
-<?php
-
-namespace App\Models;
-
-use CodeIgniter\Model;
-
-class StatsModel extends Model
-{
-    protected $table = 'stats';
-    protected $primaryKey = 'id';
-    protected $allowedFields = [
-        'students', 
-        'teachers', 
-        'admins', 
-        'users'
-    ];
-
-    public function getStats()
-    {
-        return $this->findAll();
-    }
-}

app/Models/StudentClassModel.php

@@ -2,229 +2,529 @@
 
 namespace App\Models;
 
+use CodeIgniter\Database\BaseBuilder;
 use CodeIgniter\Model;
 
 class StudentClassModel extends Model
 {
     protected $table      = 'student_class';
     protected $primaryKey = 'id';
+    protected $returnType = 'array';
+
+    protected $useAutoIncrement = true;
+    protected $protectFields    = true;
 
     protected $allowedFields = [
         'student_id',
-        'school_id',
         'class_section_id',
         'school_year',
         'is_event_only',
         'description',
         'updated_by',
+        'created_at',
         'updated_at',
-        'created_at'
     ];
 
     protected $useTimestamps = true;
     protected $createdField  = 'created_at';
     protected $updatedField  = 'updated_at';
 
+    protected $validationRules = [
+        'student_id'      => 'required|integer',
+        'class_section_id'=> 'permit_empty|integer',
+        'school_year'     => 'required|max_length[20]',
+        'is_event_only'   => 'permit_empty|in_list[0,1]',
+        'updated_by'      => 'permit_empty|integer',
+    ];
+
+    protected $validationMessages = [];
+    protected $skipValidation     = false;
+    protected $cleanValidationRules = true;
+
     /**
-     * Scope: only students active for classes/attendance.
+     * Create a fresh builder for student_class.
+     *
+     * Using a fresh builder prevents WHERE conditions from a previous model
+     * query from leaking into another query.
      */
-    public function active(): self
+    private function freshBuilder(): BaseBuilder
     {
-        return $this->select('student_class.*')
+        return $this->db->table($this->table);
-            ->join('students', 'students.id = student_class.student_id', 'inner')
+    }
+
+    /**
+     * Create a fresh builder scoped to active students.
+     */
+    private function activeStudentsBuilder(): BaseBuilder
+    {
+        return $this->freshBuilder()
+            ->select('student_class.*')
+            ->join(
+                'students',
+                'students.id = student_class.student_id',
+                'inner'
+            )
             ->where('students.is_active', 1);
     }
 
-    public function getClassSectionNameByStudentId(int $studentId): ?string
+    /**
+     * Apply the active-student scope to the model.
+     *
+     * Prefer the dedicated retrieval methods below for new code because they
+     * use fresh builders and cannot inherit stale model query state.
+     */
+    public function active(): self
     {
-        return $this->db->table($this->table)
+        return $this
+            ->select('student_class.*')
+            ->join(
+                'students',
+                'students.id = student_class.student_id',
+                'inner'
+            )
+            ->where('students.is_active', 1);
+    }
+
+    /**
+     * Return the most recent class section name assigned to a student.
+     */
+    public function getClassSectionNameByStudentId(
+        int $studentId,
+        ?string $schoolYear = null
+    ): ?string {
+        $builder = $this->freshBuilder()
             ->select('cs.class_section_name')
-            ->join('classSection cs', 'cs.class_section_id = student_class.class_section_id')
+            ->join(
+                'classSection cs',
+                'cs.class_section_id = student_class.class_section_id',
+                'inner'
+            )
             ->where('student_class.student_id', $studentId)
-            ->get()
+            ->where(
-            ->getRow('class_section_name');
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            );
+
+        if ($schoolYear !== null && trim($schoolYear) !== '') {
+            $builder->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            );
         }
 
-
+        $row = $builder
-    // Custom findAll() method
+            ->orderBy('student_class.created_at', 'DESC')
-    public function findAll($limit = 0, $offset = 0)
-    {
-        // Optional: Add custom logic before fetching all records
-
-        // Then call the parent findAll method to fetch the records
-        return parent::findAll($limit, $offset);
-    }
-
-    // Method to get all students in a specific class section (optionally scoped to term)
-    public function getClassStudents($classSectionId, ?string $schoolYear = null)
-    {
-        $qb = $this->active()->where('student_class.class_section_id', $classSectionId);
-        if ($schoolYear !== null && $schoolYear !== '') {
-            $qb->where('student_class.school_year', $schoolYear);
-        }
-        return $qb->findAll();
-    }
-
-    /**
-     * Return class section names for a student in a given school year.
-     *
-     * @param int|string $studentId
-     * @param string $schoolYear
-     * @param bool $asArray When true, return an array of names; otherwise a comma-separated string.
-     * @return array|string
-     */
-    public function getClassSectionsByStudentId($studentId, string $schoolYear, bool $asArray = false)
-    {
-        $rows = $this->db->table('student_class')
-            ->select('cs.class_section_name')
-            ->join('classSection cs', 'student_class.class_section_id = cs.class_section_id')
-            ->where('student_class.student_id', $studentId)
-            ->where('student_class.class_section_id IS NOT NULL', null, false)
-            ->where('student_class.school_year', $schoolYear)
-            ->orderBy('cs.class_section_name', 'ASC')
-            ->get()
-            ->getResultArray();
-
-        $names = array_values(array_unique(array_filter(array_map(static function ($row) {
-            return $row['class_section_name'] ?? null;
-        }, $rows))));
-
-        if ($asArray) {
-            return $names;
-        }
-
-        return !empty($names) ? implode(', ', $names) : '';
-    }
-
-    /**
-     * Return class section names for a student in a given school year, with optional event flag.
-     *
-     * @param int|string $studentId
-     * @param string $schoolYear
-     * @param bool $asArray When true, return an array of names; otherwise a comma-separated string.
-     * @return array|string
-     */
-    public function getClassSectionsByStudentIdWithFlags($studentId, string $schoolYear, bool $asArray = false)
-    {
-        $rows = $this->db->table('student_class')
-            ->select('cs.class_section_name, student_class.is_event_only')
-            ->join('classSection cs', 'student_class.class_section_id = cs.class_section_id')
-            ->where('student_class.student_id', $studentId)
-            ->where('student_class.class_section_id IS NOT NULL', null, false)
-            ->where('student_class.school_year', $schoolYear)
-            ->orderBy('cs.class_section_name', 'ASC')
-            ->get()
-            ->getResultArray();
-
-        $names = array_values(array_unique(array_filter(array_map(static function ($row) {
-            $name = $row['class_section_name'] ?? null;
-            if (!$name) return null;
-            $isEvent = (int)($row['is_event_only'] ?? 0) === 1;
-            return $isEvent ? ($name . ' (Event)') : $name;
-        }, $rows))));
-
-        if ($asArray) {
-            return $names;
-        }
-
-        return !empty($names) ? implode(', ', $names) : '';
-    }
-
-    /**
-     * Return class_section_id values for a student/year.
-     *
-     * @param int|string $studentId
-     * @param string $schoolYear
-     * @return int[]
-     */
-    public function getClassSectionIdsByStudentId($studentId, string $schoolYear): array
-    {
-        $rows = $this->db->table('student_class')
-            ->select('class_section_id')
-            ->where('student_id', $studentId)
-            ->where('class_section_id IS NOT NULL', null, false)
-            ->where('school_year', $schoolYear)
-            ->get()
-            ->getResultArray();
-
-        $ids = array_map(static fn($r) => (int)($r['class_section_id'] ?? 0), $rows);
-        return array_values(array_filter(array_unique($ids), static fn($v) => $v > 0));
-    }
-
-
-    public function getStudentsByClassSectionIds(array $classSectionIds)
-    {
-        return $this->select('students.id as student_id, students.firstname, students.lastname, students.school_id, students.is_new,students.photo_consent ,students.age, student_class.school_year, student_class.class_section_id')
-            ->join('students', 'students.id = student_class.student_id')
-            ->whereIn('student_class.class_section_id', $classSectionIds)
-            ->where('students.is_active', 1)
-            ->get()
-            ->getResultArray();
-    }
-
-    /**
-     * Get the class ID (grade) for a specific student.
-     *
-     * @param int $studentId
-     * @return string class_id or 'N/A'
-     */
-    public function getStudentGrade(int $studentId): string
-    {
-        $studentClass = $this->where('student_id', $studentId)
-            ->where('is_event_only', 0)
-            ->orderBy('created_at', 'DESC') // in case of multiple entries, get latest
-            ->first();
-
-        if ($studentClass && isset($studentClass['class_section_id'])) {
-            $classSection = $this->db->table('classSection')
-                ->where('class_section_id', $studentClass['class_section_id'])
             ->get()
             ->getRowArray();
 
-            return $classSection['class_id'] ?? 'N/A';
+        if (!$row) {
+            return null;
         }
 
-        log_message('error', "Student class or section not found for student ID: $studentId");
+        $name = trim((string) ($row['class_section_name'] ?? ''));
+
+        return $name !== '' ? $name : null;
+    }
+
+    /**
+     * Get active students assigned to a class section.
+     *
+     * student_class is scoped by school year only. It has no semester column.
+     */
+    public function getClassStudents(
+        int $classSectionId,
+        ?string $schoolYear = null
+    ): array {
+        $builder = $this->activeStudentsBuilder()
+            ->where(
+                'student_class.class_section_id',
+                $classSectionId
+            )
+            ->where(
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            );
+
+        if ($schoolYear !== null && trim($schoolYear) !== '') {
+            $builder->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            );
+        }
+
+        return $builder
+            ->orderBy('students.lastname', 'ASC')
+            ->orderBy('students.firstname', 'ASC')
+            ->get()
+            ->getResultArray();
+    }
+
+    /**
+     * Return student IDs assigned to a class section for a school year.
+     */
+    public function getStudentIdsByClassSection(
+        int $classSectionId,
+        string $schoolYear
+    ): array {
+        $schoolYear = trim($schoolYear);
+
+        $builder = $this->freshBuilder()
+            ->select('student_class.student_id')
+            ->join(
+                'students',
+                'students.id = student_class.student_id',
+                'inner'
+            )
+            ->where(
+                'student_class.class_section_id',
+                $classSectionId
+            )
+            ->where('students.is_active', 1)
+            ->where(
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            );
+
+        if ($schoolYear !== '') {
+            $builder->where(
+                'student_class.school_year',
+                $schoolYear
+            );
+        }
+
+        $rows = $builder
+            ->groupBy('student_class.student_id')
+            ->get()
+            ->getResultArray();
+
+        $studentIds = array_map(
+            static fn(array $row): int =>
+                (int) ($row['student_id'] ?? 0),
+            $rows
+        );
+
+        return array_values(array_unique(array_filter(
+            $studentIds,
+            static fn(int $studentId): bool => $studentId > 0
+        )));
+    }
+
+    /**
+     * Return class section names assigned to a student for a school year.
+     *
+     * @return array|string
+     */
+    public function getClassSectionsByStudentId(
+        int|string $studentId,
+        string $schoolYear,
+        bool $asArray = false
+    ): array|string {
+        $rows = $this->freshBuilder()
+            ->select('cs.class_section_name')
+            ->join(
+                'classSection cs',
+                'student_class.class_section_id = cs.class_section_id',
+                'inner'
+            )
+            ->where('student_class.student_id', $studentId)
+            ->where(
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            )
+            ->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            )
+            ->orderBy('cs.class_section_name', 'ASC')
+            ->get()
+            ->getResultArray();
+
+        $names = array_map(
+            static fn(array $row): string =>
+                trim((string) ($row['class_section_name'] ?? '')),
+            $rows
+        );
+
+        $names = array_values(array_unique(array_filter(
+            $names,
+            static fn(string $name): bool => $name !== ''
+        )));
+
+        return $asArray ? $names : implode(', ', $names);
+    }
+
+    /**
+     * Return class section names and indicate event-only assignments.
+     *
+     * @return array|string
+     */
+    public function getClassSectionsByStudentIdWithFlags(
+        int|string $studentId,
+        string $schoolYear,
+        bool $asArray = false
+    ): array|string {
+        $rows = $this->freshBuilder()
+            ->select(
+                'cs.class_section_name, student_class.is_event_only'
+            )
+            ->join(
+                'classSection cs',
+                'student_class.class_section_id = cs.class_section_id',
+                'inner'
+            )
+            ->where('student_class.student_id', $studentId)
+            ->where(
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            )
+            ->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            )
+            ->orderBy('cs.class_section_name', 'ASC')
+            ->get()
+            ->getResultArray();
+
+        $names = [];
+
+        foreach ($rows as $row) {
+            $name = trim(
+                (string) ($row['class_section_name'] ?? '')
+            );
+
+            if ($name === '') {
+                continue;
+            }
+
+            if ((int) ($row['is_event_only'] ?? 0) === 1) {
+                $name .= ' (Event)';
+            }
+
+            $names[] = $name;
+        }
+
+        $names = array_values(array_unique($names));
+
+        return $asArray ? $names : implode(', ', $names);
+    }
+
+    /**
+     * Return class-section IDs assigned to a student for a school year.
+     *
+     * @return int[]
+     */
+    public function getClassSectionIdsByStudentId(
+        int|string $studentId,
+        string $schoolYear
+    ): array {
+        $rows = $this->freshBuilder()
+            ->select('student_class.class_section_id')
+            ->where('student_class.student_id', $studentId)
+            ->where(
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            )
+            ->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            )
+            ->get()
+            ->getResultArray();
+
+        $ids = array_map(
+            static fn(array $row): int =>
+                (int) ($row['class_section_id'] ?? 0),
+            $rows
+        );
+
+        return array_values(array_unique(array_filter(
+            $ids,
+            static fn(int $id): bool => $id > 0
+        )));
+    }
+
+    /**
+     * Return active students for a set of class-section IDs.
+     */
+    public function getStudentsByClassSectionIds(
+        array $classSectionIds,
+        ?string $schoolYear = null
+    ): array {
+        $classSectionIds = array_values(array_unique(array_filter(
+            array_map('intval', $classSectionIds),
+            static fn(int $id): bool => $id > 0
+        )));
+
+        if ($classSectionIds === []) {
+            return [];
+        }
+
+        $builder = $this->freshBuilder()
+            ->select([
+                'students.id AS student_id',
+                'students.firstname',
+                'students.lastname',
+                'students.school_id',
+                'students.is_new',
+                'students.photo_consent',
+                'students.age',
+                'student_class.school_year',
+                'student_class.class_section_id',
+                'student_class.is_event_only',
+            ])
+            ->join(
+                'students',
+                'students.id = student_class.student_id',
+                'inner'
+            )
+            ->whereIn(
+                'student_class.class_section_id',
+                $classSectionIds
+            )
+            ->where('students.is_active', 1);
+
+        if ($schoolYear !== null && trim($schoolYear) !== '') {
+            $builder->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            );
+        }
+
+        return $builder
+            ->orderBy('students.lastname', 'ASC')
+            ->orderBy('students.firstname', 'ASC')
+            ->get()
+            ->getResultArray();
+    }
+
+    /**
+     * Return the student's most recent non-event class grade.
+     */
+    public function getStudentGrade(
+        int $studentId,
+        ?string $schoolYear = null
+    ): string {
+        $builder = $this->freshBuilder()
+            ->select('classSection.class_id')
+            ->join(
+                'classSection',
+                'classSection.class_section_id = student_class.class_section_id',
+                'inner'
+            )
+            ->where('student_class.student_id', $studentId)
+            ->where('student_class.is_event_only', 0)
+            ->where(
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            );
+
+        if ($schoolYear !== null && trim($schoolYear) !== '') {
+            $builder->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            );
+        }
+
+        $row = $builder
+            ->orderBy('student_class.created_at', 'DESC')
+            ->get()
+            ->getRowArray();
+
+        $classId = trim((string) ($row['class_id'] ?? ''));
+
+        if ($classId !== '') {
+            return $classId;
+        }
+
+        log_message(
+            'warning',
+            'Student class or section not found for student ID: {studentId}',
+            ['studentId' => $studentId]
+        );
+
         return 'N/A';
     }
 
     /**
-     * Return true if the student has at least one non-event class assignment for the year.
+     * Determine whether a student has a non-event assignment for a year.
      */
-    public function hasNonEventAssignment(int $studentId, string $schoolYear): bool
+    public function hasNonEventAssignment(
-    {
+        int $studentId,
-        return (bool) $this->where('student_id', $studentId)
+        string $schoolYear
-            ->where('school_year', $schoolYear)
+    ): bool {
-            ->where('is_event_only', 0)
+        return $this->freshBuilder()
-            ->first();
+            ->select('student_class.id')
+            ->where('student_class.student_id', $studentId)
+            ->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            )
+            ->where('student_class.is_event_only', 0)
+            ->limit(1)
+            ->get()
+            ->getRowArray() !== null;
     }
 
     /**
-     * Return student counts per class_section_id, optionally scoped to a school year.
+     * Return active student counts by class section.
      *
-     * @param string|null $schoolYear
+     * @return array<int, int>
-     * @return array<int|string,int> Map of class_section_id => count
      */
-    public function getStudentCountsBySection(?string $schoolYear = null): array
+    public function getStudentCountsBySection(
-    {
+        ?string $schoolYear = null
-        $qb = $this->db->table($this->table)
+    ): array {
-            ->select('class_section_id, COUNT(*) AS total')
+        $builder = $this->freshBuilder()
-            ->join('students', 'students.id = student_class.student_id', 'inner')
+            ->select(
+                'student_class.class_section_id, ' .
+                'COUNT(DISTINCT student_class.student_id) AS total'
+            )
+            ->join(
+                'students',
+                'students.id = student_class.student_id',
+                'inner'
+            )
             ->where('students.is_active', 1)
-            ->where('student_class.class_section_id IS NOT NULL', null, false)
+            ->where(
+                'student_class.class_section_id IS NOT NULL',
+                null,
+                false
+            )
             ->groupBy('student_class.class_section_id');
 
-        if ($schoolYear !== null && $schoolYear !== '') {
+        if ($schoolYear !== null && trim($schoolYear) !== '') {
-            $qb->where('student_class.school_year', $schoolYear);
+            $builder->where(
+                'student_class.school_year',
+                trim($schoolYear)
+            );
         }
 
-        $rows = $qb->get()->getResultArray();
+        $rows = $builder
-        $out = [];
+            ->get()
-        foreach ($rows as $r) {
+            ->getResultArray();
-            $cid = $r['class_section_id'] ?? null;
+
-            if ($cid === null || $cid === '') continue;
+        $counts = [];
-            $out[$cid] = (int)($r['total'] ?? 0);
+
+        foreach ($rows as $row) {
+            $classSectionId = (int) (
+                $row['class_section_id'] ?? 0
+            );
+
+            if ($classSectionId <= 0) {
+                continue;
             }
-        return $out;
+
+            $counts[$classSectionId] = (int) (
+                $row['total'] ?? 0
+            );
+        }
+
+        return $counts;
     }
 }

app/Models/StudentModel.php

@@ -19,7 +19,6 @@ class StudentModel extends Model
         'photo_consent',
         'is_new',
         'parent_id',
-        'school_year',
         'registration_date',
         'tuition_paid',
         'year_of_registration',
@@ -122,7 +121,9 @@ class StudentModel extends Model
 
         // school_year filter (skip if null or "all")
         if ($schoolYear !== null && strtolower($schoolYear) !== 'all') {
-            $builder->where('students.school_year', $schoolYear);
+            $builder
+                ->join('student_class sc_filter', 'sc_filter.student_id = students.id', 'inner')
+                ->where('sc_filter.school_year', $schoolYear);
         }
 
         return $builder
@@ -171,12 +172,13 @@ class StudentModel extends Model
                 users.cellphone as phone,
                 students.registration_grade,
                 classSection.class_section_name as current_class,
-                "' . $schoolYear . '" as school_year,
+                student_class.school_year,
                 "' . $semester . '" as semester
             ')
             ->join('users', 'users.id = students.parent_id', 'left')
             ->join('student_class', 'student_class.student_id = students.id', 'left')
-            ->join('class_section', 'student_class.class_section_id = classSection.id', 'left')
+            ->join('classSection', 'student_class.class_section_id = classSection.class_section_id', 'left')
+            ->where('student_class.school_year', $schoolYear)
             ->get()
             ->getResultArray();
     }
@@ -199,7 +201,7 @@ class StudentModel extends Model
             ')
             ->join('student_class', 'students.id = student_class.student_id', 'left')
             ->join('teacher_class', 'student_class.class_section_id = teacher_class.class_section_id', 'left')
-            ->join('class_section', 'student_class.class_section_id = classSection.id', 'left')
+            ->join('classSection', 'student_class.class_section_id = classSection.class_section_id', 'left')
             ->where('teacher_class.teacher_id', $teacherId)
             ->get()
             ->getResultArray();

app/Models/UserModel.php

@@ -28,7 +28,6 @@ class UserModel extends Model
         'failed_attempts',
         'last_failed_at',
         'semester',
-        'school_year',
         'status',
         'is_suspended',
         'is_verified',
@@ -237,13 +236,21 @@ class UserModel extends Model
 
     public function getUsersByRoleAndSchoolYear(string $roleName, string $schoolYear): array
     {
-        return $this->select('users.*')
+        $builder = $this->select('users.*')
             ->join('user_roles', 'user_roles.user_id = users.id')
             ->join('roles', 'roles.id = user_roles.role_id')
             ->where('roles.name', $roleName)
-            ->where('users.school_year', $schoolYear)
+            ->where('user_roles.deleted_at', null);
-            ->where('user_roles.deleted_at', null)
+
-            ->findAll();
+        if ($this->roleUsesTeacherAssignments($roleName)) {
+            $builder->join(
+                'teacher_class tc_year',
+                'tc_year.teacher_id = users.id AND tc_year.school_year = ' . $this->db->escape($schoolYear),
+                'inner'
+            );
+        }
+
+        return $builder->groupBy('users.id')->findAll();
     }
 
     public function countAdminsBySchoolYear(string $schoolYear): int
@@ -260,8 +267,8 @@ class UserModel extends Model
             )
             ->join('user_roles ur', 'ur.user_id = u.id', 'inner')
             ->join('roles r', 'r.id = ur.role_id', 'inner')
-            ->where('u.school_year', $schoolYear)
             ->where('r.is_active', 1)
+            ->where('LOWER(r.name) NOT IN ("guest","teacher","teacher_assistant","parent")', null, false)
             ->groupBy('u.id')
             ->having('is_admin', 1);
 
@@ -357,28 +364,13 @@ class UserModel extends Model
         }
 
         if (!empty($schoolYear)) {
-            // Prefer user_roles.school_year if present (true year-scoped roles)
-            $userRolesFields = [];
-            try {
-                $userRolesFields = $this->db->getFieldNames('user_roles');
-            } catch (\Throwable $e) {
-                // ignore
-            }
-
-            if (in_array('school_year', $userRolesFields, true)) {
-                $builder->where('ur.school_year', $schoolYear);
-            } else {
-                // Fallback: role-aware filter
             $escYear = $this->db->escape($schoolYear);
 
             // Include user if:
             //  (A) They are assigned as teacher/TA in teacher_class for that year
             //   OR
-                //  (B) They have at least one non-teacher-ish (and non-parent) role (admin/staff/etc.)
+            //  (B) They have at least one non-teacher-ish (and non-parent) global role.
-                //
-                // Teacher-ish detection: name contains 'teacher' OR equals 'ta'
             $builder->groupStart()
-                    // A) has teacher assignment that year
                 ->where("
                 EXISTS (
                     SELECT 1
@@ -387,7 +379,6 @@ class UserModel extends Model
                       AND tc.school_year = {$escYear}
                 )
             ", null, false)
-                    // B) OR has any non-teacher-ish, non-parent role
                 ->orWhere("
                 EXISTS (
                     SELECT 1
@@ -400,11 +391,16 @@ class UserModel extends Model
             ", null, false)
                 ->groupEnd();
         }
-        }
 
         return $builder
             ->groupBy('users.id')
             ->orderBy($sort === 'roles' ? 'roles' : $sort, $order)
             ->findAll();
     }
+
+    private function roleUsesTeacherAssignments(string $roleName): bool
+    {
+        $role = strtolower(trim($roleName));
+        return $role === 'ta' || str_contains($role, 'teacher');
+    }
 }

app/Models/UserRoleModel.php

@@ -13,8 +13,6 @@ class UserRoleModel extends Model
     protected $allowedFields = [
         'user_id',
         'role_id',
-        'semester',
-        'school_year',
         'created_at',
         'updated_at',
         'updated_by',

app/Models/WhatsappGroupLinkModel.php

@@ -11,7 +11,6 @@ class WhatsappGroupLinkModel extends Model
         'class_section_id',
         'class_section_name',
         'school_year',
-        'semester',
         'invite_link',
         'active',
     ];
@@ -22,9 +21,9 @@ class WhatsappGroupLinkModel extends Model
      *
      * @param int    $sectionId          Class/section code (not PK).
      * @param string $year               School year, e.g. "2025-2026".
-     * @param string $sem                Semester, e.g. "Fall".
+     * @param string $sem                Deprecated/ignored; links are scoped by school year.
      * @param bool   $onlyActive         If true, require active=1.
-     * @param bool   $allowNullSemester  If true, accept rows with semester IS NULL as well.
+     * @param bool   $allowNullSemester  Deprecated/ignored.
      */
     public function getLinkForSection(
         int $sectionId,
@@ -40,17 +39,6 @@ class WhatsappGroupLinkModel extends Model
             ->where('class_section_id', $sectionId)
             ->where('school_year', $year);
 
-        if ($sem !== '') {
-            if ($allowNullSemester) {
-                $b = $b->groupStart()
-                        ->where('semester', $sem)
-                        ->orWhere('semester IS NULL', null, false)
-                     ->groupEnd();
-            } else {
-                $b = $b->where('semester', $sem);
-            }
-        }
-
         if ($onlyActive) {
             $b = $b->where('active', 1);
         }
@@ -70,7 +58,7 @@ class WhatsappGroupLinkModel extends Model
      * @param string   $year
      * @param string   $sem
      * @param bool|null $onlyActive  true: active only, false: inactive only, null: both
-     * @param bool     $allowNullSemester  If true, include rows with semester IS NULL.
+     * @param bool     $allowNullSemester  Deprecated/ignored.
      */
     public function getAllForTerm(
         string $year,
@@ -84,17 +72,6 @@ class WhatsappGroupLinkModel extends Model
         $b = $this->asArray()
             ->where('school_year', $year);
 
-        if ($sem !== '') {
-            if ($allowNullSemester) {
-                $b = $b->groupStart()
-                        ->where('semester', $sem)
-                        ->orWhere('semester IS NULL', null, false)
-                     ->groupEnd();
-            } else {
-                $b = $b->where('semester', $sem);
-            }
-        }
-
         if ($onlyActive === true) {
             $b = $b->where('active', 1);
         } elseif ($onlyActive === false) {
@@ -122,7 +99,6 @@ class WhatsappGroupLinkModel extends Model
             'class_section_id'   => $sectionId,
             'class_section_name' => $sectionName,
             'school_year'        => trim($year),
-            'semester'           => trim($sem),
             'invite_link'        => trim($inviteLink),
             'active'             => $active ? 1 : 0,
         ];
@@ -131,7 +107,6 @@ class WhatsappGroupLinkModel extends Model
         $existing = $this->asArray()
             ->where('class_section_id', $sectionId)
             ->where('school_year', $payload['school_year'])
-            ->where('semester', $payload['semester'])
             ->first();
 
         if ($existing) {

app/Models/WhatsappInviteLogModel.php

@@ -42,7 +42,7 @@ class WhatsappInviteLogModel extends Model
     /**
      * Log a successful send.
      */
-    public function logSuccess(int $parentId, string $email, int $classSectionId = null, int $linkId = null): bool
+    public function logSuccess(int $parentId, string $email, ?int $classSectionId = null, ?int $linkId = null): bool
     {
         return (bool) $this->insert([
             'parent_id'        => $parentId,
@@ -58,7 +58,7 @@ class WhatsappInviteLogModel extends Model
     /**
      * Log a failure with error message.
      */
-    public function logFailure(int $parentId, string $email, string $error, int $classSectionId = null, int $linkId = null): bool
+    public function logFailure(int $parentId, string $email, string $error, ?int $classSectionId = null, ?int $linkId = null): bool
     {
         return (bool) $this->insert([
             'parent_id'        => $parentId,

app/Services/SchoolYearClosingService.php

@@ -0,0 +1,430 @@
+<?php
+
+namespace App\Services;
+
+use App\Models\SchoolYearClosingBatchModel;
+use App\Models\SchoolYearClosingItemModel;
+use App\Models\SchoolYearModel;
+use App\Support\SchoolYear\SchoolYearStatus;
+use CodeIgniter\Database\BaseConnection;
+use InvalidArgumentException;
+use RuntimeException;
+
+final class SchoolYearClosingService
+{
+    public function __construct(
+        private readonly SchoolYearModel $schoolYearModel,
+        private readonly SchoolYearClosingBatchModel $batchModel,
+        private readonly SchoolYearClosingItemModel $itemModel,
+        private readonly SchoolYearManagementService $managementService,
+        private readonly BaseConnection $db,
+    ) {
+    }
+
+    public function preview(int $sourceYearId, ?int $targetYearId = null): array
+    {
+        $source = $this->requireYear($sourceYearId);
+        $target = $targetYearId !== null ? $this->schoolYearModel->find($targetYearId) : $this->nextDraftYear((string) $source['name']);
+        $sourceName = (string) $source['name'];
+
+        $finance = $this->financialSummary($sourceName);
+        $overview = [
+            'students' => $this->countBySchoolYear('student_class', $sourceName, 'student_id'),
+            'families' => $this->countFamilies($sourceName),
+            'classes' => $this->countBySchoolYear('classSection', $sourceName, 'class_id'),
+            'teachers' => $this->countBySchoolYear('teacher_class', $sourceName, 'teacher_id'),
+            'invoices' => $finance['invoice_count'],
+            'total_invoiced' => $finance['total_invoiced'],
+            'total_paid' => $finance['total_paid'],
+            'total_outstanding' => $finance['total_outstanding'],
+        ];
+
+        $findings = [];
+        if ($target === null) {
+            $findings[] = $this->finding('blocking', 'Missing target year', 'Create a draft target school year before starting closing.');
+        } elseif (! in_array((string) $target['status'], [SchoolYearStatus::DRAFT, SchoolYearStatus::ACTIVE], true)) {
+            $findings[] = $this->finding('blocking', 'Invalid target year', 'The target school year must be draft or active.');
+        }
+
+        foreach ($this->unpaidInvoiceFindings($sourceName) as $finding) {
+            $findings[] = $finding;
+        }
+
+        if ($this->countMissingSchoolYearRows('invoices') > 0) {
+            $findings[] = $this->finding('blocking', 'Invoices missing school year', 'Some invoice records are not assigned to a school year.');
+        }
+
+        $carryForward = $this->carryForwardFamilies($sourceName);
+        $warnings = array_values(array_filter($findings, static fn (array $f): bool => $f['severity'] === 'warning'));
+        $blockers = array_values(array_filter($findings, static fn (array $f): bool => $f['severity'] === 'blocking'));
+
+        $result = [
+            'source' => $source,
+            'target' => $target,
+            'overview' => $overview,
+            'finance' => $finance,
+            'findings' => $findings,
+            'blockers' => $blockers,
+            'warnings' => $warnings,
+            'carry_forward' => $carryForward,
+            'generated_at' => date('Y-m-d H:i:s'),
+        ];
+        $result['hash'] = $this->hashPreview($result);
+
+        return $result;
+    }
+
+    public function start(int $sourceYearId, int $targetYearId, ?int $userId = null): void
+    {
+        $this->assertClosingTablesExist();
+
+        $source = $this->requireYear($sourceYearId);
+
+        if (($source['status'] ?? '') !== SchoolYearStatus::ACTIVE) {
+            throw new InvalidArgumentException('Only an active school year can begin closing.');
+        }
+
+        $preview = $this->preview($sourceYearId, $targetYearId);
+        if ($preview['blockers'] !== []) {
+            throw new InvalidArgumentException('Resolve blocking closing issues before starting closing.');
+        }
+
+        $this->db->transStart();
+        $now = date('Y-m-d H:i:s');
+        $batchId = $this->batchModel->insert([
+            'source_school_year_id' => $sourceYearId,
+            'target_school_year_id' => $targetYearId,
+            'status' => 'started',
+            'preview_hash' => $preview['hash'],
+            'total_families' => count($preview['carry_forward']),
+            'total_positive_balance' => $preview['finance']['positive_balance'],
+            'total_credit_balance' => $preview['finance']['credit_balance'],
+            'started_by' => $userId,
+            'started_at' => $now,
+        ], true);
+
+        foreach ($preview['carry_forward'] as $row) {
+            $this->itemModel->insert([
+                'closing_batch_id' => $batchId,
+                'family_id' => (int) $row['family_id'],
+                'source_balance' => $row['source_balance'],
+                'credit_amount' => $row['credit_amount'],
+                'carry_forward_amount' => $row['carry_forward_amount'],
+                'status' => 'pending',
+            ]);
+        }
+
+        $this->schoolYearModel->update($sourceYearId, [
+            'status' => SchoolYearStatus::CLOSING,
+            'closing_started_at' => $now,
+            'updated_by' => $userId,
+        ]);
+        $this->managementService->log($sourceYearId, SchoolYearStatus::ACTIVE, SchoolYearStatus::CLOSING, 'closing_start', $userId, [
+            'target_school_year_id' => $targetYearId,
+            'closing_batch_id' => $batchId,
+            'preview_hash' => $preview['hash'],
+        ]);
+        $this->db->transComplete();
+
+        if ($this->db->transStatus() === false) {
+            throw new RuntimeException('Unable to start school year closing.');
+        }
+    }
+
+    public function execute(int $sourceYearId, ?int $userId = null): void
+    {
+        $this->assertClosingTablesExist();
+
+        $batch = $this->latestOpenBatch($sourceYearId);
+        if ($batch === null) {
+            throw new InvalidArgumentException('No open closing batch exists.');
+        }
+
+        $preview = $this->preview($sourceYearId, (int) $batch['target_school_year_id']);
+        if ($preview['hash'] !== (string) $batch['preview_hash']) {
+            throw new InvalidArgumentException('Closing preview has changed. Refresh and restart closing before executing carry-forward.');
+        }
+
+        $this->db->transStart();
+        $items = $this->itemModel->where('closing_batch_id', (int) $batch['id'])->findAll();
+        foreach ($items as $item) {
+            if (($item['status'] ?? '') === 'completed') {
+                continue;
+            }
+
+            $this->itemModel->update((int) $item['id'], ['status' => 'completed']);
+        }
+        $this->batchModel->update((int) $batch['id'], ['status' => 'executed']);
+        $this->managementService->log($sourceYearId, SchoolYearStatus::CLOSING, SchoolYearStatus::CLOSING, 'carry_forward_execute', $userId, [
+            'closing_batch_id' => (int) $batch['id'],
+            'note' => 'Marked previewed carry-forward items complete. Target accounting records require the dedicated opening-balance schema.',
+        ]);
+        $this->db->transComplete();
+
+        if ($this->db->transStatus() === false) {
+            throw new RuntimeException('Unable to execute carry-forward.');
+        }
+    }
+
+    public function complete(int $sourceYearId, ?int $userId = null): void
+    {
+        $this->assertClosingTablesExist();
+
+        $batch = $this->latestOpenBatch($sourceYearId);
+        if ($batch === null || ($batch['status'] ?? '') !== 'executed') {
+            throw new InvalidArgumentException('Carry-forward must be executed before completing closing.');
+        }
+
+        $pending = $this->itemModel
+            ->where('closing_batch_id', (int) $batch['id'])
+            ->where('status !=', 'completed')
+            ->countAllResults();
+        if ($pending > 0) {
+            throw new InvalidArgumentException('All closing batch items must complete before the year can be closed.');
+        }
+
+        $this->db->transStart();
+        $now = date('Y-m-d H:i:s');
+        $this->batchModel->update((int) $batch['id'], [
+            'status' => 'completed',
+            'completed_by' => $userId,
+            'completed_at' => $now,
+        ]);
+        $this->schoolYearModel->update($sourceYearId, [
+            'status' => SchoolYearStatus::CLOSED,
+            'closed_at' => $now,
+            'updated_by' => $userId,
+        ]);
+        $this->managementService->log($sourceYearId, SchoolYearStatus::CLOSING, SchoolYearStatus::CLOSED, 'closing_complete', $userId, [
+            'closing_batch_id' => (int) $batch['id'],
+        ]);
+        $this->db->transComplete();
+
+        if ($this->db->transStatus() === false) {
+            throw new RuntimeException('Unable to complete closing.');
+        }
+    }
+
+    public function cancel(int $sourceYearId, ?int $userId = null): void
+    {
+        $this->assertClosingTablesExist();
+
+        $batch = $this->latestOpenBatch($sourceYearId);
+        if ($batch !== null && in_array((string) $batch['status'], ['executed', 'completed'], true)) {
+            throw new InvalidArgumentException('Closing cannot be cancelled after carry-forward has executed.');
+        }
+
+        $this->db->transStart();
+        if ($batch !== null) {
+            $this->batchModel->update((int) $batch['id'], ['status' => 'cancelled']);
+        }
+        $this->schoolYearModel->update($sourceYearId, [
+            'status' => SchoolYearStatus::ACTIVE,
+            'updated_by' => $userId,
+        ]);
+        $this->managementService->log($sourceYearId, SchoolYearStatus::CLOSING, SchoolYearStatus::ACTIVE, 'closing_cancel', $userId, [
+            'closing_batch_id' => $batch['id'] ?? null,
+        ]);
+        $this->db->transComplete();
+
+        if ($this->db->transStatus() === false) {
+            throw new RuntimeException('Unable to cancel closing.');
+        }
+    }
+
+    public function latestBatch(int $sourceYearId): ?array
+    {
+        if (! $this->db->tableExists('school_year_closing_batches')) {
+            return null;
+        }
+
+        return $this->batchModel
+            ->where('source_school_year_id', $sourceYearId)
+            ->orderBy('id', 'DESC')
+            ->first();
+    }
+
+    private function requireYear(int $id): array
+    {
+        $year = $this->schoolYearModel->find($id);
+        if ($year === null) {
+            throw new InvalidArgumentException('School year not found.');
+        }
+
+        return $year;
+    }
+
+    private function nextDraftYear(string $sourceName): ?array
+    {
+        if (! preg_match('/^(\d{4})-(\d{4})$/', $sourceName, $matches)) {
+            return null;
+        }
+
+        $nextName = $matches[2] . '-' . ((int) $matches[2] + 1);
+
+        return $this->schoolYearModel
+            ->where('name', $nextName)
+            ->first();
+    }
+
+    private function financialSummary(string $schoolYear): array
+    {
+        $summary = [
+            'invoice_count' => 0,
+            'total_invoiced' => 0.0,
+            'total_paid' => 0.0,
+            'total_outstanding' => 0.0,
+            'positive_balance' => 0.0,
+            'credit_balance' => 0.0,
+        ];
+
+        if (! $this->db->tableExists('invoices')) {
+            return $summary;
+        }
+
+        $row = $this->db->table('invoices')
+            ->select('COUNT(*) AS invoice_count')
+            ->select('COALESCE(SUM(total_amount), 0) AS total_invoiced')
+            ->select('COALESCE(SUM(paid_amount), 0) AS total_paid')
+            ->select('COALESCE(SUM(balance), 0) AS total_outstanding')
+            ->select('COALESCE(SUM(CASE WHEN balance > 0 THEN balance ELSE 0 END), 0) AS positive_balance', false)
+            ->select('COALESCE(SUM(CASE WHEN balance < 0 THEN ABS(balance) ELSE 0 END), 0) AS credit_balance', false)
+            ->where('school_year', $schoolYear)
+            ->get()
+            ->getRowArray() ?? [];
+
+        foreach ($summary as $key => $value) {
+            $summary[$key] = $key === 'invoice_count' ? (int) ($row[$key] ?? 0) : round((float) ($row[$key] ?? 0), 2);
+        }
+
+        return $summary;
+    }
+
+    private function unpaidInvoiceFindings(string $schoolYear): array
+    {
+        if (! $this->db->tableExists('invoices')) {
+            return [];
+        }
+
+        $count = $this->db->table('invoices')
+            ->where('school_year', $schoolYear)
+            ->where('balance >', 0)
+            ->countAllResults();
+
+        return $count > 0
+            ? [$this->finding('warning', 'Outstanding balances exist', "{$count} invoice(s) still have a positive balance and will require carry-forward review.")]
+            : [];
+    }
+
+    private function carryForwardFamilies(string $schoolYear): array
+    {
+        if (! $this->db->tableExists('invoices')) {
+            return [];
+        }
+
+        $rows = $this->db->table('invoices i')
+            ->select('i.parent_id AS family_id')
+            ->select('COALESCE(SUM(i.balance), 0) AS source_balance')
+            ->where('i.school_year', $schoolYear)
+            ->groupBy('i.parent_id')
+            ->having('source_balance !=', 0)
+            ->orderBy('i.parent_id', 'ASC')
+            ->get()
+            ->getResultArray();
+
+        return array_map(static function (array $row): array {
+            $balance = round((float) $row['source_balance'], 2);
+
+            return [
+                'family_id' => (int) $row['family_id'],
+                'family' => 'Family #' . (int) $row['family_id'],
+                'source_balance' => $balance,
+                'credit_amount' => $balance < 0 ? abs($balance) : 0.0,
+                'adjustment_amount' => 0.0,
+                'carry_forward_amount' => $balance,
+            ];
+        }, $rows);
+    }
+
+    private function countBySchoolYear(string $table, string $schoolYear, string $distinctField): int
+    {
+        if (! $this->db->tableExists($table) || ! $this->db->fieldExists('school_year', $table)) {
+            return 0;
+        }
+
+        $row = $this->db->table($table)
+            ->select("COUNT(DISTINCT {$distinctField}) AS total", false)
+            ->where('school_year', $schoolYear)
+            ->get()
+            ->getRowArray();
+
+        return (int) ($row['total'] ?? 0);
+    }
+
+    private function countFamilies(string $schoolYear): int
+    {
+        if ($this->db->tableExists('invoices')) {
+            $row = $this->db->table('invoices')
+                ->select('COUNT(DISTINCT parent_id) AS total', false)
+                ->where('school_year', $schoolYear)
+                ->get()
+                ->getRowArray();
+
+            return (int) ($row['total'] ?? 0);
+        }
+
+        return 0;
+    }
+
+    private function countMissingSchoolYearRows(string $table): int
+    {
+        if (! $this->db->tableExists($table) || ! $this->db->fieldExists('school_year', $table)) {
+            return 0;
+        }
+
+        return $this->db->table($table)
+            ->groupStart()
+            ->where('school_year', null)
+            ->orWhere('school_year', '')
+            ->groupEnd()
+            ->countAllResults();
+    }
+
+    private function latestOpenBatch(int $sourceYearId): ?array
+    {
+        return $this->batchModel
+            ->where('source_school_year_id', $sourceYearId)
+            ->whereIn('status', ['started', 'executed'])
+            ->orderBy('id', 'DESC')
+            ->first();
+    }
+
+    private function assertClosingTablesExist(): void
+    {
+        foreach (['school_year_closing_batches', 'school_year_closing_items', 'school_year_transition_logs'] as $table) {
+            if (! $this->db->tableExists($table)) {
+                throw new RuntimeException('School year lifecycle tables are missing. Run database migrations before closing a school year.');
+            }
+        }
+    }
+
+    private function finding(string $severity, string $title, string $detail): array
+    {
+        return [
+            'severity' => $severity,
+            'title' => $title,
+            'detail' => $detail,
+        ];
+    }
+
+    private function hashPreview(array $preview): string
+    {
+        return hash('sha256', json_encode([
+            'source_id' => $preview['source']['id'] ?? null,
+            'target_id' => $preview['target']['id'] ?? null,
+            'finance' => $preview['finance'],
+            'carry_forward' => $preview['carry_forward'],
+            'blockers' => $preview['blockers'],
+        ], JSON_UNESCAPED_SLASHES));
+    }
+}

app/Services/SchoolYearContextService.php

@@ -0,0 +1,95 @@
+<?php
+
+namespace App\Services;
+
+use App\Models\SchoolYearModel;
+use App\Support\SchoolYear\SchoolYearContext;
+use CodeIgniter\HTTP\IncomingRequest;
+use RuntimeException;
+
+final class SchoolYearContextService
+{
+    public function __construct(
+        private readonly SchoolYearModel $schoolYearModel,
+    ) {
+    }
+
+    public function resolve(
+        IncomingRequest $request,
+        ?int $routeSchoolYearId = null
+    ): SchoolYearContext {
+        $requestedId = $routeSchoolYearId
+            ?? $this->normalizeInt($request->getGet('school_year_id'));
+
+        $requestedName = trim((string) $request->getGet('school_year'));
+
+        if ($requestedId !== null && $requestedName !== '') {
+            $row = $this->schoolYearModel->find($requestedId);
+
+            if ($row === null || (string) ($row['name'] ?? '') !== $requestedName) {
+                throw new RuntimeException('Selected school-year parameters conflict.');
+            }
+
+            return $this->fromRow($row, true);
+        }
+
+        if ($requestedId !== null) {
+            $row = $this->schoolYearModel->find($requestedId);
+
+            if ($row === null) {
+                throw new RuntimeException('Selected school year was not found.');
+            }
+
+            return $this->fromRow($row, true);
+        }
+
+        if ($requestedName !== '') {
+            $row = $this->schoolYearModel
+                ->where('name', $requestedName)
+                ->first();
+
+            if ($row === null) {
+                throw new RuntimeException('Selected school year was not found.');
+            }
+
+            return $this->fromRow($row, true);
+        }
+
+        $sessionYearId = session('selected_school_year_id');
+
+        if (is_numeric($sessionYearId)) {
+            $row = $this->schoolYearModel->find((int) $sessionYearId);
+
+            if ($row !== null) {
+                return $this->fromRow($row, false);
+            }
+        }
+
+        $active = $this->schoolYearModel->active();
+
+        if ($active === null) {
+            throw new RuntimeException('No active school year is configured.');
+        }
+
+        return $this->fromRow($active, false);
+    }
+
+    private function normalizeInt(mixed $value): ?int
+    {
+        if ($value === null || $value === '' || ! ctype_digit((string) $value)) {
+            return null;
+        }
+
+        return (int) $value;
+    }
+
+    private function fromRow(array $row, bool $explicit): SchoolYearContext
+    {
+        return new SchoolYearContext(
+            id: (int) $row['id'],
+            yearName: (string) $row['name'],
+            status: (string) $row['status'],
+            explicitSelection: $explicit,
+        );
+    }
+}

app/Services/SchoolYearManagementService.php

@@ -0,0 +1,313 @@
+<?php
+
+namespace App\Services;
+
+use App\Models\ConfigurationModel;
+use App\Models\SchoolYearClosingBatchModel;
+use App\Models\SchoolYearModel;
+use App\Models\SchoolYearTransitionLogModel;
+use App\Support\SchoolYear\SchoolYearStatus;
+use CodeIgniter\Database\BaseConnection;
+use InvalidArgumentException;
+use RuntimeException;
+
+final class SchoolYearManagementService
+{
+    public function __construct(
+        private readonly SchoolYearModel $schoolYearModel,
+        private readonly ConfigurationModel $configurationModel,
+        private readonly SchoolYearTransitionLogModel $transitionLogModel,
+        private readonly SchoolYearClosingBatchModel $closingBatchModel,
+        private readonly SchoolYearValidationService $validationService,
+        private readonly BaseConnection $db,
+    ) {
+    }
+
+    public function createDraft(array $payload, ?int $userId = null): int
+    {
+        $payload = $this->metadataPayload($payload);
+        $payload['status'] = SchoolYearStatus::DRAFT;
+        $payload['created_by'] = $userId;
+        $payload['updated_by'] = $userId;
+
+        $this->validationService->validateMetadata($payload);
+
+        $this->db->transStart();
+        $id = $this->schoolYearModel->insert($payload, true);
+        if ($id !== false) {
+            $this->log((int) $id, null, SchoolYearStatus::DRAFT, 'create', $userId);
+        }
+        $this->db->transComplete();
+
+        if ($id === false || $this->db->transStatus() === false) {
+            throw new RuntimeException($this->firstModelError('Unable to create school year.'));
+        }
+
+        return (int) $id;
+    }
+
+    public function updateMetadata(int $id, array $payload, ?int $userId = null): void
+    {
+        $year = $this->requireYear($id);
+        $status = (string) $year['status'];
+
+        if (SchoolYearStatus::isReadonly($status) || $status === SchoolYearStatus::CLOSING) {
+            throw new InvalidArgumentException('This school year is read-only and cannot be edited.');
+        }
+
+        $payload = $this->metadataPayload($payload);
+        $payload['updated_by'] = $userId;
+        $this->validationService->validateMetadata($payload, $id);
+
+        $this->db->transStart();
+        $updated = $this->schoolYearModel->update($id, $payload);
+        if ($updated !== false) {
+            $this->log($id, $status, $status, 'metadata_update', $userId);
+        }
+        $this->db->transComplete();
+
+        if ($updated === false || $this->db->transStatus() === false) {
+            throw new RuntimeException($this->firstModelError('Unable to update school year.'));
+        }
+    }
+
+    public function activate(int $id, ?int $userId = null): void
+    {
+        $year = $this->requireYear($id);
+        $from = (string) $year['status'];
+
+        if (! SchoolYearStatus::canTransition($from, SchoolYearStatus::ACTIVE)) {
+            throw new InvalidArgumentException('Only draft or approved reopened school years can be activated.');
+        }
+
+        $this->db->transStart();
+        $activeYears = $this->schoolYearModel->where('status', SchoolYearStatus::ACTIVE)->findAll();
+        $now = date('Y-m-d H:i:s');
+
+        foreach ($activeYears as $activeYear) {
+            if ((int) $activeYear['id'] === $id) {
+                continue;
+            }
+
+            $this->schoolYearModel->update((int) $activeYear['id'], [
+                'status' => SchoolYearStatus::CLOSING,
+                'closing_started_at' => $now,
+                'updated_by' => $userId,
+            ]);
+            $this->log((int) $activeYear['id'], SchoolYearStatus::ACTIVE, SchoolYearStatus::CLOSING, 'activation_displaced_active_year', $userId, [
+                'activated_school_year_id' => $id,
+            ]);
+        }
+
+        $this->schoolYearModel->update($id, [
+            'status' => SchoolYearStatus::ACTIVE,
+            'activated_at' => $now,
+            'updated_by' => $userId,
+        ]);
+        $this->configurationModel->setConfigValueByKey('school_year', (string) $year['name']);
+        $this->log($id, $from, SchoolYearStatus::ACTIVE, 'activate', $userId);
+
+        $this->db->transComplete();
+
+        if ($this->db->transStatus() === false) {
+            throw new RuntimeException('Unable to activate school year.');
+        }
+    }
+
+    public function deleteDraft(int $id, ?int $userId = null): void
+    {
+        $year = $this->requireYear($id);
+        if (($year['status'] ?? '') !== SchoolYearStatus::DRAFT) {
+            throw new InvalidArgumentException('Only unused draft school years can be deleted. Archive historical years instead.');
+        }
+
+        if ($this->hasDependentRecords($id, (string) $year['name'])) {
+            throw new InvalidArgumentException('This school year cannot be deleted because related data exists. Archive historical years instead.');
+        }
+
+        $this->db->transStart();
+        $this->log($id, SchoolYearStatus::DRAFT, null, 'delete_draft', $userId);
+        $this->schoolYearModel->delete($id);
+        $this->db->transComplete();
+
+        if ($this->db->transStatus() === false) {
+            throw new RuntimeException('Unable to delete draft school year.');
+        }
+    }
+
+    public function archive(int $id, ?int $userId = null): void
+    {
+        $this->transition($id, SchoolYearStatus::ARCHIVED, 'archive', $userId, [
+            'archived_at' => date('Y-m-d H:i:s'),
+        ]);
+    }
+
+    public function reopen(int $id, string $reason, ?int $userId = null): void
+    {
+        if (trim($reason) === '') {
+            throw new InvalidArgumentException('A reopen reason is required.');
+        }
+
+        $this->transition($id, SchoolYearStatus::ACTIVE, 'reopen', $userId, [
+            'metadata' => ['reason' => trim($reason)],
+        ]);
+    }
+
+    public function latestTransitionByYear(): array
+    {
+        if (! $this->db->tableExists('school_year_transition_logs')) {
+            return [];
+        }
+
+        $rows = $this->transitionLogModel
+            ->orderBy('created_at', 'DESC')
+            ->findAll();
+        $latest = [];
+
+        foreach ($rows as $row) {
+            $yearId = (int) ($row['school_year_id'] ?? 0);
+            if ($yearId > 0 && ! isset($latest[$yearId])) {
+                $latest[$yearId] = $row;
+            }
+        }
+
+        return $latest;
+    }
+
+    public function log(int $schoolYearId, ?string $from, ?string $to, string $action, ?int $userId = null, array $metadata = []): void
+    {
+        if (! $this->db->tableExists('school_year_transition_logs')) {
+            throw new RuntimeException('School year lifecycle tables are missing. Run database migrations before changing school-year status.');
+        }
+
+        $this->transitionLogModel->insert([
+            'school_year_id' => $schoolYearId,
+            'from_status' => $from,
+            'to_status' => $to,
+            'action' => $action,
+            'performed_by' => $userId,
+            'metadata_json' => $metadata !== [] ? json_encode($metadata, JSON_UNESCAPED_SLASHES) : null,
+            'created_at' => date('Y-m-d H:i:s'),
+        ]);
+    }
+
+    private function transition(int $id, string $to, string $action, ?int $userId, array $options = []): void
+    {
+        $year = $this->requireYear($id);
+        $from = (string) $year['status'];
+
+        if (! SchoolYearStatus::canTransition($from, $to)) {
+            throw new InvalidArgumentException("Cannot transition school year from {$from} to {$to}.");
+        }
+
+        if ($to === SchoolYearStatus::ARCHIVED && ! $this->hasFinalizedClosingBatch($id)) {
+            throw new InvalidArgumentException('A school year can be archived only after a finalized closing batch exists.');
+        }
+
+        if ($to === SchoolYearStatus::ACTIVE) {
+            $otherActive = $this->schoolYearModel
+                ->where('status', SchoolYearStatus::ACTIVE)
+                ->where('id !=', $id)
+                ->first();
+            if ($otherActive !== null) {
+                throw new InvalidArgumentException('Another school year is already active. Activate or close years through the controlled lifecycle first.');
+            }
+        }
+
+        $this->db->transStart();
+        $data = [
+            'status' => $to,
+            'updated_by' => $userId,
+        ];
+
+        foreach (['archived_at', 'closed_at', 'closing_started_at'] as $field) {
+            if (isset($options[$field])) {
+                $data[$field] = $options[$field];
+            }
+        }
+
+        $this->schoolYearModel->update($id, $data);
+        $this->log($id, $from, $to, $action, $userId, $options['metadata'] ?? []);
+        $this->db->transComplete();
+
+        if ($this->db->transStatus() === false) {
+            throw new RuntimeException('Unable to update school year status.');
+        }
+    }
+
+    private function requireYear(int $id): array
+    {
+        $year = $this->schoolYearModel->find($id);
+        if ($year === null) {
+            throw new InvalidArgumentException('School year not found.');
+        }
+
+        return $year;
+    }
+
+    private function metadataPayload(array $payload): array
+    {
+        return [
+            'name' => trim((string) ($payload['name'] ?? '')),
+            'starts_on' => $this->nullableDate($payload['starts_on'] ?? null),
+            'ends_on' => $this->nullableDate($payload['ends_on'] ?? null),
+            'description' => trim((string) ($payload['description'] ?? '')) ?: null,
+            'registration_starts_on' => $this->nullableDate($payload['registration_starts_on'] ?? null),
+            'registration_ends_on' => $this->nullableDate($payload['registration_ends_on'] ?? null),
+            'previous_school_year_id' => $this->nullableInt($payload['previous_school_year_id'] ?? null),
+        ];
+    }
+
+    private function nullableDate(mixed $value): ?string
+    {
+        $value = trim((string) $value);
+
+        return $value !== '' ? $value : null;
+    }
+
+    private function nullableInt(mixed $value): ?int
+    {
+        return is_numeric($value) && (int) $value > 0 ? (int) $value : null;
+    }
+
+    private function hasDependentRecords(int $id, string $name): bool
+    {
+        if (
+            $this->db->tableExists('school_year_closing_batches')
+            && $this->closingBatchModel->where('source_school_year_id', $id)->orWhere('target_school_year_id', $id)->first() !== null
+        ) {
+            return true;
+        }
+
+        foreach (['invoices', 'payments', 'student_class', 'teacher_class', 'calendar_events', 'events'] as $table) {
+            if ($this->db->tableExists($table) && $this->db->fieldExists('school_year', $table)) {
+                $count = $this->db->table($table)->where('school_year', $name)->countAllResults();
+                if ($count > 0) {
+                    return true;
+                }
+            }
+        }
+
+        return false;
+    }
+
+    private function hasFinalizedClosingBatch(int $sourceSchoolYearId): bool
+    {
+        if (! $this->db->tableExists('school_year_closing_batches')) {
+            return false;
+        }
+
+        return $this->closingBatchModel
+            ->where('source_school_year_id', $sourceSchoolYearId)
+            ->whereIn('status', ['completed', 'closed'])
+            ->first() !== null;
+    }
+
+    private function firstModelError(string $fallback): string
+    {
+        $errors = $this->schoolYearModel->errors();
+        $first = reset($errors);
+
+        return is_string($first) && $first !== '' ? $first : $fallback;
+    }
+}

app/Services/SchoolYearValidationService.php

@@ -0,0 +1,68 @@
+<?php
+
+namespace App\Services;
+
+use App\Models\SchoolYearModel;
+use DateTimeImmutable;
+use InvalidArgumentException;
+
+final class SchoolYearValidationService
+{
+    public function __construct(
+        private readonly SchoolYearModel $schoolYearModel,
+    ) {
+    }
+
+    public function validateMetadata(array $payload, ?int $exceptId = null): void
+    {
+        $name = trim((string) ($payload['name'] ?? ''));
+
+        if (! $this->isValidYearName($name)) {
+            throw new InvalidArgumentException('School year must use YYYY-YYYY with consecutive years.');
+        }
+
+        $existing = $this->schoolYearModel->where('name', $name);
+        if ($exceptId !== null) {
+            $existing->where('id !=', $exceptId);
+        }
+        if ($existing->first() !== null) {
+            throw new InvalidArgumentException('That school year already exists.');
+        }
+
+        $startsOn = $this->dateOrNull($payload['starts_on'] ?? null);
+        $endsOn = $this->dateOrNull($payload['ends_on'] ?? null);
+        if ($startsOn !== null && $endsOn !== null && $startsOn >= $endsOn) {
+            throw new InvalidArgumentException('School year start date must be before the end date.');
+        }
+
+        $registrationStarts = $this->dateOrNull($payload['registration_starts_on'] ?? null);
+        $registrationEnds = $this->dateOrNull($payload['registration_ends_on'] ?? null);
+        if ($registrationStarts !== null && $registrationEnds !== null && $registrationStarts > $registrationEnds) {
+            throw new InvalidArgumentException('Registration start date must be on or before the registration end date.');
+        }
+    }
+
+    public function isValidYearName(string $value): bool
+    {
+        if (! preg_match('/^(\d{4})-(\d{4})$/', $value, $matches)) {
+            return false;
+        }
+
+        return (int) $matches[2] === (int) $matches[1] + 1;
+    }
+
+    private function dateOrNull(mixed $value): ?DateTimeImmutable
+    {
+        $value = trim((string) $value);
+        if ($value === '') {
+            return null;
+        }
+
+        $date = DateTimeImmutable::createFromFormat('!Y-m-d', $value);
+        if (! $date instanceof DateTimeImmutable || $date->format('Y-m-d') !== $value) {
+            throw new InvalidArgumentException('Dates must use YYYY-MM-DD.');
+        }
+
+        return $date;
+    }
+}

app/Services/SchoolYearWriteGuard.php

@@ -0,0 +1,25 @@
+<?php
+
+namespace App\Services;
+
+use App\Support\SchoolYear\SchoolYearContext;
+use RuntimeException;
+
+final class SchoolYearWriteGuard
+{
+    public function assertWritable(
+        SchoolYearContext $context,
+        bool $allowDraftForAdmin = false,
+        bool $isAdmin = false
+    ): void {
+        if ($context->status() === 'active') {
+            return;
+        }
+
+        if ($context->status() === 'draft' && $allowDraftForAdmin && $isAdmin) {
+            return;
+        }
+
+        throw new RuntimeException('The selected school year is read-only.');
+    }
+}

app/Support/SchoolYear/SchoolYearContext.php

@@ -0,0 +1,55 @@
+<?php
+
+namespace App\Support\SchoolYear;
+
+final class SchoolYearContext
+{
+    public function __construct(
+        private readonly int $id,
+        private readonly string $yearName,
+        private readonly string $status,
+        private readonly bool $explicitSelection = false,
+    ) {
+    }
+
+    public function id(): int
+    {
+        return $this->id;
+    }
+
+    public function yearName(): string
+    {
+        return $this->yearName;
+    }
+
+    public function status(): string
+    {
+        return $this->status;
+    }
+
+    public function isActive(): bool
+    {
+        return $this->status === 'active';
+    }
+
+    public function isReadonly(): bool
+    {
+        return in_array($this->status, ['closed', 'archived'], true);
+    }
+
+    public function isExplicitSelection(): bool
+    {
+        return $this->explicitSelection;
+    }
+
+    public function toArray(): array
+    {
+        return [
+            'id' => $this->id,
+            'name' => $this->yearName,
+            'status' => $this->status,
+            'readonly' => $this->isReadonly(),
+            'explicitSelection' => $this->explicitSelection,
+        ];
+    }
+}

app/Support/SchoolYear/SchoolYearStatus.php

@@ -0,0 +1,38 @@
+<?php
+
+namespace App\Support\SchoolYear;
+
+final class SchoolYearStatus
+{
+    public const DRAFT = 'draft';
+    public const ACTIVE = 'active';
+    public const CLOSING = 'closing';
+    public const CLOSED = 'closed';
+    public const ARCHIVED = 'archived';
+
+    public const ALL = [
+        self::DRAFT,
+        self::ACTIVE,
+        self::CLOSING,
+        self::CLOSED,
+        self::ARCHIVED,
+    ];
+
+    public const TRANSITIONS = [
+        self::DRAFT => [self::ACTIVE],
+        self::ACTIVE => [self::CLOSING],
+        self::CLOSING => [self::ACTIVE, self::CLOSED],
+        self::CLOSED => [self::ACTIVE, self::ARCHIVED],
+        self::ARCHIVED => [],
+    ];
+
+    public static function canTransition(string $from, string $to): bool
+    {
+        return in_array($to, self::TRANSITIONS[$from] ?? [], true);
+    }
+
+    public static function isReadonly(string $status): bool
+    {
+        return in_array($status, [self::CLOSED, self::ARCHIVED], true);
+    }
+}

app/Support/SchoolYear/SchoolYearTableRegistry.php

@@ -0,0 +1,154 @@
+<?php
+
+namespace App\Support\SchoolYear;
+
+use InvalidArgumentException;
+
+final class SchoolYearTableRegistry
+{
+    public const YEAR_SCOPED = [
+        'additional_charges',
+        'archived_paypal_transactions',
+        'attendance_data',
+        'attendance_day',
+        'attendance_record',
+        'attendance_tracking',
+        'badge_print_logs',
+        'below_sixty_decisions',
+        'calendar_events',
+        'certificate_records',
+        'classSection',
+        'class_progress_reports',
+        'competitions',
+        'current_flag',
+        'discount_vouchers',
+        'early_dismissal_signatures',
+        'enrollments',
+        'events',
+        'exams',
+        'exam_drafts',
+        'expenses',
+        'final_exam',
+        'final_score',
+        'flag',
+        'grading_locks',
+        'homework',
+        'inventory_movements',
+        'invoices',
+        'late_slip_logs',
+        'manual_payments',
+        'midterm_exam',
+        'missing_score_overrides',
+        'parent_attendance_reports',
+        'parent_meeting_schedules',
+        'parent_notifications',
+        'participation',
+        'payments',
+        'payment_transactions',
+        'placement_batches',
+        'print_requests',
+        'project',
+        'quiz',
+        'refunds',
+        'reimbursements',
+        'reimbursement_batches',
+        'report_card_acknowledgements',
+        'scan_log',
+        'score_comments',
+        'semester_scores',
+        'staff_attendance',
+        'student_class',
+        'student_decisions',
+        'teacher_attendance_data',
+        'teacher_class',
+        'teacher_submission_notification_history',
+        'whatsapp_group_links',
+        'whatsapp_group_memberships',
+    ];
+
+    public const GLOBAL = [
+        'authorized_users',
+        'cache',
+        'cache_locks',
+        'configuration',
+        'email_templates',
+        'ip_attempts',
+        'login_activity',
+        'migrations',
+        'nav_items',
+        'parent_accounts',
+        'password_reset_requests',
+        'password_resets',
+        'permissions',
+        'personal_access_tokens',
+        'preferences',
+        'role_nav_items',
+        'role_permissions',
+        'roles',
+        'school_years',
+        'sessions',
+        'settings',
+        'user_preferences',
+        'user_roles',
+        'users',
+    ];
+
+    public const IDENTITY_WITH_YEAR_RELATION = [
+        'emergency_contacts',
+        'families',
+        'family_guardians',
+        'family_students',
+        'parents',
+        'staff',
+        'student_allergies',
+        'student_medical_conditions',
+        'students',
+        'teachers',
+    ];
+
+    public const CONTEXT = [
+        'audit_logs',
+        'communication_logs',
+        'contactus',
+        'finance_notification_logs',
+        'messages',
+        'notification_recipients',
+        'notifications',
+        'payment_notification_logs',
+        'support_requests',
+        'user_notifications',
+    ];
+
+    public static function isYearScoped(string $table): bool
+    {
+        return in_array($table, self::YEAR_SCOPED, true);
+    }
+
+    public static function isGlobal(string $table): bool
+    {
+        return in_array($table, self::GLOBAL, true);
+    }
+
+    public static function isIdentityWithYearRelation(string $table): bool
+    {
+        return in_array($table, self::IDENTITY_WITH_YEAR_RELATION, true);
+    }
+
+    public static function isContext(string $table): bool
+    {
+        return in_array($table, self::CONTEXT, true);
+    }
+
+    public static function categoryOf(string $table): string
+    {
+        return match (true) {
+            self::isYearScoped($table) => 'YEAR_SCOPED',
+            self::isGlobal($table) => 'GLOBAL',
+            self::isIdentityWithYearRelation($table) => 'IDENTITY_WITH_YEAR_RELATION',
+            self::isContext($table) => 'CONTEXT',
+            default => throw new InvalidArgumentException(
+                "Table '{$table}' is not registered for school-year behavior."
+            ),
+        };
+    }
+}

app/Views/attendance/violations_notified.php

@@ -262,9 +262,9 @@ $show_actions = false; // notified page hides actions by design
 
   // Initialize Bootstrap tooltips (for "View full" note)
   document.addEventListener('DOMContentLoaded', function () {
-    if (window.bootstrap) {
+    if (window.bootstrap && bootstrap.Tooltip) {
       const tooltipTriggerList = [].slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'));
-      tooltipTriggerList.forEach(function (el) { new bootstrap.Tooltip(el); });
+      tooltipTriggerList.forEach(function (el) { bootstrap.Tooltip.getOrCreateInstance(el); });
     }
   });
 

app/Views/family/card.php

@@ -350,19 +350,20 @@ if ($returnUrl === '') {
                 <table class="table table-sm table-striped table-hover align-middle fc-table-stack">
                   <thead class="table-light">
                     <tr>
-                      <th>Invoice</th><th>Amount</th><th>Balance</th><th>Method</th><th>Date</th><th>Status</th>
+                      <th>Invoice</th><th>Amount</th><th>Invoice Balance</th><th>Method</th><th>Date</th><th>Payment Status</th><th>Invoice Status</th>
                     </tr>
                   </thead>
                   <tbody>
                     <?php $imap = $f['invoice_map'] ?? []; ?>
                     <?php foreach ($f['payments'] as $p): ?>
                       <tr>
-                        <td data-label="Invoice"><?php $iid = (int)($p['invoice_id'] ?? 0); echo esc($imap[$iid] ?? ('#'.$iid)); ?></td>
+                        <td data-label="Invoice"><?php $iid = (int)($p['invoice_id'] ?? 0); echo esc($p['invoice_number'] ?? $imap[$iid] ?? ('#'.$iid)); ?></td>
                         <td data-label="Amount">$<?= number_format((float)($p['paid_amount'] ?? 0), 2) ?></td>
-                        <td data-label="Balance">$<?= number_format((float)($p['balance'] ?? 0), 2) ?></td>
+                        <td data-label="Invoice Balance">$<?= number_format((float)($p['invoice_current_balance'] ?? 0), 2) ?></td>
                         <td data-label="Method"><?= esc($p['payment_method'] ?? '') ?></td>
                         <td data-label="Date"><?= esc(!empty($p['payment_date']) ? local_date($p['payment_date'], 'm-d-Y') : '') ?></td>
-                        <td data-label="Status"><?= esc($p['status'] ?? '') ?></td>
+                        <td data-label="Payment Status"><?= esc($p['payment_status'] ?? '') ?></td>
+                        <td data-label="Invoice Status"><?= esc($p['invoice_status'] ?? '') ?></td>
                       </tr>
                     <?php endforeach; ?>
                   </tbody>

app/Views/family/index.php

@@ -263,19 +263,20 @@
                     <table class="table table-sm table-striped table-hover align-middle">
                       <thead class="table-light">
                         <tr>
-                          <th>Invoice</th><th>Amount</th><th>Balance</th><th>Method</th><th>Date</th><th>Status</th>
+                          <th>Invoice</th><th>Amount</th><th>Invoice Balance</th><th>Method</th><th>Date</th><th>Payment Status</th><th>Invoice Status</th>
                         </tr>
                       </thead>
                       <tbody>
                         <?php $imap = $f['invoice_map'] ?? []; ?>
                         <?php foreach ($f['payments'] as $p): ?>
                           <tr>
-                            <td><?php $iid = (int)($p['invoice_id'] ?? 0); echo esc($imap[$iid] ?? ('#'.$iid)); ?></td>
+                            <td><?php $iid = (int)($p['invoice_id'] ?? 0); echo esc($p['invoice_number'] ?? $imap[$iid] ?? ('#'.$iid)); ?></td>
                             <td>$<?= number_format((float)($p['paid_amount'] ?? 0), 2) ?></td>
-                            <td>$<?= number_format((float)($p['balance'] ?? 0), 2) ?></td>
+                            <td>$<?= number_format((float)($p['invoice_current_balance'] ?? 0), 2) ?></td>
                             <td><?= esc($p['payment_method'] ?? '') ?></td>
                             <td><?= esc(!empty($p['payment_date']) ? local_date($p['payment_date'], 'm-d-Y') : '') ?></td>
-                            <td><?= esc($p['status'] ?? '') ?></td>
+                            <td><?= esc($p['payment_status'] ?? '') ?></td>
+                            <td><?= esc($p['invoice_status'] ?? '') ?></td>
                           </tr>
                         <?php endforeach; ?>
                       </tbody>
@@ -304,6 +305,23 @@
 
     const normalize = s => (s||'').toString().toLowerCase();
     const debounce = (fn, wait=200) => { let t; return (...args)=>{ clearTimeout(t); t=setTimeout(()=>fn(...args), wait); }; };
+    const openFamilyResult = (type, id) => {
+      const params = {};
+      if (type === 'student') {
+        params.student_id = id;
+      } else if (type === 'guardian') {
+        params.guardian_id = id;
+      }
+      if (Object.keys(params).length && typeof window.openFamilyCard === 'function') {
+        window.openFamilyCard(params);
+        return;
+      }
+      if (type === 'student') {
+        window.location.href = '<?= site_url('family') ?>' + '?student_id=' + encodeURIComponent(id);
+      } else if (type === 'guardian') {
+        window.location.href = '<?= site_url('family') ?>' + '?guardian_id=' + encodeURIComponent(id);
+      }
+    };
 
     // Preloaded datasets for suggestions (students + guardians)
     const STATIC_ITEMS = [
@@ -367,12 +385,7 @@
           btn.className = 'list-group-item list-group-item-action';
           btn.innerHTML = `<div class="d-flex justify-content-between"><span>${(it.label||'').replace(/</g,'&lt;')}</span><span class="text-muted small">${(it.sub||'').replace(/</g,'&lt;')}</span></div>`;
           btn.addEventListener('click', () => {
-            // Redirect to show only the selected family
+            openFamilyResult(it.type, it.id);
-            if (it.type === 'student') {
-              window.location.href = '<?= site_url('family') ?>' + '?student_id=' + encodeURIComponent(it.id);
-            } else if (it.type === 'guardian') {
-              window.location.href = '<?= site_url('family') ?>' + '?guardian_id=' + encodeURIComponent(it.id);
-            }
           });
           suggest.appendChild(btn);
         });
@@ -454,7 +467,7 @@
       }
       if (best) {
         e.preventDefault();
-        window.location.href = '<?= site_url('family') ?>' + '?student_id=' + encodeURIComponent(best.value);
+        openFamilyResult('student', best.value);
       }
     });
     // Hide suggestions on outside click

app/Views/layout/main.php

@@ -41,8 +41,6 @@
 
     <!-- Bootstrap Bundle JS (for modal, tooltips, etc.) -->
     <script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/js/bootstrap.bundle.min.js"></script>
-    <!-- Optional: Your other custom scripts -->
-    <script src="<?= base_url('assets/js/bootstrap.min.js') ?>"></script>
     <!-- ✅ Your validation logic (depends on jQuery & Select2) -->
     <script type="module" src="/public/assets/js/validate_student.js"></script>
     <script src="<?= base_url('assets/js/modal_validation.js') ?>"></script>

app/Views/layout/main_layout.php

@@ -153,6 +153,13 @@
 /* Prevent horizontal scrollbars from stray overflows */
 html, body { overflow-x: hidden; }
 
+#familyCardModal .family-card-modal-dialog {
+    margin-top: 1rem;
+}
+#familyCardModal .modal-content {
+    max-height: calc(100vh - 2rem);
+}
+
 </style>
 </head>
 
@@ -237,7 +244,7 @@ html, body { overflow-x: hidden; }
 
     <!-- Family Card Modal -->
     <div class="modal fade" id="familyCardModal" tabindex="-1" aria-hidden="true">
-      <div class="modal-dialog modal-xl modal-dialog-scrollable">
+      <div class="modal-dialog modal-xl modal-dialog-scrollable family-card-modal-dialog">
         <div class="modal-content">
           <div class="modal-header">
             <h5 class="modal-title">Family</h5>
@@ -250,18 +257,19 @@ html, body { overflow-x: hidden; }
       </div>
     </div>
 
+    <!-- Bootstrap Bundle (with Popper) -->
+    <script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/js/bootstrap.bundle.min.js"></script>
+
     <script>
         document.addEventListener('DOMContentLoaded', function() {
+            if (!window.bootstrap || !bootstrap.Tooltip) return;
             const tooltipTriggerList = [].slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'));
             tooltipTriggerList.forEach(function(tooltipTriggerEl) {
-                new bootstrap.Tooltip(tooltipTriggerEl);
+                bootstrap.Tooltip.getOrCreateInstance(tooltipTriggerEl);
             });
         });
     </script>
 
-    <!-- Bootstrap Bundle (with Popper) -->
-    <script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/js/bootstrap.bundle.min.js"></script>
-
     <!-- Additional JS Libraries -->
     <script src="https://cdn.jsdelivr.net/npm/jquery@3.6.0/dist/jquery.min.js"></script>
     <script src="<?= base_url('lib/wow/wow.min.js') ?>"></script>
@@ -311,33 +319,60 @@ html, body { overflow-x: hidden; }
         }
         window.openFamilyCard = loadFamilyCard;
 
-        document.addEventListener('click', function(e){
+        function familyParamsFromTarget(target) {
-          const link = e.target.closest('[data-family-student-id], [data-family-guardian-id], [data-family-id]');
+          const link = target.closest('[data-family-student-id], [data-family-guardian-id], [data-family-id], a[href]');
-          if (!link) return;
+          if (!link) return null;
-          e.preventDefault();
+          let sid = link.getAttribute('data-family-student-id');
-          const sid = link.getAttribute('data-family-student-id');
+          let gid = link.getAttribute('data-family-guardian-id');
-          const gid = link.getAttribute('data-family-guardian-id');
+          let fid = link.getAttribute('data-family-id');
-          const fid = link.getAttribute('data-family-id');
+          if (!sid && !gid && !fid && link.matches('a[href]')) {
+            try {
+              const url = new URL(link.getAttribute('href'), window.location.origin);
+              const familyPath = new URL('<?= site_url('family') ?>', window.location.origin).pathname.replace(/\/+$/, '');
+              const linkPath = url.pathname.replace(/\/+$/, '');
+              if (url.origin !== window.location.origin || linkPath !== familyPath) return null;
+              sid = url.searchParams.get('student_id');
+              gid = url.searchParams.get('guardian_id');
+              fid = url.searchParams.get('family_id');
+            } catch (_) {
+              return null;
+            }
+          }
           const params = {};
           if (fid) params.family_id = fid;
           else if (sid) params.student_id = sid;
           else if (gid) params.guardian_id = gid;
-          if (Object.keys(params).length) loadFamilyCard(params);
+          return Object.keys(params).length ? params : null;
-        });
+        }
 
-        document.addEventListener('click', function(e){
+        function handleFamilyCardClick(e) {
-          const link = e.target.closest('[data-family-student-id], [data-family-guardian-id], [data-family-id]');
+          const params = familyParamsFromTarget(e.target);
-          if (!link) return;
+          if (!params) return;
+          if (e.familyCardHandled) return;
+          e.familyCardHandled = true;
           e.preventDefault();
-          const sid = link.getAttribute('data-family-student-id');
+          loadFamilyCard(params);
-          const gid = link.getAttribute('data-family-guardian-id');
+        }
-          const fid = link.getAttribute('data-family-id');
+
+        document.addEventListener('click', handleFamilyCardClick);
+        document.addEventListener('click', handleFamilyCardClick, true);
+
+        document.addEventListener('DOMContentLoaded', function(){
+          try {
+            const current = new URL(window.location.href);
+            const familyPath = new URL('<?= site_url('family') ?>', window.location.origin).pathname.replace(/\/+$/, '');
+            const currentPath = current.pathname.replace(/\/+$/, '');
+            if (currentPath !== familyPath) return;
+            const sid = current.searchParams.get('student_id');
+            const gid = current.searchParams.get('guardian_id');
+            const fid = current.searchParams.get('family_id');
             const params = {};
             if (fid) params.family_id = fid;
             else if (sid) params.student_id = sid;
             else if (gid) params.guardian_id = gid;
             if (Object.keys(params).length) loadFamilyCard(params);
-        }, true);
+          } catch (_) {}
+        });
       })();
     </script>
 

app/Views/layout/management_layout.php

@@ -274,6 +274,12 @@
       }
       .card-header { background-color: #f0f7ff; }
       .form-text { color: var(--mgmt-muted); }
+      #familyCardModal .family-card-modal-dialog {
+        margin-top: 1rem;
+      }
+      #familyCardModal .modal-content {
+        max-height: calc(100vh - 2rem);
+      }
     </style>
     <?= $this->renderSection('styles') ?>
 </head>
@@ -295,7 +301,7 @@
 
     <!-- Family Card Modal -->
     <div class="modal fade" id="familyCardModal" tabindex="-1" aria-hidden="true">
-      <div class="modal-dialog modal-xl modal-dialog-scrollable">
+      <div class="modal-dialog modal-xl modal-dialog-scrollable family-card-modal-dialog">
         <div class="modal-content">
           <div class="modal-header">
             <h5 class="modal-title">Family</h5>
@@ -594,36 +600,63 @@
         // Expose for programmatic usage
         window.openFamilyCard = loadFamilyCard;
 
-        // Bubble-phase handler (general)
+        function familyParamsFromTarget(target) {
-        document.addEventListener('click', function(e){
+          const link = target.closest('[data-family-student-id], [data-family-guardian-id], [data-family-id], a[href]');
-          const link = e.target.closest('[data-family-student-id], [data-family-guardian-id], [data-family-id]');
+          if (!link) return null;
-          if (!link) return;
+          let sid = link.getAttribute('data-family-student-id');
-          e.preventDefault();
+          let gid = link.getAttribute('data-family-guardian-id');
-          const sid = link.getAttribute('data-family-student-id');
+          let fid = link.getAttribute('data-family-id');
-          const gid = link.getAttribute('data-family-guardian-id');
+          if (!sid && !gid && !fid && link.matches('a[href]')) {
-          const fid = link.getAttribute('data-family-id');
+            try {
+              const url = new URL(link.getAttribute('href'), window.location.origin);
+              const familyPath = new URL('<?= site_url('family') ?>', window.location.origin).pathname.replace(/\/+$/, '');
+              const linkPath = url.pathname.replace(/\/+$/, '');
+              if (url.origin !== window.location.origin || linkPath !== familyPath) return null;
+              sid = url.searchParams.get('student_id');
+              gid = url.searchParams.get('guardian_id');
+              fid = url.searchParams.get('family_id');
+            } catch (_) {
+              return null;
+            }
+          }
           const params = {};
           if (fid) params.family_id = fid;
           else if (sid) params.student_id = sid;
           else if (gid) params.guardian_id = gid;
-          if (Object.keys(params).length) loadFamilyCard(params);
+          return Object.keys(params).length ? params : null;
-        });
+        }
+
+        function handleFamilyCardClick(e) {
+          const params = familyParamsFromTarget(e.target);
+          if (!params) return;
+          if (e.familyCardHandled) return;
+          e.familyCardHandled = true;
+          e.preventDefault();
+          loadFamilyCard(params);
+        }
+
+        // Bubble-phase handler (general)
+        document.addEventListener('click', handleFamilyCardClick);
 
         // Capture-phase safety net so components that stop propagation (e.g., DataTables) don't block us
-        document.addEventListener('click', function(e){
+        document.addEventListener('click', handleFamilyCardClick, true);
-          const link = e.target.closest('[data-family-student-id], [data-family-guardian-id], [data-family-id]');
+
-          if (!link) return;
+        document.addEventListener('DOMContentLoaded', function(){
-          // If already prevented by other handler, still proceed
+          try {
-          e.preventDefault();
+            const current = new URL(window.location.href);
-          const sid = link.getAttribute('data-family-student-id');
+            const familyPath = new URL('<?= site_url('family') ?>', window.location.origin).pathname.replace(/\/+$/, '');
-          const gid = link.getAttribute('data-family-guardian-id');
+            const currentPath = current.pathname.replace(/\/+$/, '');
-          const fid = link.getAttribute('data-family-id');
+            if (currentPath !== familyPath) return;
+            const sid = current.searchParams.get('student_id');
+            const gid = current.searchParams.get('guardian_id');
+            const fid = current.searchParams.get('family_id');
             const params = {};
             if (fid) params.family_id = fid;
             else if (sid) params.student_id = sid;
             else if (gid) params.guardian_id = gid;
             if (Object.keys(params).length) loadFamilyCard(params);
-        }, true);
+          } catch (_) {}
+        });
       })();
     </script>
 

app/Views/layout/register_layout.php

@@ -36,7 +36,6 @@
     <script src="https://cdnjs.cloudflare.com/ajax/libs/OwlCarousel2/2.3.4/owl.carousel.min.js"></script>
 
     <!-- ✅ Custom JS -->
-    <script src="<?= base_url('assets/js/bootstrap.min.js') ?>"></script>    
     <script src="<?= base_url('assets/js/modal_validation.js') ?>"></script>
     <script src="<?= base_url('assets/js/validation.js') ?>" defer></script>
 

app/Views/parent/add_second_parent.php

@@ -118,11 +118,6 @@
 </div>
 
 <br>
-<?php include(__DIR__ . '/../partials/footer.php'); ?>
-
-<!-- Bootstrap JS -->
-<script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.0-alpha1/dist/js/bootstrap.bundle.min.js"></script>
-
 <!-- JavaScript for Phone and Email Validation -->
 <script>
     document.addEventListener('DOMContentLoaded', function() {

app/Views/payment/_list_modal.php

@@ -21,15 +21,17 @@
             <th>Date</th>
             <th>Invoice</th>
             <th class="text-end">Paid</th>
-            <th class="text-end">Balance</th>
+            <th class="text-end">Invoice Balance</th>
             <th>Method</th>
-            <th>Status</th>
+            <th>Payment Status</th>
+            <th>Invoice Status</th>
+            <th>School Year</th>
           </tr>
         </thead>
         <tbody>
           <?php if (empty($payments)): ?>
             <tr>
-              <td colspan="6" class="text-center text-muted py-4">No payment</td>
+              <td colspan="8" class="text-center text-muted py-4">No payment</td>
             </tr>
           <?php else: ?>
             <?php foreach ($payments as $p): ?>
@@ -40,9 +42,11 @@
                   <?= $inv !== '' ? esc($inv) : ('#' . (int)($p['invoice_id'] ?? 0)) ?>
                 </td>
                 <td class="text-end">$<?= number_format((float)($p['paid_amount'] ?? 0), 2) ?></td>
-                <td class="text-end">$<?= number_format((float)($p['balance'] ?? 0), 2) ?></td>
+                <td class="text-end">$<?= number_format((float)($p['invoice_current_balance'] ?? 0), 2) ?></td>
                 <td><?= esc($p['payment_method'] ?? '') ?></td>
-                <td><?= esc($p['status'] ?? '') ?></td>
+                <td><?= esc($p['payment_status'] ?? '') ?></td>
+                <td><?= esc($p['invoice_status'] ?? '') ?></td>
+                <td><?= esc($p['school_year'] ?? '') ?></td>
               </tr>
             <?php endforeach; ?>
           <?php endif; ?>

app/Views/payment/manual_pay.php

@@ -219,11 +219,12 @@
         <tr>
           <th>Paid Date</th>
           <th>Paid Amount</th>
-          <th>Balance</th>
+          <th>Invoice Balance</th>
           <th>Payment Method</th>
           <th>Check Number</th>
           <th>Installments</th>
-          <th>Status</th>
+          <th>Payment Status</th>
+          <th>Invoice Status</th>
           <th>Check/Card File</th>
           <th>Action</th>
         </tr>
@@ -242,7 +243,7 @@
             ?>
             <td><?= esc($paidAt) ?></td>
             <td>$<?= number_format((float)($payment['paid_amount'] ?? 0), 2) ?></td>
-            <td>$<?= number_format((float)($payment['balance'] ?? 0), 2) ?></td>
+            <td>$<?= number_format((float)($payment['invoice_current_balance'] ?? 0), 2) ?></td>
             <td>
               <?php if ($method === 'cash'): ?>
                 <span class="badge" style="background-color:#28a745;color:#fff;">Cash</span>
@@ -255,8 +256,9 @@
               <?php endif; ?>
             </td>
             <td><?= !empty($payment['check_number']) ? esc($payment['check_number']) : '-' ?></td>
-            <td><?= esc($payment['number_of_installments'] ?? '') ?></td>
+            <td><?= esc($payment['installment_seq'] ?? $payment['number_of_installments'] ?? '') ?></td>
-            <td><?= esc($payment['status'] ?? '') ?></td>
+            <td><?= esc($payment['payment_status'] ?? '') ?></td>
+            <td><?= esc($payment['invoice_status'] ?? '') ?></td>
             <td>
               <?php if (!empty($payment['check_file'])): ?>
                 <a href="#" data-bs-toggle="modal" data-bs-target="#checkModal<?= (int)$payment['id'] ?>">View</a> /

app/Views/payment_list.php

@@ -19,25 +19,29 @@
               <th>Date</th>
               <th>Invoice #</th>
               <th class="text-end">Paid Amount</th>
-              <th class="text-end">Balance</th>
+              <th class="text-end">Invoice Balance</th>
               <th>Method</th>
-              <th>Status</th>
+              <th>Payment Status</th>
+              <th>Invoice Status</th>
+              <th>School Year</th>
             </tr>
           </thead>
           <tbody>
           <?php if (empty($payments)): ?>
             <tr>
-              <td colspan="6" class="text-center text-muted py-4">No payment</td>
+              <td colspan="8" class="text-center text-muted py-4">No payment</td>
             </tr>
           <?php else: ?>
             <?php foreach ($payments as $p): ?>
               <tr>
                 <td><?= esc(!empty($p['payment_date']) ? local_date($p['payment_date'], 'm-d-Y') : '') ?></td>
-                <td><?= esc($p['invoice_id'] ?? '') ?></td>
+                <td><?= esc($p['invoice_number'] ?? ('#' . (int)($p['invoice_id'] ?? 0))) ?></td>
                 <td class="text-end">$<?= number_format((float)($p['paid_amount'] ?? 0), 2) ?></td>
-                <td class="text-end">$<?= number_format((float)($p['balance'] ?? 0), 2) ?></td>
+                <td class="text-end">$<?= number_format((float)($p['invoice_current_balance'] ?? 0), 2) ?></td>
                 <td><?= esc($p['payment_method'] ?? '') ?></td>
-                <td><?= esc($p['status'] ?? '') ?></td>
+                <td><?= esc($p['payment_status'] ?? '') ?></td>
+                <td><?= esc($p['invoice_status'] ?? '') ?></td>
+                <td><?= esc($p['school_year'] ?? '') ?></td>
               </tr>
             <?php endforeach; ?>
           <?php endif; ?>