-
Notifications
You must be signed in to change notification settings - Fork 254
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
pymongo 3.0 #153
Comments
Anyone here? |
I'm not sure that mongoengine support for pymongo 3.0 has been released yet. |
They are waiting for us in that matter... I'm asking here as a suggestion from them |
Since mongoengine has updated its dependencies to pymongo 3.0 this extension doesn't work anymore |
Currently, you'd have to lock mongoengine versions until someone from the community investigates and fixes the issue. This project open source and as such is looking for active users from the community to step up and become maintainers. |
I know and understand open source but here we need to be honest and if this volutaries don't show up more soon than later will be better that someone put a huge warning telling this extension is out of date We, as open source users, need to be patience but open source creators need to be honest since the users are making a de facto contract of trust when they choose to use the piece of software |
If you change the line in flask_mongoengine/operation_tracker.py from |
The solution is to change the line 140 of operation_tracker.py from def privar(name):
return getattr(cursor_self, '_Cursor__{0}'.format(name)) to def privar(name):
return getattr(cursor_self, '_Cursor__{0}'.format(name), None) I am sending the PR |
Fix #153 privar() return None when attr is not found
Hi!
I would like to update pymongo to 3.0
Did you know when will be flask-mongoengine ready to do so?
As soon as I update it I raises and error
I've tryed to install masters versions of mongoengine (proposed by the people in charge) and yours but the errors is still there
Let me put the traceback here to:
Any ideas?
Thanks!
The text was updated successfully, but these errors were encountered: