bedezign / yii2-audit

src/migrations/m150626_000005_create_audit_javascript.php

Spacing +5 added lines, -5 removed lines

@@ -11,11 +11,11 @@
     {
         $this->createTable(self::TABLE, [
             'id'         => Schema::TYPE_PK,
-            'entry_id'   => Schema::TYPE_INTEGER . ' NOT NULL',
-            'created'    => Schema::TYPE_DATETIME . ' NOT NULL',
-            'type'       => Schema::TYPE_STRING . '(20) NOT NULL',
-            'message'    => Schema::TYPE_TEXT . ' NOT NULL',
-            'origin'     => Schema::TYPE_STRING . '(512)',
+            'entry_id'   => Schema::TYPE_INTEGER.' NOT NULL',
+            'created'    => Schema::TYPE_DATETIME.' NOT NULL',
+            'type'       => Schema::TYPE_STRING.'(20) NOT NULL',
+            'message'    => Schema::TYPE_TEXT.' NOT NULL',
+            'origin'     => Schema::TYPE_STRING.'(512)',
             'data'       => Schema::TYPE_BINARY,
         ], $this->db->driverName === 'mysql' ? 'CHARACTER SET utf8 COLLATE utf8_general_ci ENGINE=InnoDB' : null);
 

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

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

Spacing +2 added lines, -2 removed lines

@@ -122,7 +122,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];
         }
@@ -232,7 +232,7 @@ 
             if ($this->getRequestMethodIsCli()) {
                 $route = $this->route;
             } else {
-                $route = '/' . $this->route;
+                $route = '/'.$this->route;
             }
             return Url::to([$route ?: '/'], 'https');
         }

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/panels/CurlPanel.php

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->data) . ')</small>';
+        return $this->getName().' <small>('.count($this->data).')</small>';
     }
 
     /**
@@ -137,7 +137,7 @@ 
         }
 
         if ($this->content)
-            $this->data[$id]['content']= curl_multi_getcontent($handle);
+            $this->data[$id]['content'] = curl_multi_getcontent($handle);
 
         // Cleanup empty things
         $this->data[$id] = array_filter($this->data[$id]);
@@ -208,7 +208,7 @@ 
         if (!is_resource($resource))
             return false;
 
-        $parts = explode('#', (string)$resource);
+        $parts = explode('#', (string) $resource);
         return array_pop($parts);
     }
 

Braces +18 added lines, -12 removed lines

@@ -96,19 +96,22 @@ 
             curl_setopt($handle, CURLOPT_URL, $url);
         }
 
-        if ($postData)
-            $this->data[$id]['post'] = $postData;
+        if ($postData) {
+                    $this->data[$id]['post'] = $postData;
+        }
 
-        if ($this->headers)
-            curl_setopt($handle, CURLOPT_HEADERFUNCTION, [$this, 'captureHeader']);
+        if ($this->headers) {
+                    curl_setopt($handle, CURLOPT_HEADERFUNCTION, [$this, 'captureHeader']);
+        }
 
         if ($this->log) {
             curl_setopt($handle, CURLOPT_VERBOSE, true);
             curl_setopt($handle, CURLOPT_STDERR, $this->_logHandles[$id] = fopen('php://temp', 'rw+'));
         }
 
-        if ($this->content)
-            curl_setopt($handle, CURLOPT_RETURNTRANSFER, 1);
+        if ($this->content) {
+                    curl_setopt($handle, CURLOPT_RETURNTRANSFER, 1);
+        }
 
         return true;
     }
@@ -136,8 +139,9 @@ 
             unset($this->_logHandles[$id]);
         }
 
-        if ($this->content)
-            $this->data[$id]['content']= curl_multi_getcontent($handle);
+        if ($this->content) {
+                    $this->data[$id]['content']= curl_multi_getcontent($handle);
+        }
 
         // Cleanup empty things
         $this->data[$id] = array_filter($this->data[$id]);
@@ -192,8 +196,9 @@ 
     public function captureHeader($handle, $header)
     {
         $id = $this->id($handle);
-        if (!isset($this->data[$id]['headers']))
-            $this->data[$id]['headers'] = [];
+        if (!isset($this->data[$id]['headers'])) {
+                    $this->data[$id]['headers'] = [];
+        }
 
         $this->data[$id]['headers'][] = $header;
         return strlen($header);
@@ -205,8 +210,9 @@ 
      */
     protected function id($resource)
     {
-        if (!is_resource($resource))
-            return false;
+        if (!is_resource($resource)) {
+                    return false;
+        }
 
         $parts = explode('#', (string)$resource);
         return array_pop($parts);

src/panels/DbPanel.php

Spacing +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
     {
         $timings = $this->calculateTimings();
         $queryCount = count($timings);
-        $queryTime = number_format($this->getTotalQueryTime($timings) * 1000) . ' ms';
-        return $this->getName() . ' <small>(' . $queryCount . ' / ' . $queryTime . ')</small>';
+        $queryTime = number_format($this->getTotalQueryTime($timings) * 1000).' ms';
+        return $this->getName().' <small>('.$queryCount.' / '.$queryTime.')</small>';
     }
 
     /**

src/panels/ErrorPanel.php

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"))
         ]);

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,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;
         });
@@ -63,7 +63,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;
 
@@ -90,7 +90,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;
     }
 
@@ -115,7 +115,7 @@ 
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . $this->_model->getLinkedErrors()->count() . ')</small>';
+        return $this->getName().' <small>('.$this->_model->getLinkedErrors()->count().')</small>';
     }
 
     /**