datasift / storyplayer

src/php/DataSift/Storyplayer/ConfigLib/WrappedConfig.php

Braces +1 added lines, -2 removed lines

@@ -234,8 +234,7 @@
         //
         // if this fails, we do not know why
         $success = mkdir($configDir, 0700, true);
-        if (!$success)
-        {
+        if (!$success) {
             // cannot create it - bail out now
             throw new E4xx_ConfigPathCannotBeCreated($configDir);
         }

src/php/DataSift/Storyplayer/Injectables/RunningTestEnvironmentsListSupport.php

Indentation +3 added lines, -3 removed lines

@@ -63,9 +63,9 @@
 {
     public $runningTestEnvironmentsList;
 
-	public function initRunningTestEnvironmentsListSupport()
-	{
+    public function initRunningTestEnvironmentsListSupport()
+    {
         $this->runningTestEnvironmentsList = new RunningTestEnvironmentsList();
         $this->runningTestEnvironmentsList->findConfigs();
-	}
+    }
 }

src/php/Prose/FromArray.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
  * @license   http://www.opensource.org/licenses/bsd-license.php  BSD License
  * @link      http://datasift.github.io/storyplayer
  */
-class FromArray extends Prose{
+class FromArray extends Prose {
 
     /**
      * Sets a value in an array for a . delimited path
@@ -68,7 +68,7 @@ 
     public function setValueInArray(&$array, $path, $val)
     {
         $pathAsArray = fromString()->splitDotSeparatedPath($path);
-        for ($i=&$array; $key=array_shift($pathAsArray); $i=&$i[$key]) {
+        for ($i = &$array; $key = array_shift($pathAsArray); $i = &$i[$key]) {
             if (!isset($i[$key])) {
                 $i[$key] = array();
             }

Braces +2 added lines, -1 removed lines

@@ -55,7 +55,8 @@
  * @license   http://www.opensource.org/licenses/bsd-license.php  BSD License
  * @link      http://datasift.github.io/storyplayer
  */
-class FromArray extends Prose{
+class FromArray extends Prose
+{
 
     /**
      * Sets a value in an array for a . delimited path

src/php/Prose/FromString.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
  * @license   http://www.opensource.org/licenses/bsd-license.php  BSD License
  * @link      http://datasift.github.io/storyplayer
  */
-class FromString extends Prose{
+class FromString extends Prose {
 
     /**
      * Reduces a dot separated path e.g. fb.parent.context by one from

Braces +6 added lines, -3 removed lines

@@ -55,7 +55,8 @@ 
  * @license   http://www.opensource.org/licenses/bsd-license.php  BSD License
  * @link      http://datasift.github.io/storyplayer
  */
-class FromString extends Prose{
+class FromString extends Prose
+{
 
     /**
      * Reduces a dot separated path e.g. fb.parent.context by one from
@@ -71,9 +72,11 @@ 
         $parts = $this->splitDotSeparatedPath($pathToReduce);
         if (count($parts) == 0) {
             return null;
-        } else if (count($parts) == 1) {
+        }
+        else if (count($parts) == 1) {
             return $parts[0];
-        } else {
+        }
+        else {
             array_pop($parts);
             return implode('.', $parts);
         }

src/php/Prose/UsingTargetsTable.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
         $log = usingLog()->startAction("add current test environment to targets table");
 
         // get the details to add
-        $testEnvSig  = $this->st->getTestEnvironmentSignature();
+        $testEnvSig = $this->st->getTestEnvironmentSignature();
 
         // add it
         usingRuntimeTable($this->entryKey)->addItem(FromTargetsTable::SIGNATURE_KEY, $testEnvSig);

storyplayer/test-environments/vagrant-vbox-centos7-empty/main.php

Indentation +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
 $group1 = $testEnv->newGroup('vagrant', new Vagrant_GroupAdapter);
 $group1->newHost('default', new Vagrant_VirtualboxHostAdapter)
-       ->setOperatingSystem(new CentOS_7_HostAdapter);
+        ->setOperatingSystem(new CentOS_7_HostAdapter);
 
 $testEnv->setModuleSettings((object)[
     "http" => (object)[

src/php/Storyplayer/SPv2/Modules/Asserts.php

Doc Comments +6 added lines, -6 removed lines

@@ -73,7 +73,7 @@ 
      *
      * @param  array $actual
      *         the array to be tested
-     * @return \Prose\AssertsArray
+     * @return AssertsArray
      */
     public static function assertsArray($actual)
     {
@@ -85,7 +85,7 @@ 
      *
      * @param  boolean $actual
      *         the data to be tested
-     * @return \Prose\AssertsBoolean
+     * @return AssertsBoolean
      */
     public static function assertsBoolean($actual)
     {
@@ -97,7 +97,7 @@ 
      *
      * @param  double $actual
      *         the data to be tested
-     * @return \Prose\AssertsDouble
+     * @return AssertsDouble
      */
     public static function assertsDouble($actual)
     {
@@ -109,7 +109,7 @@ 
      *
      * @param  int $actual
      *         the data to be tested
-     * @return \Prose\AssertsInteger
+     * @return AssertsInteger
      */
     public static function assertsInteger($actual)
     {
@@ -121,7 +121,7 @@ 
      *
      * @param  object $actual
      *         the data to be tested
-     * @return \Prose\AssertsObject
+     * @return AssertsObject
      */
     public static function assertsObject($actual)
     {
@@ -133,7 +133,7 @@ 
      *
      * @param  string $actual
      *         the data to be tested
-     * @return \Prose\AssertsString
+     * @return AssertsString
      */
     public static function assertsString($actual)
     {

src/php/Storyplayer/SPv2/Stories/BuildStory.php

Doc Comments -1 removed lines

@@ -50,7 +50,6 @@
     /**
      * Create a new story object
      *
-     * @param  string $category the category that the story belongs to
      * @return Story            the new story object to use
      */
     public static function newStory()

src/php/Storyplayer/SPv2/Modules/Browser/UsingBrowser.php

Doc Comments +8 added lines, -5 removed lines

@@ -72,7 +72,7 @@ 
     /**
      * tick a checkbox or radio button if it has not yet been checked
      *
-     * @return \DataSift\Storyplayer\BrowserLib\SingleElementAction
+     * @return SingleElementAction
      */
     public function check()
     {
@@ -101,7 +101,7 @@ 
      * remove any content from an input box, or untick a checkbox or
      * radio button
      *
-     * @return \DataSift\Storyplayer\BrowserLib\SingleElementAction
+     * @return SingleElementAction
      */
     public function clear()
     {
@@ -132,7 +132,7 @@ 
     /**
      * Send a 'click' to the selected element
      *
-     * @return \DataSift\Storyplayer\BrowserLib\SingleElementAction
+     * @return SingleElementAction
      */
     public function click()
     {
@@ -154,7 +154,7 @@ 
      *
      * @param  string $label
      *         the human-readable text of the option to select
-     * @return \DataSift\Storyplayer\BrowserLib\SingleElementAction
+     * @return SingleElementAction
      */
     public function select($label)
     {
@@ -185,7 +185,7 @@ 
      *
      * @param  string $text
      *         the text to type
-     * @return \DataSift\Storyplayer\BrowserLib\SingleElementAction
+     * @return SingleElementAction
      */
     public function type($text)
     {
@@ -214,6 +214,9 @@ 
     //
     // ------------------------------------------------------------------
 
+    /**
+     * @param string $url
+     */
     public function gotoPage($url)
     {
         // some shorthand to make things easier to read

Spacing +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
      */
     public function select($label)
     {
-        $action = function ($element, $elementName, $elementDesc) use ($label) {
+        $action = function($element, $elementName, $elementDesc) use ($label) {
 
             // what are we doing?
             $log = usingLog()->startAction("choose option '$label' from $elementDesc '$elementName'");

Braces +1 added lines, -2 removed lines

@@ -77,8 +77,7 @@
 
             return $return;
         }
-        catch (Exception $e)
-        {
+        catch (Exception $e) {
             throw new E5xx_ActionFailed(__METHOD__, $e->getMessage());
         }
     }