Page MenuHomePhabricator
Differential D19115

Provide the document content as a context object when rendering Phriction documents
ClosedPublic
Actions

Authored by epriestley on Feb 16 2018, 8:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 13, 6:11 PM
Unknown Object (File)
Sun, Feb 9, 5:30 AM
Unknown Object (File)
Sun, Feb 9, 5:29 AM
Unknown Object (File)
Sun, Feb 9, 5:29 AM
Unknown Object (File)
Wed, Jan 29, 7:15 AM
Unknown Object (File)
Tue, Jan 28, 11:01 AM
Unknown Object (File)
Sat, Jan 25, 3:23 PM
Unknown Object (File)
Sat, Jan 25, 3:23 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13077: Plans: Phriction modern "v3" APIs, Drafts
Commits
rPcb2f7106069b: Provide the document content as a context object when rendering Phriction…
Summary

Ref T13077. The context object wasn't being passed into the engine properly here, affecting relative link rendering in Phriction.

Test Plan

Viewed rendered Phriction documents with relative links, got clean renders.

Diff Detail

Repository
rP Phabricator
Branch
phriction15
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19609
Build 26546: Run Core Tests
Build 26545: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 16 2018, 8:41 PM
Harbormaster completed remote builds in B19609: Diff 45805.Feb 16 2018, 8:42 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 16 2018, 8:42 PM
epriestley requested review of this revision.
Closed by commit rPcb2f7106069b: Provide the document content as a context object when rendering Phriction… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
phriction/
storage/
1 line

Diff 45805

View Options

src/applications/phriction/storage/PhrictionContent.php

Loading...
Phabricator · Built by Phacility