Page MenuHomePhabricator

Ensure edit pen visibility is conditional
Closed, ResolvedPublic

Description

So far, edit pen visibility is controlled through the proprietary <wb:sectionedit> tag (see Sectionedit component). Ensure that the solution implemented for T215788 keeps this a viable option.

Bildschirmfoto von 2019-02-13 22-46-03.png (1×1 px, 261 KB)

(dev system with https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/490369 checked out)

Event Timeline

Ohh, well spotted. It appears that the html blob we add to ParserOutput through setExtensionData does not pass through RepoHooks::onParserOutputPostCacheTransform.

Change 493419 had a related patch set uploaded (by Jakob; owner: Jakob):
[mediawiki/extensions/Wikibase@master] Ensure edit pen visibility is conditional

https://gerrit.wikimedia.org/r/493419

Change 494198 had a related patch set uploaded (by Jakob; owner: Jakob):
[mediawiki/extensions/Wikibase@master] Ensure edit pen visibility is conditional

https://gerrit.wikimedia.org/r/494198

Change 493419 abandoned by Jakob:
Ensure edit pen visibility is conditional

https://gerrit.wikimedia.org/r/493419

Change 494198 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Ensure edit pen visibility is conditional

https://gerrit.wikimedia.org/r/494198

@Pablo-WMDE, @Jakob_WMDE: All patches merged/abandoned and no active project tags assigned. If there is nothing left to do here, please feel free to resolve this task via the Add Action...Change Status dropdown. If there is work left, please add an active project tag. Thanks!