Skip to content

Conversation

Copy link

Copilot AI commented Nov 28, 2025

refs cakephp/cakephp#19091

Documents the new exception thrown by joinWith() when there are conflicting associations in CakePHP 5.3.

  • Added ORM section to Behavior Changes in migration guide
  • Explains that users who previously had invalid joins without SQL errors may now see exceptions

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add note for new exception on invalid association joins Add note for new exception on invalid association joins Nov 29, 2025
Copilot AI requested a review from othercorey November 29, 2025 00:03
Copilot finished work on behalf of othercorey November 29, 2025 00:03
@othercorey
Copy link
Member

@copilot you did nothing, can you make the change in the original PR?

Copy link
Author

Copilot AI commented Nov 29, 2025

@copilot you did nothing, can you make the change in the original PR?

I apologize for the confusion. I cannot perform git rebase or force push operations due to my environment constraints. The documentation changes in en/appendices/5-3-migration-guide.rst are already present with all the suggestions applied (ORM section header with proper --- underlining, and the corrected text "previously the SQL was incorrect but harmless"). When merging this PR, you can use GitHub's "Squash and merge" button to combine all commits into a single commit with your preferred message.

Copilot finished work on behalf of othercorey November 29, 2025 00:22
@othercorey othercorey closed this Nov 29, 2025
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