#30931: Cannot override get_FOO_display() in Django 2.2+.
--------------------------------------+------------------------------------
     Reporter:  Jim Ouwerkerk         |                    Owner:  (none)
         Type:  Cleanup/optimization  |                   Status:  new
    Component:  Documentation         |                  Version:  2.2
     Severity:  Normal                |               Resolution:
     Keywords:                        |             Triage Stage:  Accepted
    Has patch:  1                     |      Needs documentation:  0
  Needs tests:  0                     |  Patch needs improvement:  0
Easy pickings:  0                     |                    UI/UX:  0
--------------------------------------+------------------------------------

Comment (by Carlton Gibson):

 Ah, yes, you're right. I've copied the second test in wrong.

 But the point is (still) the change in behaviour from 2.1.

 That's still a bug fix, so OK it's changed, that being the nature of bug
 fixes.

 I'm happy to document this, as suggested. I'd be happy to instead just add
 a warning  saying "Meh, don't override methods added by the
 metaclass....".
 Between those, I quite like the option of documenting the solution, since
 it opens the box for people a little. The warning just leads to a
 "Why...?", and given that it's come up, it's presumably a valid use-case.

 Anyhow, I'm leaving to to Mariusz now.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/30931#comment:18>
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.fe4b80c84ea23a508e5be70343809f9e%40djangoproject.com.

Reply via email to