MediaWiki r96014 - Code Review

Repository:	MediaWiki
Revision:	< r96013‎ \| r96014 (on ViewVC)‎ \| 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)

Diff [purge]

Index: branches/RL2/extensions/Gadgets/api/ApiQueryGadgetPages.php
—	—	@@ -75,7 +75,7 @@
76	76
77	77	$title = Title::makeTitle( $row->gpl_namespace, $row->gpl_title );
78	78	if ( is_null( $resultPageSet ) ) {
79		~~- $vals = array();~~
	79	+ $vals = array( 'pagename' => $title->getPrefixedText() );
80	80	self::addTitleInfo( $vals, $title );
81	81	$fit = $result->addValue( array( 'query', $this->getModuleName() ), null, $vals );
82	82	if ( !$fit ) {

User	Flag	Date
Krinkle	inspected	15:30, 3 September 2011
Krinkle	tested	15:30, 3 September 2011

Rev.	Commit summary	Author	Date
r96200	[RL2] Follow-up r96014: getText instead of getPrefixedText, we need it withou...	krinkle	15:30, 3 September 2011

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

getPrefixedText contains the namespace too. getText ;-)

15:30, 3 September 2011 Krinkle (talk | contribs) changed the status of r96014 [removed: fixme added: resolved]
15:29, 3 September 2011 Krinkle (talk | contribs) changed the status of r96014 [removed: new added: fixme]