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

DeprecationWarning: get_db_prep_lookup #1

Open
airstrike opened this issue Mar 10, 2011 · 3 comments
Open

DeprecationWarning: get_db_prep_lookup #1

airstrike opened this issue Mar 10, 2011 · 3 comments

Comments

@airstrike
Copy link

I get these messages with manage.py runserver running trunk (1.3 beta 1 SVN-15564):

C:\django\Lib\site-packages\django\db\models\fields\subclassing.py:80: DeprecationWarning: A Field class whose get_db_prep_lookup method hasn't been updated to take connection and prepared arguments.
new_class = super(SubfieldBase, cls).new(cls, name, bases, attrs)

C:\django\Lib\site-packages\django\db\models\fields\subclassing.py:80: DeprecationWarning: A Field class whose get_db_prep_value method hasn't been updated to take connection and prepared arguments.
new_class = super(SubfieldBase, cls).new(cls, name, bases, attrs)

grep -irn "get_db_prep_value" . yields:

./utils/picklefield.py:119: def get_db_prep_value(self, value):
./utils/picklefield.py:142: return self.get_db_prep_value(value)
./utils/picklefield.py:150: # The Field model already calls get_db_prep_value before doing the

@nowells
Copy link
Owner

nowells commented Mar 12, 2011

Hey Andre,

I have not had time to bring this project in-line with Django 1.3 or with the addition of namespaces in the URL system of Django 1.2 (we added our own namespace solution since it was not available at the time) If you would like to take a stab at fixing these issues and resolving and blockers from this working in Django 1.3 I would happily review a Pull Request. Thanks for the report.

Cheers!
Nowell

@pombredanne
Copy link
Collaborator

@airstrike can you check if you are still having this with the latest master?

@airstrike
Copy link
Author

Unfortunately, I haven't used this app in so long I'm afraid it would be impossible to reproduce the issue.

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

3 participants