ImportTask::run() - Code Metrics - dynamic/silverstripe-salsify - Measure and Improve Code Quality continuously with Scrutinizer

ImportTask::run() A
last analyzed 2021-10-21 15:43 UTC

↳ Parent: ImportTask

Complexity

Conditions	3
Paths	4

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	9
c	0
b	0
f	0
nc	4
nop	1
dl	0
loc	16
rs	9.9666

<?php

namespace Dynamic\Salsify\Task;

use Dynamic\Salsify\Model\Importer;
use Dynamic\Salsify\Traits\Yieldable;
use SilverStripe\Control\Director;
use SilverStripe\Core\Config\Config;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\Dev\BuildTask;

/**
 * Class ImportTask
 * @package Dynamic\Salsify\Task
 * @mixin Yieldable
 */
class ImportTask extends BuildTask
{
    use Yieldable;

    /**
     * @var string
     */
    private static $segment = 'SalsifyImportTask';


    /**
     * @var string
     */
    protected $title = 'Import products from salsify';

    /**
     * @var string
     */
    protected $description = 'Imports products from salsify into silverstripe';

    /**
     * @var bool
     */
    protected $enabled = true;

    /**
     * @var
     */
    private static $lineEnding;

    /**
     * @var bool
     */
    private static $output = true;


    /**
     * @param \SilverStripe\Control\HTTPRequest $request
     */
    public function run($request)
    {
        static::$lineEnding = Director::is_cli() ? PHP_EOL : '<br />';


        // gets all importers
        $injectorConfig = array_keys(Config::inst()->get(Injector::class));
        $importers = array_filter(
            $injectorConfig,
            function ($element) {
                return strncmp($element, Importer::class, strlen(Importer::class)) === 0;
            }
        );

        foreach ($this->yieldSingle($importers) as $importerClass) {
            $importer = Injector::inst()->create($importerClass);
            $importer->run();
        }
    }

    /**
     * @param $string
     */
    public static function output($string)
    {
        if (static::config()->get('output')) {
            echo $string . static::$lineEnding;

        }
    }
}


1			<?php
2
3			namespace Dynamic\Salsify\Task;
4
5			use Dynamic\Salsify\Model\Importer;
6			use Dynamic\Salsify\Traits\Yieldable;
7			use SilverStripe\Control\Director;
8			use SilverStripe\Core\Config\Config;
9			use SilverStripe\Core\Injector\Injector;
10			use SilverStripe\Dev\BuildTask;
11
12			/**
13			* Class ImportTask
14			* @package Dynamic\Salsify\Task
15			* @mixin Yieldable
16			*/
17			class ImportTask extends BuildTask
18			{
19			use Yieldable;
20
21			/**
22			* @var string
23			*/
24			private static $segment = 'SalsifyImportTask';
			0 ignored issues – show introduced 2019-05-18 02:04 UTC by Report Bug Copy Issue Report The private property `$segment` is not used, and could be removed. Loading history...
25
26			/**
27			* @var string
28			*/
29			protected $title = 'Import products from salsify';
30
31			/**
32			* @var string
33			*/
34			protected $description = 'Imports products from salsify into silverstripe';
35
36			/**
37			* @var bool
38			*/
39			protected $enabled = true;
40
41			/**
42			* @var
43			*/
44			private static $lineEnding;
45
46			/**
47			* @var bool
48			*/
49			private static $output = true;
			0 ignored issues – show introduced 2019-05-18 02:04 UTC by Report Bug Copy Issue Report The private property `$output` is not used, and could be removed. Loading history...
50
51			/**
52			* @param \SilverStripe\Control\HTTPRequest $request
53			*/
54			public function run($request)
55			{
56			static::$lineEnding = Director::is_cli() ? PHP_EOL : '<br />';
			0 ignored issues – show Bug introduced 2019-05-18 02:04 UTC by Report Bug Copy Issue Report Since `$lineEnding` is declared private, accessing it with `static` will lead to errors in possible sub-classes; you can either use `self`, or increase the visibility of `$lineEnding` to at least protected. Loading history...
57
58			// gets all importers
59			$injectorConfig = array_keys(Config::inst()->get(Injector::class));
60			$importers = array_filter(
61			$injectorConfig,
62			function ($element) {
63			return strncmp($element, Importer::class, strlen(Importer::class)) === 0;
64			}
65			);
66
67			foreach ($this->yieldSingle($importers) as $importerClass) {
68			$importer = Injector::inst()->create($importerClass);
69			$importer->run();
70			}
71			}
72
73			/**
74			* @param $string
75			*/
76			public static function output($string)
77			{
78			if (static::config()->get('output')) {
79			echo $string . static::$lineEnding;
			0 ignored issues – show Bug introduced 2019-05-18 02:04 UTC by Report Bug Copy Issue Report Since `$lineEnding` is declared private, accessing it with `static` will lead to errors in possible sub-classes; you can either use `self`, or increase the visibility of `$lineEnding` to at least protected. Loading history...
80			}
81			}
82			}
83

dynamic / silverstripe-salsify

ImportTask::run() A last analyzed 2021-10-21 15:43 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ImportTask::run() A
last analyzed 2021-10-21 15:43 UTC