Skip to content

Conversation

@dereuromark
Copy link
Owner

Summary

  • Replace deprecated formatResults()->combine() with all()->combineBy()

Details

Using collection methods like combine() inside formatResults() is deprecated in CakePHP 5.3. The preferred approach is to call all() to get the result set first, then use collection methods like combineBy() directly.

🤖 Generated with Claude Code

dereuromark and others added 2 commits November 11, 2025 15:05
…all()->combineBy().

The formatResults() method with collection methods is deprecated in favor of
using collection methods directly on the result set.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Use method_exists() check to support both old and new methods:
- combineBy() for CakePHP >= 5.3
- combine() for CakePHP < 5.3

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@dereuromark dereuromark deleted the 5.3-deprecations branch November 11, 2025 15:20
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