revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-01-10 14:51:06 +01:00
Code Issues Releases Wiki Activity

Change Pholio editor access modifier

Browse source 
Summary: Used by `PholioImageFileTransaction::mergeTransactions()`. I forgot to test adding multiple images to a Mock at the same time after migrating `mergeTransactions` over to the modular framework.

Test Plan: Added multiple images in a single transaction and didn't get an exception about accessing a protected function.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D17946
This commit is contained in:
Austin McKinley 2017-05-17 16:11:04 -07:00
parent 0e8f72a0d9
commit d12be0fc21
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
View file

@ -1743,7 +1743,7 @@ abstract class PhabricatorApplicationTransactionEditor
return array_values($result);
}
protected function mergePHIDOrEdgeTransactions(
public function mergePHIDOrEdgeTransactions(
PhabricatorApplicationTransaction $u,
PhabricatorApplicationTransaction $v) {