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?