Testing Done: |
|
|---|
Add support for Django 1.5's index_together property.
Review Request #5121 — Created Dec. 16, 2013 and submitted
Add support for Django 1.5's index_together property.
Django 1.5 introduced index_together for creating an index that spans
columns without being unique. This provides support for that new
property through ChangeMeta.Unlike with unique_together, we don't have old signature state to worry
about, so we can safely compare values and assume they're correct
(though the actual code to perform the index_together will still
sanity-check the database before generating error-prone SQL).
Unit tests pass.
Tested that it doesn't cause any issues with our existing evolutions.
| Description | From | Last Updated |
|---|---|---|
|
This indentation is very, very weird. How about: self.changed\ .setdefault(app_name, {})\ .setdefault('changed', {})\ .setdefault(model_name, {})\ .setdefault('meta_changed', [])\ .append('index_together') |
 david
|
-
-
django_evolution/diff.py (Diff revision 1) This indentation is very, very weird.
How about:
self.changed\ .setdefault(app_name, {})\ .setdefault('changed', {})\ .setdefault(model_name, {})\ .setdefault('meta_changed', [])\ .append('index_together')