r96014 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r96013‎ | r96014 | r96015 >
Date:17:12, 1 September 2011
Author:catrope
Status:resolved (Comments)
Tags:
Comment:
RL2: Add pagename property to list=gadgetpages containing the page title without the namespace prefix. Timo needs this for his AJAX stuff
Modified paths:
  • /branches/RL2/extensions/Gadgets/api/ApiQueryGadgetPages.php (modified) (history)

Diff [purge]

Index: branches/RL2/extensions/Gadgets/api/ApiQueryGadgetPages.php
@@ -75,7 +75,7 @@
7676
7777 $title = Title::makeTitle( $row->gpl_namespace, $row->gpl_title );
7878 if ( is_null( $resultPageSet ) ) {
79 - $vals = array();
 79+ $vals = array( 'pagename' => $title->getPrefixedText() );
8080 self::addTitleInfo( $vals, $title );
8181 $fit = $result->addValue( array( 'query', $this->getModuleName() ), null, $vals );
8282 if ( !$fit ) {

Sign-offs

UserFlagDate
Krinkleinspected15:30, 3 September 2011
Krinkletested15:30, 3 September 2011

Follow-up revisions

RevisionCommit summaryAuthorDate
r96200[RL2] Follow-up r96014: getText instead of getPrefixedText, we need it withou...krinkle15:30, 3 September 2011

Comments

#Comment by Krinkle (talk | contribs)   15:29, 3 September 2011

getPrefixedText contains the namespace too. getText ;-)

Status & tagging log