Ticket #2618: mid_air_collision_retry-r3341.patch

trac/ticket/web_ui.py

@@ -252 +252 @@
 
         if req.method == 'POST':
             if not req.args.has_key('preview'):
-                self._do_save(req, db, ticket)
+                return self._do_save(req, db, ticket)
             else:
                 # Use user supplied values
                 ticket.populate(req.args)
                 self._validate_ticket(req, ticket)
-
-                req.hdf['ticket.action'] = action
-                req.hdf['ticket.ts'] = req.args.get('ts')
-                req.hdf['ticket.reassign_owner'] = req.args.get('reassign_owner') \
-                                                   or req.authname
-                req.hdf['ticket.resolve_resolution'] = req.args.get('resolve_resolution')
-                reporter_id = req.args.get('author')
-                comment = req.args.get('comment')
-                if comment:
-                    req.hdf['ticket.comment'] = comment
-                    # Wiki format a preview of comment
-                    req.hdf['ticket.comment_preview'] = wiki_to_html(comment,
-                                                                     self.env,
-                                                                     req, db)
+                self._populate_misc(req, db)
         else:
             req.hdf['ticket.reassign_owner'] = req.authname
             # Store a timestamp in order to detect "mid air collisions"
@@ -484 +471 @@
 
         # Mid air collision?
         if int(req.args.get('ts')) != ticket.time_changed:
-            raise TracError("Sorry, can not save your changes. "
-                            "This ticket has been modified by someone else "
-                            "since you started", 'Mid Air Collision')
+            self._insert_ticket_data(req, None, ticket,
+                                     req.args.get('author'))
+            self._populate_misc(req, db)
+            return 'ticket_error.cs', None
 
         self._validate_ticket(req, ticket)
 
@@ -590 +578 @@
         actions = TicketSystem(self.env).get_available_actions(ticket, req.perm)
         for action in actions:
             req.hdf['ticket.actions.' + action] = '1'
+
+    def _populate_misc(self, req, db):
+        req.hdf['ticket.action'] = req.args.get('action', 'view')
+        req.hdf['ticket.ts'] = req.args.get('ts')
+        req.hdf['ticket.reassign_owner'] = req.args.get('reassign_owner') \
+                                           or req.authname
+        req.hdf['ticket.resolve_resolution'] = req.args.get('resolve_resolution')
+        reporter_id = req.args.get('author')
+        comment = req.args.get('comment')
+        if comment:
+            req.hdf['ticket.comment'] = comment
+            # Wiki format a preview of comment
+            req.hdf['ticket.comment_preview'] = wiki_to_html(
+                comment, self.env, req, db)

templates/ticket_error.cs

@@ +1 @@
+<?cs include "header.cs"?>
+
+<div id="ctxtnav" class="nav"></div>
+
+<div id="content" class="error">
+ <h1>Mid Air Collision</h1>
+  <div class="message">
+   <strong>Sorry, can not save your changes.</strong>
+   <p>This ticket has been modified by someone else since you started.</p>
+  </div>
+
+ <form action="<?cs var:ticket.href ?>#preview" method="post">
+  <input type="hidden" name="author" value="<?cs var:ticket.reporter_id ?>" /><?cs
+  each:field = ticket.fields ?>
+   <input type="hidden" name="<?cs var:name(field) ?>" value="<?cs var:ticket[name(field)] ?>" /><?cs
+  /each ?>
+  <input type="hidden" name="reassign_owner" value="<?cs var:ticket.reassign_owner ?>" />
+  <input type="hidden" name="action" value="<?cs var:ticket.action ?>" />
+  <input type="hidden" name="ts" value="<?cs var:ticket.ts ?>" />
+  <input type="hidden" name="comment" value="<?cs var:ticket.comment ?>" />
+
+  <div class="buttons">
+   <input type="submit" name="preview" value="Retry"  />&nbsp;
+  </div>
+ </form>
+</div>
+
+<?cs include "footer.cs"?>