Edgewall Software

Changes between Version 3 and Version 4 of Ticket #12414, comment 11


Ignore:
Timestamp:
May 7, 2020, 8:37:14 PM (3 years ago)
Author:
Ryan J Ollos

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #12414, comment 11

    v3 v4  
    88`ITicketManipulator.validate_ticket` is primarily document as being for identifying problems and rejecting changes, but it's location in the request processing makes it very useful for modifying the ticket changes.
    99
    10 I think it's been discussed before, `ITicketChangeListener` might be more useful if the request was passed as an argument. One way would be to move the call to `ITicketChangeListener` out of the model object. Also, it might be more user friendly if ticket changes were passed in a dict that included the `cnum` and `cdate`, rather than passing the `ticket` and `old_values`.
     10I think it's been discussed before, `ITicketChangeListener` might be more useful if the request was passed as an argument. One way would be to move the call to `ITicketChangeListener` out of the model object. Also, it might be more user friendly if ticket changes were passed in a dict that included the `cnum` and `cdate`, rather than passing the `ticket` and `old_values`. But those are just ideas, maybe not even good ones, that won't happen soon.