petrica / php-statsd-system

Config/ConfigLoader.php

Braces +1 added lines, -2 removed lines

@@ -57,8 +57,7 @@
             }
 
             return $processedConfiguration;
-        }
-        else {
+        } else {
             throw new \RuntimeException(sprintf('Configuration file does not exist or is not accessible %s',
                 $this->filepath));
         }

Command/NotifyCommand.php

Braces +2 added lines, -4 removed lines

@@ -150,12 +150,10 @@
                     if ($reflection->getConstructor()) {
                         $this->gauges[$path] = $reflection->newInstanceArgs(
                             $details['arguments'] ? $details['arguments'] : array());
-                    }
-                    else {
+                    } else {
                         $this->gauges[$path] = $reflection->newInstance();
                     }
-                }
-                else {
+                } else {
                     throw new \RuntimeException(sprintf(
                         'Class does not exists %s'
                     ), $className);

Spacing +4 added lines, -4 removed lines

@@ -88,9 +88,9 @@ 
 
         $iterations = $input->getOption('iterations');
         $count = 0;
-        while($count < $iterations)
+        while ($count < $iterations)
         {
-            foreach($gauges as $path => $gauge) {
+            foreach ($gauges as $path => $gauge) {
                 // Sampling period attained for current gauge?
                 if (fmod($count, $gauge->getSamplingPeriod()) == 0) {
 
@@ -99,7 +99,7 @@ 
                     $statsd->startBatch();
 
                     foreach ($values as $key => $value) {
-                        $composed = $path . '.' . $key;
+                        $composed = $path.'.'.$key;
 
                         if (null !== $value) {
                             $statsd->gauge($composed, $value);
@@ -115,7 +115,7 @@ 
             }
 
             sleep(1);
-            $count ++;
+            $count++;
         }
     }
 

Tests/Command/NotifyCommandTest.php

Spacing +3 added lines, -3 removed lines

@@ -124,9 +124,9 @@
         $input = $this->getInputInterface();
 
         $map = array(
-            [ 'statsd-host', 'localhost' ],
-            [ 'statsd-port', '8125' ],
-            [ 'statsd-namespace', 'test' ]
+            ['statsd-host', 'localhost'],
+            ['statsd-port', '8125'],
+            ['statsd-namespace', 'test']
         );
 
         $input->expects($this->atLeastOnce())

Model/TopCommand.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         foreach ($lines as $line) {
             $data = explode(' ', $line);
             array_walk($data, 'trim');
-            $stats[] = array_values(array_filter($data, function ($value) {
+            $stats[] = array_values(array_filter($data, function($value) {
                 return strlen($value) > 0;
             }));
         }

Gauge/ProcessesGauge.php

Spacing +3 added lines, -3 removed lines

@@ -63,11 +63,11 @@ 
         $collection = new ValuesCollection();
 
         foreach ($cpu as $name => $value) {
-            $collection->add($name . '.cpu.value', $value);
+            $collection->add($name.'.cpu.value', $value);
         }
 
         foreach ($memory as $name => $value) {
-            $collection->add($name . '.memory.value', $value);
+            $collection->add($name.'.memory.value', $value);
         }
 
         return $collection;
@@ -94,7 +94,7 @@ 
      */
     protected function filterAbove($processes, $gate)
     {
-        return array_filter($processes, function ($item) use ($gate) {
+        return array_filter($processes, function($item) use ($gate) {
             return $item >= $gate;
         });
     }

Braces +2 added lines, -4 removed lines

@@ -111,8 +111,7 @@ 
         foreach ($processes as $process) {
             if (isset($cpus[$process->getName()])) {
                 $cpus[$process->getName()] += $process->getCpu();
-            }
-            else {
+            } else {
                 $cpus[$process->getName()] = $process->getCpu();
             }
         }
@@ -133,8 +132,7 @@ 
         foreach ($processes as $process) {
             if (isset($memory[$process->getName()])) {
                 $memory[$process->getName()] += $process->getMemory();
-            }
-            else {
+            } else {
                 $memory[$process->getName()] = $process->getMemory();
             }
         }

Doc Comments +6 added lines, -5 removed lines

@@ -107,6 +107,7 @@ 
      * Return only those processes above certail value
      *
      * @param $processes Process[]
+     * @param double $gate
      * @return array
      */
     protected function filterAbove($processes, $gate)
@@ -119,7 +120,7 @@ 
     /**
      * Aggregate CPU for processes with the same name
      *
-     * @param $processes Process[]
+     * @param Process[] $processes Process[]
      */
     protected function aggregateCpu($processes)
     {
@@ -140,7 +141,7 @@ 
     /**
      * Aggregate memory for processes with the same name
      *
-     * @param $processes Process[]
+     * @param Process[] $processes Process[]
      * @return array
      */
     protected function aggregateMemory($processes)
@@ -173,7 +174,7 @@ 
      * Persist collection values to a temporary storage
      * key is the command string value
      *
-     * @param $collection
+     * @param ValuesCollection $collection
      */
     protected function persistCollection($collection)
     {
@@ -210,7 +211,7 @@ 
      *
      * These are in fact previous collection values that needs to be sent to statsd as 0
      *
-     * @param $collection
+     * @param ValuesCollection $collection
      */
     protected function resetValues($collection) {
         foreach ($collection as $key => $value) {
@@ -223,7 +224,7 @@ 
     /**
      * Remove empty values from collection
      *
-     * @param $collection
+     * @param ValuesCollection $collection
      */
     protected function removeEmpty($collection) {
         foreach ($collection as $key => $value) {

Gauge/RemoteProcessesGauge.php

Spacing +1 added lines, -1 removed lines

@@ -44,6 +44,6 @@
         $this->command = new RemoteTopCommand($this->sshString, $this->sshPort, $this->sshIdentityFile);
 
 
-        $this->namespace = preg_replace('/[^-+_.A-Za-z0-9]*/i', '', 'statsd.' . $this->sshString . '.' . $this->sshPort);
+        $this->namespace = preg_replace('/[^-+_.A-Za-z0-9]*/i', '', 'statsd.'.$this->sshString.'.'.$this->sshPort);
     }
 }
\ No newline at end of file