mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/TemplateData
synced 2024-11-15 11:58:51 +00:00
6f362c6d8a
Follows-up 956367fb90.
* Fix typo.
* Remove <del>footgun</del><ins>dangerous return value</ins>
from this unabortable hook.
* Document the weird PageProps workaround.
This should be fixed in core.
Change-Id: I6b22e9c2112039e5703e6a62252f1909b15c8887
234 lines
8 KiB
PHP
234 lines
8 KiB
PHP
<?php
|
|
/**
|
|
* Hooks for TemplateData extension
|
|
*
|
|
* @file
|
|
* @ingroup Extensions
|
|
*/
|
|
|
|
class TemplateDataHooks {
|
|
/**
|
|
* Register parser hooks
|
|
* @param Parser &$parser
|
|
* @return bool
|
|
*/
|
|
public static function onParserFirstCallInit( &$parser ) {
|
|
$parser->setHook( 'templatedata', [ 'TemplateDataHooks', 'render' ] );
|
|
return true;
|
|
}
|
|
|
|
/**
|
|
* Register qunit unit tests
|
|
* @param array &$testModules
|
|
* @param ResourceLoader &$resourceLoader
|
|
* @return bool
|
|
*/
|
|
public static function onResourceLoaderTestModules(
|
|
array &$testModules,
|
|
ResourceLoader &$resourceLoader
|
|
) {
|
|
$testModules['qunit']['ext.templateData.test'] = [
|
|
'scripts' => [ 'tests/qunit/ext.templateData.tests.js' ],
|
|
'dependencies' => [ 'ext.templateDataGenerator.data' ],
|
|
'localBasePath' => dirname( __DIR__ ) ,
|
|
'remoteExtPath' => 'TemplateData',
|
|
];
|
|
return true;
|
|
}
|
|
|
|
/**
|
|
* Conditionally register the jquery.uls.data module, in case they've already been
|
|
* registered by the UniversalLanguageSelector extension or the VisualEditor extension.
|
|
*
|
|
* @param ResourceLoader &$resourceLoader
|
|
*/
|
|
public static function onResourceLoaderRegisterModules( ResourceLoader &$resourceLoader ) {
|
|
$resourceModules = $resourceLoader->getConfig()->get( 'ResourceModules' );
|
|
$name = 'jquery.uls.data';
|
|
if ( !isset( $resourceModules[$name] ) && !$resourceLoader->isModuleRegistered( $name ) ) {
|
|
$resourceLoader->register( [
|
|
'jquery.uls.data' => [
|
|
'localBasePath' => dirname( __DIR__ ),
|
|
'remoteExtPath' => 'TemplateData',
|
|
'scripts' => [
|
|
'lib/jquery.uls/src/jquery.uls.data.js',
|
|
'lib/jquery.uls/src/jquery.uls.data.utils.js',
|
|
],
|
|
'targets' => [ 'desktop', 'mobile' ],
|
|
]
|
|
] );
|
|
}
|
|
}
|
|
|
|
/**
|
|
* @param WikiPage &$page
|
|
* @param User &$user
|
|
* @param Content &$content
|
|
* @param string &$summary
|
|
* @param bool $minor
|
|
* @param bool|null $watchthis
|
|
* @param string $sectionanchor
|
|
* @param int &$flags
|
|
* @param Status &$status
|
|
* @return bool
|
|
*/
|
|
public static function onPageContentSave( WikiPage &$page, &$user, &$content, &$summary, $minor,
|
|
$watchthis, $sectionanchor, &$flags, &$status
|
|
) {
|
|
// The PageContentSave hook provides raw $text, but not $parser because at this stage
|
|
// the page is not actually parsed yet. Which means we can't know whether self::render()
|
|
// got a valid tag or not. Looking at $text directly is not a solution either as
|
|
// it may not be in the current page (it can be transcluded).
|
|
// Since there is no later hook that allows aborting the save and showing an error,
|
|
// we will have to trigger the parser ourselves.
|
|
// Fortunately this causes no overhead since the below (copied from WikiPage::doEditContent,
|
|
// right after this hook is ran) has guards that lazy-init and return early if called again
|
|
// later by the real WikiPage.
|
|
|
|
// Specify format the same way the API and EditPage do to avoid extra parsing
|
|
$format = $content->getContentHandler()->getDefaultFormat();
|
|
$editInfo = $page->prepareContentForEdit( $content, null, $user, $format );
|
|
|
|
$templateDataStatus = $editInfo->output->getExtensionData( 'TemplateDataStatus' );
|
|
if ( $templateDataStatus instanceof Status && !$templateDataStatus->isOK() ) {
|
|
// Abort edit, show error message from TemplateDataBlob::getStatus
|
|
$status->merge( $templateDataStatus );
|
|
return false;
|
|
}
|
|
return true;
|
|
}
|
|
|
|
/**
|
|
* Parser hook registering the GUI module only in edit pages.
|
|
*
|
|
* @param EditPage $editPage
|
|
* @param OutputPage $output
|
|
* @return bool
|
|
*/
|
|
public static function onEditPage( EditPage $editPage, OutputPage $output ) {
|
|
global $wgTemplateDataUseGUI;
|
|
if ( $wgTemplateDataUseGUI ) {
|
|
if ( $output->getTitle()->inNamespace( NS_TEMPLATE ) ) {
|
|
$output->addModules( 'ext.templateDataGenerator.editTemplatePage' );
|
|
}
|
|
}
|
|
return true;
|
|
}
|
|
|
|
/**
|
|
* Parser hook for <templatedata>.
|
|
* If there is any JSON provided, render the template documentation on the page.
|
|
*
|
|
* @param string $input The content of the tag.
|
|
* @param array $args The attributes of the tag.
|
|
* @param Parser $parser Parser instance available to render
|
|
* wikitext into html, or parser methods.
|
|
* @param PPFrame $frame Can be used to see what template parameters ("{{{1}}}", etc.)
|
|
* this hook was used with.
|
|
*
|
|
* @return string HTML to insert in the page.
|
|
*/
|
|
public static function render( $input, $args, Parser $parser, $frame ) {
|
|
$ti = TemplateDataBlob::newFromJSON( wfGetDB( DB_REPLICA ), $input );
|
|
|
|
$status = $ti->getStatus();
|
|
if ( !$status->isOK() ) {
|
|
$parser->getOutput()->setExtensionData( 'TemplateDataStatus', $status );
|
|
return '<div class="errorbox">' . $status->getHTML() . '</div>';
|
|
}
|
|
|
|
// Store the blob as page property for retrieval by ApiTemplateData.
|
|
// But, don't store it if we're parsing a doc sub page, because:
|
|
// - The doc subpage should not appear in ApiTemplateData as a documented
|
|
// template itself, which is confusing to users (T54448).
|
|
// - The doc subpage should not appear at Special:PagesWithProp.
|
|
// - Storing the blob twice in the database is inefficient (T52512).
|
|
$title = $parser->getTitle();
|
|
$docPage = wfMessage( 'templatedata-doc-subpage' )->inContentLanguage();
|
|
if ( !$title->isSubpage() || $title->getSubpageText() !== $docPage->plain() ) {
|
|
$parser->getOutput()->setProperty( 'templatedata', $ti->getJSONForDatabase() );
|
|
}
|
|
|
|
$parser->getOutput()->addModuleStyles( [
|
|
'ext.templateData',
|
|
'ext.templateData.images',
|
|
'jquery.tablesorter.styles',
|
|
] );
|
|
$parser->getOutput()->addModules( 'jquery.tablesorter' );
|
|
$parser->enableOOUI();
|
|
|
|
return $ti->getHtml( $parser->getOptions()->getUserLangObj() );
|
|
}
|
|
|
|
/**
|
|
* Fetch templatedata for an array of titles.
|
|
*
|
|
* @todo Document this hook
|
|
*
|
|
* The following questions are yet to be resolved.
|
|
* (a) Should we extend functionality to looking up an array of titles instead of one?
|
|
* The signature allows for an array of titles to be passed in, but the
|
|
* current implementation is not optimized for the multiple-title use case.
|
|
* (b) Should this be a lookup service instead of this faux hook?
|
|
* This will be resolved separately.
|
|
*
|
|
* @param array $tplTitles
|
|
* @param stdclass[] &$tplData
|
|
*/
|
|
public static function onParserFetchTemplateData( array $tplTitles, array &$tplData ): void {
|
|
$tplData = [];
|
|
|
|
// This inefficient implementation is currently tuned for
|
|
// Parsoid's use case where it requests info for exactly one title.
|
|
// For a real batch use case, this code will need an overhaul.
|
|
foreach ( $tplTitles as $tplTitle ) {
|
|
$title = Title::newFromText( $tplTitle );
|
|
if ( !$title ) {
|
|
// Invalid title
|
|
$tplData[$tplTitle] = null;
|
|
continue;
|
|
}
|
|
|
|
if ( $title->isRedirect() ) {
|
|
$title = ( new WikiPage( $title ) )->getRedirectTarget();
|
|
if ( !$title ) {
|
|
// Invalid redirecting title
|
|
$tplData[$tplTitle] = null;
|
|
continue;
|
|
}
|
|
}
|
|
|
|
if ( !$title->exists() ) {
|
|
$tplData[$tplTitle] = (object)[ "missing" => true ];
|
|
continue;
|
|
}
|
|
|
|
// FIXME: PageProps returns takes titles but returns by page id.
|
|
// This means we need to do our own look up and hope it matches.
|
|
// Spoiler, sometimes it won't. When that happens, the user won't
|
|
// get any templatedata-based interfaces for that template.
|
|
// The fallback is to not serve data for that template, which
|
|
// the clients have to support anyway, so the impact is minimal.
|
|
// It is also expected that such race conditions resolve themselves
|
|
// after a few seconds so the old "try again later" should cover this.
|
|
$pageId = $title->getArticleID();
|
|
$props = PageProps::getInstance()->getProperties( $title, 'templatedata' );
|
|
if ( !isset( $props[$pageId] ) ) {
|
|
// No templatedata
|
|
$tplData[$tplTitle] = (object)[ "notemplatedata" => true ];
|
|
continue;
|
|
}
|
|
|
|
$tdb = TemplateDataBlob::newFromDatabase( wfGetDB( DB_REPLICA ), $props[$pageId] );
|
|
$status = $tdb->getStatus();
|
|
if ( !$status->isOK() ) {
|
|
// Invalid data. Parsoid has no use for the error.
|
|
$tplData[$tplTitle] = (object)[ "notemplatedata" => true ];
|
|
continue;
|
|
}
|
|
|
|
$tplData[$tplTitle] = $tdb->getData();
|
|
}
|
|
}
|
|
}
|