r67530 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r67529‎ | r67530 | r67531 >
Date:13:35, 7 June 2010
Author:werdna
Status:deferred
Tags:
Comment:
Fix bug in ActiveStrategy extension affecting link pointing and identification of task forces
Modified paths:
  • /trunk/extensions/StrategyWiki/ActiveStrategy/ActiveStrategy_body.php (modified) (history)

Diff [purge]

Index: trunk/extensions/StrategyWiki/ActiveStrategy/ActiveStrategy_body.php
@@ -9,7 +9,7 @@
1010 'page_id',
1111 'page_namespace',
1212 'page_title',
13 - "substring_index(page_title, '/', -1) AS tf_name"
 13+ "substring_index(page_title, '/', 2) AS tf_name"
1414 ),
1515 array(
1616 'page_namespace' => 0,
@@ -24,6 +24,7 @@
2525
2626 $title = Title::newFromText( $taskForce );
2727 $text = $wgContLang->convert( $title->getPrefixedText() );
 28+ $text = substr( $text, strpos($text, '/') + 1 );
2829 $pageLink = $skin->linkKnown( $title, $text );
2930 $colors = null;
3031 $color = null;
@@ -146,7 +147,7 @@
147148
148149 foreach( $taskForces as $row ) {
149150 $title = Title::makeTitle( $row->page_namespace, $row->page_title );
150 - $memberCount[$row->title] =
 151+ $memberCount[$row->tf_name] =
151152 self::getMemberCount( $title->getPrefixedText() );
152153 }
153154

Status & tagging log