revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-26 00:32:42 +01:00
Code Issues Releases Wiki Activity

Remove "HarbormasterBuildableTransaction::TYPE_CREATE"

Browse source 
Summary: Ref T13072. This transaction type has no writers and is mooted by EditEngine.

Test Plan: Grepped for transaction constant.

Subscribers: PHID-OPKG-gm6ozazyms6q6i22gyam

Maniphest Tasks: T13072

Differential Revision: https://secure.phabricator.com/D21693
This commit is contained in:
epriestley 2021-07-16 08:33:10 -07:00
parent 1a1395579f
commit 706b91adf6
2 changed files with 0 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/applications/harbormaster/editor/HarbormasterBuildableTransactionEditor.php
View file

@ -14,7 +14,6 @@ final class HarbormasterBuildableTransactionEditor
public function getTransactionTypes() { public function getTransactionTypes() {
$types = parent::getTransactionTypes(); $types = parent::getTransactionTypes();
$types[] = HarbormasterBuildableTransaction::TYPE_CREATE;
$types[] = HarbormasterBuildableTransaction::TYPE_COMMAND; $types[] = HarbormasterBuildableTransaction::TYPE_COMMAND;
return $types; return $types;
@ -25,7 +24,6 @@ final class HarbormasterBuildableTransactionEditor
PhabricatorApplicationTransaction $xaction) { PhabricatorApplicationTransaction $xaction) {
switch ($xaction->getTransactionType()) { switch ($xaction->getTransactionType()) {
case HarbormasterBuildableTransaction::TYPE_CREATE:
case HarbormasterBuildableTransaction::TYPE_COMMAND: case HarbormasterBuildableTransaction::TYPE_COMMAND:
return null; return null;
} }
@ -38,8 +36,6 @@ final class HarbormasterBuildableTransactionEditor
PhabricatorApplicationTransaction $xaction) { PhabricatorApplicationTransaction $xaction) {
switch ($xaction->getTransactionType()) { switch ($xaction->getTransactionType()) {
case HarbormasterBuildableTransaction::TYPE_CREATE:
return true;
case HarbormasterBuildableTransaction::TYPE_COMMAND: case HarbormasterBuildableTransaction::TYPE_COMMAND:
return $xaction->getNewValue(); return $xaction->getNewValue();
} }
@ -52,7 +48,6 @@ final class HarbormasterBuildableTransactionEditor
PhabricatorApplicationTransaction $xaction) { PhabricatorApplicationTransaction $xaction) {
switch ($xaction->getTransactionType()) { switch ($xaction->getTransactionType()) {
case HarbormasterBuildableTransaction::TYPE_CREATE:
case HarbormasterBuildableTransaction::TYPE_COMMAND: case HarbormasterBuildableTransaction::TYPE_COMMAND:
return; return;
} }
@ -65,7 +60,6 @@ final class HarbormasterBuildableTransactionEditor
PhabricatorApplicationTransaction $xaction) { PhabricatorApplicationTransaction $xaction) {
switch ($xaction->getTransactionType()) { switch ($xaction->getTransactionType()) {
case HarbormasterBuildableTransaction::TYPE_CREATE:
case HarbormasterBuildableTransaction::TYPE_COMMAND: case HarbormasterBuildableTransaction::TYPE_COMMAND:
return; return;
} }

9
src/applications/harbormaster/storage/HarbormasterBuildableTransaction.php
View file

@ -3,7 +3,6 @@
final class HarbormasterBuildableTransaction final class HarbormasterBuildableTransaction
extends PhabricatorApplicationTransaction { extends PhabricatorApplicationTransaction {
const TYPE_CREATE = 'harbormaster:buildable:create';
const TYPE_COMMAND = 'harbormaster:buildable:command'; const TYPE_COMMAND = 'harbormaster:buildable:command';
public function getApplicationName() { public function getApplicationName() {
@ -21,10 +20,6 @@ final class HarbormasterBuildableTransaction
$new = $this->getNewValue(); $new = $this->getNewValue();
switch ($this->getTransactionType()) { switch ($this->getTransactionType()) {
case self::TYPE_CREATE:
return pht(
'%s created this buildable.',
$this->renderHandleLink($author_phid));
case self::TYPE_COMMAND: case self::TYPE_COMMAND:
switch ($new) { switch ($new) {
case HarbormasterBuildMessageRestartTransaction::MESSAGETYPE: case HarbormasterBuildMessageRestartTransaction::MESSAGETYPE:
@ -55,8 +50,6 @@ final class HarbormasterBuildableTransaction
$new = $this->getNewValue(); $new = $this->getNewValue();
switch ($this->getTransactionType()) { switch ($this->getTransactionType()) {
case self::TYPE_CREATE:
return 'fa-plus';
case self::TYPE_COMMAND: case self::TYPE_COMMAND:
switch ($new) { switch ($new) {
case HarbormasterBuildMessageRestartTransaction::MESSAGETYPE: case HarbormasterBuildMessageRestartTransaction::MESSAGETYPE:
@ -80,8 +73,6 @@ final class HarbormasterBuildableTransaction
$new = $this->getNewValue(); $new = $this->getNewValue();
switch ($this->getTransactionType()) { switch ($this->getTransactionType()) {
case self::TYPE_CREATE:
return 'green';
case self::TYPE_COMMAND: case self::TYPE_COMMAND:
switch ($new) { switch ($new) {
case HarbormasterBuildMessagePauseTransaction::MESSAGETYPE: case HarbormasterBuildMessagePauseTransaction::MESSAGETYPE: