-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix quoting when using inheritance as well #6416
Conversation
* @param string $type | ||
* | ||
* @return string | ||
*/ | ||
protected function getSelectJoinColumnSQL($tableAlias, $joinColumnName, $type) | ||
protected function getSelectJoinColumnSQL($tableAlias, $joinColumnName, $quotedColumnName, $type) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Theoretically this is as minor BC break, should we add to UPGRADING.md
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a valid BC break since we do not allow custom persisters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, change is fine here
In order to keep that information for inheritance mapping.
c869347
to
0aef63b
Compare
|
@guilhermeblanco just a note: rebasing on this will likely be annoying. Still, the fix is valid IMO, and it shall be merged in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
* @param string $type | ||
* | ||
* @return string | ||
*/ | ||
protected function getSelectJoinColumnSQL($tableAlias, $joinColumnName, $type) | ||
protected function getSelectJoinColumnSQL($tableAlias, $joinColumnName, $quotedColumnName, $type) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, change is fine here
Related to #6402 and #6404