UI to associate bugs with an analysis task is crappy
[WebKit-https.git] / Websites / perf.webkit.org / public / privileged-api / associate-bug.php
index d4bd2608688344d12a271c402af4c90436ed6e47..4318d4ab7f59728c125bdf9758fc1a14dfddfb9c 100644 (file)
@@ -8,25 +8,23 @@ function main() {
     $analysis_task_id = array_get($data, 'task');
     $bug_tracker_id = array_get($data, 'bugTracker');
     $bug_number = array_get($data, 'number');
     $analysis_task_id = array_get($data, 'task');
     $bug_tracker_id = array_get($data, 'bugTracker');
     $bug_number = array_get($data, 'number');
-
-    require_format('AnalysisTask', $analysis_task_id, '/^\d+$/');
-    require_format('BugTracker', $bug_tracker_id, '/^\d+$/');
-    require_format('BugNumber', $bug_number, '/^\d*$/');
+    $bug_id = array_get($data, 'bugToDelete');
 
     $db = connect();
     $db->begin_transaction();
 
 
     $db = connect();
     $db->begin_transaction();
 
-    $bug_id = NULL;
-    if (!$bug_number) {
-        $count = $db->query_and_get_affected_rows("DELETE FROM bugs WHERE bug_task = $1 AND bug_tracker = $2",
-            array($analysis_task_id, $bug_tracker_id));
-        if ($count > 1) {
+    if ($bug_id) {
+        require_format('BugToDelete', $bug_id, '/^\d+$/');
+        $count = $db->query_and_get_affected_rows("DELETE FROM bugs WHERE bug_id = $1", array($bug_id));
+        if ($count != 1) {
             $db->rollback_transaction();
             exit_with_error('UnexpectedNumberOfAffectedRows', array('affectedRows' => $count));
         }
     } else {
             $db->rollback_transaction();
             exit_with_error('UnexpectedNumberOfAffectedRows', array('affectedRows' => $count));
         }
     } else {
-        $bug_id = $db->update_or_insert_row('bugs', 'bug', array('task' => $analysis_task_id, 'tracker' => $bug_tracker_id),
-            array('task' => $analysis_task_id, 'tracker' => $bug_tracker_id, 'number' => $bug_number));
+        require_format('AnalysisTask', $analysis_task_id, '/^\d+$/');
+        require_format('BugTracker', $bug_tracker_id, '/^\d+$/');
+        require_format('BugNumber', $bug_number, '/^\d+$/');
+        $bug_id = $db->insert_row('bugs', 'bug', array('task' => $analysis_task_id, 'tracker' => $bug_tracker_id, 'number' => $bug_number));
     }
     $db->commit_transaction();
 
     }
     $db->commit_transaction();