Page MenuHomePhabricator
Differential D19095

Mostly use PhrictionContentQuery to load PhrictionContent objects
ClosedPublic
Actions

Authored by epriestley on Feb 15 2018, 1:33 PM.
Tags
None
Referenced Files
F15387887: D19095.id45775.diff
Sat, Mar 15, 3:04 AM
F15342052: D19095.id45775.diff
Sun, Mar 9, 9:02 PM
F15342051: D19095.id45761.diff
Sun, Mar 9, 9:02 PM
F15342049: D19095.id.diff
Sun, Mar 9, 9:02 PM
F15320904: D19095.id45775.diff
Thu, Mar 6, 10:29 PM
Unknown Object (File)
Thu, Feb 27, 10:09 PM
Unknown Object (File)
Sun, Feb 23, 7:06 PM
Unknown Object (File)
Feb 9 2025, 5:27 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13077: Plans: Phriction modern "v3" APIs, Drafts
Commits
rPf742d00c2894: Mostly use PhrictionContentQuery to load PhrictionContent objects
Summary

Depends on D19094. Ref T13077. Use modern infrastructure to perform these loads. I left a couple of calls in the older API methods unconverted.

Test Plan

Viewed documents. Viewed older versions. Viewed diffs. Did revert edits to older versions.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 15 2018, 1:33 PM
Harbormaster completed remote builds in B19565: Diff 45761.Feb 15 2018, 1:35 PM
epriestley requested review of this revision.Feb 15 2018, 1:35 PM
epriestley added a child revision: D19096: Make PhrictionContent "description" non-nullable.Feb 15 2018, 1:50 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 16 2018, 1:44 AM
Closed by commit rPf742d00c2894: Mostly use PhrictionContentQuery to load PhrictionContent objects (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 45775

View Options

src/applications/phriction/controller/PhrictionDiffController.php

Loading...
View Options

src/applications/phriction/controller/PhrictionDocumentController.php

Loading...
View Options

src/applications/phriction/controller/PhrictionEditController.php

Loading...
View Options

src/applications/phriction/query/PhrictionContentQuery.php

Loading...
View Options

src/applications/phriction/query/PhrictionDocumentQuery.php

Loading...
View Options

src/applications/phriction/storage/PhrictionDocument.php

Loading...
Phabricator · Built by Phacility