#23581: Migration shouldn't generate "DROP DEFAULT" SQL if no database changes 
are
required
--------------------------------------+------------------------------------
     Reporter:  john_scott            |                    Owner:  nobody
         Type:  Cleanup/optimization  |                   Status:  new
    Component:  Migrations            |                  Version:  1.7
     Severity:  Normal                |               Resolution:
     Keywords:                        |             Triage Stage:  Accepted
    Has patch:  0                     |      Needs documentation:  0
  Needs tests:  0                     |  Patch needs improvement:  0
Easy pickings:  0                     |                    UI/UX:  0
--------------------------------------+------------------------------------

Comment (by andrewgodwin):

 I agree it's a pain but there's not much of a way round this; the
 alternative is to then break fields inside data migrations, at which point
 someone else will come and tell me how much of a pain it is.

 If choices is genuinely not important to ANYTHING, then they should just
 use a custom field subclass that omits it from the deconstruct response
 (deconstruct is an official API now and everything).

 It's not that django-cms did anything wrong, it's just that we had to
 tighten up a few previously-loose things to get migrations working
 properly. South had these same issues, it's just that it decided to
 abandon choices completely (which itself caused a few bugs where people
 got angry as they were using choices to populate database columns and
 stuff).

 There's no "core developer approved" method of how to do this stuff; we're
 not all-powerful or all-knowledgeable, and I personally at least have not
 run into this case (this is why we had such a long alpha and beta period
 to get feedback, and changed quite a lot of stuff during it).

 I can give suggestions, but ultimately it's up to the community to work
 out good solutions and/or write code to support them. If you want to
 revert to the old South way of always throwing away choices, that's just a
 small patch to django-cms away.

--
Ticket URL: <https://code.djangoproject.com/ticket/23581#comment:15>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.99c3209adaa426510405faa16cc5ac2e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to