Page MenuHomePhabricator

Change Pholio editor access modifier
ClosedPublic

Authored by amckinley on May 17 2017, 11:15 PM.
Tags
None
Referenced Files
F19289267: D17946.id.diff
Tue, Dec 23, 8:13 AM
F19262771: D17946.diff
Mon, Dec 22, 5:59 PM
F18865026: D17946.id.diff
Nov 3 2025, 9:54 AM
F18786733: D17946.id43161.diff
Oct 14 2025, 8:12 PM
F18780599: D17946.diff
Oct 12 2025, 5:46 PM
F18736460: D17946.id43162.diff
Oct 1 2025, 6:01 AM
F18391903: D17946.id.diff
Aug 29 2025, 7:53 AM
F18380709: D17946.diff
Aug 28 2025, 6:11 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