datasift / storyplayer

src/php/DataSift/Storyplayer/HostLib/Blackboxes.php

Spacing +4 added lines, -4 removed lines

@@ -88,7 +88,7 @@ 
     public function createHost($groupDef, $provisioningVars = array())
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction('register blackbox(es)');
+        $log = Log::usingLog()->startAction('register blackbox(es)');
 
         // make sure we like the provided details
         if (!isset($groupDef->details)) {
@@ -100,7 +100,7 @@ 
         if (empty($groupDef->details->machines)) {
             throw Exceptions::newActionFailedException(__METHOD__, "groupDef->details->machines cannot be empty");
         }
-        foreach($groupDef->details->machines as $hostId => $machine) {
+        foreach ($groupDef->details->machines as $hostId => $machine) {
             // TODO: it would be great to autodetect this one day
             if (!isset($machine->roles)) {
                 throw Exceptions::newActionFailedException(__METHOD__, "missing groupDef->details->machines['$hostId']->roles");
@@ -121,7 +121,7 @@ 
         // if it is not, that is NOT our responsibility
 
         // store the details
-        foreach($groupDef->details->machines as $hostId => $machine)
+        foreach ($groupDef->details->machines as $hostId => $machine)
         {
             // we want all the details from the config file
             $vmDetails = clone $machine;
@@ -200,7 +200,7 @@ 
     public function destroyHost($groupDef)
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction("de-register blackbox(es)");
+        $log = Log::usingLog()->startAction("de-register blackbox(es)");
 
         // de-register all the hosts
         foreach ($groupDef->details->machines as $hostId => $machine)

src/php/DataSift/Storyplayer/HostLib/Ec2Vm.php

Spacing +9 added lines, -9 removed lines

@@ -89,10 +89,10 @@ 
     public function createHost($vmDetails, $provisioningVars = array())
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction('provision new VM');
+        $log = Log::usingLog()->startAction('provision new VM');
 
         // make sure we like the provided details
-        foreach(array('name', 'environment', 'osName', 'amiId', 'securityGroup') as $param) {
+        foreach (array('name', 'environment', 'osName', 'amiId', 'securityGroup') as $param) {
             if (!isset($vmDetails->$param)) {
                 throw Exceptions::newActionFailedException(__METHOD__, "missing vmDetails['{$param}']");
             }
@@ -132,7 +132,7 @@ 
                     $client->createTags(array(
                         'Resources' => array($response['Instances'][0]['InstanceId']),
                         'Tags' => array(
-                            array (
+                            array(
                                 'Key' => 'Name',
                                 'Value' => $vmDetails->ec2Name
                             )
@@ -199,7 +199,7 @@ 
     public function startHost($vmDetails)
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction("start VM");
+        $log = Log::usingLog()->startAction("start VM");
 
         // is the VM actually running?
         if ($this->isRunning($vmDetails)) {
@@ -264,7 +264,7 @@ 
     public function stopHost($vmDetails)
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction("stop VM");
+        $log = Log::usingLog()->startAction("stop VM");
 
         // is the VM actually running?
         if (!$this->isRunning($vmDetails)) {
@@ -319,7 +319,7 @@ 
     public function restartHost($vmDetails)
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction("restart VM");
+        $log = Log::usingLog()->startAction("restart VM");
 
         // stop and start
         $this->stopHost($vmDetails);
@@ -350,7 +350,7 @@ 
     public function destroyHost($vmDetails)
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction("destroy VM");
+        $log = Log::usingLog()->startAction("destroy VM");
 
         // get our Ec2 client from the SDK
         $client = fromAws()->getEc2Client();
@@ -419,7 +419,7 @@ 
     public function isRunning($vmDetails)
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction("determine status of EC2 VM '{$vmDetails->hostId}'");
+        $log = Log::usingLog()->startAction("determine status of EC2 VM '{$vmDetails->hostId}'");
 
         // get the instance data
         $instance = fromEc2()->getInstance($vmDetails->hostId);
@@ -446,7 +446,7 @@ 
     public function determineIpAddress($vmDetails)
     {
         // what are we doing?
-        $log =Log::usingLog()->startAction("determine IP address of EC2 VM '{$vmDetails->hostId}'");
+        $log = Log::usingLog()->startAction("determine IP address of EC2 VM '{$vmDetails->hostId}'");
 
         // we need to get a fresh copy of the instance details
         $dnsName = fromEc2Instance($vmDetails->hostId)->getPublicDnsName();

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

Indentation +135 added lines, -135 removed lines

@@ -59,143 +59,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 = 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 = 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 = 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 = 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);
+    }
 
     // ====================================================================
     //