Fixed aliasing of join conditions when passing a 'class' param #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* It provides the following new features: * -Plays nice with Containable which means that you can force INNER JOINS for hasOne/belongsTo and at the same time do a query on a hasMany/HABTM relationship. * * -The original code required the relationship to be established from the target to the source. * (e.g. if you are linking Post => User then User would have to define a hasOne Post relationship. * However, this proves problematic when doing on-the-fly binds as you would have to bind on more than just the model you are querying from) *
…rror in CakePHP 1.3 (RC2)
…l's fields when none given in the find options
…iling white spaces
…he best syntax to do these kind of joins
…ip (hasMany, hasOne) is aliased and the alias is related using a belongsTo, the aliased relationship is not treated as belongsTo and additional records may be returned
…uses default conection for tests
|
Pinging @giulianob to get everyone upstream to update |
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Lots of fixes, upgrades, updates and other crap. I'm hoping everyone else in the chain will see this and update their references too.