Title: [214707] trunk/Websites/perf.webkit.org
Revision
214707
Author
[email protected]
Date
2017-03-31 17:45:51 -0700 (Fri, 31 Mar 2017)

Log Message

Build fix. For OS versions, we can end up with non-alphanumeric revision.
Delete the code path only used by the v2 UI since nobody uses that now.

* public/api/commits.php:
(main):

Modified Paths

Diff

Modified: trunk/Websites/perf.webkit.org/ChangeLog (214706 => 214707)


--- trunk/Websites/perf.webkit.org/ChangeLog	2017-04-01 00:31:34 UTC (rev 214706)
+++ trunk/Websites/perf.webkit.org/ChangeLog	2017-04-01 00:45:51 UTC (rev 214707)
@@ -1,3 +1,11 @@
+2017-03-31  Ryosuke Niwa  <[email protected]>
+
+        Build fix. For OS versions, we can end up with non-alphanumeric revision.
+        Delete the code path only used by the v2 UI since nobody uses that now.
+
+        * public/api/commits.php:
+        (main):
+
 2017-03-30  Ryosuke Niwa  <[email protected]>
 
         sync-buildbot.js can schedule more than one build per builder

Modified: trunk/Websites/perf.webkit.org/public/api/commits.php (214706 => 214707)


--- trunk/Websites/perf.webkit.org/public/api/commits.php	2017-04-01 00:31:34 UTC (rev 214706)
+++ trunk/Websites/perf.webkit.org/public/api/commits.php	2017-04-01 00:45:51 UTC (rev 214707)
@@ -38,14 +38,8 @@
             $commits = $fetcher->fetch_last_reported_between_orders($repository_id, $from, $to);
         else
             $commits = $fetcher->fetch_last_reported($repository_id);
-    } else if (ctype_alnum($filter)) {
+    } else
         $commits = $fetcher->fetch_revision($repository_id, $filter);
-    } else { // V2 UI compatibility.
-        $matches = array();
-        if (!preg_match('/([A-Za-z0-9]+)[\:\-]([A-Za-z0-9]+)/', $filter, $matches))
-            exit_with_error('UnknownFilter', array('repositoryName' => $repository_name, 'filter' => $filter));
-        $commits = $fetcher->fetch_between($repository_id, $matches[1], $matches[2]);
-    }
 
     if (!is_array($commits))
         exit_with_error('FailedToFetchCommits', array('repository' => $repository_id, 'filter' => $filter));
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to