<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Create analysis task should use build time as fallback when commit time is not available."
   href="https://bugs.webkit.org/show_bug.cgi?id=183309#c9">Comment # 9</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Create analysis task should use build time as fallback when commit time is not available."
   href="https://bugs.webkit.org/show_bug.cgi?id=183309">bug 183309</a>
              from <span class="vcard"><a class="email" href="mailto:dewei_zhu@apple.com" title="dewei_zhu@apple.com">dewei_zhu</a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=334946&action=diff" name="attach_334946" title="Patch">attachment 334946</a> <a href="attachment.cgi?id=334946&action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=334946&action=review">https://bugs.webkit.org/attachment.cgi?id=334946&action=review</a>

<span class="quote">>>>>>> Websites/perf.webkit.org/public/privileged-api/create-analysis-task.php:97
>>>>>> +    return $result ? ($result[0]['time'] ? $result[0]['time'] : $result[0]['build_time']) : null;
>>>>> 
>>>>> Can't we just do $result[0]['time'] || $result[0]['build_time'] ?
>>>> 
>>>> No. Logical operator will return true / false in PHP. <a href="http://php.net/manual/en/language.operators.logical.php">http://php.net/manual/en/language.operators.logical.php</a>
>>> 
>>> Okay. In that case, do
>>> $first_result = array_get($result, 0, array());
>>> return array_key_exists('time', $first_result) : $first_result['time'] : $first_result['build_time'];
>> 
>> Sounds a good idea. The 'time' key will always exist as long as there is a matching entry. Changing `array_key_exists('time', $first_result) : $first_result['time'] : $first_result['build_time']` to `$first_result['time'] ? $first_result['time'] : $first_result['build_time']` would make it work.

> Okay. We should be careful for the case when time is 0 and NULL.</span >

We've already have some code that assumes time is never to be 0. For example: line 29 of create-analysis-task.php `if (!$start_run_time || !$end_run_time || $start_run_time == $end_run_time)`</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>