chrl / myhome

src/AppBundle/Action/Executor/Movement.php

Spacing +2 added lines, -2 removed lines

@@ -17,8 +17,8 @@
         $variable = false;
 
         foreach ($params as $param) {
-            list($p,$v) = explode(':', $param);
-            if ($p=='variable') {
+            list($p, $v) = explode(':', $param);
+            if ($p == 'variable') {
                 $variable = $this->
                                 getDoctrine()->
                                 getManager()->

src/AppBundle/Action/Service.php

Doc Comments +3 added lines

@@ -34,6 +34,9 @@
         return $this->execute($action, $source, $changeSet, true);
     }
 
+    /**
+     * @param string $source
+     */
     public function executeReal(Action $action, $source, $arguments)
     {
         return $this->execute($action, $source, $arguments, false);

Spacing +4 added lines, -4 removed lines

@@ -47,10 +47,10 @@ 
         if (!$virtual) {
             list($executor, $method) = explode(':', $action->getExecutor());
 
-            $executor = 'AppBundle\Action\Executor\\' . ucfirst($executor);
+            $executor = 'AppBundle\Action\Executor\\'.ucfirst($executor);
 
             if (!class_exists($executor)) {
-                throw new \Exception('Unknown executor: ' . $executor);
+                throw new \Exception('Unknown executor: '.$executor);
             }
 
             /** @var ExecutorInterface $executor */
@@ -63,7 +63,7 @@ 
             }
 
             if (!method_exists($executor, $method)) {
-                throw new \Exception('Unknown executor method: ' . $action->getExecutor().'()');
+                throw new \Exception('Unknown executor method: '.$action->getExecutor().'()');
             }
 
             $executor->setParameters($changeSet);
@@ -73,7 +73,7 @@ 
             } catch (\Exception $exception) {
                 $result = $exception->getMessage();
             }
-            $changeSet['result']=$result;
+            $changeSet['result'] = $result;
         }
 
         $state = new ActionHistory();

src/AppBundle/Action/Executor/Saytext.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 {
     public function say(Action $action)
     {
-        $args =  json_decode($action->getArguments(), true);
+        $args = json_decode($action->getArguments(), true);
 
         if (isset($this->parameters['text'])) {
             $text = $this->parameters['text'];
@@ -36,7 +36,7 @@ 
                 urlencode($text)."&tl=De_de");
             file_put_contents($file, $res);
 
-            $result= 'Saved in cache';
+            $result = 'Saved in cache';
         }
 
         $res = exec('which mplayer');

src/AppBundle/Command/RunCronsCommand.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
                 continue;
             }
 
-            if (time()-$args['last'] >= $args['every']) {
+            if (time() - $args['last'] >= $args['every']) {
                 $output->writeln('Running '.$cronAction->getExecutor().' for "'.
                     $cronAction->getDevice()->getName().'" device');
                 $this->

src/AppBundle/Action/Executor/Noolite.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
         $deviceParams = $device->getParams();
         $ch = 0;
         foreach ($deviceParams as $deviceParam) {
-            list($param,$value) = explode(':', $deviceParam);
+            list($param, $value) = explode(':', $deviceParam);
             if ($param == 'channel') {
                 $ch = $value;
                 break;

src/AppBundle/Controller/NooliteController.php

Spacing +21 added lines, -21 removed lines

@@ -13,15 +13,15 @@ 
 
     private $channelMap = [
 
-        1 => ['name'=>'noolite.main','source'=>'human'],
-        2 => ['name'=>'noolite.kitchen','source'=>'human'],
-        3 => ['name'=>'noolite.diningroom','source'=>'human'],
-        4 => ['name'=>'noolite.main','source'=>'pi'],
-        5 => ['name'=>'noolite.toilet','source'=>'pi'],
-        6 => ['name'=>'noolite.diningroom','source'=>'pi'],
-        7 => ['name'=>'noolite.kitchen','source'=>'pi'],
-        9 => ['name'=>'noolite.toilet','source'=>'human,upper'],
-        10 => ['name'=>'noolite.toilet','source'=>'human,lower'],
+        1 => ['name'=>'noolite.main', 'source'=>'human'],
+        2 => ['name'=>'noolite.kitchen', 'source'=>'human'],
+        3 => ['name'=>'noolite.diningroom', 'source'=>'human'],
+        4 => ['name'=>'noolite.main', 'source'=>'pi'],
+        5 => ['name'=>'noolite.toilet', 'source'=>'pi'],
+        6 => ['name'=>'noolite.diningroom', 'source'=>'pi'],
+        7 => ['name'=>'noolite.kitchen', 'source'=>'pi'],
+        9 => ['name'=>'noolite.toilet', 'source'=>'human,upper'],
+        10 => ['name'=>'noolite.toilet', 'source'=>'human,lower'],
     ];
 
     /**
@@ -60,7 +60,7 @@ 
             $changeSet = [];
 
 
-            if ($request->get('cmd')==4) {
+            if ($request->get('cmd') == 4) {
                 // reverse state, only by human
 
                 $oldState = $device->getState();
@@ -73,7 +73,7 @@ 
                     /** @var Action $action */
                     foreach ($actions as $action) {
                         $args = json_decode($action->getArguments(), true);
-                        if (isset($args['state']) && ($args['state']=='on')) {
+                        if (isset($args['state']) && ($args['state'] == 'on')) {
                             $resultAction = $action;
                             break;
                         }
@@ -84,7 +84,7 @@ 
                     /** @var Action $action */
                     foreach ($actions as $action) {
                         $args = json_decode($action->getArguments(), true);
-                        if (isset($args['state']) && ($args['state']=='off')) {
+                        if (isset($args['state']) && ($args['state'] == 'off')) {
                             $resultAction = $action;
                             break;
                         }
@@ -92,44 +92,44 @@ 
                 }
             }
 
-            if ($request->get('cmd')==0) {
+            if ($request->get('cmd') == 0) {
                 // turn off, only by pi
 
                 $changeSet = ['state'=>'off'];
                 /** @var Action $action */
                 foreach ($actions as $action) {
                     $args = json_decode($action->getArguments(), true);
-                    if (isset($args['state']) && ($args['state']=='off')) {
+                    if (isset($args['state']) && ($args['state'] == 'off')) {
                         $resultAction = $action;
                         break;
                     }
                 }
             }
 
-            if ($request->get('cmd')==6) {
+            if ($request->get('cmd') == 6) {
                 // turn off, only by pi
 
-                $percent = round((($request->get('d0') - 34)/123) * 100);
+                $percent = round((($request->get('d0') - 34)/123)*100);
 
-                $changeSet = ['state'=>'on','percent'=>$percent];
+                $changeSet = ['state'=>'on', 'percent'=>$percent];
                 /** @var Action $action */
                 foreach ($actions as $action) {
                     $args = json_decode($action->getArguments(), true);
-                    if (isset($args['state']) && ($args['state']=='on')) {
+                    if (isset($args['state']) && ($args['state'] == 'on')) {
                         $resultAction = $action;
                         break;
                     }
                 }
             }
 
-            if ($request->get('cmd')==2) {
+            if ($request->get('cmd') == 2) {
                 // turn on, only by pi
 
                 $changeSet = ['state'=>'on'];
                 /** @var Action $action */
                 foreach ($actions as $action) {
                     $args = json_decode($action->getArguments(), true);
-                    if (isset($args['state']) && ($args['state']=='on')) {
+                    if (isset($args['state']) && ($args['state'] == 'on')) {
                         $resultAction = $action;
                         break;
                     }
@@ -141,7 +141,7 @@ 
             }
 
             if (($changeSet['state'] == 'on') && !isset($changeSet['percent'])) {
-                $changeSet['percent'] = 100;  // ? I don't really know if they restore their states
+                $changeSet['percent'] = 100; // ? I don't really know if they restore their states
             }
 
             if (!$resultAction) {

src/AppBundle/Variable/Service.php

Spacing +4 added lines, -4 removed lines

@@ -70,7 +70,7 @@ 
 
         if ($this->needSync) {
             if ($var->needSync) {
-                @file_get_contents($this->syncHost . 'set/' . $varName . '?value=' . $value);
+                @file_get_contents($this->syncHost.'set/'.$varName.'?value='.$value);
             }
         }
 
@@ -99,10 +99,10 @@ 
         foreach ($hooks as $varHook) {
             list($executor, $method) = explode(':', $varHook->getExecutor());
 
-            $executor = 'AppBundle\Action\Executor\\' . ucfirst($executor);
+            $executor = 'AppBundle\Action\Executor\\'.ucfirst($executor);
 
             if (!class_exists($executor)) {
-                throw new \Exception('Unknown executor: ' . $executor);
+                throw new \Exception('Unknown executor: '.$executor);
             }
 
             /** @var ExecutorInterface $executor */
@@ -111,7 +111,7 @@ 
 
 
             if (!method_exists($executor, $method)) {
-                throw new \Exception('Unknown executor method: ' . $varHook->getExecutor().'()');
+                throw new \Exception('Unknown executor method: '.$varHook->getExecutor().'()');
             }
 
             $executor->setParameters(json_decode($varHook->getArguments(), true));