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.

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

Investigate new mongo extension #403

Closed
rexxars opened this issue Nov 15, 2015 · 4 comments · Fixed by #436
Closed

Investigate new mongo extension #403

rexxars opened this issue Nov 15, 2015 · 4 comments · Fixed by #436
Assignees
Milestone

Comments

@rexxars
Copy link
Member

rexxars commented Nov 15, 2015

There seems to be a new mongo extension - the new one is called mongodb while the old one is mongo. I'm still a little unclear on the differences, but it seems like the old one won't be maintained in the future, so we should probably see if we can upgrade - especially if there is some sort of compatibility layer for the old extension.

@rexxars
Copy link
Member Author

rexxars commented Nov 15, 2015

@rexxars rexxars self-assigned this Dec 7, 2015
@rexxars
Copy link
Member Author

rexxars commented Dec 7, 2015

I've started investigating this in a branch: https://github.com/rexxars/imbo/tree/new-mongo

Got mostly everything working, database-wise. There is one bug in the official PHP driver that will be fixed shortly, which will fix the one test that is currently not passing.

I have not touched storage yet (GridFS) - not sure what the story is there, cause I see no classes for this so far. Will investigate further and possibly inquire if I can't find any more information.

@rexxars
Copy link
Member Author

rexxars commented Dec 7, 2015

FYI: The reason why I've started working on this is that the old Mongo extension won't be updated to work with PHP7, it seems.

@rexxars
Copy link
Member Author

rexxars commented Dec 28, 2015

For tracking purposes:

mongodb/mongo-php-library#57 - is required for GridFS
mongodb/mongo-php-library#78 - would clean up parts of the code where we translate between the objects returned and arrays to maintain backwards compatibility

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants