FileBackendGroup::get() - Code Metrics - Inspection of "Daily Inspection: Merge "Add new type SearchIndexF..." - wikimedia/mediawiki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (098997)

unknown

created 2016-09-26 17:59 UTC

FileBackendGroup::get() C

↳ Parent: FileBackendGroup

Complexity

Conditions	7
Paths	6

Size

Total Lines	40
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	7
eloc	28
nc	6
nop	1
dl	0
loc	40
rs	6.7272
c	0
b	0
f	0

<?php
/**
 * File backend registration handling.
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License along
 * with this program; if not, write to the Free Software Foundation, Inc.,
 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 * http://www.gnu.org/copyleft/gpl.html
 *
 * @file
 * @ingroup FileBackend
 * @author Aaron Schulz
 */
use \MediaWiki\Logger\LoggerFactory;

/**
 * Class to handle file backend registration
 *
 * @ingroup FileBackend
 * @since 1.19
 */
class FileBackendGroup {
	/** @var FileBackendGroup */
	protected static $instance = null;

	/** @var array (name => ('class' => string, 'config' => array, 'instance' => object)) */
	protected $backends = [];

	protected function __construct() {
	}

	/**
	 * @return FileBackendGroup
	 */
	public static function singleton() {
		if ( self::$instance == null ) {
			self::$instance = new self();
			self::$instance->initFromGlobals();
		}

		return self::$instance;
	}

	/**
	 * Destroy the singleton instance
	 */
	public static function destroySingleton() {
		self::$instance = null;
	}

	/**
	 * Register file backends from the global variables
	 */
	protected function initFromGlobals() {
		global $wgLocalFileRepo, $wgForeignFileRepos, $wgFileBackends, $wgDirectoryMode;

		// Register explicitly defined backends
		$this->register( $wgFileBackends, wfConfiguredReadOnlyReason() );


		$autoBackends = [];
		// Automatically create b/c backends for file repos...
		$repos = array_merge( $wgForeignFileRepos, [ $wgLocalFileRepo ] );
		foreach ( $repos as $info ) {
			$backendName = $info['backend'];
			if ( is_object( $backendName ) || isset( $this->backends[$backendName] ) ) {
				continue; // already defined (or set to the object for some reason)
			}
			$repoName = $info['name'];
			// Local vars that used to be FSRepo members...
			$directory = $info['directory'];
			$deletedDir = isset( $info['deletedDir'] )
				? $info['deletedDir']
				: false; // deletion disabled
			$thumbDir = isset( $info['thumbDir'] )
				? $info['thumbDir']
				: "{$directory}/thumb";
			$transcodedDir = isset( $info['transcodedDir'] )
				? $info['transcodedDir']
				: "{$directory}/transcoded";
			// Get the FS backend configuration
			$autoBackends[] = [
				'name' => $backendName,
				'class' => 'FSFileBackend',
				'lockManager' => 'fsLockManager',
				'containerPaths' => [
					"{$repoName}-public" => "{$directory}",
					"{$repoName}-thumb" => $thumbDir,
					"{$repoName}-transcoded" => $transcodedDir,
					"{$repoName}-deleted" => $deletedDir,
					"{$repoName}-temp" => "{$directory}/temp"
				],
				'fileMode' => isset( $info['fileMode'] ) ? $info['fileMode'] : 0644,
				'directoryMode' => $wgDirectoryMode,
			];
		}

		// Register implicitly defined backends
		$this->register( $autoBackends, wfConfiguredReadOnlyReason() );

	}

	/**
	 * Register an array of file backend configurations
	 *
	 * @param array $configs
	 * @param string|null $readOnlyReason
	 * @throws InvalidArgumentException
	 */
	protected function register( array $configs, $readOnlyReason = null ) {
		foreach ( $configs as $config ) {
			if ( !isset( $config['name'] ) ) {
				throw new InvalidArgumentException( "Cannot register a backend with no name." );
			}
			$name = $config['name'];
			if ( isset( $this->backends[$name] ) ) {
				throw new LogicException( "Backend with name `{$name}` already registered." );
			} elseif ( !isset( $config['class'] ) ) {
				throw new InvalidArgumentException( "Backend with name `{$name}` has no class." );
			}
			$class = $config['class'];

			$config['readOnly'] = !empty( $config['readOnly'] )
				? $config['readOnly']
				: $readOnlyReason;

			unset( $config['class'] ); // backend won't need this
			$this->backends[$name] = [
				'class' => $class,
				'config' => $config,
				'instance' => null
			];
		}
	}

	/**
	 * Get the backend object with a given name
	 *
	 * @param string $name
	 * @return FileBackend
	 * @throws InvalidArgumentException
	 */
	public function get( $name ) {
		if ( !isset( $this->backends[$name] ) ) {
			throw new InvalidArgumentException( "No backend defined with the name `$name`." );
		}
		// Lazy-load the actual backend instance
		if ( !isset( $this->backends[$name]['instance'] ) ) {
			$class = $this->backends[$name]['class'];
			$config = $this->backends[$name]['config'];
			$config += [
				'wikiId' => wfWikiID(), // e.g. "my_wiki-en_"
				'mimeCallback' => [ $this, 'guessMimeInternal' ],
				'obResetFunc' => 'wfResetOutputBuffers',
				'streamMimeFunc' => [ 'StreamFile', 'contentTypeFromPath' ]
			];
			$config['lockManager'] =
				LockManagerGroup::singleton( $config['wikiId'] )->get( $config['lockManager'] );
			$config['fileJournal'] = isset( $config['fileJournal'] )
				? FileJournal::factory( $config['fileJournal'], $name )
				: FileJournal::factory( [ 'class' => 'NullFileJournal' ], $name );
			$config['wanCache'] = ObjectCache::getMainWANInstance();
			$config['srvCache'] = ObjectCache::getLocalServerInstance( 'hash' );
			$config['statusWrapper'] = [ 'Status', 'wrap' ];
			$config['tmpDirectory'] = wfTempDir();
			$config['logger'] = LoggerFactory::getInstance( 'FileOperation' );
			$config['profiler'] = Profiler::instance();
			if ( $class === 'FileBackendMultiWrite' ) {
				foreach ( $config['backends'] as $index => $beConfig ) {
					if ( isset( $beConfig['template'] ) ) {
						// Config is just a modified version of a registered backend's.
						// This should only be used when that config is used only by this backend.
						$config['backends'][$index] += $this->config( $beConfig['template'] );
					}
				}
			}

			$this->backends[$name]['instance'] = new $class( $config );
		}

		return $this->backends[$name]['instance'];
	}

	/**
	 * Get the config array for a backend object with a given name
	 *
	 * @param string $name
	 * @return array
	 * @throws InvalidArgumentException
	 */
	public function config( $name ) {
		if ( !isset( $this->backends[$name] ) ) {
			throw new InvalidArgumentException( "No backend defined with the name `$name`." );
		}
		$class = $this->backends[$name]['class'];

		return [ 'class' => $class ] + $this->backends[$name]['config'];
	}

	/**
	 * Get an appropriate backend object from a storage path
	 *
	 * @param string $storagePath
	 * @return FileBackend|null Backend or null on failure
	 */
	public function backendFromPath( $storagePath ) {
		list( $backend, , ) = FileBackend::splitStoragePath( $storagePath );
		if ( $backend !== null && isset( $this->backends[$backend] ) ) {
			return $this->get( $backend );
		}

		return null;
	}

	/**
	 * @param string $storagePath
	 * @param string|null $content
	 * @param string|null $fsPath
	 * @return string
	 * @since 1.27
	 */
	public function guessMimeInternal( $storagePath, $content, $fsPath ) {
		$magic = MimeMagic::singleton();
		// Trust the extension of the storage path (caller must validate)
		$ext = FileBackend::extensionFromPath( $storagePath );
		$type = $magic->guessTypesForExtension( $ext );
		// For files without a valid extension (or one at all), inspect the contents
		if ( !$type && $fsPath ) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			$type = $magic->guessMimeType( $fsPath, false );
		} elseif ( !$type && strlen( $content ) ) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			$tmpFile = TempFSFile::factory( 'mime_', '', wfTempDir() );
			file_put_contents( $tmpFile->getPath(), $content );
			$type = $magic->guessMimeType( $tmpFile->getPath(), false );
		}
		return $type ?: 'unknown/unknown';
	}
}


1			<?php
2			/**
3			* File backend registration handling.
4			*
5			* This program is free software; you can redistribute it and/or modify
6			* it under the terms of the GNU General Public License as published by
7			* the Free Software Foundation; either version 2 of the License, or
8			* (at your option) any later version.
9			*
10			* This program is distributed in the hope that it will be useful,
11			* but WITHOUT ANY WARRANTY; without even the implied warranty of
12			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13			* GNU General Public License for more details.
14			*
15			* You should have received a copy of the GNU General Public License along
16			* with this program; if not, write to the Free Software Foundation, Inc.,
17			* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
18			* http://www.gnu.org/copyleft/gpl.html
19			*
20			* @file
21			* @ingroup FileBackend
22			* @author Aaron Schulz
23			*/
24			use \MediaWiki\Logger\LoggerFactory;
25
26			/**
27			* Class to handle file backend registration
28			*
29			* @ingroup FileBackend
30			* @since 1.19
31			*/
32			class FileBackendGroup {
33			/** @var FileBackendGroup */
34			protected static $instance = null;
35
36			/** @var array (name => ('class' => string, 'config' => array, 'instance' => object)) */
37			protected $backends = [];
38
39			protected function __construct() {
40			}
41
42			/**
43			* @return FileBackendGroup
44			*/
45			public static function singleton() {
46			if ( self::$instance == null ) {
47			self::$instance = new self();
48			self::$instance->initFromGlobals();
49			}
50
51			return self::$instance;
52			}
53
54			/**
55			* Destroy the singleton instance
56			*/
57			public static function destroySingleton() {
58			self::$instance = null;
59			}
60
61			/**
62			* Register file backends from the global variables
63			*/
64			protected function initFromGlobals() {
65			global $wgLocalFileRepo, $wgForeignFileRepos, $wgFileBackends, $wgDirectoryMode;
66
67			// Register explicitly defined backends
68			$this->register( $wgFileBackends, wfConfiguredReadOnlyReason() );
			0 ignored issues – show Security Bug introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report It seems like `wfConfiguredReadOnlyReason()` can also be of type `false`; however, `FileBackendGroup::register()` does only seem to accept `string\|null`, did you maybe forget to handle an error condition? Loading history...
69
70			$autoBackends = [];
71			// Automatically create b/c backends for file repos...
72			$repos = array_merge( $wgForeignFileRepos, [ $wgLocalFileRepo ] );
73			foreach ( $repos as $info ) {
74			$backendName = $info['backend'];
75			if ( is_object( $backendName ) \|\| isset( $this->backends[$backendName] ) ) {
76			continue; // already defined (or set to the object for some reason)
77			}
78			$repoName = $info['name'];
79			// Local vars that used to be FSRepo members...
80			$directory = $info['directory'];
81			$deletedDir = isset( $info['deletedDir'] )
82			? $info['deletedDir']
83			: false; // deletion disabled
84			$thumbDir = isset( $info['thumbDir'] )
85			? $info['thumbDir']
86			: "{$directory}/thumb";
87			$transcodedDir = isset( $info['transcodedDir'] )
88			? $info['transcodedDir']
89			: "{$directory}/transcoded";
90			// Get the FS backend configuration
91			$autoBackends[] = [
92			'name' => $backendName,
93			'class' => 'FSFileBackend',
94			'lockManager' => 'fsLockManager',
95			'containerPaths' => [
96			"{$repoName}-public" => "{$directory}",
97			"{$repoName}-thumb" => $thumbDir,
98			"{$repoName}-transcoded" => $transcodedDir,
99			"{$repoName}-deleted" => $deletedDir,
100			"{$repoName}-temp" => "{$directory}/temp"
101			],
102			'fileMode' => isset( $info['fileMode'] ) ? $info['fileMode'] : 0644,
103			'directoryMode' => $wgDirectoryMode,
104			];
105			}
106
107			// Register implicitly defined backends
108			$this->register( $autoBackends, wfConfiguredReadOnlyReason() );
			0 ignored issues – show Security Bug introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report It seems like `wfConfiguredReadOnlyReason()` can also be of type `false`; however, `FileBackendGroup::register()` does only seem to accept `string\|null`, did you maybe forget to handle an error condition? Loading history...
109			}
110
111			/**
112			* Register an array of file backend configurations
113			*
114			* @param array $configs
115			* @param string\|null $readOnlyReason
116			* @throws InvalidArgumentException
117			*/
118			protected function register( array $configs, $readOnlyReason = null ) {
119			foreach ( $configs as $config ) {
120			if ( !isset( $config['name'] ) ) {
121			throw new InvalidArgumentException( "Cannot register a backend with no name." );
122			}
123			$name = $config['name'];
124			if ( isset( $this->backends[$name] ) ) {
125			throw new LogicException( "Backend with name `{$name}` already registered." );
126			} elseif ( !isset( $config['class'] ) ) {
127			throw new InvalidArgumentException( "Backend with name `{$name}` has no class." );
128			}
129			$class = $config['class'];
130
131			$config['readOnly'] = !empty( $config['readOnly'] )
132			? $config['readOnly']
133			: $readOnlyReason;
134
135			unset( $config['class'] ); // backend won't need this
136			$this->backends[$name] = [
137			'class' => $class,
138			'config' => $config,
139			'instance' => null
140			];
141			}
142			}
143
144			/**
145			* Get the backend object with a given name
146			*
147			* @param string $name
148			* @return FileBackend
149			* @throws InvalidArgumentException
150			*/
151			public function get( $name ) {
152			if ( !isset( $this->backends[$name] ) ) {
153			throw new InvalidArgumentException( "No backend defined with the name `$name`." );
154			}
155			// Lazy-load the actual backend instance
156			if ( !isset( $this->backends[$name]['instance'] ) ) {
157			$class = $this->backends[$name]['class'];
158			$config = $this->backends[$name]['config'];
159			$config += [
160			'wikiId' => wfWikiID(), // e.g. "my_wiki-en_"
161			'mimeCallback' => [ $this, 'guessMimeInternal' ],
162			'obResetFunc' => 'wfResetOutputBuffers',
163			'streamMimeFunc' => [ 'StreamFile', 'contentTypeFromPath' ]
164			];
165			$config['lockManager'] =
166			LockManagerGroup::singleton( $config['wikiId'] )->get( $config['lockManager'] );
167			$config['fileJournal'] = isset( $config['fileJournal'] )
168			? FileJournal::factory( $config['fileJournal'], $name )
169			: FileJournal::factory( [ 'class' => 'NullFileJournal' ], $name );
170			$config['wanCache'] = ObjectCache::getMainWANInstance();
171			$config['srvCache'] = ObjectCache::getLocalServerInstance( 'hash' );
172			$config['statusWrapper'] = [ 'Status', 'wrap' ];
173			$config['tmpDirectory'] = wfTempDir();
174			$config['logger'] = LoggerFactory::getInstance( 'FileOperation' );
175			$config['profiler'] = Profiler::instance();
176			if ( $class === 'FileBackendMultiWrite' ) {
177			foreach ( $config['backends'] as $index => $beConfig ) {
178			if ( isset( $beConfig['template'] ) ) {
179			// Config is just a modified version of a registered backend's.
180			// This should only be used when that config is used only by this backend.
181			$config['backends'][$index] += $this->config( $beConfig['template'] );
182			}
183			}
184			}
185
186			$this->backends[$name]['instance'] = new $class( $config );
187			}
188
189			return $this->backends[$name]['instance'];
190			}
191
192			/**
193			* Get the config array for a backend object with a given name
194			*
195			* @param string $name
196			* @return array
197			* @throws InvalidArgumentException
198			*/
199			public function config( $name ) {
200			if ( !isset( $this->backends[$name] ) ) {
201			throw new InvalidArgumentException( "No backend defined with the name `$name`." );
202			}
203			$class = $this->backends[$name]['class'];
204
205			return [ 'class' => $class ] + $this->backends[$name]['config'];
206			}
207
208			/**
209			* Get an appropriate backend object from a storage path
210			*
211			* @param string $storagePath
212			* @return FileBackend\|null Backend or null on failure
213			*/
214			public function backendFromPath( $storagePath ) {
215			list( $backend, , ) = FileBackend::splitStoragePath( $storagePath );
216			if ( $backend !== null && isset( $this->backends[$backend] ) ) {
217			return $this->get( $backend );
218			}
219
220			return null;
221			}
222
223			/**
224			* @param string $storagePath
225			* @param string\|null $content
226			* @param string\|null $fsPath
227			* @return string
228			* @since 1.27
229			*/
230			public function guessMimeInternal( $storagePath, $content, $fsPath ) {
231			$magic = MimeMagic::singleton();
232			// Trust the extension of the storage path (caller must validate)
233			$ext = FileBackend::extensionFromPath( $storagePath );
234			$type = $magic->guessTypesForExtension( $ext );
235			// For files without a valid extension (or one at all), inspect the contents
236			if ( !$type && $fsPath ) {
			0 ignored issues – show Bug Best Practice introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report The expression `$type` of type `null\|string` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history... Bug Best Practice introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report The expression `$fsPath` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
237			$type = $magic->guessMimeType( $fsPath, false );
238			} elseif ( !$type && strlen( $content ) ) {
			0 ignored issues – show Bug Best Practice introduced 2016-01-16 18:00 UTC by Report Bug Copy Issue Report The expression `$type` of type `null\|string` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
239			$tmpFile = TempFSFile::factory( 'mime_', '', wfTempDir() );
240			file_put_contents( $tmpFile->getPath(), $content );
241			$type = $magic->guessMimeType( $tmpFile->getPath(), false );
242			}
243			return $type ?: 'unknown/unknown';
244			}
245			}
246

wikimedia / mediawiki

Branch — master (098997)

FileBackendGroup::get() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like