bedezign / yii2-audit

src/Audit.php

Indentation +6 added lines, -6 removed lines

@@ -373,12 +373,12 @@
 
         // We now need one more iteration to add core classes to the panels added via the merge, if needed
         array_walk($this->panels, function(&$value, $key) {
-           if (!isset($value['class'])) {
-               if (isset($this->_corePanels[$key]))
-                   $value = ArrayHelper::merge($value, $this->_corePanels[$key]);
-               else
-                   throw new InvalidConfigException("Invalid configuration for '$key'. No 'class' specified.");
-           }
+            if (!isset($value['class'])) {
+                if (isset($this->_corePanels[$key]))
+                    $value = ArrayHelper::merge($value, $this->_corePanels[$key]);
+                else
+                    throw new InvalidConfigException("Invalid configuration for '$key'. No 'class' specified.");
+            }
         });
     }
 

Braces +44 added lines, -33 removed lines

@@ -239,8 +239,9 @@ 
      */
     public function registerFunction($name, $callback)
     {
-        if (isset($this->_panelFunctions[$name]))
-            throw new InvalidParamException("The '$name'-function has already been defined.");
+        if (isset($this->_panelFunctions[$name])) {
+                    throw new InvalidParamException("The '$name'-function has already been defined.");
+        }
 
         $this->_panelFunctions[$name] = $callback;
     }
@@ -260,8 +261,9 @@ 
      */
     public function __call($name, $params)
     {
-        if (!isset($this->_panelFunctions[$name]))
-            throw new \yii\base\InvalidCallException("Unknown panel function '$name'");
+        if (!isset($this->_panelFunctions[$name])) {
+                    throw new \yii\base\InvalidCallException("Unknown panel function '$name'");
+        }
 
         return call_user_func_array($this->_panelFunctions[$name], $params);
     }
@@ -333,13 +335,15 @@ 
     public function getPanel($identifier)
     {
         $config = null;
-        if (isset($this->panels[$identifier]))
-            $config = $this->panels[$identifier];
-        elseif (isset($this->_corePanels[$identifier]))
-            $config = $this->_corePanels[$identifier];
+        if (isset($this->panels[$identifier])) {
+                    $config = $this->panels[$identifier];
+        } elseif (isset($this->_corePanels[$identifier])) {
+                    $config = $this->_corePanels[$identifier];
+        }
 
-        if (!$config)
-            throw new InvalidConfigException("'$identifier' is not a valid panel identifier");
+        if (!$config) {
+                    throw new InvalidConfigException("'$identifier' is not a valid panel identifier");
+        }
 
         if (is_array($config)) {
             $config['module'] = $this;
@@ -360,11 +364,11 @@ 
         foreach ($this->panels as $key => $value) {
             if (is_numeric($key)) {
                 // The $value contains the identifier of a core panel
-                if (!isset($this->_corePanels[$value]))
-                    throw new InvalidConfigException("'$value' is not a valid panel identifier");
+                if (!isset($this->_corePanels[$value])) {
+                                    throw new InvalidConfigException("'$value' is not a valid panel identifier");
+                }
                 $panels[$value] = $this->_corePanels[$value];
-            }
-            else {
+            } else {
                 // The key contains the identifier and the value is either a class name or a full array
                 $panels[$key] = is_string($value) ? ['class' => $value] : $value;
             }
@@ -374,10 +378,11 @@ 
         // We now need one more iteration to add core classes to the panels added via the merge, if needed
         array_walk($this->panels, function(&$value, $key) {
            if (!isset($value['class'])) {
-               if (isset($this->_corePanels[$key]))
-                   $value = ArrayHelper::merge($value, $this->_corePanels[$key]);
-               else
-                   throw new InvalidConfigException("Invalid configuration for '$key'. No 'class' specified.");
+               if (isset($this->_corePanels[$key])) {
+                                  $value = ArrayHelper::merge($value, $this->_corePanels[$key]);
+               } else {
+                                  throw new InvalidConfigException("Invalid configuration for '$key'. No 'class' specified.");
+               }
            }
         });
     }
@@ -389,18 +394,21 @@ 
     {
         foreach (Yii::$app->modules as $name => $module) {
             $class = null;
-            if (is_string($module))
-                $class = $module;
-            elseif (is_array($module)) {
-                if (isset($module['class']))
-                    $class = $module['class'];
-            } else
-                /** @var Module $module */
+            if (is_string($module)) {
+                            $class = $module;
+            } elseif (is_array($module)) {
+                if (isset($module['class'])) {
+                                    $class = $module['class'];
+                }
+            } else {
+                            /** @var Module $module */
                 $class = $module::className();
+            }
 
             $parts = explode('\\', $class);
-            if ($class && strtolower(end($parts)) == 'audit')
-                return $name;
+            if ($class && strtolower(end($parts)) == 'audit') {
+                            return $name;
+            }
         }
         return null;
     }
@@ -436,18 +444,21 @@ 
             $len = strlen($compare);
             if ($compare[$len - 1] == '*') {
                 $compare = rtrim($compare, '*');
-                if (substr($route, 0, $len - 1) === $compare)
-                    return true;
+                if (substr($route, 0, $len - 1) === $compare) {
+                                    return true;
+                }
             }
 
             if ($compare[0] == '*') {
                 $compare = ltrim($compare, '*');
-                if (substr($route, - ($len - 1)) === $compare)
-                    return true;
+                if (substr($route, - ($len - 1)) === $compare) {
+                                    return true;
+                }
             }
 
-            if ($route === $compare)
-                return true;
+            if ($route === $compare) {
+                            return true;
+            }
         }
         return false;
     }

src/LogTarget.php

Braces +12 added lines, -9 removed lines

@@ -36,13 +36,15 @@ 
      */
     public function export()
     {
-        if (!\Yii::$app)
-            // Things like this can happen in tests etc, but it is too late for us to do anything about that.
+        if (!\Yii::$app) {
+                    // Things like this can happen in tests etc, but it is too late for us to do anything about that.
             return;
+        }
 
         $module = $this->module;
-        if (!$module->entry || empty($module->panels))
-            return;
+        if (!$module->entry || empty($module->panels)) {
+                    return;
+        }
 
         $entry = $module->entry;
 
@@ -53,11 +55,12 @@ 
         $records = array_filter($records);
 
         if (!empty($records)) {
-            if ($module->batchSave)
-                $entry->addBatchData($records, false);
-            else {
-                foreach ($records as $type => $record)
-                    $entry->addData($type, $record, false);
+            if ($module->batchSave) {
+                            $entry->addBatchData($records, false);
+            } else {
+                foreach ($records as $type => $record) {
+                                    $entry->addData($type, $record, false);
+                }
             }
         }
         $this->messages = [];

src/components/SoapClient.php

Braces +17 added lines, -14 removed lines

@@ -14,8 +14,7 @@ 
 
         try {
             parent::__construct($wsdl, $options);
-        }
-        catch (\SoapFault $e) {
+        } catch (\SoapFault $e) {
             // Catches errors like being unable to read the WSDL and so on
             $this->logSoapFault($e);
             throw $e;
@@ -46,22 +45,24 @@ 
         try {
             $result = parent::__soapCall($function_name, $arguments, $options, $input_headers, $output_headers);
             $this->_data['duration'] = microtime(true) - $started;
-            if ($output_headers)
-                $this->_data['output_headers'] = (array) $output_headers;
+            if ($output_headers) {
+                            $this->_data['output_headers'] = (array) $output_headers;
+            }
 
-            if (is_soap_fault($result))
-                // Cover non-exception variant
+            if (is_soap_fault($result)) {
+                            // Cover non-exception variant
                 return $this->logSoapFault($result);
+            }
 
             $this->_data['result_object'] = $result;
             $this->finalize();
 
             return $result;
-        }
-        catch (\Exception $error) {
+        } catch (\Exception $error) {
             $this->_data['duration'] = microtime(true) - $started;
-            if ($output_headers)
-                $this->_data['output_headers'] = (array) $output_headers;
+            if ($output_headers) {
+                            $this->_data['output_headers'] = (array) $output_headers;
+            }
 
             $this->logSoapFault($error);
             throw $error;
@@ -82,8 +83,9 @@ 
 
     public function __setCookie ($name, $value = null)
     {
-        if (!isset($this->_data['cookies']))
-            $this->_data['cookies'] = [];
+        if (!isset($this->_data['cookies'])) {
+                    $this->_data['cookies'] = [];
+        }
         $this->_data['cookies'][$name] = $value;
 
         parent::__setCookie($name, $value);
@@ -99,7 +101,8 @@ 
     protected function finalize()
     {
         $_panel = Audit::getInstance()->getPanel('audit/soap');
-        if ($_panel)
-            $_panel->logSoapRequest($this->_data);
+        if ($_panel) {
+                    $_panel->logSoapRequest($this->_data);
+        }
     }
 }
\ No newline at end of file

src/components/db/ActiveRecord.php

Braces +12 added lines, -8 removed lines

@@ -37,13 +37,15 @@ 
      */
     public function beforeSave($insert)
     {
-        if ($insert && $this->hasAttribute('created'))
-            $this->created = date('Y-m-d H:i:s');
+        if ($insert && $this->hasAttribute('created')) {
+                    $this->created = date('Y-m-d H:i:s');
+        }
 
-        if ($this->autoSerialize)
-            foreach ($this->serializeAttributes as $attribute)
+        if ($this->autoSerialize) {
+                    foreach ($this->serializeAttributes as $attribute)
                 if ($this->hasAttribute($attribute))
                     $this->$attribute = [Helper::serialize($this->$attribute, false), \PDO::PARAM_LOB];
+        }
 
         return parent::beforeSave($insert);
     }
@@ -56,12 +58,13 @@ 
     {
         parent::afterSave($insert, $changedAttributes);
 
-        if ($this->autoSerialize)
-            foreach ($this->serializeAttributes as $attribute)
+        if ($this->autoSerialize) {
+                    foreach ($this->serializeAttributes as $attribute)
                 if ($this->hasAttribute($attribute)) {
                     if (is_array($this->$attribute)) {
                         // Get rid of the extra PDO parameter type if needed
                         $array = $this->$attribute;
+        }
                         $this->$attribute = $array[0];
                     }
                     $this->$attribute = Helper::unserialize($this->$attribute);
@@ -75,9 +78,10 @@ 
     {
         parent::afterFind();
 
-        if ($this->autoSerialize)
-            foreach ($this->serializeAttributes as $attribute)
+        if ($this->autoSerialize) {
+                    foreach ($this->serializeAttributes as $attribute)
                 if ($this->hasAttribute($attribute))
                     $this->$attribute = Helper::unserialize($this->$attribute);
+        }
     }
 }
\ No newline at end of file

src/components/panels/PanelTrait.php

Braces +3 added lines, -2 removed lines

@@ -101,8 +101,9 @@
     public function cleanup($maxAge = null)
     {
         $maxAge = $maxAge !== null ? $maxAge : $this->maxAge;
-        if ($maxAge === null)
-            return false;
+        if ($maxAge === null) {
+                    return false;
+        }
         return AuditData::deleteAll('type = :type AND created <= :created', [
             ':type' => $this->id,
             ':created' => date('Y-m-d 23:59:59', strtotime("-$maxAge days")),

src/controllers/EntryController.php

Braces +6 added lines, -5 removed lines

@@ -46,9 +46,9 @@ 
     public function actionView($id, $panel = '')
     {
         list($model, $panels) = $this->loadData($id);
-        if (isset($panels[$panel]))
-            $activePanel = $panel;
-        else {
+        if (isset($panels[$panel])) {
+                    $activePanel = $panel;
+        } else {
             reset($panels);
             $activePanel = key($panels);
         }
@@ -97,9 +97,10 @@ 
         $panels = $this->module->loadPanels($this->module->getPanelIdentifiers());
         $activePanels = [];
         $data = ArrayHelper::getColumn($model->data, 'data');
-        foreach ($panels as $panelId => $panel)
-            if ($panel->hasEntryData($model)) {
+        foreach ($panels as $panelId => $panel) {
+                    if ($panel->hasEntryData($model)) {
                 $panel->tag = $id;
+        }
                 $panel->model = $model;
                 $panel->load(isset($data[$panelId]) ? $data[$panelId] : null);
                 $activePanels[$panelId] = $panel;

src/controllers/JsLogController.php

Braces +15 added lines, -8 removed lines

@@ -51,21 +51,28 @@
                 $javascript->origin .= ':' . $value;
             },
             'data'       => function ($value) use ($javascript) {
-                if (count($value)) $javascript->data = $value;
+                if (count($value)) {
+                    $javascript->data = $value;
+                }
             },
         ];
 
-        foreach ($map as $key => $target)
-            if (isset($data[$key])) {
+        foreach ($map as $key => $target) {
+                    if (isset($data[$key])) {
                 if (is_callable($target)) $target($data[$key]);
-                else $javascript->$target = $data[$key];
+        }
+                else {
+                    $javascript->$target = $data[$key];
+                }
             }
 
-        if (!$javascript->type)
-            $javascript->type = 'unknown';
+        if (!$javascript->type) {
+                    $javascript->type = 'unknown';
+        }
 
-        if ($javascript->save())
-            return ['result' => 'ok', 'entry' => $data['auditEntry']];
+        if ($javascript->save()) {
+                    return ['result' => 'ok', 'entry' => $data['auditEntry']];
+        }
 
         return ['result' => 'error', 'errors' => $javascript->getErrors()];
     }

src/messages/config.php

Indentation +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     // and the '.svn' will match all files and directories named exactly '.svn'.
     // Note, the '/' characters in a pattern matches both '/' and '\'.
     // See helpers/FileHelper::findFiles() description for more details on pattern matching rules.
-     'only' => ['*.php', '*.js'],
+        'only' => ['*.php', '*.js'],
     // array, list of patterns that specify which files (not directories) should be processed.
     // If empty or not set, all files will be processed.
     // Please refer to "except" for details about the patterns.

src/models/AuditEntry.php

Braces +3 added lines, -2 removed lines

@@ -50,8 +50,9 @@
     public static function create($initialise = true)
     {
         $entry = new static;
-        if ($initialise)
-            $entry->record();
+        if ($initialise) {
+                    $entry->record();
+        }
 
         return $entry;
     }