datasift / storyplayer

src/php/DataSift/Storyplayer/OutputLib/E4xx/CannotOpenOutputFile.php

Braces +2 added lines, -1 removed lines

@@ -58,7 +58,8 @@
  */
 class E4xx_OutputFilenameIsAReservedName extends Exxx_Exception
 {
-    public function __construct($filename) {
+    public function __construct($filename)
+    {
         $msg = "'{$filename}' is reserved for internal use; please use a different filename";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/Phases/ShutdownHandlersPhase.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
     public function doPhase($phaseResults)
     {
         // shorthand
-        $st      = $this->st;
+        $st = $this->st;
 
         // our result object
         $phaseResult = $this->getNewPhaseResult();

src/php/DataSift/Storyplayer/PlayerLib/E5xx/CheckpointIsReadOnly.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/E5xx/NoRemoteWebDriverConfig.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/E5xx/NoRemoteWebDriverUrl.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/E5xx/NoSauceLabsApiKey.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/E5xx/NoSauceLabsConfig.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/E5xx/NoSauceLabsUsername.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/Story.php

Spacing +12 added lines, -12 removed lines

@@ -622,18 +622,18 @@
         // tell the template which story it is being used with
         $tmpl->setStory($this);
 
-        $tmpl->hasTestCanRunCheck()         && $this->addTestCanRunCheck($tmpl->getTestCanRunCheck());
-        $tmpl->hasTestSetup()               && $this->addTestSetup($tmpl->getTestSetup());
-        $tmpl->hasTestTeardown()            && $this->addTestTeardown($tmpl->getTestTeardown());
-        $tmpl->hasPerPhaseSetup()           && $this->addPerPhaseSetup($tmpl->getPerPhaseSetup());
-        $tmpl->hasPerPhaseTeardown()        && $this->addPerPhaseTeardown($tmpl->getPerPhaseTeardown());
-        $tmpl->hasDeviceSetup()             && $this->addDeviceSetup($tmpl->getDeviceSetup());
-        $tmpl->hasDeviceTeardown()          && $this->addDeviceTeardown($tmpl->getDeviceTeardown());
-        $tmpl->hasHints()                   && $this->addHints($tmpl->getHints());
-        $tmpl->hasPreTestPrediction()       && $this->addPreTestPrediction($tmpl->getPreTestPrediction());
-        $tmpl->hasPreTestInspection()       && $this->addPreTestInspection($tmpl->getPreTestInspection());
-        $tmpl->hasAction()                  && $this->addAction($tmpl->getAction());
-        $tmpl->hasPostTestInspection()      && $this->addPostTestInspection($tmpl->getPostTestInspection());
+        $tmpl->hasTestCanRunCheck() && $this->addTestCanRunCheck($tmpl->getTestCanRunCheck());
+        $tmpl->hasTestSetup() && $this->addTestSetup($tmpl->getTestSetup());
+        $tmpl->hasTestTeardown() && $this->addTestTeardown($tmpl->getTestTeardown());
+        $tmpl->hasPerPhaseSetup() && $this->addPerPhaseSetup($tmpl->getPerPhaseSetup());
+        $tmpl->hasPerPhaseTeardown() && $this->addPerPhaseTeardown($tmpl->getPerPhaseTeardown());
+        $tmpl->hasDeviceSetup() && $this->addDeviceSetup($tmpl->getDeviceSetup());
+        $tmpl->hasDeviceTeardown() && $this->addDeviceTeardown($tmpl->getDeviceTeardown());
+        $tmpl->hasHints() && $this->addHints($tmpl->getHints());
+        $tmpl->hasPreTestPrediction() && $this->addPreTestPrediction($tmpl->getPreTestPrediction());
+        $tmpl->hasPreTestInspection() && $this->addPreTestInspection($tmpl->getPreTestInspection());
+        $tmpl->hasAction() && $this->addAction($tmpl->getAction());
+        $tmpl->hasPostTestInspection() && $this->addPostTestInspection($tmpl->getPostTestInspection());
 
         // remember this template for future use
         $this->storyTemplates[] = $tmpl;

Braces +1 added lines, -2 removed lines

@@ -1137,8 +1137,7 @@
     public function getOneAction()
     {
         // do we have any callbacks to pick from?
-        if (count($this->actionsCallbacks) == 0)
-        {
+        if (count($this->actionsCallbacks) == 0) {
             throw new E5xx_NoStoryActions($this->getName());
         }
 

Doc Comments +1 added lines

@@ -700,6 +700,7 @@
     }
 
     /**
+     * @param integer $version
      * @return Story
      */
     public function requiresStoryplayerVersion($version)