(follow-up r108145) Mark test as needing a database, and fix bug number in comment.

diff --git a/includes/Skin.php b/includes/Skin.php
index bb81782..ec6c95e 100644 (file)
--- 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 1734fd2..3edcbbb 100644 (file)
--- 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(