Skip to content

fix: find() now uses getKeyName() for primary key #1392

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Dec 22, 2017
Merged

fix: find() now uses getKeyName() for primary key #1392

merged 1 commit into from
Dec 22, 2017

Conversation

StevePorter92
Copy link
Contributor

fixes #1354

@StevePorter92 StevePorter92 mentioned this pull request Dec 19, 2017
Closed
@jenssegers jenssegers merged commit 5d55933 into mongodb:master Dec 22, 2017
@danjohnson95
Copy link
Contributor

🎉

@StevePorter92 StevePorter92 deleted the 1354 branch September 9, 2018 07:32
@paouriel paouriel mentioned this pull request Feb 11, 2020
Closed
mnphpexpert added a commit to mnphpexpert/laravel-mongodb that referenced this pull request Sep 2, 2024
@mnphpexpert
Merge pull request mongodb#1392 from DMNSteve/1354
3238ec3 
fix: find() now uses getKeyName() for primary key
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Overriding $primaryKey does not work with find($id)
3 participants
@StevePorter92 @danjohnson95 @jenssegers