ExportTest: Simplify code that failed on 5.3 because of reserved word 'namespace'
authorBartosz Dziewoński <matma.rex@gmail.com>
Wed, 17 Feb 2016 19:39:46 +0000 (20:39 +0100)
committerUmherirrender <umherirrender_de.wp@web.de>
Wed, 17 Feb 2016 21:23:49 +0000 (21:23 +0000)
Reserved words, like 'namespace', are perfectly fine now to use as object
keys, but it seems they used to not be in PHP 5.3, necessitating workarounds.

Change-Id: Iecb79fda19803461dfb3800b8af4d31e971551b8

tests/phpunit/includes/ExportTest.php

index 4a9918c..d451e21 100644 (file)
@@ -43,28 +43,26 @@ class ExportTest extends MediaWikiLangTestCase {
 
                /**
                 * Check namespaces match xml
-                * FIXME: PHP 5.3 support. When we don't support PHP 5.3,
-                * add ->namespace to object and remove from array
                 */
-               $xmlNamespaces = (array) $xmlObject->siteinfo->namespaces;
-               $xmlNamespaces = str_replace( ' ', '_', $xmlNamespaces['namespace'] );
-               unset ( $xmlNamespaces[ '@attributes' ] );
+               $xmlNamespaces = (array)$xmlObject->siteinfo->namespaces->namespace;
+               $xmlNamespaces = str_replace( ' ', '_', $xmlNamespaces );
+               unset( $xmlNamespaces[ '@attributes' ] );
                foreach ( $xmlNamespaces as &$namespaceObject ) {
                        if ( is_object( $namespaceObject ) ) {
                                $namespaceObject = '';
                        }
                }
 
-               $actualNamespaces = (array) $wgContLang->getNamespaces();
+               $actualNamespaces = (array)$wgContLang->getNamespaces();
                $actualNamespaces = array_values( $actualNamespaces );
                $this->assertEquals( $actualNamespaces, $xmlNamespaces );
 
                // Check xml page title correct
-               $xmlTitle = (array) $xmlObject->page->title;
+               $xmlTitle = (array)$xmlObject->page->title;
                $this->assertEquals( $pageTitle, $xmlTitle[0] );
 
                // Check xml page text is not empty
-               $text = (array) $xmlObject->page->revision->text;
+               $text = (array)$xmlObject->page->revision->text;
                $this->assertNotEquals( '', $text[0] );
        }