Differential D11288 Diff 27128 src/applications/aphlict/management/PhabricatorAphlictManagementRestartWorkflow.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/aphlict/management/PhabricatorAphlictManagementRestartWorkflow.php
| <?php | <?php | ||||
| final class PhabricatorAphlictManagementRestartWorkflow | final class PhabricatorAphlictManagementRestartWorkflow | ||||
| extends PhabricatorAphlictManagementWorkflow { | extends PhabricatorAphlictManagementWorkflow { | ||||
| public function didConstruct() { | public function didConstruct() { | ||||
| parent::didConstruct(); | |||||
| $this | $this | ||||
| ->setName('restart') | ->setName('restart') | ||||
| ->setSynopsis(pht('Stop, then start the notifications server.')) | ->setSynopsis(pht('Stop, then start the notifications server.')); | ||||
| ->setArguments(array()); | |||||
| } | } | ||||
| public function execute(PhutilArgumentParser $args) { | public function execute(PhutilArgumentParser $args) { | ||||
| parent::execute($args); | |||||
| $err = $this->executeStopCommand(); | $err = $this->executeStopCommand(); | ||||
epriestley: Maybe move this spec into the superclass? | |||||
Not Done Inline ActionsSure. This might be problematic if any of the workflows want to implement their own arguments, but we can deal with this later. joshuaspence: Sure. This might be problematic if any of the workflows want to implement their own arguments… | |||||
| if ($err) { | if ($err) { | ||||
| return $err; | return $err; | ||||
| } | } | ||||
| return $this->executeStartCommand(); | return $this->executeStartCommand(); | ||||
| } | } | ||||
| } | } | ||||
Maybe move this spec into the superclass?