datasift / storyplayer

src/php/DataSift/Storyplayer/OutputLib/OutputWriter.php

Doc Comments +6 added lines

@@ -128,6 +128,9 @@ 
         ];
     }
 
+    /**
+     * @param string $filename
+     */
     public function addOutputToFile($filename)
     {
         // can we open the file?
@@ -169,6 +172,9 @@ 
     //
     // ------------------------------------------------------------------
 
+    /**
+     * @param integer $mode
+     */
     public function setColourMode($mode)
     {
         switch ($mode)

src/php/DataSift/Storyplayer/Output.php

Braces +22 added lines, -44 removed lines

@@ -179,8 +179,7 @@ 
      */
     public function resetSilentMode()
     {
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->resetSilentMode();
         }
     }
@@ -196,8 +195,7 @@ 
      */
     public function setSilentMode()
     {
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->setSilentMode();
         }
     }
@@ -213,8 +211,7 @@ 
      */
     public function setIsVerbose($isVerbose)
     {
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->setIsVerbose($isVerbose);
         }
     }
@@ -226,8 +223,7 @@ 
      */
     public function disableColourSupport()
     {
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->disableColourSupport();
         }
     }
@@ -239,8 +235,7 @@ 
      */
     public function enforceColourSupport()
     {
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->enforceColourSupport();
         }
     }
@@ -257,8 +252,7 @@ 
      */
     public function enableColourSupport()
     {
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->enableColourSupport();
         }
     }
@@ -281,8 +275,7 @@ 
         Contract::RequiresValue($license,   is_string($license));
 
         // call all of our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->startStoryplayer($version, $url, $copyright, $license);
         }
     }
@@ -296,8 +289,7 @@ 
      */
     public function endStoryplayer($duration)
     {
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->endStoryplayer($duration);
         }
     }
@@ -321,8 +313,7 @@ 
         Contract::RequiresValue($name,     is_string($name));
 
         // call our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->startPhaseGroup($activity, $name, $details);
         }
     }
@@ -343,8 +334,7 @@ 
         Contract::Requires($result instanceof PhaseGroup_Result);
 
         // call our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->endPhaseGroup($result);
         }
     }
@@ -361,8 +351,7 @@ 
         // enforce our input type
         Contract::Requires($phase instanceof Phase);
 
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->startPhase($phase);
         }
     }
@@ -387,8 +376,7 @@ 
         $this->activityLog=[];
 
         // pass the phase on
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->endPhase($phase, $phaseResult);
         }
     }
@@ -420,8 +408,7 @@ 
         ];
 
         // call all of our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logPhaseActivity($msg, $codeLine);
         }
     }
@@ -448,8 +435,7 @@ 
         ];
 
         // call all of our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logPhaseSubprocessOutput($msg);
         }
     }
@@ -478,8 +464,7 @@ 
         ];
 
         // call all of our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logPhaseError($phaseName, $msg);
         }
     }
@@ -508,8 +493,7 @@ 
         ];
 
         // call all of our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logPhaseSkipped($phaseName, $msg);
         }
     }
@@ -528,8 +512,7 @@ 
         Contract::RequiresValue($codeLine, is_array($codeLine));
 
         // pass it on to all of our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logPhaseCodeLine($codeLine);
         }
     }
@@ -548,8 +531,7 @@ 
         Contract::RequiresValue($msg, is_string($msg));
 
         // pass it on to our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logCliError($msg);
         }
     }
@@ -570,8 +552,7 @@ 
         Contract::RequiresValue($e, $e instanceof Exception);
 
         // pass this on to our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logCliErrorWithException($msg, $e);
         }
     }
@@ -590,8 +571,7 @@ 
         Contract::RequiresValue($msg, is_string($msg));
 
         // pass this on to our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logCliWarning($msg);
         }
     }
@@ -610,8 +590,7 @@ 
         Contract::RequiresValue($msg, is_string($msg));
 
         // pass this on to our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logCliInfo($msg);
         }
     }
@@ -634,8 +613,7 @@ 
         // $var can be anything, so there is no contract to enforce
 
         // pass this on to our plugins
-        foreach ($this->plugins as $plugin)
-        {
+        foreach ($this->plugins as $plugin) {
             $plugin->logVardump($name, $var);
         }
     }