datasift / storyplayer

src/php/DataSift/Storyplayer/Phases/TestEnvironmentDestructionPhase.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     public function doPhase($thingBeingPlayed = null)
     {
         // shorthand
-        $st    = $this->st;
+        $st = $this->st;
 
         // our return value
         $phaseResult = $this->getNewPhaseResult();

src/php/DataSift/Storyplayer/Phases/TestSetupPhase.php

Braces +2 added lines, -4 removed lines

@@ -73,8 +73,7 @@ 
         $phaseResult = $this->getNewPhaseResult();
 
         // do we have anything to do?
-        if (!$story->hasTestSetup())
-        {
+        if (!$story->hasTestSetup()) {
             $phaseResult->setContinuePlaying(
                 $phaseResult::HASNOACTIONS,
                 "story has no test setup instructions"
@@ -122,8 +121,7 @@ 
             );
             $storyResult->setStoryIsIncomplete($phaseResult);
         }
-        catch (Exception $e)
-        {
+        catch (Exception $e) {
             // something went wrong ... the test cannot continue
             $phaseResult->setPlayingFailed(
                 $phaseResult::ERROR,

src/php/DataSift/Storyplayer/Phases/TestTeardownPhase.php

Braces +2 added lines, -4 removed lines

@@ -73,8 +73,7 @@ 
         $phaseResult = $this->getNewPhaseResult();
 
         // do we have anything to do?
-        if (!$story->hasTestTeardown())
-        {
+        if (!$story->hasTestTeardown()) {
             $phaseResult->setContinuePlaying(
                 $phaseResult::HASNOACTIONS,
                 "story has no test teardown instructions"
@@ -124,8 +123,7 @@ 
             );
             $storyResult->setStoryIsIncomplete($phaseResult);
         }
-        catch (Exception $e)
-        {
+        catch (Exception $e) {
             // we still want to continue at this stage
             $phaseResult->setContinuePlaying(
                 $phaseResult::ERROR,

src/php/DataSift/Storyplayer/PlayerLib/Action/Logger.php

Braces +2 added lines, -4 removed lines

@@ -78,8 +78,7 @@ 
     public function startAction($message, $codeLine = null)
     {
         // do we have an open action?
-        if (!$this->action || !$this->action->getIsOpen())
-        {
+        if (!$this->action || !$this->action->getIsOpen()) {
             $openItem = $this->action = new Action_LogItem($this->injectables, 1);
         }
         else {
@@ -93,8 +92,7 @@ 
     public function closeAllOpenActions()
     {
         // do we have any empty log items?
-        if (!$this->action)
-        {
+        if (!$this->action) {
             return;
         }
 

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

Braces +2 added lines, -1 removed lines

@@ -60,7 +60,8 @@
     /**
      * @param string $err
      */
-    public function __construct($err) {
+    public function __construct($err)
+    {
         $msg = "Invalid config - $err";
         parent::__construct(500, $msg, $msg);
     }

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

Braces +2 added lines, -1 removed lines

@@ -61,7 +61,8 @@
     /**
      * @param string $key
      */
-    public function __construct($key) {
+    public function __construct($key)
+    {
         $msg = "No such data '{$key}' in the checkpoint";
         parent::__construct(500, $msg, $msg);
     }

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

Braces +2 added lines, -1 removed lines

@@ -61,7 +61,8 @@
     /**
      * @param string $className
      */
-    public function __construct($className) {
+    public function __construct($className)
+    {
         $msg = "Class '{$className}' does not inherit from Prose\\Prose";
         parent::__construct(500, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/Phase/Loader.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
         //
         // we always search the generic 'Phases' namespace first, in case
         // users don't want to uniquely namespace their Phase classes
-        $this->namespaces = array ("Phases");
+        $this->namespaces = array("Phases");
 
         // add in any additional namespaces we've been asked to search
         foreach ($namespaces as $namespace) {

src/php/DataSift/Storyplayer/PlayerLib/PhaseGroup/Player.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
     public function playPhases($activity, StoryTeller $st, Injectables $injectables, $phases, $thingBeingPlayed)
     {
         // shorthand
-        $output  = $st->getOutput();
+        $output = $st->getOutput();
 
         // we are going to need something to help us load each of our
         // phases
@@ -98,7 +98,7 @@ 
 
         // the result of playing this group of phases
         $groupResult = null;
-        if ($thingBeingPlayed){
+        if ($thingBeingPlayed) {
             $groupResult = $thingBeingPlayed->getResult();
             $groupResult->setActivity($activity);
         }

Braces +3 added lines, -5 removed lines

@@ -98,7 +98,7 @@ 
 
         // the result of playing this group of phases
         $groupResult = null;
-        if ($thingBeingPlayed){
+        if ($thingBeingPlayed) {
             $groupResult = $thingBeingPlayed->getResult();
             $groupResult->setActivity($activity);
         }
@@ -110,8 +110,7 @@ 
         //
         // 1. all listed phases have been executed, or
         // 2. one of the phases says that the story has failed
-        foreach ($phasesToPlay as $phaseName => $phaseData)
-        {
+        foreach ($phasesToPlay as $phaseName => $phaseData) {
             // shorthand
             $phase    = $phaseData['phase'];
             $isActive = $phaseData['isActive'];
@@ -128,8 +127,7 @@ 
 
                 // now, what do we do?
                 $nextAction = $phaseResult->getNextAction();
-                switch ($nextAction)
-                {
+                switch ($nextAction) {
                     case self::NEXT_SKIP:
                         // why?
                         if ($phaseResult->getPhaseIsBlacklisted()) {