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
F19509186: D17868.diff
Sat, Jan 10, 4:15 PM
F19216239: D17868.diff
Wed, Dec 17, 7:02 AM
F19130711: D17868.diff
Dec 10 2025, 5:24 AM
F19031101: D17868.id.diff
Nov 24 2025, 11:10 PM
F19015152: D17868.diff
Nov 22 2025, 11:30 PM
F18994211: D17868.diff
Nov 19 2025, 10:44 AM
F18824752: D17868.id.diff
Oct 23 2025, 5:43 PM
F18816749: D17868.id42989.diff
Oct 21 2025, 9:08 AM
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