Changeset View
Changeset View
Standalone View
Standalone View
src/workflow/ArcanistLandWorkflow.php
| Show First 20 Lines • Show All 1,370 Lines • ▼ Show 20 Lines | |||||
| private function checkForBuildables($diff_phid) { | private function checkForBuildables($diff_phid) { | ||||
| // Try to use the more modern check which respects the "Warn on Land" | // Try to use the more modern check which respects the "Warn on Land" | ||||
| // behavioral flag on build plans if we can. This newer check won't work | // behavioral flag on build plans if we can. This newer check won't work | ||||
| // unless the server is running code from March 2019 or newer since the | // unless the server is running code from March 2019 or newer since the | ||||
| // API methods we need won't exist yet. We'll fall back to the older check | // API methods we need won't exist yet. We'll fall back to the older check | ||||
| // if this one doesn't work out. | // if this one doesn't work out. | ||||
| try { | try { | ||||
| $this->checkForBuildablesWithPlanBehaviors($diff_phid); | $this->checkForBuildablesWithPlanBehaviors($diff_phid); | ||||
| return; | |||||
| } catch (ArcanistUserAbortException $abort_ex) { | } catch (ArcanistUserAbortException $abort_ex) { | ||||
| throw $abort_ex; | throw $abort_ex; | ||||
| } catch (Exception $ex) { | } catch (Exception $ex) { | ||||
| // Continue with the older approach, below. | // Continue with the older approach, below. | ||||
| } | } | ||||
| // NOTE: Since Harbormaster is still beta and this stuff all got added | // NOTE: Since Harbormaster is still beta and this stuff all got added | ||||
| // recently, just bail if we can't find a buildable. This is just an | // recently, just bail if we can't find a buildable. This is just an | ||||
| ▲ Show 20 Lines • Show All 294 Lines • Show Last 20 Lines | |||||