Skip to content

Migrate to CodeIgniter 4 and PHP 8.5 - #80

Merged
Addono merged 3 commits into
Addono:masterfrom
omit01:upgrade/codeigniter4-php85
Oct 21, 2026
Merged

Addono merged 3 commits into
Addono:masterfrom
omit01:upgrade/codeigniter4-php85

Conversation

@omit01

@omit01 omit01 commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Migrate the CI3 application to Composer-managed CodeIgniter 4.7.5 and PHP 8.5.
  • Move application code to app/ and the document root to public/; replace legacy loader, database, session, validation, CSRF, view, and email APIs with native CI4 services.
  • Preserve historical model-version keys, table naming, password hashes, transaction workflows, and the existing inactive-user filtering from the fork.
  • Update PHP-FPM/Nginx containers, configuration examples, migration documentation, CI, and regression coverage.

Upstream master was fetched and checked before publication. This includes all 38 upstream commits missing from the fork, through 477d204; no newer master commits were found beyond the already pending merge.

Validation

  • docker compose exec -T php-fpm composer test: passed on PHP 8.5.11, using an isolated in-memory database.
  • npx cypress run --record false: all 8 tests passed against the local MariaDB-backed Docker stack, covering installer rerun, login validation, administrator views, CSRF, API access, user creation, consumption transactions, and leaderboard rendering.
  • git diff --cached --check: passed before committing.

Deployment Caveats

This is a staging-ready application conversion, not certification of a production-database upgrade.

  • Back up and verify restoration, rehearse against a production database copy, and compare users, roles, balances, transactions, and migration versions. Schema changes must not be assumed transactionally reversible.
  • Configure explicit persistent database storage and replace development credentials before production. The Compose database remains MariaDB 10.4.
  • Restrict /Install to operators in production. Existing sessions are replaced; users must log in again.
  • Production SMTP delivery, concurrent transaction workloads, and a real production-database upgrade have not been verified.
  • Existing frontend dependency advisories remain outside the scope of this migration.

Comment thread README.md

```bash
# Clone the repository
git clone https://github.com/Addono/lisa && cd lisa

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's some things in here which rewrite the origin repo to your fork, doesn't make sense to merge it like this :)

There's something similar happening in the composer.json file.

@Addono
Addono merged commit 475c064 into Addono:master Oct 21, 2026
1 check passed
@cypress

cypress Bot commented Oct 21, 2026

Copy link
Copy Markdown

LISA    Run #77

Run Properties:  status check passed Passed #77  •  git commit 475c064b1e: feat: migrate to CodeIgniter 4 and PHP 8.5 (#80)
Project LISA
Branch Review master
Run status status check passed Passed #77
Run duration 00m 15s
Commit git commit 475c064b1e: feat: migrate to CodeIgniter 4 and PHP 8.5 (#80)
Committer Timo
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 8
View all changes introduced in this branch ↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants