Page MenuHomePhabricator

Make rendered table of contents private in Phriction
ClosedPublic

Authored by epriestley on Nov 22 2015, 9:17 PM.
Tags
None
Referenced Files
F18093722: D14538.id35166.diff
Thu, Aug 7, 8:09 PM
F18088081: D14538.id35167.diff
Wed, Aug 6, 6:45 AM
F17927012: D14538.id.diff
Wed, Jul 30, 1:59 PM
F17906691: D14538.diff
Tue, Jul 29, 12:44 PM
Unknown Object (File)
Jun 26 2025, 3:49 AM
Unknown Object (File)
Jun 19 2025, 4:32 PM
Unknown Object (File)
May 18 2025, 4:11 AM
Unknown Object (File)
May 8 2025, 12:41 AM
Subscribers
None

Details

Summary

Oops, I missed this -- when properties are protected, Lisk assumes they database properties which should be stored and read from the database. To make Lisk ignore a property, make it private.

Test Plan

Should fix this:

Screen Shot 2015-11-22 at 1.15.56 PM.png (877×1 px, 108 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make rendered table of contents private in Phriction.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

Repro for error is to edit a Phriction page.

chad edited edge metadata.
This revision is now accepted and ready to land.Nov 22 2015, 9:17 PM
This revision was automatically updated to reflect the committed changes.

oh haha, I guess I didn't re-test saving a document.