Fix for bug 74394: editmilestones.cgi can't edit the name and sortkey at the same...

Patch by slee@uk.bnsmc.com (Stephen Lee)

diff --git a/editmilestones.cgi b/editmilestones.cgi
index 9eadeeba2dcbb343c9382f91127ae6971f17b69a..57fe78830d514a99cabf5caf6645e13652f460af 100755 (executable)
--- a/editmilestones.cgi
+++ b/editmilestones.cgi
@@ -513,6 +513,13 @@ if ($action eq 'update') {
                          milestones WRITE,
                          products WRITE");
 
+    if ($sortkey != $sortkeyold) {
+        SendSQL("UPDATE milestones SET sortkey=$sortkey
+                 WHERE product=" . SqlQuote($product) . "
+                   AND value=" . SqlQuote($milestoneold));
+        unlink "data/versioncache";
+        print "Updated sortkey.<BR>\n";
+    }
     if ($milestone ne $milestoneold) {
         unless ($milestone) {
             print "Sorry, I can't delete the milestone text.";
@@ -541,13 +548,6 @@ if ($action eq 'update') {
         unlink "data/versioncache";
         print "Updated milestone.<BR>\n";
     }
-    if ($sortkey != $sortkeyold) {
-        SendSQL("UPDATE milestones SET sortkey=$sortkey
-                 WHERE product=" . SqlQuote($product) . "
-                   AND value=" . SqlQuote($milestoneold));
-        unlink "data/versioncache";
-        print "Updated sortkey.<BR>\n";
-    }
     SendSQL("UNLOCK TABLES");
 
     PutTrailer($localtrailer);