mirror of
https://we.phorge.it/source/phorge.git
synced 2025-01-31 08:58:20 +01:00
Link "continue" action to confirm dialog in bulk jobs that are unconfirmed
Summary: See Q266. Test Plan: Created a bulk job, clicked "Details" instead of "Confirm", clicked "Continue" to get back to confirmation dialog. Reviewers: chad Reviewed By: chad Differential Revision: https://secure.phabricator.com/D14985
This commit is contained in:
parent
01fba34327
commit
96b1665eaa
2 changed files with 11 additions and 1 deletions
|
@ -71,9 +71,15 @@ final class PhabricatorDaemonBulkJobViewController
|
||||||
->setUser($viewer)
|
->setUser($viewer)
|
||||||
->setObject($job);
|
->setObject($job);
|
||||||
|
|
||||||
|
if ($job->isConfirming()) {
|
||||||
|
$continue_uri = $job->getMonitorURI();
|
||||||
|
} else {
|
||||||
|
$continue_uri = $job->getDoneURI();
|
||||||
|
}
|
||||||
|
|
||||||
$actions->addAction(
|
$actions->addAction(
|
||||||
id(new PhabricatorActionView())
|
id(new PhabricatorActionView())
|
||||||
->setHref($job->getDoneURI())
|
->setHref($continue_uri)
|
||||||
->setIcon('fa-arrow-circle-o-right')
|
->setIcon('fa-arrow-circle-o-right')
|
||||||
->setName(pht('Continue')));
|
->setName(pht('Continue')));
|
||||||
|
|
||||||
|
|
|
@ -130,6 +130,10 @@ final class PhabricatorWorkerBulkJob
|
||||||
return idx($map, $this->getStatus(), $this->getStatus());
|
return idx($map, $this->getStatus(), $this->getStatus());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function isConfirming() {
|
||||||
|
return ($this->getStatus() == self::STATUS_CONFIRM);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
/* -( Job Implementation )------------------------------------------------- */
|
/* -( Job Implementation )------------------------------------------------- */
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue