Deleting first ticket change does not reset changetime correctly

[osimons]

The change in #3641 / [9270] works fine as long as earlier ticket changes exist. However, if no such ticket change then exist, the changetime will be set to NULL.

If so, the # Fix the last modification time part of Ticket.delete_change() needs to use time from ticket instead of ticket_change so that last modified will be set to time of ticket creation.

[osimons]

Meaning "first and only" ticket change of course - which also makes it the "last" change I suppose :-)

Anyway…:

1. Create a ticket
2. Make one change
3. Delete the change
4. Observe that ticket is "last modified" 42 years ago

[Remy Blank]

Good catch Simon. We could maybe get away with simply adding UNION SELECT time FROM ticket WHERE id=%s to the sub-query (and passing the additional self.id argument), or possibly use COALESCE().

Do you want to fix that yourself, or should I look into it?

[osimons]

Using UNION works for me.

[osimons]

Might as well attempt to commit again…

[osimons]

Done. [10792] for 0.12-stable, merged to trunk in [10793].