Differential D7498 Diff 16921 src/applications/harbormaster/controller/HarbormasterBuildableViewController.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/harbormaster/controller/HarbormasterBuildableViewController.php
Show All 31 Lines | $builds = id(new HarbormasterBuildQuery()) | ||||
->execute(); | ->execute(); | ||||
$build_list = id(new PHUIObjectItemListView()) | $build_list = id(new PHUIObjectItemListView()) | ||||
->setUser($viewer); | ->setUser($viewer); | ||||
foreach ($builds as $build) { | foreach ($builds as $build) { | ||||
$item = id(new PHUIObjectItemView()) | $item = id(new PHUIObjectItemView()) | ||||
->setObjectName(pht('Build %d', $build->getID())) | ->setObjectName(pht('Build %d', $build->getID())) | ||||
->setHeader($build->getName()); | ->setHeader($build->getName()); | ||||
switch ($build->getBuildStatus()) { | |||||
case HarbormasterBuild::STATUS_INACTIVE: | |||||
$item->setBarColor('grey'); | |||||
$item->addAttribute(pht('Inactive')); | |||||
break; | |||||
case HarbormasterBuild::STATUS_PENDING: | |||||
$item->setBarColor('blue'); | |||||
$item->addAttribute(pht('Pending')); | |||||
break; | |||||
case HarbormasterBuild::STATUS_WAITING: | |||||
$item->setBarColor('blue'); | |||||
$item->addAttribute(pht('Waiting on Resource')); | |||||
break; | |||||
case HarbormasterBuild::STATUS_BUILDING: | |||||
$item->setBarColor('yellow'); | |||||
$item->addAttribute(pht('Building')); | |||||
break; | |||||
case HarbormasterBuild::STATUS_PASSED: | |||||
$item->setBarColor('green'); | |||||
$item->addAttribute(pht('Passed')); | |||||
break; | |||||
case HarbormasterBuild::STATUS_FAILED: | |||||
$item->setBarColor('red'); | |||||
$item->addAttribute(pht('Failed')); | |||||
break; | |||||
} | |||||
$build_list->addItem($item); | $build_list->addItem($item); | ||||
} | } | ||||
$title = pht("Buildable %d", $id); | $title = pht("Buildable %d", $id); | ||||
$header = id(new PHUIHeaderView()) | $header = id(new PHUIHeaderView()) | ||||
->setHeader($title) | ->setHeader($title) | ||||
->setUser($viewer) | ->setUser($viewer) | ||||
Show All 27 Lines | private function buildActionList(HarbormasterBuildable $buildable) { | ||||
$viewer = $request->getUser(); | $viewer = $request->getUser(); | ||||
$id = $buildable->getID(); | $id = $buildable->getID(); | ||||
$list = id(new PhabricatorActionListView()) | $list = id(new PhabricatorActionListView()) | ||||
->setUser($viewer) | ->setUser($viewer) | ||||
->setObject($buildable) | ->setObject($buildable) | ||||
->setObjectURI("/B{$id}"); | ->setObjectURI("/B{$id}"); | ||||
$apply_uri = $this->getApplicationURI('/buildable/apply/'.$id.'/'); | |||||
$list->addAction( | |||||
id(new PhabricatorActionView()) | |||||
->setName(pht('Apply Build Plan')) | |||||
->setIcon('edit') | |||||
->setHref($apply_uri) | |||||
->setWorkflow(true)); | |||||
epriestley: I think this should be on the detail view, not the main view? In the long run, I think/hope it… | |||||
return $list; | return $list; | ||||
} | } | ||||
private function buildPropertyLists( | private function buildPropertyLists( | ||||
PHUIObjectBoxView $box, | PHUIObjectBoxView $box, | ||||
HarbormasterBuildable $buildable, | HarbormasterBuildable $buildable, | ||||
PhabricatorActionListView $actions) { | PhabricatorActionListView $actions) { | ||||
$request = $this->getRequest(); | $request = $this->getRequest(); | ||||
Show All 15 Lines |
I think this should be on the detail view, not the main view? In the long run, I think/hope it will be a rare action.