Page MenuHomePhabricator

Multiple indicators of status of transcluded page
Closed, ResolvedPublic

Description

Author: beau

Description:
When I view a page in main namespace which transcludes from Page: namespace I can see status of transcluded pages under the title. When I edit that page, there are two quality indicators, when I preview my edit, there are three. There is no need for such duplication.


Version: unspecified
Severity: normal

Details

Reference
bz23363

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:04 PM
bzimport added a project: ProofreadPage.
bzimport set Reference to bz23363.

beau wrote:

The patch removes incorrect OutputPageBeforeHTML hook

I have created a patch, which removes OutputPageBeforeHTML hook (it gets called every time addParserOutput method is used).

The status of transcluded pages should be added in the BeforePageDisplay hook.

Attached:

thomasV1 wrote:

Thanks for the patch. However, this is going to make the hook function return without calling pr_prepareArticle; it is better to setup the subtitle in pr_prepareArticle.