#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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.fe4b80c84ea23a508e5be70343809f9e%40djangoproject.com.

Reply via email to