PlatformFactory::getPlatform() - Code Metrics - LukeCarrier/moodle-componentmgr - Measure and Improve Code Quality continuously with Scrutinizer

PlatformFactory::getPlatform() B
last analyzed 2018-03-07 12:10 UTC

↳ Parent: PlatformFactory

Complexity

Conditions	6
Paths	6

Size

Total Lines	19
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	19
ccs	0
cts	11
cp	0
rs	8.8571
cc	6
eloc	15
nc	6
nop	1
crap	42

<?php

/**
 * Moodle component manager.
 *
 * @author Luke Carrier <[email protected]>
 * @copyright 2016 Luke Carrier
 * @license GPL-3.0+
 */

namespace ComponentManager\Platform;

use ComponentManager\Exception\PlatformException;
use Symfony\Component\Filesystem\Filesystem;

class PlatformFactory {
    /**
     * Filesystem.
     *
     * @var Filesystem
     */
    protected $filesystem;

    /**
     * Initialiser.
     *
     * @param Filesystem $filesystem
     */
    public function __construct(Filesystem $filesystem) {
        $this->filesystem = $filesystem;
    }

    /**
     * Get the platform with the specified name.
     *
     * @param string $platformName The name of the platform (retrieved from
     *                             PHP_OS).
     *
     * @return Platform
     *
     * @throws PlatformException
     */
    public function getPlatform($platformName) {
        switch ($platformName) {
            case 'Darwin':
            case 'FreeBSD':
            case 'Linux':
                return new LinuxPlatform($this->filesystem);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            case 'WINNT':
            case 'Windows':
                return new WindowsPlatform($this->filesystem);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            default:
                throw new PlatformException(
                        sprintf('Unsupported platform %s', PHP_OS),
                        PlatformException::CODE_UNKNOWN_PLATFORM);
        }
    }
}


1			<?php
2
3			/**
4			* Moodle component manager.
5			*
6			* @author Luke Carrier <[email protected]>
7			* @copyright 2016 Luke Carrier
8			* @license GPL-3.0+
9			*/
10
11			namespace ComponentManager\Platform;
12
13			use ComponentManager\Exception\PlatformException;
14			use Symfony\Component\Filesystem\Filesystem;
15
16			class PlatformFactory {
17			/**
18			* Filesystem.
19			*
20			* @var Filesystem
21			*/
22			protected $filesystem;
23
24			/**
25			* Initialiser.
26			*
27			* @param Filesystem $filesystem
28			*/
29			public function __construct(Filesystem $filesystem) {
30			$this->filesystem = $filesystem;
31			}
32
33			/**
34			* Get the platform with the specified name.
35			*
36			* @param string $platformName The name of the platform (retrieved from
37			* PHP_OS).
38			*
39			* @return Platform
40			*
41			* @throws PlatformException
42			*/
43			public function getPlatform($platformName) {
44			switch ($platformName) {
45			case 'Darwin':
46			case 'FreeBSD':
47			case 'Linux':
48			return new LinuxPlatform($this->filesystem);
49			break;
			0 ignored issues – show Unused Code introduced 2016-06-08 11:33 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
50
51			case 'WINNT':
52			case 'Windows':
53			return new WindowsPlatform($this->filesystem);
54			break;
			0 ignored issues – show Unused Code introduced 2016-06-08 11:33 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
55
56			default:
57			throw new PlatformException(
58			sprintf('Unsupported platform %s', PHP_OS),
59			PlatformException::CODE_UNKNOWN_PLATFORM);
60			}
61			}
62			}
63

LukeCarrier / moodle-componentmgr

PlatformFactory::getPlatform() B last analyzed 2018-03-07 12:10 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

PlatformFactory::getPlatform() B
last analyzed 2018-03-07 12:10 UTC