django-autofixture icon indicating copy to clipboard operation
django-autofixture copied to clipboard

Fix `process_field` so that it works for m2m fields in Django 2.0

Open volrath opened this issue 7 years ago • 3 comments

Support for direct assignment in manytomany fields was removed in Django 2.0 (see: https://docs.djangoproject.com/en/2.0/releases/2.0/#features-removed-in-2-0).

This commit adds an extra param to process_field to indicate that the field being processed is a m2m and assignment should be handled through .set instead.

volrath avatar Feb 14 '18 09:02 volrath

While waiting for this to be merged, I found this library a pretty easy drop-in replacement: https://model-mommy.readthedocs.io/en/latest/index.html.

panchr avatar Apr 23 '18 13:04 panchr

Perhaps the tox matrix needs to updated to include Django 1.10, 1.11, and 2.0. And perhaps a further clean up to only bother with Django 1.11 under Python 2.7, and remove support for Python 3.3 since that is now EOL.

This PR is not likely to be merged while tests are failing, even if it is because the travis environment is not supporting those versions of python.

rhelms avatar May 21 '18 04:05 rhelms

i prefer PR #110 fix the gis module too

zodman avatar Aug 24 '18 21:08 zodman