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

Require higher version of doctrine/mongodb-odm #83

Closed
DmitryVarennikov opened this issue Aug 13, 2013 · 9 comments
Closed

Require higher version of doctrine/mongodb-odm #83

DmitryVarennikov opened this issue Aug 13, 2013 · 9 comments
Milestone

Comments

@DmitryVarennikov
Copy link

Recently I was struggling with sorting until I finally realize that this bug was fixed in Doctrine MongoDB but current module still sticks to the older ~1.0.0-beta5@dev version of Doctrine MongoDB

Suggest to require "doctrine/mongodb-odm": "1.0.x-dev"

@DmitryVarennikov
Copy link
Author

doctrine/mongodb-odm#658

@jmikola
Copy link
Member

jmikola commented Aug 13, 2013

@Ocramius: If the dependency hasn't been bumped, then #70 (comment) is likely still open for users.

@Ocramius
Copy link
Member

@superdweebie any problem with bumping this?

@superdweebie
Copy link
Contributor

@Ocramius no, go for it. I'm a little uncertain about going all the way to 1.0.x-dev, just because mongodb-odm is still in beta, and subtle changes that may be bc breaks for some are introduced into the code from time to time.

@jmikola
Copy link
Member

jmikola commented Aug 13, 2013

BETA9 would be safest for now. There will be a few more betas before 1.0, as I'm still updating doctrine/mongodb and mean to get ODM over to doctrine/mongodb 1.1.

@DmitryVarennikov
Copy link
Author

Does BETA9 include cursor sorting fix?

@jmikola
Copy link
Member

jmikola commented Aug 15, 2013

No; it includes the fix for #70 only. Taking another look at composer.json for this module, and there should be nothing preventing you from manually requiring a higher version of ODM (this just requires something >= BETA5).

@DmitryVarennikov
Copy link
Author

@jmikola Well, already did

@Ocramius
Copy link
Member

Also got this in composer.json @85714a0e85e36f6516a0e00f679fcdbf4e414e10

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

No branches or pull requests

4 participants