Skip to content

Fixed PHPLIB-108: Library relies on BSON being a vendor prefix, while our vendor should be MongoDB #19

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 2 commits into from

Conversation

derickr
Copy link
Contributor

@derickr derickr commented Jun 10, 2015

@jmikola
Copy link
Member

jmikola commented Jun 10, 2015

This needs to wait until PHPC-309 is implemented and released, since we'll also need to bump the ext-mongo dependency.

@derickr derickr force-pushed the issue108-vendor-prefix branch 2 times, most recently from 03ff930 to c10f2e2 Compare June 19, 2015 22:19
* Internal helper for running a command
* @internal
*/
final protected function _runCommand($dbname, array $cmd, ReadPreference $rp = null)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this method added?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That wasn't me... NFC where it came from.

@derickr derickr force-pushed the issue108-vendor-prefix branch from c10f2e2 to be1df64 Compare June 30, 2015 08:08
@jmikola
Copy link
Member

jmikola commented Jun 30, 2015

Rebased and merged in #22.

@jmikola jmikola closed this Jun 30, 2015
@jmikola
Copy link
Member

jmikola commented Jun 30, 2015

@derickr derickr deleted the issue108-vendor-prefix branch August 21, 2018 10:27
GromNaN pushed a commit to GromNaN/mongo-php-library that referenced this pull request Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants