Page MenuHomePhabricator

Change Pholio editor access modifier
ClosedPublic

Authored by amckinley on May 17 2017, 11:15 PM.

Details

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.

Diff Detail

Repository
rP Phabricator
Branch
remove-protected (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17077
Build 22837: Run Core Tests
Build 22836: arc lint + arc unit