Changes between Initial Version and Version 1 of Ticket #27317, comment 3


Ignore:
Timestamp:
Oct 7, 2016, 5:00:26 PM (8 years ago)
Author:
James Pic

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #27317, comment 3

    initial v1  
    1 Thanks for re-centering the discussion. I understand this design decision. However,  explicit inheritance doesn't fix it:
     1Thanks for re-centering the discussion. I understand this design decision.
     2
     3> ​As the documentation says about Meta, "Normal Python name resolution rules apply. If you have multiple base classes that declare a Meta inner class, only the first one will be used.
     4
     5Oh, I was talking about Media... I don't see any case where you'd want to render a widget or form without a certain script that it requires, users would just end with broken forms :D
     6
     7But, let's admit that such a paralogism is valid, even then, explicit inheritance doesn't fix it:
    28
    39{{{
     
    2329
    2430Shouldn't we be expecting `MediaC.js = ('jquery.js', 'a.js', 'b.js')` here ? I can't think of any case where we wouldn't: this multiple inheritance is after all, explicit.
     31
     32> By the way, we're thinking about ways to replace Media with a better solution, although no solution seems imminent (#22298).
     33
     34There's no hurry, I got a solution that works until them: using a Widget to render additional script without breaking the user Form class yaayyyy :D it will be easy to change when we have another solution.
     35
     36Meanwhile, should we fix inheritance in Media.js and Media.css attributes ?
Back to Top