Page MenuHomePhabricator

Update Phriction for PHUIDocumentViewPro
ClosedPublic

Authored by chad on Nov 4 2015, 2:27 AM.
Tags
None
Referenced Files
F14357428: D14399.diff
Fri, Dec 20, 4:45 AM
F14351129: D14399.id34785.diff
Thu, Dec 19, 7:17 AM
Unknown Object (File)
Wed, Dec 11, 5:34 AM
Unknown Object (File)
Tue, Dec 10, 11:51 PM
Unknown Object (File)
Sun, Dec 8, 1:25 AM
Unknown Object (File)
Nov 19 2024, 4:12 PM
Unknown Object (File)
Nov 14 2024, 6:28 AM
Unknown Object (File)
Nov 14 2024, 6:28 AM
Subscribers

Details

Summary

Moves Phriction to use PHUIDocumentViewPro

Test Plan

Read lots of documents, tablet, mobile, and desktop. Check ToC, non ToC, Edit a Maniphest Task, New Phriction Document, edit Phriction Document.

Diff Detail

Repository
rP Phabricator
Branch
phriction-pro
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8971
Build 10532: Run Core Tests
Build 10531: arc lint + arc unit

Event Timeline

chad retitled this revision from to Update Phriction for PHUIDocumentViewPro.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
src/applications/phriction/controller/PhrictionDocumentController.php
467

Having issues getting this to kick back the Table of Contents... but I pulled it from PhrictionContent. What's the best place to get the ToC as a separate entity?

src/applications/phriction/controller/PhrictionDocumentController.php
468

The $engine you're using here is empty, while the $engine in the original is one that has been used to process the text.

If you do a purely standalone generation of the TOC, you'll have to do the work for generating it twice -- once to generate the body, then once to regenerate it to get the TOC.

Easiest fix for now at least is probably making didMarkupText() just have a side effect like:

$this->renderedTableOfContents = ...

...then add a method like:

public function getRenderedTableOfContents() {
  if ($this->renderedTableOfContents === null) {
    throw new PhutiIInalidStateException('didMarkupText');
  }
  return $this-> renderedTableOfContents;
}

...to fetch it. I think that will work until I can clean up Remarkup more generally.

chad planned changes to this revision.Nov 5 2015, 5:34 PM
  • Toc Renders
  • Add DOCUMENT to PHUIRemarkupPreviewView
chad edited the test plan for this revision. (Show Details)
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/phriction/controller/PhrictionDocumentController.php
90

(Weird indent?)

src/applications/phriction/storage/PhrictionContent.php
121

(Weird indent?)

This revision is now accepted and ready to land.Nov 22 2015, 9:03 PM
This revision was automatically updated to reflect the committed changes.
chad marked 2 inline comments as done.