Page MenuHomePhabricator

Migrate Pholio image description and mock status to modular transactions
ClosedPublic

Authored by amckinley on May 11 2017, 12:05 AM.

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
Branch
pholio-images-modular
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16900
Build 22577: Run Core Tests
Build 22576: arc lint + arc unit

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.