datasift / storyplayer

src/php/DataSift/Storyplayer/BrowserLib/BaseElementAction.php

Braces +2 added lines, -1 removed lines

@@ -323,7 +323,8 @@
         $searchTerm = $methodArgs[0];
 
         $searchType = $this->determineSearchType($words);
-        if ($searchType === null) {             // we do not understand how to find the target field
+        if ($searchType === null) {
+// we do not understand how to find the target field
             throw new E5xx_ActionFailed(__CLASS__ . '::' . $methodName, "could not work out how to find the target to action upon");
         }
 

src/php/DataSift/Storyplayer/BrowserLib/DomElementSearch.php

Braces +2 added lines, -4 removed lines

@@ -245,8 +245,7 @@ 
 
         // search all of the label elements to find an associated input
         // element that we can safely use
-        foreach ($labelElements as $labelElement)
-        {
+        foreach ($labelElements as $labelElement) {
             try {
                 // add each element that matches this label
                 $retval[] = $this->getElementAssociatedWithLabelElement($labelElement, $labelText);
@@ -289,8 +288,7 @@ 
         }
 
         // what do we do next?
-        if ($inputElementId !== null)
-        {
+        if ($inputElementId !== null) {
             // where does the 'for' attribute go?
             try {
                 $inputElement = $log->addStep("find the input element with the id '{$inputElementId}'", function() use($topElement, $inputElementId) {

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

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/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/PersistReuseTargetSupport.php

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 = $st->fromTargetsTable()->hasCurrentTestEnvironment();
@@ -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

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

Braces +1 added lines, -2 removed lines

@@ -112,8 +112,7 @@
         if (isset($engine->options->readOnlyTestUsers) && $engine->options->readOnlyTestUsers) {
             usingUsers()->setUsersFileIsReadOnly();
         }
-        else
-        {
+        else {
             $setting = fromTestEnvironment()->getModuleSetting("users.readOnly");
             if ($setting === true) {
                 usingUsers()->setUsersFileIsReadOnly();

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

Braces +11 added lines, -8 removed lines

@@ -123,20 +123,22 @@ 
         $downloader = new FileDownloader();
 
         // let's get the files downloaded
-        foreach ($filesToDownload as $file){
+        foreach ($filesToDownload as $file) {
 
-            if (!is_object($file->url)){
+            if (!is_object($file->url)) {
                 $url = $file->url;
-            } else {
+            }
+            else {
                 $platform = strtolower(php_uname("s") . '/' . php_uname("m"));
-                if (isset($file->url->{$platform})){
+                if (isset($file->url->{$platform})) {
                     $url = $file->url->{$platform};
-                } else if (isset($file->url->generic)){
+                }
+                else if (isset($file->url->generic)) {
                     $url =  $file->url->generic;
                 }
             }
 
-            if (!isset($url)){
+            if (!isset($url)) {
                 throw new Exception("No supported downloads for ".$file->name);
             }
 
@@ -145,7 +147,8 @@ 
             $headers = array_change_key_case(get_headers($url, 1),CASE_LOWER);
             if ( !preg_match('/HTTP\/1\.(0|1) 200 OK/', $headers[0] ) ) {
                 $fileSize = $headers['content-length'][1];
-            } else {
+            }
+            else {
                 $fileSize = $headers['content-length'];
             }
 
@@ -158,7 +161,7 @@ 
 
             // Make sure that the relevant files are executable
             if (isset($file->makeExecutable)) {
-                foreach ($file->makeExecutable as $exec){
+                foreach ($file->makeExecutable as $exec) {
                     chmod("./vendor/bin/".$exec, 0755);
                 }
             }

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

Braces +3 added lines, -6 removed lines

@@ -261,8 +261,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
@@ -314,8 +313,7 @@ 
 
         // setup the reports that have been requested
         $injectables->initReportLoaderSupport($injectables);
-        foreach ($engine->options->reports as $reportName => $reportFilename)
-        {
+        foreach ($engine->options->reports as $reportName => $reportFilename) {
             try {
                 $report = $injectables->reportLoader->loadReport($reportName, [ 'filename' => $reportFilename]);
             }
@@ -532,8 +530,7 @@ 
         echo "FINAL RESULTS\n";
         echo "\n";
 
-        foreach ($storyResults as $result)
-        {
+        foreach ($storyResults as $result) {
             echo Story_Player::$outcomeToText[$result->resultCode] . " :: " . $result->story->getName() . "\n";
         }
     }

src/php/DataSift/Storyplayer/Cli/RuntimeConfigManager.php

Braces +3 added lines, -6 removed lines

@@ -69,8 +69,7 @@ 
         static $configDir = null;
 
         // do we have a configDir remembered yet?
-        if (!$configDir)
-        {
+        if (!$configDir) {
             $configDir = getcwd() . '/.storyplayer';
         }
 
@@ -87,11 +86,9 @@ 
         $configDir = $this->getConfigDir();
 
         // does it exist?
-        if (!file_exists($configDir))
-        {
+        if (!file_exists($configDir)) {
             $success = mkdir($configDir, 0700, true);
-            if (!$success)
-            {
+            if (!$success) {
                 // cannot create it - bail out now
                 $output->logCliError("unable to create config directory '{$configDir}'");
                 exit(1);