Skip to content

Update plugin for CakePHP 6 and PHP 8.5 - #42

Merged
ADmad merged 4 commits into
ADmad:cake6from
josbeir:cakephp-6-compatibility
Oct 31, 2026
Merged

ADmad merged 4 commits into
ADmad:cake6from
josbeir:cakephp-6-compatibility

Conversation

@josbeir

@josbeir josbeir commented Oct 30, 2026 •

Copy link
Copy Markdown

This updates the plugin for CakePHP 6 on PHP 8.5, dropping CakePHP 5 support. The PR targets the cake6 branch.

  • Require CakePHP ^6.0, PHP >=8.5, and PHPUnit ^13.0. Allow development dependencies with stable versions preferred while CakePHP 6 and Chronos 4 are unreleased.
  • Rename configuration properties and protected members to follow CakePHP 6 conventions. The configuration rename fixes middleware initialization and missing helper defaults.
  • Update failure-event examples and tests to use $event->setResult($response), and restore parent test setup for state isolation.
  • Run CI on PHP 8.5 with Glide 3 and 4, CakePHP 6 coding standards, PHPStan, and Psalm. Document the requirements and subclass migration.

Validation: PHP 8.5.12, CakePHP 6.x-dev at d5d303d, PHPUnit 13.4.5. All 19 tests / 42 assertions pass with both Glide 3.2.0 and 4.2.0 using --display-all-issues --fail-on-all-issues. PHPCS, PHPStan level 8, Psalm, composer validate --strict, and git diff --check pass.

Upgrade-tool validation: ran the local cakephp/upgrade checkout at 7a0d340 with Rector 2.6.7 and the cakephp60 ruleset in dry-run mode over src and tests. No further changes were suggested on this branch. Also checked the original source with CakePHP 5.4.3 dependencies: the tool suggested the helper default-configuration property rename, already included here.

@josbeir
josbeir changed the base branch from master to cake6 October 30, 2026 13:18
Comment thread .github/workflows/ci.yml Resolved
@ADmad
ADmad merged commit 8a4f614 into ADmad:cake6 Oct 31, 2026
3 checks passed
@josbeir
josbeir deleted the cakephp-6-compatibility branch October 31, 2026 06:09
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