bedezign / yii2-audit

src/AuditTrailBehavior.php

Braces +6 added lines, -4 removed lines

@@ -237,13 +237,15 @@
         $oldAttributes = $this->cleanAttributes($this->getOldAttributes());
 
         // ensure to handle serialized attributes properly
-        foreach($newAttributes as $key => $value)
-            if(is_array($newAttributes[$key]))
+        foreach($newAttributes as $key => $value) {
+                    if(is_array($newAttributes[$key]))
                 $newAttributes[$key] = Json::encode($newAttributes[$key]);
+        }
 
-        foreach($oldAttributes as $key => $value)
-            if(is_array($oldAttributes[$key]))
+        foreach($oldAttributes as $key => $value) {
+                    if(is_array($oldAttributes[$key]))
                 $oldAttributes[$key] = Json::encode($oldAttributes[$key]);
+        }
 
         // If no difference then get out of here
         if (count(array_diff_assoc($newAttributes, $oldAttributes)) <= 0) {

src/views/entry/panels/request/table.php

Braces +5 added lines, -2 removed lines

@@ -12,7 +12,8 @@ 
 
     <p>Empty.</p>
 
-<?php else: ?>
+<?php else {
+    : ?>
 
     <div class="table-responsive">
         <table class="table table-condensed table-bordered table-striped table-hover request-table"
@@ -30,7 +31,9 @@ 
                     <td><?= htmlspecialchars(VarDumper::dumpAsString($value), ENT_QUOTES | ENT_SUBSTITUTE,
                             \Yii::$app->charset, true) ?></td>
                 </tr>
-            <?php endforeach; ?>
+            <?php endforeach;
+}
+?>
             </tbody>
         </table>
     </div>

src/views/mail/view-html.php

Braces +1 added lines, -2 removed lines

@@ -15,8 +15,7 @@
 
 if (Audit::getInstance()->hasMethod('renderEmail')) {
     Audit::getInstance()->renderEmail($this, $model);
-}
-elseif (class_exists('\PhpMimeMailParser\Parser')) {
+} elseif (class_exists('\PhpMimeMailParser\Parser')) {
     $parser = new \PhpMimeMailParser\Parser();
     $parser->setText($model->data);
     echo $parser->getMessageBody('htmlEmbedded');

src/LogTarget.php

Braces +13 added lines, -11 removed lines

@@ -43,13 +43,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;
 
@@ -60,11 +62,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);
+                }
             }
         }
     }
@@ -77,8 +80,7 @@ 
     {
         if ($this->exportAtIntervals) {
             parent::collect($messages, $final);
-        }
-        else {
+        } else {
             $this->messages = array_merge($this->messages, static::filterMessages($messages, $this->getLevels(), $this->categories, $this->except));
             if ($final) {
                 $this->export();

src/Audit.php

Braces +44 added lines, -33 removed lines

@@ -291,8 +291,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;
     }
@@ -320,8 +321,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);
     }
@@ -459,13 +461,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;
@@ -486,11 +490,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;
             }
@@ -500,10 +504,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.");
+                }
             }
         });
     }
@@ -515,18 +520,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;
     }
@@ -562,18 +570,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;
     }