Skip to content

Fix ambiguous column for "where conditions" in hybrid relations with a "join" #1840

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

Closed
wants to merge 4 commits into from

Conversation

rdehaas-comvex
Copy link

@rdehaas-comvex rdehaas-comvex commented Oct 4, 2019

Description

Resolve keyname with a prefix of the table name for models which are not of Jenssegers\Mongodb\Eloquent\Model.

Fix #1876

@Smolevich
Copy link
Contributor

@rdehaas-comvex can you add test case for this changes?

@anderson-comvex
Copy link

anderson-comvex commented Oct 21, 2019

@rdehaas-comvex can you add test case for this changes?

@Smolevich getRelatedConstraintKey() is a protected method, the public method is already being tested on the HybridRelationsTest.php and RelationsTest.php, or am i mistaken?

@rdehaas-comvex
Copy link
Author

ping @jenssegers

@stephandesouza
Copy link
Contributor

The HasOneOrMany part will conflict when merging #1837

@Smolevich Smolevich added the Needs investigation Need investigation about bugs described in issue label Jan 20, 2020
@divine
Copy link
Contributor

divine commented Feb 23, 2020

Sorry to disturb you @rdehaas-comvex but it has been almost 5 months.

Please let us know when your PR will be ready with resolved conflicting file and tests of course, closing for now.

@divine divine closed this Feb 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs investigation Need investigation about bugs described in issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ambiguous column for "whereHas conditions" in hybrid relations with a "join"
5 participants