Page MenuHomePhabricator

Change Pholio editor access modifier
ClosedPublic

Authored by amckinley on May 17 2017, 11:15 PM.
Tags
None
Referenced Files
F18391903: D17946.id.diff
Fri, Aug 29, 7:53 AM
F18380709: D17946.diff
Thu, Aug 28, 6:11 PM
F18139192: D17946.id43161.diff
Thu, Aug 14, 2:49 AM
F18106110: D17946.id43162.diff
Aug 10 2025, 9:17 PM
F17901953: D17946.id.diff
Jul 29 2025, 10:03 AM
F17876681: D17946.diff
Jul 28 2025, 7:19 PM
Unknown Object (File)
Jun 18 2025, 9:10 AM
Unknown Object (File)
Jun 17 2025, 7:03 PM
Subscribers

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
Lint
Lint Not Applicable
Unit
Tests Not Applicable