Script_Command::addScriptsFromFolder() - Code Metrics - datasift/storyplayer - Measure and Improve Code Quality continuously with Scrutinizer

Script_Command::addScriptsFromFolder() A
last analyzed 2016-04-05 10:50 UTC

↳ Parent: Script_Command

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	4
rs	10
cc	1
eloc	1
nc	1
nop	3

<?php

/**
 * Copyright (c) 2011-present Mediasift Ltd
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions
 * are met:
 *
 *   * Redistributions of source code must retain the above copyright
 *     notice, this list of conditions and the following disclaimer.
 *
 *   * Redistributions in binary form must reproduce the above copyright
 *     notice, this list of conditions and the following disclaimer in
 *     the documentation and/or other materials provided with the
 *     distribution.
 *
 *   * Neither the names of the copyright holders nor the names of his
 *     contributors may be used to endorse or promote products derived
 *     from this software without specific prior written permission.
 *
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
 * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
 * COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
 * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
 * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
 * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
 * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
 * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 * POSSIBILITY OF SUCH DAMAGE.
 *
 * @category  Libraries
 * @package   Storyplayer/Cli
 * @author    Stuart Herbert <[email protected]>
 * @copyright 2011-present Mediasift Ltd www.datasift.com
 * @license   http://www.opensource.org/licenses/bsd-license.php  BSD License
 * @link      http://datasift.github.io/storyplayer
 */

namespace DataSift\Storyplayer\Cli;

use Exception;
use stdClass;
use Phix_Project\CliEngine;
use Phix_Project\CliEngine\CliCommand;
use Phix_Project\ExceptionsLib1\Legacy_ErrorHandler;
use Phix_Project\ExceptionsLib1\Legacy_ErrorException;
use DataSift\Stone\ConfigLib\E5xx_ConfigFileNotFound;
use DataSift\Stone\ConfigLib\E5xx_InvalidConfigFile;
use DataSift\Storyplayer\PlayerLib\E4xx_NoSuchReport;
use DataSift\Storyplayer\PlayerLib\PhaseGroup_Player;
use DataSift\Storyplayer\PlayerLib\StoryTeller;
use DataSift\Storyplayer\PlayerLib\Script_Player;
use DataSift\Storyplayer\Console\ScriptConsole;
use DataSift\Storyplayer\Injectables;

/**
 * A command to play a script
 *
 * @category  Libraries
 * @package   Storyplayer/Cli
 * @author    Stuart Herbert <[email protected]>
 * @copyright 2011-present Mediasift Ltd www.datasift.com
 * @license   http://www.opensource.org/licenses/bsd-license.php  BSD License
 * @link      http://datasift.github.io/storyplayer
 */
class Script_Command extends BaseCommand implements CliSignalHandler
{
    /**
     * should we let background processes survive when we shutdown?
     * @var boolean
     */
    protected $persistProcesses = false;

    // we need to track this for handling CTRL-C
    protected $st;

    // we track this for convenience
    protected $output;

    // our list of scripts to execute
    protected $scriptList;

    // our injected data / services
    // needed for when user presses CTRL+C
    protected $injectables;

    public function __construct($injectables)
    {
        parent::__construct($injectables);

        // define the command
        $this->setName('script');
        $this->setShortDescription('run an automation script');
        $this->setLongDescription(
            "Use this command to play an automation script."
            .PHP_EOL
        );
        $this->setArgsList(array(
            "[<script.php>]" => "run a script"
        ));

        // the switches that this command supports
        $this->addFeature(new Feature_ConsoleSupport);
        $this->addFeature(new Feature_DeviceSupport);
        $this->addFeature(new Feature_TestEnvironmentConfigSupport);
        $this->addFeature(new Feature_SystemUnderTestConfigSupport);
        $this->addFeature(new Feature_LocalhostSupport);
        $this->addFeature(new Feature_ActiveConfigSupport);
        $this->addFeature(new Feature_DefinesSupport);
        $this->addFeature(new Feature_PhaseLoaderSupport);
        $this->addFeature(new Feature_ProseLoaderSupport);
        $this->addFeature(new Feature_TestUsersSupport);

        // now setup all of the switches that we support
        $this->addFeatureSwitches();
    }

    /**
     *
     * @param  CliEngine $engine
     * @param  array     $params
     * @param  Injectables|null $injectables
     * @return integer
     */
    public function processCommand(CliEngine $engine, $params = array(), $injectables = null)
    {
        // we need to wrap our code to catch old-style PHP errors
        $legacyHandler = new Legacy_ErrorHandler();
        try {
            $returnCode = $legacyHandler->run([$this, 'processInsideLegacyHandler'], [$engine, $params, $injectables]);
            return $returnCode;
        }
        catch (Exception $e) {
            $injectables->output->logCliError($e->getMessage());
            $engine->options->dev = true;
            if (isset($engine->options->dev) && $engine->options->dev) {
                $injectables->output->logCliError("Stack trace is:\n\n" . $e->getTraceAsString());
            }
            exit(1);

        }
    }

    public function processInsideLegacyHandler(CliEngine $engine, $params = array(), $injectables = null)
    {
        // make sure we're using the ScriptConsole by default
        $injectables->output->usePluginInSlot(new ScriptConsole, 'console');

        // process the common functionality
        $this->initFeaturesBeforeModulesAvailable($engine);

        // now it is safe to create our shorthand
        $runtimeConfig        = $injectables->getRuntimeConfig();
        $runtimeConfigManager = $injectables->getRuntimeConfigManager();
        $output               = $injectables->output;

        // save the output for use in other methods
        $this->output = $output;

        // at this point, all of the services / data held in $injectables
        // has been initialised and is ready for use

        // create a new StoryTeller object
        $st = new StoryTeller($injectables);

        // remember our $st object, as we'll need it for our
        // shutdown function
        $this->st = $st;

        // now that we have $st, we can initialise any feature that
        // wants to use our modules
        $this->initFeaturesAfterModulesAvailable($st, $engine, $injectables);

        // install signal handling, now that $this->st is defined
        //
        // we wouldn't want signal handling called out of order :)
        $this->initSignalHandling($injectables);

        // build our list of stories to run
        $this->initScriptList($engine, $injectables, $params);

        // and we're ready to tell the world that we're here
        $output->startStoryplayer(
            $engine->getAppVersion(),
            $engine->getAppUrl(),
            $engine->getAppCopyright(),
            $engine->getAppLicense()
        );

        // $this->scriptList contains one or more things to run
        //
        // let's play each of them in order
        foreach ($this->scriptList as $player)
        {
            // play the story(ies)
            $player->play($st, $injectables);
        }

        // write out any changed runtime config to disk
        $runtimeConfigManager->saveRuntimeConfig($runtimeConfig, $output);

        // tell the output plugins that we're all done
        $output->endStoryplayer(0);

        // all done
        return 0;
    }

    // ==================================================================
    //
    // the individual initX() methods
    //
    // these are processed *after* the objects defined in the
    // CommonFunctionalitySupport trait have been initialised
    //
    // ------------------------------------------------------------------

    /**
     *
     * @param  Injectables $injectables
     * @return void
     */
    protected function initSignalHandling(Injectables $injectables)
    {
        // we need to remember the injectables, for when we handle CTRL+C
        $this->injectables = $injectables;

        // setup signal handling
        pcntl_signal(SIGTERM, array($this, 'sigtermHandler'));
        pcntl_signal(SIGINT , array($this, 'sigtermHandler'));
    }

    /**
     *
     * @param  CliEngine   $cliEngine
     * @param  Injectables $injectables
     * @param  array       $cliParams
     * @return void
     */
    protected function initScriptList(CliEngine $cliEngine, Injectables $injectables, $cliParams)
    {
        // our list of stories to play
        $this->scriptList = [];

        foreach ($cliParams as $cliParam) {
            // figure out what to do?
            if (is_dir($cliParam)) {
                $this->scriptList = $this->scriptList + $this->addScriptsFromFolder($cliEngine, $injectables, $cliParam);
            }
            else if (is_file($cliParam)) {
                // are we loading a story, or a list of stories?
                $paramParts  = explode('.', $cliParams[0]);
                $paramSuffix = end($paramParts);

                switch ($paramSuffix) {
                    case 'php':
                        $this->scriptList = $this->scriptList + $this->addScriptFromFile($cliEngine, $injectables, $cliParam);
                        break;

                    case 'json':
                        $this->scriptList = $this->scriptList + $this->addScriptsFromFile($cliEngine, $injectables, $cliParam);
                        break;

                    default:
                        $this->output->logCliError("unsupported script file '{$cliParam}'");
                        exit(1);

                }
            }
            else {
                // if we get here, we've no idea what to do
                $this->output->logCliError("no such file: '{$cliParam}'");
                exit(1);

            }
        }
    }

    // ==================================================================
    //
    // Story loading
    //
    // ------------------------------------------------------------------

    protected function addScriptFromFile(CliEngine $engine, Injectables $injectables, $storyFile)

    {
        // these are the players we want to execute for the story
        $return = [
            new Script_Player($storyFile, $injectables),
        ];

        // all done
        return $return;
    }

    protected function addScriptsFromFolder(CliEngine $engine, Injectables $injectables, $folder)

    {
        // tbd
    }

    protected function addScriptsFromFile(CliEngine $engine, Injectables $injectables, $file)

    {
        // tbd
    }

    // ==================================================================
    //
    // SIGNAL handling
    //
    // ------------------------------------------------------------------

    /**
     *
     * @param  integer $signo
     * @return void
     */
    public function sigtermHandler($signo)
    {
        // tell the user what is happening
        echo "\n";
        echo "============================================================\n";
        echo "USER ABORT!!\n";
        echo "============================================================\n";
        echo "\n";

        // cleanup
        $phasesPlayer = new PhaseGroup_Player();
        $phasesPlayer->playPhases(
            "user abort",
            $this->st,
            $this->injectables,
            $this->injectables->activeConfig->getData('storyplayer.phases.userAbort'),
            null
        );

        // force a clean shutdown
        exit(1);

    }
}

1		<?php
2
3		/**
4		* Copyright (c) 2011-present Mediasift Ltd
5		* All rights reserved.
6		*
7		* Redistribution and use in source and binary forms, with or without
8		* modification, are permitted provided that the following conditions
9		* are met:
10		*
11		* * Redistributions of source code must retain the above copyright
12		* notice, this list of conditions and the following disclaimer.
13		*
14		* * Redistributions in binary form must reproduce the above copyright
15		* notice, this list of conditions and the following disclaimer in
16		* the documentation and/or other materials provided with the
17		* distribution.
18		*
19		* * Neither the names of the copyright holders nor the names of his
20		* contributors may be used to endorse or promote products derived
21		* from this software without specific prior written permission.
22		*
23		* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
24		* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
25		* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
26		* FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
27		* COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
28		* INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
29		* BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
30		* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
31		* CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
32		* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
33		* ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
34		* POSSIBILITY OF SUCH DAMAGE.
35		*
36		* @category Libraries
37		* @package Storyplayer/Cli
38		* @author Stuart Herbert <[email protected]>
39		* @copyright 2011-present Mediasift Ltd www.datasift.com
40		* @license http://www.opensource.org/licenses/bsd-license.php BSD License
41		* @link http://datasift.github.io/storyplayer
42		*/
43
44		namespace DataSift\Storyplayer\Cli;
45
46		use Exception;
47		use stdClass;
48		use Phix_Project\CliEngine;
49		use Phix_Project\CliEngine\CliCommand;
50		use Phix_Project\ExceptionsLib1\Legacy_ErrorHandler;
51		use Phix_Project\ExceptionsLib1\Legacy_ErrorException;
52		use DataSift\Stone\ConfigLib\E5xx_ConfigFileNotFound;
53		use DataSift\Stone\ConfigLib\E5xx_InvalidConfigFile;
54		use DataSift\Storyplayer\PlayerLib\E4xx_NoSuchReport;
55		use DataSift\Storyplayer\PlayerLib\PhaseGroup_Player;
56		use DataSift\Storyplayer\PlayerLib\StoryTeller;
57		use DataSift\Storyplayer\PlayerLib\Script_Player;
58		use DataSift\Storyplayer\Console\ScriptConsole;
59		use DataSift\Storyplayer\Injectables;
60
61		/**
62		* A command to play a script
63		*
64		* @category Libraries
65		* @package Storyplayer/Cli
66		* @author Stuart Herbert <[email protected]>
67		* @copyright 2011-present Mediasift Ltd www.datasift.com
68		* @license http://www.opensource.org/licenses/bsd-license.php BSD License
69		* @link http://datasift.github.io/storyplayer
70		*/
71		class Script_Command extends BaseCommand implements CliSignalHandler
72		{
73		/**
74		* should we let background processes survive when we shutdown?
75		* @var boolean
76		*/
77		protected $persistProcesses = false;
78
79		// we need to track this for handling CTRL-C
80		protected $st;
81
82		// we track this for convenience
83		protected $output;
84
85		// our list of scripts to execute
86		protected $scriptList;
87
88		// our injected data / services
89		// needed for when user presses CTRL+C
90		protected $injectables;
91
92	View Code Duplication	public function __construct($injectables)
93		{
94		parent::__construct($injectables);
95
96		// define the command
97		$this->setName('script');
98		$this->setShortDescription('run an automation script');
99		$this->setLongDescription(
100		"Use this command to play an automation script."
101		.PHP_EOL
102		);
103		$this->setArgsList(array(
104		"[<script.php>]" => "run a script"
105		));
106
107		// the switches that this command supports
108		$this->addFeature(new Feature_ConsoleSupport);
109		$this->addFeature(new Feature_DeviceSupport);
110		$this->addFeature(new Feature_TestEnvironmentConfigSupport);
111		$this->addFeature(new Feature_SystemUnderTestConfigSupport);
112		$this->addFeature(new Feature_LocalhostSupport);
113		$this->addFeature(new Feature_ActiveConfigSupport);
114		$this->addFeature(new Feature_DefinesSupport);
115		$this->addFeature(new Feature_PhaseLoaderSupport);
116		$this->addFeature(new Feature_ProseLoaderSupport);
117		$this->addFeature(new Feature_TestUsersSupport);
118
119		// now setup all of the switches that we support
120		$this->addFeatureSwitches();
121		}
122
123		/**
124		*
125		* @param CliEngine $engine
126		* @param array $params
127		* @param Injectables\|null $injectables
128		* @return integer
129		*/
130		public function processCommand(CliEngine $engine, $params = array(), $injectables = null)
131		{
132		// we need to wrap our code to catch old-style PHP errors
133		$legacyHandler = new Legacy_ErrorHandler();
134		try {
135		$returnCode = $legacyHandler->run([$this, 'processInsideLegacyHandler'], [$engine, $params, $injectables]);
136		return $returnCode;
137		}
138		catch (Exception $e) {
139		$injectables->output->logCliError($e->getMessage());
140		$engine->options->dev = true;
141		if (isset($engine->options->dev) && $engine->options->dev) {
142		$injectables->output->logCliError("Stack trace is:\n\n" . $e->getTraceAsString());
143		}
144		exit(1);
		0 ignored issues – show Coding Style Compatibility introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The method `processCommand()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
145		}
146		}
147
148		public function processInsideLegacyHandler(CliEngine $engine, $params = array(), $injectables = null)
149		{
150		// make sure we're using the ScriptConsole by default
151		$injectables->output->usePluginInSlot(new ScriptConsole, 'console');
152
153		// process the common functionality
154		$this->initFeaturesBeforeModulesAvailable($engine);
155
156		// now it is safe to create our shorthand
157		$runtimeConfig = $injectables->getRuntimeConfig();
158		$runtimeConfigManager = $injectables->getRuntimeConfigManager();
159		$output = $injectables->output;
160
161		// save the output for use in other methods
162		$this->output = $output;
163
164		// at this point, all of the services / data held in $injectables
165		// has been initialised and is ready for use
166
167		// create a new StoryTeller object
168		$st = new StoryTeller($injectables);
169
170		// remember our $st object, as we'll need it for our
171		// shutdown function
172		$this->st = $st;
173
174		// now that we have $st, we can initialise any feature that
175		// wants to use our modules
176		$this->initFeaturesAfterModulesAvailable($st, $engine, $injectables);
177
178		// install signal handling, now that $this->st is defined
179		//
180		// we wouldn't want signal handling called out of order :)
181		$this->initSignalHandling($injectables);
182
183		// build our list of stories to run
184		$this->initScriptList($engine, $injectables, $params);
185
186		// and we're ready to tell the world that we're here
187		$output->startStoryplayer(
188		$engine->getAppVersion(),
189		$engine->getAppUrl(),
190		$engine->getAppCopyright(),
191		$engine->getAppLicense()
192		);
193
194		// $this->scriptList contains one or more things to run
195		//
196		// let's play each of them in order
197		foreach ($this->scriptList as $player)
198		{
199		// play the story(ies)
200		$player->play($st, $injectables);
201		}
202
203		// write out any changed runtime config to disk
204		$runtimeConfigManager->saveRuntimeConfig($runtimeConfig, $output);
205
206		// tell the output plugins that we're all done
207		$output->endStoryplayer(0);
208
209		// all done
210		return 0;
211		}
212
213		// ==================================================================
214		//
215		// the individual initX() methods
216		//
217		// these are processed after the objects defined in the
218		// CommonFunctionalitySupport trait have been initialised
219		//
220		// ------------------------------------------------------------------
221
222		/**
223		*
224		* @param Injectables $injectables
225		* @return void
226		*/
227	View Code Duplication	protected function initSignalHandling(Injectables $injectables)
228		{
229		// we need to remember the injectables, for when we handle CTRL+C
230		$this->injectables = $injectables;
231
232		// setup signal handling
233		pcntl_signal(SIGTERM, array($this, 'sigtermHandler'));
234		pcntl_signal(SIGINT , array($this, 'sigtermHandler'));
235		}
236
237		/**
238		*
239		* @param CliEngine $cliEngine
240		* @param Injectables $injectables
241		* @param array $cliParams
242		* @return void
243		*/
244		protected function initScriptList(CliEngine $cliEngine, Injectables $injectables, $cliParams)
245		{
246		// our list of stories to play
247		$this->scriptList = [];
248
249		foreach ($cliParams as $cliParam) {
250		// figure out what to do?
251		if (is_dir($cliParam)) {
252		$this->scriptList = $this->scriptList + $this->addScriptsFromFolder($cliEngine, $injectables, $cliParam);
253		}
254		else if (is_file($cliParam)) {
255		// are we loading a story, or a list of stories?
256		$paramParts = explode('.', $cliParams[0]);
257		$paramSuffix = end($paramParts);
258
259		switch ($paramSuffix) {
260		case 'php':
261		$this->scriptList = $this->scriptList + $this->addScriptFromFile($cliEngine, $injectables, $cliParam);
262		break;
263
264		case 'json':
265		$this->scriptList = $this->scriptList + $this->addScriptsFromFile($cliEngine, $injectables, $cliParam);
266		break;
267
268		default:
269		$this->output->logCliError("unsupported script file '{$cliParam}'");
270		exit(1);
		0 ignored issues – show Coding Style Compatibility introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The method `initScriptList()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
271		}
272		}
273		else {
274		// if we get here, we've no idea what to do
275		$this->output->logCliError("no such file: '{$cliParam}'");
276		exit(1);
		0 ignored issues – show Coding Style Compatibility introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The method `initScriptList()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
277		}
278		}
279		}
280
281		// ==================================================================
282		//
283		// Story loading
284		//
285		// ------------------------------------------------------------------
286
287		protected function addScriptFromFile(CliEngine $engine, Injectables $injectables, $storyFile)
		0 ignored issues – show Unused Code introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The parameter `$engine` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
288		{
289		// these are the players we want to execute for the story
290		$return = [
291		new Script_Player($storyFile, $injectables),
292		];
293
294		// all done
295		return $return;
296		}
297
298		protected function addScriptsFromFolder(CliEngine $engine, Injectables $injectables, $folder)
		0 ignored issues – show Unused Code introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The parameter `$engine` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The parameter `$injectables` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The parameter `$folder` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
299		{
300		// tbd
301		}
302
303		protected function addScriptsFromFile(CliEngine $engine, Injectables $injectables, $file)
		0 ignored issues – show Unused Code introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The parameter `$engine` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The parameter `$injectables` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The parameter `$file` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
304		{
305		// tbd
306		}
307
308		// ==================================================================
309		//
310		// SIGNAL handling
311		//
312		// ------------------------------------------------------------------
313
314		/**
315		*
316		* @param integer $signo
317		* @return void
318		*/
319		public function sigtermHandler($signo)
320		{
321		// tell the user what is happening
322		echo "\n";
323		echo "============================================================\n";
324		echo "USER ABORT!!\n";
325		echo "============================================================\n";
326		echo "\n";
327
328		// cleanup
329		$phasesPlayer = new PhaseGroup_Player();
330		$phasesPlayer->playPhases(
331		"user abort",
332		$this->st,
333		$this->injectables,
334		$this->injectables->activeConfig->getData('storyplayer.phases.userAbort'),
335		null
336		);
337
338		// force a clean shutdown
339		exit(1);
		0 ignored issues – show Coding Style Compatibility introduced 2015-12-07 15:44 UTC by Report Bug Copy Issue Report The method `sigtermHandler()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
340		}
341		}

datasift / storyplayer

GitHub Access Token became invalid

Script_Command::addScriptsFromFolder() A last analyzed 2016-04-05 10:50 UTC

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like

Script_Command::addScriptsFromFolder() A
last analyzed 2016-04-05 10:50 UTC