bedezign / yii2-audit

src/migrations/m150626_000002_create_audit_data.php

Spacing +2 added lines, -2 removed lines

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

src/migrations/m150626_000003_create_audit_error.php

Spacing +9 added lines, -9 removed lines

@@ -12,15 +12,15 @@ 
         $driver = $this->db->driverName;
         $this->createTable(self::TABLE, [
             'id'         => Schema::TYPE_PK,
-            'entry_id'   => Schema::TYPE_INTEGER . ' NOT NULL',
-            'created'    => Schema::TYPE_DATETIME . ' NOT NULL',
-            'message'    => Schema::TYPE_TEXT . ' NOT NULL',
-            'code'       => Schema::TYPE_INTEGER . " DEFAULT '0'",
-            'file'       => Schema::TYPE_STRING . '(512)',
-            'line'       => Schema::TYPE_INTEGER ,
+            'entry_id'   => Schema::TYPE_INTEGER.' NOT NULL',
+            'created'    => Schema::TYPE_DATETIME.' NOT NULL',
+            'message'    => Schema::TYPE_TEXT.' NOT NULL',
+            'code'       => Schema::TYPE_INTEGER." DEFAULT '0'",
+            'file'       => Schema::TYPE_STRING.'(512)',
+            'line'       => Schema::TYPE_INTEGER,
             'trace'      => Schema::TYPE_BINARY,
-            'hash'       => Schema::TYPE_STRING . '(32)',
-            'emailed'    => Schema::TYPE_BOOLEAN . " NOT NULL DEFAULT '0'",
+            'hash'       => Schema::TYPE_STRING.'(32)',
+            'emailed'    => Schema::TYPE_BOOLEAN." NOT NULL DEFAULT '0'",
         ], ($driver === 'mysql' ? 'CHARACTER SET utf8 COLLATE utf8_general_ci ENGINE=InnoDB' : null));
 
         if ($driver != 'sqlite') {
@@ -28,7 +28,7 @@ 
         }
 
         // Issue #122: Specified key was too long; max key length is 767 bytes - http://stackoverflow.com/q/1814532/50158
-        $this->createIndex('idx_file', self::TABLE, ['file' . ($driver === 'mysql' ? '(180)' : '')]);
+        $this->createIndex('idx_file', self::TABLE, ['file'.($driver === 'mysql' ? '(180)' : '')]);
         $this->createIndex('idx_emailed', self::TABLE, ['emailed']);
     }
 

src/migrations/m150626_000004_create_audit_trail.php

Spacing +5 added lines, -5 removed lines

@@ -12,14 +12,14 @@
         $this->createTable(self::TABLE, [
             'id'        => Schema::TYPE_PK,
             'entry_id'  => Schema::TYPE_INTEGER,
-            'user_id'   => Schema::TYPE_INTEGER . ' NULL',
-            'action'    => Schema::TYPE_STRING . ' NOT NULL',
-            'model'     => Schema::TYPE_STRING . ' NOT NULL',
-            'model_id'  => Schema::TYPE_STRING . ' NOT NULL',
+            'user_id'   => Schema::TYPE_INTEGER.' NULL',
+            'action'    => Schema::TYPE_STRING.' NOT NULL',
+            'model'     => Schema::TYPE_STRING.' NOT NULL',
+            'model_id'  => Schema::TYPE_STRING.' NOT NULL',
             'field'     => Schema::TYPE_STRING,
             'old_value' => Schema::TYPE_TEXT,
             'new_value' => Schema::TYPE_TEXT,
-            'created'   => Schema::TYPE_DATETIME . ' NOT NULL',
+            'created'   => Schema::TYPE_DATETIME.' NOT NULL',
         ], ($this->db->driverName === 'mysql' ? 'CHARACTER SET utf8 COLLATE utf8_general_ci ENGINE=InnoDB' : null));
 
         if ($this->db->driverName != 'sqlite') {

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

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();

Braces +2 added lines, -2 removed lines

@@ -106,9 +106,9 @@
         if (strlen($this->user_id)) {
             if (!is_numeric($userId) && $callback = Audit::getInstance()->userFilterCallback) {
                 $userId = call_user_func($callback, $userId);
+            } else {
+                            $userId = intval($this->user_id) ?: 0;
             }
-            else
-                $userId = intval($this->user_id) ?: 0;
         }
         $query->andFilterWhere(['user_id' => $userId]);
     }

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/AuditJavascript.php

Indentation +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 /**
- * This model allows for storing of javascript logging entries linked to a specific audit entry
- */
+     * This model allows for storing of javascript logging entries linked to a specific audit entry
+     */
 
 namespace bedezign\yii2\audit\models;