datasift / storyplayer

src/php/DataSift/Storyplayer/Reports/TapReport.php

Doc Comments -1 removed lines

@@ -177,7 +177,6 @@
     }
 
     /**
-     * @param integer $level
      * @param string $msg
      * @return void
      */

src/php/Prose/BaseCleanup.php

Doc Comments +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
      * __construct
      *
      * @param StoryTeller $st The StoryTeller object
-     * @param array $args Any arguments to be used in this Prose module
+     * @param string[] $args Any arguments to be used in this Prose module
      *
      * @return parent::__construct
      */

src/php/DataSift/Storyplayer/Cli/BuildTestEnvironment/Command.php

Indentation +1 added lines, -1 removed lines

@@ -302,7 +302,7 @@
         // do we skip destroying the test environment?
         if ($this->st->getPersistTestEnvironment()) {
             echo PHP_EOL . "* Warning: NOT destroying test environment" . PHP_EOL
-                 .         "           --reuse-target flag is set" . PHP_EOL;
+                    .         "           --reuse-target flag is set" . PHP_EOL;
         }
 
         // cleanup

Spacing +1 added lines, -1 removed lines

@@ -267,7 +267,7 @@
 
         // setup signal handling
         pcntl_signal(SIGTERM, array($this, 'sigtermHandler'));
-        pcntl_signal(SIGINT , array($this, 'sigtermHandler'));
+        pcntl_signal(SIGINT, array($this, 'sigtermHandler'));
     }
 
     /**

Braces +1 added lines, -2 removed lines

@@ -218,8 +218,7 @@
         // $this->playerList contains one or more things to play
         //
         // let's play each of them in order
-        foreach ($this->playerList as $player)
-        {
+        foreach ($this->playerList as $player) {
             // execute each player in turn
             //
             // they may also have their own list of nested players

src/php/DataSift/Storyplayer/Cli/E4xx/NoSystemUnderTestSpecified.php

Indentation +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
     public function __construct()
     {
         $msg = "You need to use -s to specify which system to test." . PHP_EOL . PHP_EOL
-             . "Use 'storyplayer --list-systems' to see the list of known systems under test.";
+                . "Use 'storyplayer --list-systems' to see the list of known systems under test.";
         parent::__construct(400, $msg, $msg);
     }
 }
\ No newline at end of file

src/php/DataSift/Storyplayer/Cli/Feature/ColorSupport.php

Indentation +12 added lines, -12 removed lines

@@ -78,20 +78,20 @@
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// which colour mode are we in?
-    	switch ($engine->options->color) {
-    		case Feature_ColorSwitch::NO_COLOR:
-    			$injectables->output->disableColourSupport();
-    			break;
+        // which colour mode are we in?
+        switch ($engine->options->color) {
+            case Feature_ColorSwitch::NO_COLOR:
+                $injectables->output->disableColourSupport();
+                break;
 
-    		case Feature_ColorSwitch::ALWAYS_COLOR:
-    			$injectables->output->enforceColourSupport();
-    			break;
+            case Feature_ColorSwitch::ALWAYS_COLOR:
+                $injectables->output->enforceColourSupport();
+                break;
 
-    		case Feature_ColorSwitch::AUTO_COLOR:
-    			$injectables->output->enableColourSupport();
-    			break;
-    	}
+            case Feature_ColorSwitch::AUTO_COLOR:
+                $injectables->output->enableColourSupport();
+                break;
+        }
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)

src/php/DataSift/Storyplayer/Cli/Feature/DefineSwitch.php

Braces +1 added lines, -2 removed lines

@@ -94,8 +94,7 @@
             $engine->options->defines = new stdClass;
         }
 
-        foreach ($params as $param)
-        {
+        foreach ($params as $param) {
             // split up the setting
             $parts = explode('=', $param);
             $key   = array_shift($parts);

src/php/DataSift/Storyplayer/Cli/Feature/PersistDeviceSupport.php

Indentation +4 added lines, -4 removed lines

@@ -70,14 +70,14 @@
 {
     public function addSwitches(CliCommand $command, $injectables)
     {
-    	$command->addSwitches([
-    		new Feature_PersistDeviceSwitch,
-    	]);
+        $command->addSwitches([
+            new Feature_PersistDeviceSwitch,
+        ]);
     }
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// no-op
+        // no-op
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)

src/php/DataSift/Storyplayer/Cli/Feature/PersistProcessesSupport.php

Indentation +4 added lines, -4 removed lines

@@ -70,14 +70,14 @@
 {
     public function addSwitches(CliCommand $command, $injectables)
     {
-    	$command->addSwitches([
-    		new Feature_PersistProcessesSwitch,
-    	]);
+        $command->addSwitches([
+            new Feature_PersistProcessesSwitch,
+        ]);
     }
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// no-op
+        // no-op
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)

src/php/DataSift/Storyplayer/Cli/Feature/PersistReuseTargetSupport.php

Indentation +7 added lines, -7 removed lines

@@ -70,21 +70,21 @@
 {
     public function addSwitches(CliCommand $command, $injectables)
     {
-    	$command->addSwitches([
-    		new Feature_PersistTargetSwitch,
-    		new Feature_ReuseTargetSwitch
-    	]);
+        $command->addSwitches([
+            new Feature_PersistTargetSwitch,
+            new Feature_ReuseTargetSwitch
+        ]);
     }
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// no-op
+        // no-op
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)
     {
-    	// shorthand
-    	$output = $injectables->output;
+        // shorthand
+        $output = $injectables->output;
 
         // are we keeping the test environment hanging around afterwards?
         if (isset($engine->options->persistTarget) && $engine->options->persistTarget)

Braces +3 added lines, -6 removed lines

@@ -87,8 +87,7 @@ 
     	$output = $injectables->output;
 
         // are we keeping the test environment hanging around afterwards?
-        if (isset($engine->options->persistTarget) && $engine->options->persistTarget)
-        {
+        if (isset($engine->options->persistTarget) && $engine->options->persistTarget) {
             $injectables->activeConfig->setData('storyplayer.phases.testEnvShutdown.TestEnvironmentDestruction', false);
             $injectables->activeConfig->unsetData('storyplayer.phases.userAbort.TestEnvironmentDestruction');
             $st->setPersistTestEnvironment();
@@ -96,8 +95,7 @@ 
 
         // are we trying to use a test environment that has previously
         // been persisted?
-        if (isset($engine->options->reuseTarget) && $engine->options->reuseTarget)
-        {
+        if (isset($engine->options->reuseTarget) && $engine->options->reuseTarget) {
             // does the target exist to be reused?
             $output->setSilentMode();
             $hasTarget = fromTargetsTable()->hasCurrentTestEnvironmentSignature();
@@ -128,8 +126,7 @@ 
             // if we get here, then we do not need to create the test environment
             $injectables->activeConfig->setData('storyplayer.phases.testEnvStartup.TestEnvironmentConstruction', false);
         }
-        else
-        {
+        else {
             // do we already have this target?
             //
             // this can happen when the test environment was previously