Page MenuHomePhabricator

Migrate Pholio image description and mock status to modular transactions
ClosedPublic

Authored by amckinley on May 11 2017, 12:05 AM.
Tags
None
Referenced Files
F18824752: D17868.id.diff
Thu, Oct 23, 5:43 PM
F18816749: D17868.id42989.diff
Tue, Oct 21, 9:08 AM
F18745689: D17868.id42974.diff
Fri, Oct 3, 8:48 AM
F18627926: D17868.diff
Sep 16 2025, 5:08 AM
F18620510: D17868.diff
Sep 15 2025, 5:29 AM
F18617299: D17868.id.diff
Sep 14 2025, 8:51 PM
F18597763: D17868.id42988.diff
Sep 13 2025, 3:36 AM
F18594373: D17868.id42989.diff
Sep 12 2025, 5:59 PM
Subscribers

Details

Summary

Also removes more now-dead code from PholioTransaction.

Test Plan

opened and closed a bunch of mocks, edited a bunch of image descriptions

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley added inline comments.
src/applications/pholio/xaction/PholioImageDescriptionTransaction.php
72–73

I think this came from elsewhere, but head(array_values($x)) is equivalent to head($x).

This revision is now accepted and ready to land.May 11 2017, 4:33 PM
This revision was automatically updated to reflect the committed changes.