From ed394c764be464d96d0a211e4117da74d64a894e Mon Sep 17 00:00:00 2001 From: Brian Wolff Date: Sat, 7 Jan 2012 15:43:21 +0000 Subject: [PATCH] (follow-up r108145) Mark test as needing a database, and fix bug number in comment. --- includes/Skin.php | 2 +- tests/phpunit/skins/SideBarTest.php | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/includes/Skin.php b/includes/Skin.php index bb817828ac..ec6c95e5a4 100644 --- a/includes/Skin.php +++ b/includes/Skin.php @@ -1246,7 +1246,7 @@ abstract class Skin extends ContextSource { $line = array_map( 'trim', explode( '|', $line, 2 ) ); if ( count( $line ) !== 2 ) { // Second sanity check, could be hit by people doing - // funky stuff with parserfuncs... (bug 3321) + // funky stuff with parserfuncs... (bug 33321) continue; } diff --git a/tests/phpunit/skins/SideBarTest.php b/tests/phpunit/skins/SideBarTest.php index 1734fd28d2..3edcbbbe9a 100644 --- a/tests/phpunit/skins/SideBarTest.php +++ b/tests/phpunit/skins/SideBarTest.php @@ -106,7 +106,10 @@ class SideBarTest extends MediaWikiLangTestCase { ); } - /** bug 33321 */ + /** + * bug 33321 - Make sure there's a | after transforming. + * @group Database + */ function testTrickyPipe() { $this->assertSidebar( array( 'Title' => array( -- 2.20.1