Changes between Initial Version and Version 1 of Ticket #29301, comment 8


Ignore:
Timestamp:
May 21, 2018, 11:34:08 AM (6 years ago)
Author:
David Foster

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #29301, comment 8

    initial v1  
    55(B) If we take the interpretation that management commands are ultimately in control of their arguments, I would expect Django to back off if sees that a management command added its own argument with a conflicting name. Adding new default arguments would then be a minor change.
    66
    7 Before the original patch for this ticket was merged, add_arguments() was allowed even more leeway than (B), potentially removing default arguments outright, setting a custom formatters, or doing other unusual things, even though those possibilities were not documented. I don't think we should necessary support arbitrary actions in add_arguments().
     7Before the original patch for this ticket was merged, add_arguments() was allowed even more leeway than (B), potentially removing default arguments outright, setting a custom formatter, or doing other unusual things, even though those possibilities were not documented. I don't think we should necessary support arbitrary actions in add_arguments().
    88
    99I'm personally +0 on approach (B). It would involve a small additional patch where Django checks whether each default argument already existed before adding its own.
Back to Top