Page MenuHomePhabricator

Change Pholio editor access modifier
ClosedPublic

Authored by amckinley on May 17 2017, 11:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 11 2026, 7:06 AM
Unknown Object (File)
Mar 11 2026, 7:00 AM
Unknown Object (File)
Jan 29 2026, 11:22 AM
Unknown Object (File)
Dec 23 2025, 8:13 AM
Unknown Object (File)
Dec 22 2025, 5:59 PM
Unknown Object (File)
Nov 3 2025, 9:54 AM
Unknown Object (File)
Oct 14 2025, 8:12 PM
Unknown Object (File)
Oct 12 2025, 5:46 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