Phuedx has uploaded a new change for review.

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

Change subject: Clear extension data in ParserClearState handler
......................................................................

Clear extension data in ParserClearState handler

While the related parser function sets the accumulated related pages as
extension data on the parser output
(see RelatedArticles\Hooks::onFuncRelated). The ParserClearState
handler, however, sets the empty list as a property, which is then
stored in the DB.

Changes:

* Make Related\Articles\Hooks::onParserClearState use
  ParserOutput#setExtensionData, mirroring ::onFuncRelated

Bug: T115698
Change-Id: I3deaf1e8ee78944250c3f26315ee2450b444a035
---
M includes/Hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/RelatedArticles 
refs/changes/36/246836/1

diff --git a/includes/Hooks.php b/includes/Hooks.php
index bcbbdac..fd1ec41 100644
--- a/includes/Hooks.php
+++ b/includes/Hooks.php
@@ -74,7 +74,7 @@
         * @return boolean Always <code>true</code>
         */
        public static function onParserClearState( Parser &$parser ) {
-               $parser->getOutput()->setProperty( 'RelatedArticles', array() );
+               $parser->getOutput()->setExtensionData( 'RelatedArticles', 
array() );
 
                return true;
        }

-- 
To view, visit https://gerrit.wikimedia.org/r/246836
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3deaf1e8ee78944250c3f26315ee2450b444a035
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/RelatedArticles
Gerrit-Branch: master
Gerrit-Owner: Phuedx <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to