datasift / storyplayer

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()) {

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

Spacing +6 added lines, -6 removed lines

@@ -78,37 +78,37 @@
      *
      * @var PhaseResult
      */
-    public $failedPhase     = null;
+    public $failedPhase = null;
 
     /**
      * did the story pass, fail, or otherwise go horribly wrong?
      * @var integer
      */
-    public $resultCode     = 0;
+    public $resultCode = 0;
 
     /**
      * when did we start playing this story?
      * @var float
      */
-    public $startTime       = null;
+    public $startTime = null;
 
     /**
      * when did we finish playing this story?
      * @var float
      */
-    public $endTime         = null;
+    public $endTime = null;
 
     /**
      * how long did the story take to play?
      * @var float
      */
-    public $durationTime    = null;
+    public $durationTime = null;
 
     /**
      * which file is this PhaseGroup_Result associated with?
      * @var string
      */
-    public $filename        = null;
+    public $filename = null;
 
     const UNKNOWN     = 0;
     const OKAY        = 1;

Braces +2 added lines, -4 removed lines

@@ -223,8 +223,7 @@ 
      */
     public function getPhaseGroupFailed()
     {
-        switch ($this->resultCode)
-        {
+        switch ($this->resultCode) {
             case self::FAIL:
             case self::ERROR:
             case self::INCOMPLETE:
@@ -240,8 +239,7 @@ 
      */
     public function getPhaseGroupSkipped()
     {
-        switch ($this->resultCode)
-        {
+        switch ($this->resultCode) {
             case self::BLACKLISTED:
             case self::SKIPPED:
                 return true;

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

Spacing +1 added lines, -1 removed lines

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

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

Spacing +1 added lines, -1 removed lines

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

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

Indentation +135 added lines, -135 removed lines

@@ -58,143 +58,143 @@
  */
 class Story_Checkpoint implements IteratorAggregate
 {
-	/**
-	 * the StoryTeller object
-	 *
-	 * @var \DataSift\Storyplayer\PlayerLib\StoryTeller
-	 */
-	private $st;
-
-	/**
-	 * keep track of whether the checkpoint is readonly (true) or
-	 * read-write(false)
-	 *
-	 * @var boolean
-	 */
-	private $readOnly = false;
-
-	/**
-	 * the data stored inside the checkpoint
-	 *
-	 * @var array
-	 */
-	private $data = array();
-
-	/**
-	 * our constructor
-	 *
-	 * @param StoryTeller $st
-	 *        The StoryTeller object (which we will cache)
-	 */
-	public function __construct(StoryTeller $st)
-	{
-		// remember the StoryTeller object for future use
-		$this->st = $st;
-	}
-
-	/**
-	 * is the checkpoint currently readonly?
-	 *
-	 * @return boolean
-	 *         TRUE if the checkpoint is currently readonly
-	 *         FALSE if you can change the data in the checkpoint
-	 */
-	public function getReadOnly()
-	{
-		return $this->readOnly;
-	}
-
-	/**
-	 * put the checkpoint into readonly mode
-	 */
-	public function setReadOnly()
-	{
-		$this->readOnly = true;
-	}
-
-	/**
-	 * put the checkpoint into read-write mode
-	 */
-	public function setReadWrite()
-	{
-		$this->readOnly = false;
-	}
-
-	/**
-	 * magic method to retrieve data from the checkpoint
-	 *
-	 * throws the E5xx_NoSuchDataInCheckpoint exception if you attempt
-	 * to get data that does not exist
-	 *
-	 * @param  string $key
-	 *         the name of the data to store
-	 * @return mixed
-	 *         the data stored in the checkpoint
-	 */
-	public function &__get($key)
-	{
-		// what are we doing?
-		$log = usingLog()->startAction("retrieve '{$key}' from the checkpoint");
-
-		// do we have the data to return?
-		if (!isset($this->data[$key])) {
-			// no, we do not
-			$log->endAction("'{$key}' is not in the checkpoint");
-			throw new E5xx_NoSuchDataInCheckpoint($key);
-		}
-
-		// yes, we do
+    /**
+     * the StoryTeller object
+     *
+     * @var \DataSift\Storyplayer\PlayerLib\StoryTeller
+     */
+    private $st;
+
+    /**
+     * keep track of whether the checkpoint is readonly (true) or
+     * read-write(false)
+     *
+     * @var boolean
+     */
+    private $readOnly = false;
+
+    /**
+     * the data stored inside the checkpoint
+     *
+     * @var array
+     */
+    private $data = array();
+
+    /**
+     * our constructor
+     *
+     * @param StoryTeller $st
+     *        The StoryTeller object (which we will cache)
+     */
+    public function __construct(StoryTeller $st)
+    {
+        // remember the StoryTeller object for future use
+        $this->st = $st;
+    }
+
+    /**
+     * is the checkpoint currently readonly?
+     *
+     * @return boolean
+     *         TRUE if the checkpoint is currently readonly
+     *         FALSE if you can change the data in the checkpoint
+     */
+    public function getReadOnly()
+    {
+        return $this->readOnly;
+    }
+
+    /**
+     * put the checkpoint into readonly mode
+     */
+    public function setReadOnly()
+    {
+        $this->readOnly = true;
+    }
+
+    /**
+     * put the checkpoint into read-write mode
+     */
+    public function setReadWrite()
+    {
+        $this->readOnly = false;
+    }
+
+    /**
+     * magic method to retrieve data from the checkpoint
+     *
+     * throws the E5xx_NoSuchDataInCheckpoint exception if you attempt
+     * to get data that does not exist
+     *
+     * @param  string $key
+     *         the name of the data to store
+     * @return mixed
+     *         the data stored in the checkpoint
+     */
+    public function &__get($key)
+    {
+        // what are we doing?
+        $log = usingLog()->startAction("retrieve '{$key}' from the checkpoint");
+
+        // do we have the data to return?
+        if (!isset($this->data[$key])) {
+            // no, we do not
+            $log->endAction("'{$key}' is not in the checkpoint");
+            throw new E5xx_NoSuchDataInCheckpoint($key);
+        }
+
+        // yes, we do
         $log->endAction($this->data[$key]);
 
-		// all done
-		return $this->data[$key];
-	}
-
-	/**
-	 * magic method to tell if the data is stored in the checkpoint or not
-	 *
-	 * @param  string  $key
-	 *         the name of the data to test for
-	 * @return boolean
-	 *         TRUE if the data exists in the checkpoint
-	 *         FALSE if the data does not exist in the checkpoint
-	 */
-	public function __isset($key)
-	{
-		return (isset($this->data[$key]));
-	}
-
-	/**
-	 * magic method to store data in the checkpoint
-	 *
-	 * throws the E5xx_CheckpointIsReadOnly exception if you attempt
-	 * to store data when the checkpoint is in readonly mode
-	 *
-	 * @param  string  $key
-	 *         the name of the data to store
-	 * @param  mixed   $value
-	 *         the value to store in the checkpoint
-	 * @return void
-	 */
-	public function __set($key, $value)
-	{
-		// what are we doing?
-		$log = usingLog()->startAction("store '{$key}' in the checkpoint");
-
-		// are we allowed to change the data at this time?
-		if ($this->readOnly)
-		{
-			// no, we are not
-			$log->endAction("checkpoint is readonly; did not store '{$key}'");
-			throw new E5xx_CheckpointIsReadOnly();
-		}
-
-		// if we get here, we're allowed to change the checkpoint
-		$this->data[$key] = $value;
-
-		// all done
-		$log->endAction($value);
-	}
+        // all done
+        return $this->data[$key];
+    }
+
+    /**
+     * magic method to tell if the data is stored in the checkpoint or not
+     *
+     * @param  string  $key
+     *         the name of the data to test for
+     * @return boolean
+     *         TRUE if the data exists in the checkpoint
+     *         FALSE if the data does not exist in the checkpoint
+     */
+    public function __isset($key)
+    {
+        return (isset($this->data[$key]));
+    }
+
+    /**
+     * magic method to store data in the checkpoint
+     *
+     * throws the E5xx_CheckpointIsReadOnly exception if you attempt
+     * to store data when the checkpoint is in readonly mode
+     *
+     * @param  string  $key
+     *         the name of the data to store
+     * @param  mixed   $value
+     *         the value to store in the checkpoint
+     * @return void
+     */
+    public function __set($key, $value)
+    {
+        // what are we doing?
+        $log = usingLog()->startAction("store '{$key}' in the checkpoint");
+
+        // are we allowed to change the data at this time?
+        if ($this->readOnly)
+        {
+            // no, we are not
+            $log->endAction("checkpoint is readonly; did not store '{$key}'");
+            throw new E5xx_CheckpointIsReadOnly();
+        }
+
+        // if we get here, we're allowed to change the checkpoint
+        $this->data[$key] = $value;
+
+        // all done
+        $log->endAction($value);
+    }
 
     // ====================================================================
     //

Braces +13 added lines, -26 removed lines

@@ -182,8 +182,7 @@ 
 		$log = usingLog()->startAction("store '{$key}' in the checkpoint");
 
 		// are we allowed to change the data at this time?
-		if ($this->readOnly)
-		{
+		if ($this->readOnly) {
 			// no, we are not
 			$log->endAction("checkpoint is readonly; did not store '{$key}'");
 			throw new E5xx_CheckpointIsReadOnly();
@@ -266,26 +265,22 @@ 
     public function getList($propName)
     {
         // do we have the property at all?
-        if (!isset($this->$propName))
-        {
+        if (!isset($this->$propName)) {
             // no ... send back an empty list
             return array();
         }
 
         // is the property already a list?
-        if (is_array($this->$propName))
-        {
+        if (is_array($this->$propName)) {
             // yes ... no conversion needed
             return $this->$propName;
         }
 
         // is the property something we can convert?
-        if (is_object($this->$propName))
-        {
+        if (is_object($this->$propName)) {
             // yes
             $return = array();
-            foreach ($this->$propName as $key => $value)
-            {
+            foreach ($this->$propName as $key => $value) {
                 $return[$key] = $value;
             }
 
@@ -307,16 +302,14 @@ 
     public function getString($propName, $default = '')
     {
         // does this property exist at all?
-        if (!isset($this->$propName))
-        {
+        if (!isset($this->$propName)) {
             // no, so return the default
             return $default;
         }
 
         // is this property something that can be auto-converted to a
         // string reliably?
-        if (is_string($this->$propName) || is_int($this->$propName) || is_double($this->$propName))
-        {
+        if (is_string($this->$propName) || is_int($this->$propName) || is_double($this->$propName)) {
             // yes
             return (string)$this->$propName;
         }
@@ -324,10 +317,8 @@ 
         // starting to clutch at straws now
 
         // a boolean, perhaps?
-        if (is_bool(($this->$propName)))
-        {
-            if ($this->$propName)
-            {
+        if (is_bool(($this->$propName))) {
+            if ($this->$propName) {
                 return 'TRUE';
             }
 
@@ -335,11 +326,9 @@ 
         }
 
         // is it an object that can convert itself to a string?
-        if (is_object($this->$propName))
-        {
+        if (is_object($this->$propName)) {
             $refObj = new ReflectionObject($this->$propName);
-            if ($refObj->hasMethod('__toString'))
-            {
+            if ($refObj->hasMethod('__toString')) {
                 return (string)$this->$propName;
             }
 
@@ -367,14 +356,12 @@ 
         $refProps = $refObj->getProperties(ReflectionProperty::IS_PUBLIC);
 
         // convert each property into an array entry
-        foreach ($refProps as $refProp)
-        {
+        foreach ($refProps as $refProp) {
             $propKey      = $refProp->getName();
             $retKey       = $propKey;
 
             // do we need to enforce the prefix?
-            if ($prefix !== null && substr($this->$propKey, 0, strlen($prefix)) !== $prefix)
-            {
+            if ($prefix !== null && substr($this->$propKey, 0, strlen($prefix)) !== $prefix) {
                 // yes we do
                 $retKey = $prefix . $propKey;
             }

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

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
      *
      * @var Story
      */
-    public $story           = null;
+    public $story = null;
 
     /**
      * is this a story where a failure is the expected outcome?