Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F14678099
D12281.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Size
21 KB
Referenced Files
None
Subscribers
None
D12281.diff
View Options
diff --git a/resources/sql/autopatches/20141222.maniphestprojtxn.php b/resources/sql/autopatches/20141222.maniphestprojtxn.php
--- a/resources/sql/autopatches/20141222.maniphestprojtxn.php
+++ b/resources/sql/autopatches/20141222.maniphestprojtxn.php
@@ -6,7 +6,8 @@
echo "Converting Maniphest project transactions to modern EDGE ".
"transactions...\n";
$metadata = array(
- 'edge:type' => PhabricatorProjectObjectHasProjectEdgeType::EDGECONST,);
+ 'edge:type' => PhabricatorProjectObjectHasProjectEdgeType::EDGECONST,
+);
foreach (new LiskMigrationIterator($table) as $txn) {
// ManiphestTransaction::TYPE_PROJECTS
if ($txn->getTransactionType() == 'projects') {
diff --git a/src/applications/auth/management/PhabricatorAuthManagementTrustOAuthClientWorkflow.php b/src/applications/auth/management/PhabricatorAuthManagementTrustOAuthClientWorkflow.php
--- a/src/applications/auth/management/PhabricatorAuthManagementTrustOAuthClientWorkflow.php
+++ b/src/applications/auth/management/PhabricatorAuthManagementTrustOAuthClientWorkflow.php
@@ -18,7 +18,8 @@
'name' => 'id',
'param' => 'id',
'help' => pht('The id of the OAuth client.'),
- ),));
+ ),
+ ));
}
public function execute(PhutilArgumentParser $args) {
diff --git a/src/applications/auth/management/PhabricatorAuthManagementUntrustOAuthClientWorkflow.php b/src/applications/auth/management/PhabricatorAuthManagementUntrustOAuthClientWorkflow.php
--- a/src/applications/auth/management/PhabricatorAuthManagementUntrustOAuthClientWorkflow.php
+++ b/src/applications/auth/management/PhabricatorAuthManagementUntrustOAuthClientWorkflow.php
@@ -18,7 +18,8 @@
'name' => 'id',
'param' => 'id',
'help' => pht('The id of the OAuth client.'),
- ),));
+ ),
+ ));
}
public function execute(PhutilArgumentParser $args) {
diff --git a/src/applications/base/controller/PhabricatorController.php b/src/applications/base/controller/PhabricatorController.php
--- a/src/applications/base/controller/PhabricatorController.php
+++ b/src/applications/base/controller/PhabricatorController.php
@@ -256,7 +256,8 @@
if ($must_sign_docs) {
$controller = new LegalpadDocumentSignController();
$this->getRequest()->setURIMap(array(
- 'id' => head($must_sign_docs)->getID(),));
+ 'id' => head($must_sign_docs)->getID(),
+ ));
$this->setCurrentApplication($legalpad);
return $this->delegateToController($controller);
} else {
diff --git a/src/applications/conduit/controller/PhabricatorConduitAPIController.php b/src/applications/conduit/controller/PhabricatorConduitAPIController.php
--- a/src/applications/conduit/controller/PhabricatorConduitAPIController.php
+++ b/src/applications/conduit/controller/PhabricatorConduitAPIController.php
@@ -283,7 +283,8 @@
pht(
'This request originates from outside of the Phabricator '.
'cluster address range. Requests signed with trusted '.
- 'device keys must originate from within the cluster.'),);
+ 'device keys must originate from within the cluster.'),
+ );
}
$user = PhabricatorUser::getOmnipotentUser();
@@ -384,7 +385,8 @@
pht(
'This request originates from outside of the Phabricator '.
'cluster address range. Requests signed with cluster API '.
- 'tokens must originate from within the cluster.'),);
+ 'tokens must originate from within the cluster.'),
+ );
}
// Flag this as an intracluster request.
diff --git a/src/applications/config/check/PhabricatorBinariesSetupCheck.php b/src/applications/config/check/PhabricatorBinariesSetupCheck.php
--- a/src/applications/config/check/PhabricatorBinariesSetupCheck.php
+++ b/src/applications/config/check/PhabricatorBinariesSetupCheck.php
@@ -107,20 +107,25 @@
case PhabricatorRepositoryType::REPOSITORY_TYPE_SVN:
$minimum_version = '1.5';
$bad_versions = array(
- '1.7.1' => pht('This version of Subversion has a bug where '.
- '"svn diff -c N" does not work for files added '.
- 'in rN (Subversion issue #2873), fixed in 1.7.2.'),);
+ '1.7.1' => pht(
+ 'This version of Subversion has a bug where `%s` does not work '.
+ 'for files added in rN (Subversion issue #2873), fixed in 1.7.2.',
+ 'svn diff -c N'),
+ );
list($err, $stdout, $stderr) = exec_manual('svn --version --quiet');
$version = trim($stdout);
break;
case PhabricatorRepositoryType::REPOSITORY_TYPE_MERCURIAL:
$minimum_version = '1.9';
$bad_versions = array(
- '2.1' => pht('This version of Mercurial returns a bad exit code '.
- 'after a successful pull.'),
- '2.2' => pht('This version of Mercurial has a significant memory '.
- 'leak, fixed in 2.2.1. Pushing fails with this '.
- 'version as well; see T3046#54922.'),);
+ '2.1' => pht(
+ 'This version of Mercurial returns a bad exit code '.
+ 'after a successful pull.'),
+ '2.2' => pht(
+ 'This version of Mercurial has a significant memory leak, fixed '.
+ 'in 2.2.1. Pushing fails with this version as well; see %s.',
+ 'T3046#54922'),
+ );
$version = PhabricatorRepositoryVersion::getMercurialVersion();
break;
}
diff --git a/src/applications/conpherence/__tests__/ConpherenceTestCase.php b/src/applications/conpherence/__tests__/ConpherenceTestCase.php
--- a/src/applications/conpherence/__tests__/ConpherenceTestCase.php
+++ b/src/applications/conpherence/__tests__/ConpherenceTestCase.php
@@ -9,7 +9,8 @@
$xactions = array(id(new ConpherenceTransaction())
->setTransactionType(ConpherenceTransactionType::TYPE_PARTICIPANTS)
- ->setNewValue(array('+' => $participant_phids)),);
+ ->setNewValue(array('+' => $participant_phids)),
+ );
$editor = id(new ConpherenceEditor())
->setActor($actor)
->setContentSource(PhabricatorContentSource::newConsoleSource())
@@ -24,7 +25,8 @@
$xactions = array(id(new ConpherenceTransaction())
->setTransactionType(ConpherenceTransactionType::TYPE_PARTICIPANTS)
- ->setNewValue(array('-' => $participant_phids)),);
+ ->setNewValue(array('-' => $participant_phids)),
+ );
$editor = id(new ConpherenceEditor())
->setActor($actor)
->setContentSource(PhabricatorContentSource::newConsoleSource())
diff --git a/src/applications/conpherence/controller/ConpherenceListController.php b/src/applications/conpherence/controller/ConpherenceListController.php
--- a/src/applications/conpherence/controller/ConpherenceListController.php
+++ b/src/applications/conpherence/controller/ConpherenceListController.php
@@ -125,7 +125,8 @@
->execute();
return array(
- 'all_participation' => $all_participation,);
+ 'all_participation' => $all_participation,
+ );
}
private function loadConpherenceThreadData($participation) {
diff --git a/src/applications/conpherence/query/ConpherenceThreadSearchEngine.php b/src/applications/conpherence/query/ConpherenceThreadSearchEngine.php
--- a/src/applications/conpherence/query/ConpherenceThreadSearchEngine.php
+++ b/src/applications/conpherence/query/ConpherenceThreadSearchEngine.php
@@ -178,7 +178,8 @@
return array(
'rooms' => pht('Rooms'),
'messages' => pht('Messages'),
- 'both' => pht('Both'),);
+ 'both' => pht('Both'),
+ );
}
}
diff --git a/src/applications/conpherence/storage/ConpherenceThread.php b/src/applications/conpherence/storage/ConpherenceThread.php
--- a/src/applications/conpherence/storage/ConpherenceThread.php
+++ b/src/applications/conpherence/storage/ConpherenceThread.php
@@ -60,7 +60,8 @@
),
self::CONFIG_KEY_SCHEMA => array(
'key_room' => array(
- 'columns' => array('isRoom', 'dateModified'),),
+ 'columns' => array('isRoom', 'dateModified'),
+ ),
'key_phid' => null,
'phid' => array(
'columns' => array('phid'),
diff --git a/src/applications/conpherence/view/ConpherenceDurableColumnView.php b/src/applications/conpherence/view/ConpherenceDurableColumnView.php
--- a/src/applications/conpherence/view/ConpherenceDurableColumnView.php
+++ b/src/applications/conpherence/view/ConpherenceDurableColumnView.php
@@ -375,8 +375,8 @@
),
$header),
$settings_button,
- $settings_menu,));
-
+ $settings_menu,
+ ));
}
private function getHeaderActionsConfig(ConpherenceThread $conpherence) {
@@ -418,7 +418,8 @@
'href' => '#',
'icon' => 'fa-times',
'key' => 'hide_column',
- ),);
+ ),
+ );
}
private function buildTransactions() {
@@ -441,7 +442,8 @@
'class' => 'button grey',
'sigil' => 'workflow',
),
- pht('Send a Message')),);
+ pht('Send a Message')),
+ );
}
$data = ConpherenceTransactionView::renderTransactions(
@@ -501,7 +503,8 @@
'type' => 'hidden',
'name' => 'action',
'value' => ConpherenceUpdateActions::MESSAGE,
- )),));
+ )),
+ ));
}
private function buildStatusText() {
diff --git a/src/applications/differential/application/PhabricatorDifferentialApplication.php b/src/applications/differential/application/PhabricatorDifferentialApplication.php
--- a/src/applications/differential/application/PhabricatorDifferentialApplication.php
+++ b/src/applications/differential/application/PhabricatorDifferentialApplication.php
@@ -179,7 +179,8 @@
phutil_tag(
'a',
array(
- 'href' => $this->getInboundEmailSupportLink(),),
+ 'href' => $this->getInboundEmailSupportLink(),
+ ),
pht('Learn More')));
}
diff --git a/src/applications/differential/conduit/DifferentialCreateDiffConduitAPIMethod.php b/src/applications/differential/conduit/DifferentialCreateDiffConduitAPIMethod.php
--- a/src/applications/differential/conduit/DifferentialCreateDiffConduitAPIMethod.php
+++ b/src/applications/differential/conduit/DifferentialCreateDiffConduitAPIMethod.php
@@ -163,11 +163,13 @@
$request->getValue('sourceControlBaseRevision'),
'arcanistProjectPHID' => $project_phid,
'lintStatus' => $lint_status,
- 'unitStatus' => $unit_status,);
+ 'unitStatus' => $unit_status,
+ );
$xactions = array(id(new DifferentialTransaction())
->setTransactionType(DifferentialDiffTransaction::TYPE_DIFF_CREATE)
- ->setNewValue($diff_data_dict),);
+ ->setNewValue($diff_data_dict),
+ );
id(new DifferentialDiffEditor())
->setActor($viewer)
diff --git a/src/applications/differential/conduit/DifferentialCreateRawDiffConduitAPIMethod.php b/src/applications/differential/conduit/DifferentialCreateRawDiffConduitAPIMethod.php
--- a/src/applications/differential/conduit/DifferentialCreateRawDiffConduitAPIMethod.php
+++ b/src/applications/differential/conduit/DifferentialCreateRawDiffConduitAPIMethod.php
@@ -53,11 +53,13 @@
'authorPHID' => $viewer->getPHID(),
'repositoryPHID' => $repository_phid,
'lintStatus' => DifferentialLintStatus::LINT_SKIP,
- 'unitStatus' => DifferentialUnitStatus::UNIT_SKIP,);
+ 'unitStatus' => DifferentialUnitStatus::UNIT_SKIP,
+ );
$xactions = array(id(new DifferentialTransaction())
->setTransactionType(DifferentialDiffTransaction::TYPE_DIFF_CREATE)
- ->setNewValue($diff_data_dict),);
+ ->setNewValue($diff_data_dict),
+ );
if ($request->getValue('viewPolicy')) {
$xactions[] = id(new DifferentialTransaction())
diff --git a/src/applications/differential/controller/DifferentialRevisionCloseDetailsController.php b/src/applications/differential/controller/DifferentialRevisionCloseDetailsController.php
--- a/src/applications/differential/controller/DifferentialRevisionCloseDetailsController.php
+++ b/src/applications/differential/controller/DifferentialRevisionCloseDetailsController.php
@@ -62,7 +62,8 @@
phutil_tag(
'a',
array(
- 'href' => $obj_handle->getURI(),),
+ 'href' => $obj_handle->getURI(),
+ ),
$obj_handle->getName()));
} else if ($revision_match_data['foundURI']) {
$body_why[] = pht(
@@ -96,7 +97,8 @@
$diff_link = phutil_tag(
'a',
array(
- 'href' => $obj_handle->getURI(),),
+ 'href' => $obj_handle->getURI(),
+ ),
$obj_handle->getName());
$body_why = pht(
'This commit and the active diff of %s had the same %s hash '.
diff --git a/src/applications/differential/mail/DifferentialCreateMailReceiver.php b/src/applications/differential/mail/DifferentialCreateMailReceiver.php
--- a/src/applications/differential/mail/DifferentialCreateMailReceiver.php
+++ b/src/applications/differential/mail/DifferentialCreateMailReceiver.php
@@ -59,7 +59,8 @@
$call = new ConduitCall(
'differential.createrawdiff',
array(
- 'diff' => $body,));
+ 'diff' => $body,
+ ));
$call->setUser($sender);
try {
$result = $call->execute();
diff --git a/src/applications/files/application/PhabricatorFilesApplication.php b/src/applications/files/application/PhabricatorFilesApplication.php
--- a/src/applications/files/application/PhabricatorFilesApplication.php
+++ b/src/applications/files/application/PhabricatorFilesApplication.php
@@ -51,7 +51,8 @@
phutil_tag(
'a',
array(
- 'href' => $this->getInboundEmailSupportLink(),),
+ 'href' => $this->getInboundEmailSupportLink(),
+ ),
pht('Learn More')));
}
diff --git a/src/applications/maniphest/application/PhabricatorManiphestApplication.php b/src/applications/maniphest/application/PhabricatorManiphestApplication.php
--- a/src/applications/maniphest/application/PhabricatorManiphestApplication.php
+++ b/src/applications/maniphest/application/PhabricatorManiphestApplication.php
@@ -122,7 +122,8 @@
phutil_tag(
'a',
array(
- 'href' => $this->getInboundEmailSupportLink(),),
+ 'href' => $this->getInboundEmailSupportLink(),
+ ),
pht('Learn More')));
}
diff --git a/src/applications/paste/application/PhabricatorPasteApplication.php b/src/applications/paste/application/PhabricatorPasteApplication.php
--- a/src/applications/paste/application/PhabricatorPasteApplication.php
+++ b/src/applications/paste/application/PhabricatorPasteApplication.php
@@ -55,7 +55,8 @@
phutil_tag(
'a',
array(
- 'href' => $this->getInboundEmailSupportLink(),),
+ 'href' => $this->getInboundEmailSupportLink(),
+ ),
pht('Learn More')));
}
diff --git a/src/applications/phriction/controller/PhrictionDocumentController.php b/src/applications/phriction/controller/PhrictionDocumentController.php
--- a/src/applications/phriction/controller/PhrictionDocumentController.php
+++ b/src/applications/phriction/controller/PhrictionDocumentController.php
@@ -376,7 +376,8 @@
$child_dict = array(
'slug' => $child->getSlug(),
'depth' => $child->getDepth(),
- 'title' => $child->getContent()->getTitle(),);
+ 'title' => $child->getContent()->getTitle(),
+ );
if ($child->getDepth() == $d_child) {
$children_dicts[] = $child_dict;
continue;
diff --git a/src/applications/phriction/editor/PhrictionTransactionEditor.php b/src/applications/phriction/editor/PhrictionTransactionEditor.php
--- a/src/applications/phriction/editor/PhrictionTransactionEditor.php
+++ b/src/applications/phriction/editor/PhrictionTransactionEditor.php
@@ -132,7 +132,8 @@
'id' => $document->getID(),
'phid' => $document->getPHID(),
'content' => $document->getContent()->getContent(),
- 'title' => $document->getContent()->getTitle(),);
+ 'title' => $document->getContent()->getTitle(),
+ );
return $dict;
case PhrictionTransaction::TYPE_MOVE_AWAY:
$document = $xaction->getNewValue();
@@ -140,7 +141,8 @@
'id' => $document->getID(),
'phid' => $document->getPHID(),
'content' => $document->getContent()->getContent(),
- 'title' => $document->getContent()->getTitle(),);
+ 'title' => $document->getContent()->getTitle(),
+ );
return $dict;
}
}
diff --git a/src/applications/policy/config/PhabricatorPolicyConfigOptions.php b/src/applications/policy/config/PhabricatorPolicyConfigOptions.php
--- a/src/applications/policy/config/PhabricatorPolicyConfigOptions.php
+++ b/src/applications/policy/config/PhabricatorPolicyConfigOptions.php
@@ -22,7 +22,8 @@
public function getOptions() {
$policy_locked_type = 'custom:PolicyLockOptionType';
$policy_locked_example = array(
- 'people.create.users' => 'admin',);
+ 'people.create.users' => 'admin',
+ );
$json = new PhutilJSON();
$policy_locked_example = $json->encodeFormatted($policy_locked_example);
diff --git a/src/applications/transactions/controller/PhabricatorApplicationTransactionShowOlderController.php b/src/applications/transactions/controller/PhabricatorApplicationTransactionShowOlderController.php
--- a/src/applications/transactions/controller/PhabricatorApplicationTransactionShowOlderController.php
+++ b/src/applications/transactions/controller/PhabricatorApplicationTransactionShowOlderController.php
@@ -49,7 +49,8 @@
return id(new AphrontAjaxResponse())
->setContent(array(
- 'timeline' => hsprintf('%s', $events),));
+ 'timeline' => hsprintf('%s', $events),
+ ));
}
}
diff --git a/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php b/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php
--- a/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php
+++ b/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php
@@ -39,7 +39,8 @@
),
array(
$icon,
- $group,));
+ $group,
+ ));
$rows[] = phutil_tag('tr', array(), $header);
foreach ($items as $item) {
diff --git a/src/view/control/AphrontTableView.php b/src/view/control/AphrontTableView.php
--- a/src/view/control/AphrontTableView.php
+++ b/src/view/control/AphrontTableView.php
@@ -305,7 +305,8 @@
$html = phutil_tag(
'table',
array(
- 'class' => implode(' ', $classes),),
+ 'class' => implode(' ', $classes),
+ ),
$table);
return phutil_tag_div('aphront-table-wrap', $html);
}
diff --git a/src/view/form/control/AphrontFormControl.php b/src/view/form/control/AphrontFormControl.php
--- a/src/view/form/control/AphrontFormControl.php
+++ b/src/view/form/control/AphrontFormControl.php
@@ -212,7 +212,8 @@
),
array(
$this->getLabel(),
- $error,));
+ $error,
+ ));
} else {
$label = null;
$custom_class .= ' aphront-form-control-nolabel';
diff --git a/src/view/layout/PhabricatorTransactionView.php b/src/view/layout/PhabricatorTransactionView.php
--- a/src/view/layout/PhabricatorTransactionView.php
+++ b/src/view/layout/PhabricatorTransactionView.php
@@ -81,7 +81,8 @@
array(
$image,
phutil_tag_div('phabricator-transaction-detail grouped',
- array($header, $content)),));
+ array($header, $content)),
+ ));
}
@@ -129,7 +130,8 @@
return phutil_tag(
'span',
array(
- 'class' => 'phabricator-transaction-info',),
+ 'class' => 'phabricator-transaction-info',
+ ),
$info);
}
diff --git a/src/view/phui/PHUIDocumentView.php b/src/view/phui/PHUIDocumentView.php
--- a/src/view/phui/PHUIDocumentView.php
+++ b/src/view/phui/PHUIDocumentView.php
@@ -174,7 +174,8 @@
'map' => array(
$this->actionListID => 'phabricator-action-list-toggle',
$icon_id => 'phuix-dropdown-open',
- ),);
+ ),
+ );
$mobile_menu = id(new PHUIButtonView())
->setTag('a')
->setText(pht('Actions'))
diff --git a/src/view/phui/PHUIObjectBoxView.php b/src/view/phui/PHUIObjectBoxView.php
--- a/src/view/phui/PHUIObjectBoxView.php
+++ b/src/view/phui/PHUIObjectBoxView.php
@@ -171,7 +171,8 @@
'map' => array(
$this->actionListID => 'phabricator-action-list-toggle',
$icon_id => 'phuix-dropdown-open',
- ),);
+ ),
+ );
$mobile_menu = id(new PHUIButtonView())
->setTag('a')
->setText(pht('Actions'))
diff --git a/src/view/phui/PHUIObjectItemView.php b/src/view/phui/PHUIObjectItemView.php
--- a/src/view/phui/PHUIObjectItemView.php
+++ b/src/view/phui/PHUIObjectItemView.php
@@ -581,7 +581,8 @@
),
array(
$icons,
- $bylines,));
+ $bylines,
+ ));
}
$table = phutil_tag(
diff --git a/src/view/phui/PHUITimelineEventView.php b/src/view/phui/PHUITimelineEventView.php
--- a/src/view/phui/PHUITimelineEventView.php
+++ b/src/view/phui/PHUITimelineEventView.php
@@ -435,7 +435,8 @@
'class' => 'phui-timeline-event-view '.
'phui-timeline-spacer '.
'phui-timeline-spacer-bold',
- '',));
+ '',
+ ));
}
return array(
@@ -453,7 +454,8 @@
'class' => implode(' ', $classes),
),
$content)),
- $major_event,);
+ $major_event,
+ );
}
private function renderExtra(array $events) {
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Mon, Jan 13, 9:03 PM (20 h, 11 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6990460
Default Alt Text
D12281.diff (21 KB)
Attached To
Mode
D12281: Fix some odd looking arrays
Attached
Detach File
Event Timeline
Log In to Comment