Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Changing primary key breaks getKey() #8

Closed
leekaiwei opened this issue Jul 26, 2013 · 2 comments
Closed

Changing primary key breaks getKey() #8

leekaiwei opened this issue Jul 26, 2013 · 2 comments

Comments

@leekaiwei
Copy link
Contributor

If I extend MongoDB Eloquent and set my own $primaryKey = 'myId', calling $this->getKey() returns null. However, I can access the key using $this->_id and not $this->myId. I think that the original primary key for some reason is not being re-declared in the extended class.
jenssegers added a commit that referenced this issue Jul 26, 2013
@jenssegers
Fixed issue #8
955b214
@jenssegers
Copy link
Contributor

Can you check if it is working with this last fix?

@leekaiwei
Copy link
Contributor Author

Yes it is working. Thanks very much!

@igormatkovic igormatkovic mentioned this issue Feb 14, 2015
Closed
@robsoncombr robsoncombr mentioned this issue May 8, 2015
Closed
@fuzeboxsoftware fuzeboxsoftware mentioned this issue Sep 29, 2015
Closed
@wayne5w wayne5w mentioned this issue Mar 22, 2016
Closed
Closed
@yuchao86 yuchao86 mentioned this issue Nov 7, 2016
Closed
This was referenced Nov 26, 2018
Closed
Closed
@irajtaghlidi irajtaghlidi mentioned this issue Dec 9, 2018
Closed
@phillmorgan28 phillmorgan28 mentioned this issue Aug 17, 2019
Closed
@paouriel paouriel mentioned this issue Feb 11, 2020
Closed
@wohcnosae wohcnosae mentioned this issue Dec 9, 2020
Closed
@slia-stryber slia-stryber mentioned this issue Feb 5, 2021
Closed
@KolyaSlisarenko KolyaSlisarenko mentioned this issue Apr 8, 2021
Closed
@georak georak mentioned this issue May 26, 2021
Open
@farmani farmani mentioned this issue Aug 28, 2021
Closed
@EfronC EfronC mentioned this issue Sep 20, 2021
Open
@jinrenjie jinrenjie mentioned this issue Dec 27, 2021
Closed
@elihaidv elihaidv mentioned this issue Jul 10, 2022
Closed
@dave2309 dave2309 mentioned this issue Sep 15, 2022
Closed
alcaeus pushed a commit to alcaeus/laravel-mongodb that referenced this issue Aug 16, 2023
@GromNaN
Create UTCDateTime from DateTimeInterface objects (mongodb#8)
843904b
alcaeus pushed a commit that referenced this issue Aug 22, 2023
@GromNaN @alcaeus
Create UTCDateTime from DateTimeInterface objects (#8)
933073d
@GromNaN GromNaN mentioned this issue Aug 21, 2024
Merged
3 tasks
mnphpexpert added a commit to mnphpexpert/laravel-mongodb that referenced this issue Sep 2, 2024
@mnphpexpert
Fixed issue mongodb#8
48fe062
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jenssegers @leekaiwei