django 2.1+ support - Githubissues

Mikhail517 commented 6 years ago

supporting django 2.1 for a project

Mixer 6.0.1 version raises an exception while creating FK items in django 2.1:

Traceback (most recent call last): File "", line 1, in File "/usr/local/lib/python3.6/site-packages/mixer/main.py", line 568, in blend return type_mixer.blend(**values) File "/usr/local/lib/python3.6/site-packages/mixer/main.py", line 116, in blend for name, value in defaults.items() File "/usr/local/lib/python3.6/site-packages/mixer/main.py", line 116, in for name, value in defaults.items() File "/usr/local/lib/python3.6/site-packages/mixer/backend/django.py", line 218, in get_value return self._get_value(name, value, field) File "/usr/local/lib/python3.6/site-packages/mixer/backend/django.py", line 233, in _get_value value = field.scheme.to_python(value) File "/usr/local/lib/python3.6/site-packages/django/db/models/fields/related.py", line 874, in to_python return self.target_field.to_python(value) File "/usr/local/lib/python3.6/site-packages/django/db/models/fields/init.py", line 945, in to_python params={'value': value}, django.core.exceptions.ValidationError: ["Значение 'Anna Stevens' должно быть целым числом."]

This patch fix it

coveralls commented 6 years ago

Coverage increased (+0.2%) to 81.986% when pulling 91b4aa66bdbb0338b769f8992583fc5e496e170c on Mikhail517:develop into 9e773265d5787fac2401f446b5a1acf827638b3d on klen:develop.

mahaffey commented 6 years ago

Thank you this is great!

For others to use:

As an addition to your requirements file:

# mixer  --> Does not support Django 2.1
git+git://github.com/Mikhail517/mixer.git@develop#egg=mixer

Or as an easy pip install script:

$ pip install git+git://github.com/Mikhail517/mixer.git@develop#egg=mixer

klen commented 5 years ago

@Mikhail517 Thank you for the PR. The problem has been fixed in version 6.1.1

klen / mixer

django 2.1+ support #104

For others to use:

As an addition to your requirements file:

Or as an easy pip install script: