Anomie has uploaded a new change for review.
https://gerrit.wikimedia.org/r/52060
Change subject: Add parser method to call parser functions
......................................................................
Add parser method to call parser functions
There is currently no straightforward way for anything to call a parser
function and get the result. This abstracts out that portion of
braceSubstitution() to allow this.
The immediate motivation for this patch is to close bug 41769 against
Scribunto, see I0138836654b0e34c5c23daaedcdf5d4f9d1c7ab2.
Bug: 41769
Change-Id: I339b882010dedd714e7965e25ad650ed8b8cd48f
---
M includes/parser/Parser.php
1 file changed, 116 insertions(+), 62 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/60/52060/1
diff --git a/includes/parser/Parser.php b/includes/parser/Parser.php
index ed5071e..09b8436 100644
--- a/includes/parser/Parser.php
+++ b/includes/parser/Parser.php
@@ -3240,70 +3240,21 @@
$colonPos = strpos( $part1, ':' );
if ( $colonPos !== false ) {
- # Case sensitive functions
- $function = substr( $part1, 0, $colonPos );
- if ( isset(
$this->mFunctionSynonyms[1][$function] ) ) {
- $function =
$this->mFunctionSynonyms[1][$function];
- } else {
- # Case insensitive functions
- $function = $wgContLang->lc( $function
);
- if ( isset(
$this->mFunctionSynonyms[0][$function] ) ) {
- $function =
$this->mFunctionSynonyms[0][$function];
- } else {
- $function = false;
- }
+ try {
+ $result = $this->callParserFunction(
$frame,
+ substr( $part1, 0, $colonPos ),
+ trim( substr( $part1, $colonPos
+ 1 ) ),
+ $args
+ );
+ } catch ( Exception $ex ) {
+ wfProfileOut( __METHOD__ . '-pfunc' );
+ throw $ex;
}
- if ( $function ) {
- wfProfileIn( __METHOD__ . '-pfunc-' .
$function );
- list( $callback, $flags ) =
$this->mFunctionHooks[$function];
- $initialArgs = array( &$this );
- $funcArgs = array( trim( substr(
$part1, $colonPos + 1 ) ) );
- if ( $flags & SFH_OBJECT_ARGS ) {
- # Add a frame parameter, and
pass the arguments as an array
- $allArgs = $initialArgs;
- $allArgs[] = $frame;
- for ( $i = 0; $i <
$args->getLength(); $i++ ) {
- $funcArgs[] =
$args->item( $i );
- }
- $allArgs[] = $funcArgs;
- } else {
- # Convert arguments to plain
text
- for ( $i = 0; $i <
$args->getLength(); $i++ ) {
- $funcArgs[] = trim(
$frame->expand( $args->item( $i ) ) );
- }
- $allArgs = array_merge(
$initialArgs, $funcArgs );
- }
- # Workaround for PHP bug 35229 and
similar
- if ( !is_callable( $callback ) ) {
- wfProfileOut( __METHOD__ .
'-pfunc-' . $function );
- wfProfileOut( __METHOD__ .
'-pfunc' );
- wfProfileOut( __METHOD__ );
- throw new MWException( "Tag
hook for $function is not callable\n" );
- }
- $result = call_user_func_array(
$callback, $allArgs );
- $found = true;
- $noparse = true;
- $preprocessFlags = 0;
-
- if ( is_array( $result ) ) {
- if ( isset( $result[0] ) ) {
- $text = $result[0];
- unset( $result[0] );
- }
-
- # Extract flags into the local
scope
- # This allows callers to set
flags such as nowiki, found, etc.
- extract( $result );
- } else {
- $text = $result;
- }
- if ( !$noparse ) {
- $text = $this->preprocessToDom(
$text, $preprocessFlags );
- $isChildObj = true;
- }
- wfProfileOut( __METHOD__ . '-pfunc-' .
$function );
- }
+ # The interface for parser functions allows for
extracting
+ # flags into the local scope. Extract any
forwarded flags
+ # here.
+ extract( $result );
}
wfProfileOut( __METHOD__ . '-pfunc' );
}
@@ -3501,6 +3452,109 @@
}
/**
+ * Call a parser function and return an array with text and flags.
+ *
+ * The returned array will always contain a boolean 'found', indicating
+ * whether the parser function was found or not. It may also contain the
+ * following:
+ * text: string|object, resulting wikitext or PP DOM object
+ * isHTML: bool, $text is HTML, armour it against wikitext
transformation
+ * isChildObj: bool, $text is a DOM node needing expansion in a child
frame
+ * isLocalObj: bool, $text is a DOM node needing expansion in the
current frame
+ * nowiki: bool, wiki markup in $text should be escaped
+ *
+ * @since 1.21
+ * @param $frame PPFrame The current frame, contains template arguments
+ * @param $function string Function name
+ * @param $arg1 string Argument after colon
+ * @param $args array Additional args, if any
+ * @return array
+ */
+ public function callParserFunction( $frame, $function, $arg1, $args =
array() ) {
+ global $wgContLang;
+
+ wfProfileIn( __METHOD__ );
+
+ # Case sensitive functions
+ if ( isset( $this->mFunctionSynonyms[1][$function] ) ) {
+ $function = $this->mFunctionSynonyms[1][$function];
+ } else {
+ # Case insensitive functions
+ $function = $wgContLang->lc( $function );
+ if ( isset( $this->mFunctionSynonyms[0][$function] ) ) {
+ $function =
$this->mFunctionSynonyms[0][$function];
+ } else {
+ wfProfileOut( __METHOD__ );
+ return array( 'found' => false );
+ }
+ }
+
+ wfProfileIn( __METHOD__ . '-pfunc-' . $function );
+ list( $callback, $flags ) = $this->mFunctionHooks[$function];
+ $initialArgs = array( &$this );
+ $funcArgs = array( $arg1 );
+ if ( $flags & SFH_OBJECT_ARGS ) {
+ # Add a frame parameter, and pass the arguments as an
array
+ $allArgs = $initialArgs;
+ $allArgs[] = $frame;
+ for ( $i = 0; $i < $args->getLength(); $i++ ) {
+ $funcArgs[] = $args->item( $i );
+ }
+ $allArgs[] = $funcArgs;
+ } else {
+ # Convert arguments to plain text
+ for ( $i = 0; $i < $args->getLength(); $i++ ) {
+ $funcArgs[] = trim( $frame->expand(
$args->item( $i ) ) );
+ }
+ $allArgs = array_merge( $initialArgs, $funcArgs );
+ }
+
+ # Workaround for PHP bug 35229 and similar
+ if ( !is_callable( $callback ) ) {
+ wfProfileOut( __METHOD__ . '-pfunc-' . $function );
+ wfProfileOut( __METHOD__ );
+ throw new MWException( "Tag hook for $function is not
callable\n" );
+ }
+ $result = call_user_func_array( $callback, $allArgs );
+
+ # The interface for function hooks allows them to return a
wikitext
+ # string or an array containing the string and any flags. This
mungs
+ # things around to match what this method should return.
+ if ( !is_array( $result ) ) {
+ $result = array(
+ 'found' => true,
+ 'text' => $result,
+ );
+ } else {
+ if ( isset( $result[0] ) && !isset( $result['text'] ) )
{
+ $result['text'] = $result[0];
+ unset( $result[0] );
+ }
+ $result += array(
+ 'found' => true,
+ );
+ }
+
+ $noparse = true;
+ $preprocessFlags = 0;
+ if ( isset( $result['noparse'] ) ) {
+ $noparse = $result['noparse'];
+ }
+ if ( isset( $result['preprocessFlags'] ) ) {
+ $preprocessFlags = $result['preprocessFlags'];
+ }
+
+ if ( !$noparse ) {
+ $result['text'] = $this->preprocessToDom(
$result['text'], $preprocessFlags );
+ $result['isChildObj'] = true;
+ }
+ wfProfileOut( __METHOD__ . '-pfunc-' . $function );
+ wfProfileOut( __METHOD__ );
+
+ return $result;
+ }
+
+ /**
* Get the semi-parsed DOM representation of a template with a given
title,
* and its redirect destination title. Cached.
*
--
To view, visit https://gerrit.wikimedia.org/r/52060
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I339b882010dedd714e7965e25ad650ed8b8cd48f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Anomie <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
