Opened 12 years ago

Closed 12 years ago

#19234 closed New feature (needsinfo)

can't adapt type error with models

Reported by: artscoop93.info@… Owned by: nobody
Component: Database layer (models, ORM) Version: 1.4
Severity: Normal Keywords: pickle, celery
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Sometimes with Django you want to perform operations which need to pickle model objects, you end up with a can't adapt type error. This is often true when one uses celery, for example.
I have a simple way to resolve this issue for model classes, but I wonder if it's relevant to add this code to Django itself.
This would be a new function in the django.db.models.base.Model class

def __reduce__(instance):
    names = [f.name for f in instance._meta.fields]
    attributes = dict([(name, getattr(instance, name)) for name in names])
    return (instance.__class__, (), attributes)

Worth adding or not ?

Change History (1)

comment:1 by Russell Keith-Magee, 12 years ago

Resolution: needsinfo
Status: newclosed

Words like "sometimes" aren't very helpful in a bug report.

Django models *should* be serializable already. If you actually look at the code for django.db.models.base.Model, there is already an implementation of __reduce__, and it's a lot more complex than what you propose (for very good reasons).

If you're having problems pickling models, you'll need to provide a specific example that reproduces your problem.

Closing needinfo; feel free to reopen if you can provide a specific example.

Note: See TracTickets for help on using tickets.
Back to Top