-
-
django_evolution/tests/test_evolution_models.py (Diff revision 1) local variable 'version1' is assigned to but never used
chipx86 got a fish trophy!
Add smarter support for grabbing the latest Version for a database.
Review Request #7777 — Created Nov. 23, 2015 and submitted
Historically, django-evolution was using
Version.objects.latest()to
grab the latest version of the database schema. However, this was
failing in the event that you had twoVersionentries created within the
same second on a database that didn't have very good precision on
date/time fields (MySQL). This easily happened when both computing a
baseline for new models and then trying to perform very simple and fast
schema updates.We now have a new function,
current_version(), which does the same thing
thatlatest()did, but with the addition of factoring in the ordering of
the IDs. In the event that two timestamps match, the newer ID will now
take precedence, ensuring we're not looking at the "older" baseline
entry.
Unit tests pass.
Simulated the original problem we've seen on a database here. Verified
the same failure conditions (using an outdated schema, resulting in
"fields have changed" errors) with the old implementation. Saw the correct
results with the new one.
| Description | From | Last Updated |
|---|---|---|
|
local variable 'version1' is assigned to but never used |
 reviewbot
|
|
|
Typo: "latet" -> "latest" |
AD adriano | |
|
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot
|
-
Tool: Pyflakes Processed Files: django_evolution/evolve.py django_evolution/tests/test_evolution_models.py django_evolution/management/__init__.py django_evolution/management/commands/evolve.py django_evolution/models.py Tool: PEP8 Style Checker Processed Files: django_evolution/evolve.py django_evolution/tests/test_evolution_models.py django_evolution/management/__init__.py django_evolution/management/commands/evolve.py django_evolution/models.py -
django_evolution/tests/test_evolution_models.py (Diff revision 2) Col: 80 E501 line too long (85 > 79 characters)
Tool: Pyflakes Processed Files: django_evolution/evolve.py django_evolution/tests/test_evolution_models.py django_evolution/management/__init__.py django_evolution/management/commands/evolve.py django_evolution/models.py Tool: PEP8 Style Checker Processed Files: django_evolution/evolve.py django_evolution/tests/test_evolution_models.py django_evolution/management/__init__.py django_evolution/management/commands/evolve.py django_evolution/models.py