Page MenuHomePhabricator
Differential D17868

Migrate Pholio image description and mock status to modular transactions
ClosedPublic
Actions

Authored by amckinley on May 11 2017, 12:05 AM.
Tags
None
Referenced Files
F14103755: D17868.diff
Tue, Nov 26, 10:43 PM
Unknown Object (File)
Sun, Nov 24, 4:56 AM
Unknown Object (File)
Sat, Nov 23, 10:37 AM
Unknown Object (File)
Fri, Nov 22, 12:57 AM
Unknown Object (File)
Sun, Nov 17, 11:38 PM
Unknown Object (File)
Thu, Nov 14, 9:07 PM
Unknown Object (File)
Wed, Nov 13, 10:22 PM
Unknown Object (File)
Sat, Nov 9, 7:06 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rP74d86f09023c: Migrate Pholio image description and mock status to modular transactions
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

amckinley created this revision.May 11 2017, 12:05 AM
Herald added a subscriber: epriestley. · View Herald TranscriptMay 11 2017, 12:05 AM
Harbormaster completed remote builds in B16900: Diff 42974.May 11 2017, 12:07 AM
epriestley accepted this revision.May 11 2017, 4:33 PM
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
amckinley updated this revision to Diff 42988.May 12 2017, 11:03 PM
  • switch to just using head
Herald added a subscriber: Korvin. · View Herald TranscriptMay 12 2017, 11:03 PM
Harbormaster completed remote builds in B16912: Diff 42988.May 12 2017, 11:04 PM
Closed by commit rP74d86f09023c: Migrate Pholio image description and mock status to modular transactions (authored by amckinley, committed by amckinley). · Explain WhyMay 12 2017, 11:07 PM
This revision was automatically updated to reflect the committed changes.
amckinley mentioned this in D17889: Fix breakage of Pholio.May 15 2017, 8:25 PM
amckinley mentioned this in rP78544334cd69: Fix breakage of Pholio.May 15 2017, 8:30 PM
epriestley mentioned this in rP73ed316c5d2e: (stable) Fix breakage of Pholio.May 16 2017, 12:52 PM

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
src/
4 lines--
applications/
pholio/
controller/
2 lines4%
Loading...
2 lines0%
Loading...
editor/
35 lines0%
Loading...
storage/
73 lines1%
Loading...
xaction/
PholioImageDescriptionTransaction.php
PholioImageNameTransaction.php
55 lines5%
Loading...
10 lines4%
Loading...
66 lines6%
Loading...

Diff 42974

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/pholio/controller/PholioMockArchiveController.php

Loading...
View Options

src/applications/pholio/controller/PholioMockEditController.php

Loading...
View Options

src/applications/pholio/editor/PholioMockEditor.php

Loading...
View Options

src/applications/pholio/storage/PholioTransaction.php

Loading...
View Options

src/applications/pholio/xaction/PholioImageDescriptionTransaction.php

Loading...
View Options

src/applications/pholio/xaction/PholioImageNameTransaction.php

Loading...
View Options

src/applications/pholio/xaction/PholioMockStatusTransaction.php

Loading...
Phabricator · Built by Phacility