Remove broken explode() from MediaWikiMediaTestCase::dataFile()
authorThiemo Kreuz <thiemo.kreuz@wikimedia.de>
Wed, 27 Mar 2019 10:22:49 +0000 (11:22 +0100)
committerThiemo Kreuz (WMDE) <thiemo.kreuz@wikimedia.de>
Thu, 28 Mar 2019 23:01:27 +0000 (23:01 +0000)
It appears this is unused. Providing false is enough. When all tests go
green, this should be proof enough that all tests are fine with this
fallback code being removed.

Change-Id: I1fdeb26c61d67f5564377770c6d5e11d789971bb

tests/phpunit/includes/media/MediaWikiMediaTestCase.php

index 7a536df..2e9acfa 100644 (file)
@@ -70,16 +70,10 @@ abstract class MediaWikiMediaTestCase extends MediaWikiTestCase {
         *
         * File must be in the path returned by getFilePath()
         * @param string $name File name
-        * @param string|null $type MIME type [optional]
+        * @param string|false $type MIME type [optional]
         * @return UnregisteredLocalFile
         */
-       protected function dataFile( $name, $type = null ) {
-               if ( !$type ) {
-                       // Autodetect by file extension for the lazy.
-                       $magic = MediaWiki\MediaWikiServices::getInstance()->getMimeAnalyzer();
-                       $parts = explode( $name, '.' );
-                       $type = $magic->guessTypesForExtension( $parts[count( $parts ) - 1] );
-               }
+       protected function dataFile( $name, $type = false ) {
                return new UnregisteredLocalFile( false, $this->repo,
                        "mwstore://localtesting/data/$name", $type );
        }