Page MenuHomePhabricator

Regression: __NOEDITSECTION__ is ignored when the page is read from the parser cache
Closed, ResolvedPublic

Description

Patch to have ParserCache check for NOEDITSECTION

After the fix for bug 31445, NOEDITSECTION is now ignored when the page is loaded from the parser cache.

The problem seems to be that ParserCache's get() is not taking into account the possibility of NOEDITSECTION when calling ParserOutput's setEditSectionTokens().

An easy patch is attached, although it may not be the best way to do it.


Version: 1.20.x
Severity: normal

Attached:

Details

Reference
bz31647
ReferenceSource BranchDest BranchAuthorTitle
repos/releng/release!69work/thcipriani/dzahn-windowmainthciprianiadd a Friday (omg) window for Gitlab version upgrades
Customize query in GitLab

Event Timeline

bzimport raised the priority of this task from to Unbreak Now!.Nov 21 2014, 11:54 PM
bzimport set Reference to bz31647.
bzimport added a subscriber: Unknown Object (MLST).

I see. the parser, which is the entity knowing about NOEDITSECTION, is no longer run there, so it doesn't come to decide on it and the default (ie. without taking NOEDITSECTION into account) is used. Still, I don't think that logic should be in ParserCache.

Should be fixed in r100756.