bedezign / yii2-audit

src/migrations/m150626_000006_create_audit_mail.php

Spacing +3 added lines, -3 removed lines

@@ -11,9 +11,9 @@
     {
         $this->createTable(self::TABLE, [
             'id'         => Schema::TYPE_PK,
-            'entry_id'   => Schema::TYPE_INTEGER . ' NOT NULL',
-            'created'    => Schema::TYPE_DATETIME . ' NOT NULL',
-            'successful' => Schema::TYPE_INTEGER . ' NOT NULL',
+            'entry_id'   => Schema::TYPE_INTEGER.' NOT NULL',
+            'created'    => Schema::TYPE_DATETIME.' NOT NULL',
+            'successful' => Schema::TYPE_INTEGER.' NOT NULL',
             'from'       => Schema::TYPE_STRING,
             'to'         => Schema::TYPE_STRING,
             'reply'      => Schema::TYPE_STRING,

src/models/AuditEntry.php

Spacing +1 added lines, -1 removed lines

@@ -119,7 +119,7 @@
         // for batch inserts the same way. This code adds a number of literals instead of the actual values
         // so that they can be bound right before insert and still get escaped correctly
         foreach ($batchData as $type => $data) {
-            $param = ':data_' . str_replace('/', '_', $type);
+            $param = ':data_'.str_replace('/', '_', $type);
             $rows[] = [$this->id, $type, $date, new Expression($param)];
             $params[$param] = [Helper::serialize($data, $compact), \PDO::PARAM_LOB];
         }

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;
     }

src/models/AuditEntrySearch.php

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
      */
     static public function routeFilter()
     {
-        $routes = AuditEntry::getDb()->cache(function () {
+        $routes = AuditEntry::getDb()->cache(function() {
             return AuditEntry::find()->distinct(true)
                 ->select('route')->where(['is not', 'route', null])
                 ->groupBy('route')->orderBy('route ASC')->column();
@@ -89,7 +89,7 @@ 
      */
     static public function methodFilter()
     {
-        $methods = AuditEntry::getDb()->cache(function () {
+        $methods = AuditEntry::getDb()->cache(function() {
             return AuditEntry::find()->distinct(true)
                 ->select('request_method')->where(['is not', 'request_method', null])
                 ->groupBy('request_method')->orderBy('request_method ASC')->column();

src/models/AuditErrorSearch.php

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
      */
     static protected function filterData()
     {
-        return AuditEntry::getDb()->cache(function () {
+        return AuditEntry::getDb()->cache(function() {
             return AuditError::find()->distinct(true)
                 ->select(['hash', 'message', 'file'])->asArray()->all();
         }, 30);

src/models/AuditJavascriptSearch.php

Spacing +2 added lines, -2 removed lines

@@ -68,7 +68,7 @@ 
      */
     public function typeFilter()
     {
-        $types = AuditJavascript::getDb()->cache(function () {
+        $types = AuditJavascript::getDb()->cache(function() {
             return AuditJavascript::find()->distinct(true)
                 ->select('type')
                 ->where(['entry_id' => $this->entry_id])
@@ -83,7 +83,7 @@ 
      */
     public function originFilter()
     {
-        $origin = AuditJavascript::getDb()->cache(function () {
+        $origin = AuditJavascript::getDb()->cache(function() {
             return AuditJavascript::find()->distinct(true)
                 ->select('origin')
                 ->where(['entry_id' => $this->entry_id])

src/models/AuditTrailSearch.php

Braces +3 added lines, -2 removed lines

@@ -57,8 +57,9 @@
 
         // adjust the query by adding the filters
         $userId = $this->user_id;
-        if (strlen($this->user_id))
-            $userId = intval($this->user_id) ?: 0;
+        if (strlen($this->user_id)) {
+                    $userId = intval($this->user_id) ?: 0;
+        }
 
         $query->andFilterWhere(['id' => $this->id]);
         $query->andFilterWhere(['entry_id' => $this->entry_id]);

src/panels/ErrorPanel.php

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
             return $entry ? $this->log($entry->id, $e) : null;
         });
 
-        $this->module->registerFunction('errorMessage', function ($message, $code = 0, $file = '', $line = 0, $trace = []) {
+        $this->module->registerFunction('errorMessage', function($message, $code = 0, $file = '', $line = 0, $trace = []) {
             $entry = $this->module->getEntry(true);
             return $entry ? $this->logMessage($entry->id, $message, $code, $file, $line, $trace) : null;
         });
@@ -61,7 +61,7 @@ 
         $error->file        = $exception->getFile();
         $error->line        = $exception->getLine();
         $error->trace       = Helper::cleanupTrace($exception->getTrace());
-        $error->hash        = Helper::hash($error->message . $error->file . $error->line);
+        $error->hash        = Helper::hash($error->message.$error->file.$error->line);
 
         $this->_exceptions[] = $exceptionId;
 
@@ -88,7 +88,7 @@ 
         $error->file        = $file;
         $error->line        = $line;
         $error->trace       = Helper::cleanupTrace($trace);
-        $error->hash        = Helper::hash($error->message . $error->file . $error->line);
+        $error->hash        = Helper::hash($error->message.$error->file.$error->line);
         return $error->save(false) ? $error : null;
     }
 
@@ -113,7 +113,7 @@ 
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->_model->linkedErrors) . ')</small>';
+        return $this->getName().' <small>('.count($this->_model->linkedErrors).')</small>';
     }
 
     /**

Braces +9 added lines, -6 removed lines

@@ -47,12 +47,14 @@ 
     {
         // Only log each exception once
         $exceptionId = spl_object_hash($exception);
-        if (in_array($exceptionId, $this->_exceptions))
-            return true;
+        if (in_array($exceptionId, $this->_exceptions)) {
+                    return true;
+        }
 
         // If this is a follow up exception, make sure to log the base exception first
-        if ($exception->getPrevious())
-            $this->log($entry_id, $exception->getPrevious());
+        if ($exception->getPrevious()) {
+                    $this->log($entry_id, $exception->getPrevious());
+        }
 
         $error = new AuditError();
         $error->entry_id    = $entry_id;
@@ -165,8 +167,9 @@ 
     public function cleanup($maxAge = null)
     {
         $maxAge = $maxAge !== null ? $maxAge : $this->maxAge;
-        if ($maxAge === null)
-            return false;
+        if ($maxAge === null) {
+                    return false;
+        }
         return AuditError::deleteAll([
             '<=', 'created', date('Y-m-d 23:59:59', strtotime("-$maxAge days"))
         ]);

src/panels/ExtraDataPanel.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->data) . ')</small>';
+        return $this->getName().' <small>('.count($this->data).')</small>';
     }
 
     /**

Braces +3 added lines, -2 removed lines

@@ -46,8 +46,9 @@
     public function trackData($type, $data)
     {
         $this->module->getEntry(true);
-        if (!is_array($this->data))
-            $this->data = [];
+        if (!is_array($this->data)) {
+                    $this->data = [];
+        }
 
         $this->data[] = ['type' => $type, 'data' => $data];
     }

src/panels/JavascriptPanel.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->_model->javascripts) . ')</small>';
+        return $this->getName().' <small>('.count($this->_model->javascripts).')</small>';
     }
 
     /**

Braces +3 added lines, -2 removed lines

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