[Trac-tickets] Re: [The Trac Project] #2618: ticket mods are lost on submit if ticket modified by someone else

[The Trac Project]

#2618: ticket mods are lost on submit if ticket modified by someone else
---------------------------------+------------------------------------------
 Reporter:  bboisvert at gmail.com  |        Owner:  cboos   
     Type:  defect               |       Status:  assigned
 Priority:  normal               |    Milestone:  0.10    
Component:  ticket system        |      Version:  0.9     
 Severity:  major                |   Resolution:          
 Keywords:                       |  
---------------------------------+------------------------------------------
Comment (by bboisvert at gmail.com):

 Shouldn't the ClearSilver template be ticket_error_midair.cs, not
 ticket_error.cs?  There are other ticket errors (such as forgetting the
 summary on a new ticket) that aren't handled by it, though it's probably
 desirable if they all are.  You don't need the merge stuff for new
 tickets, but not having to manually copy and paste your ticket data back
 in would be nice.

 One problem with it, however, is that the preview gives you a preview only
 of YOUR data, so if the collision was for anything except the addition of
 a comment/attachement, there's no way to view what the updated value is.
 E.g. if someone updates 'keywords', then you submit and get a mid-air,
 when you hit 'retry', it's going to show your keywords, with no indication
 of what the current commited keyword's value is (which you've never seen).

 What I'm thinking is maybe on the error page, it can display the committed
 values and the submitted values in two columns, and you can run down the
 list and pick which to keep.  Then when you hit 'retry' it submits the
 collection of values you picked.  You get your ability to manually merge,
 and without much more fuss.  Just need to load the committed data into the
 HDF for the template to use, and throw in a little JavaScript magic for
 dynamically valued form fields based on which side of the merge is
 requested for each field.

-- 
Ticket URL: <http://projects.edgewall.com/trac/ticket/2618>
The Trac Project <http://trac.edgewall.com/>