bedezign / yii2-audit

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")),

Spacing +2 added lines, -2 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>';
     }
 
     /**
@@ -80,7 +80,7 @@ 
      */
     public function getUrl($additionalParams = NULL)
     {
-        return Url::toRoute(['/' . $this->module->id . '/entry/view',
+        return Url::toRoute(['/'.$this->module->id.'/entry/view',
             'panel' => $this->id,
             'id' => $this->tag,
         ]);

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

Spacing +5 added lines, -5 removed lines

@@ -44,13 +44,13 @@
             'message'    => 'message',
             'type'       => 'type',
             'file'       => 'origin',
-            'line'       => function ($value) use ($javascript) {
-                $javascript->origin .= ':' . $value;
+            'line'       => function($value) use ($javascript) {
+                $javascript->origin .= ':'.$value;
             },
-            'col'        => function ($value) use ($javascript) {
-                $javascript->origin .= ':' . $value;
+            'col'        => function($value) use ($javascript) {
+                $javascript->origin .= ':'.$value;
             },
-            'data'       => function ($value) use ($javascript) {
+            'data'       => function($value) use ($javascript) {
                 if (count($value)) $javascript->data = $value;
             },
         ];

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/controllers/MailController.php

Spacing +1 added lines, -1 removed lines

@@ -58,6 +58,6 @@
         if (!$model) {
             throw new NotFoundHttpException('The requested mail does not exist.');
         }
-        Yii::$app->response->sendContentAsFile(Helper::uncompress($model->data), $model->id . '.eml');
+        Yii::$app->response->sendContentAsFile(Helper::uncompress($model->data), $model->id.'.eml');
     }
 }

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.

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 return [
     // string, required, root directory of all source files
-    'sourcePath' => __DIR__ . '/..',
+    'sourcePath' => __DIR__.'/..',
     // array, required, list of language codes that the extracted messages
     // should be translated to. For example, ['zh-CN', 'de'].
     'languages' => ['en', 'nl', 'ru', 'es', 'tr'],

src/migrations/0.1-1.0/m140829_000001_create_audit_entry.php

Spacing +12 added lines, -12 removed lines

@@ -15,19 +15,19 @@
 
         $this->createTable(self::TABLE, [
             'id'         => Schema::TYPE_PK,
-            'created'    => Schema::TYPE_DATETIME . ' NOT NULL',
-            'start_time' => Schema::TYPE_FLOAT . ' NULL',
-            'end_time'   => Schema::TYPE_FLOAT . ' NULL',
-            'duration'   => Schema::TYPE_FLOAT . ' NULL',
-            'user_id'    => Schema::TYPE_INTEGER . " DEFAULT '0'",
-            'ip'         => Schema::TYPE_STRING . '(45) NULL',
-            'referrer'   => Schema::TYPE_STRING . '(512) NULL',
-            'origin'     => Schema::TYPE_STRING . '(512) NULL',
-            'url'        => Schema::TYPE_STRING . '(512) NULL',
-            'route'      => Schema::TYPE_STRING . '(255) NULL',
+            'created'    => Schema::TYPE_DATETIME.' NOT NULL',
+            'start_time' => Schema::TYPE_FLOAT.' NULL',
+            'end_time'   => Schema::TYPE_FLOAT.' NULL',
+            'duration'   => Schema::TYPE_FLOAT.' NULL',
+            'user_id'    => Schema::TYPE_INTEGER." DEFAULT '0'",
+            'ip'         => Schema::TYPE_STRING.'(45) NULL',
+            'referrer'   => Schema::TYPE_STRING.'(512) NULL',
+            'origin'     => Schema::TYPE_STRING.'(512) NULL',
+            'url'        => Schema::TYPE_STRING.'(512) NULL',
+            'route'      => Schema::TYPE_STRING.'(255) NULL',
             'data'       => 'BLOB NULL',
-            'memory'     => Schema::TYPE_INTEGER . " NULL",
-            'memory_max' => Schema::TYPE_INTEGER . " NULL",
+            'memory'     => Schema::TYPE_INTEGER." NULL",
+            'memory_max' => Schema::TYPE_INTEGER." NULL",
 
         ], $tableOptions);
 

src/migrations/0.1-1.0/m140829_000002_create_audit_data.php

Spacing +3 added lines, -3 removed lines

@@ -15,9 +15,9 @@
 
         $this->createTable(self::TABLE, [
             'id'         => Schema::TYPE_PK,
-            'audit_id'   => Schema::TYPE_INTEGER . ' NOT NULL',
-            'name'       => Schema::TYPE_STRING . '(255) NOT NULL',
-            'type'       => Schema::TYPE_STRING . '(255) NULL',
+            'audit_id'   => Schema::TYPE_INTEGER.' NOT NULL',
+            'name'       => Schema::TYPE_STRING.'(255) NOT NULL',
+            'type'       => Schema::TYPE_STRING.'(255) NULL',
             'packed'     => "TINYINT(1) UNSIGNED NOT NULL DEFAULT '0'",
             'data'       => 'BLOB NOT NULL',
         ], $tableOptions);

src/migrations/0.1-1.0/m140829_000003_create_audit_error.php

Spacing +6 added lines, -6 removed lines

@@ -15,12 +15,12 @@
 
         $this->createTable(self::TABLE, [
             'id'         => Schema::TYPE_PK,
-            'audit_id'   => Schema::TYPE_INTEGER . ' NOT NULL',
-            'created'    => Schema::TYPE_DATETIME . ' NOT NULL',
-            'message'    => Schema::TYPE_STRING . '(512) NOT NULL',
-            'code'       => Schema::TYPE_INTEGER . " DEFAULT '0'",
-            'file'       => Schema::TYPE_STRING . '(512)',
-            'line'       => Schema::TYPE_INTEGER ,
+            'audit_id'   => Schema::TYPE_INTEGER.' NOT NULL',
+            'created'    => Schema::TYPE_DATETIME.' NOT NULL',
+            'message'    => Schema::TYPE_STRING.'(512) NOT NULL',
+            'code'       => Schema::TYPE_INTEGER." DEFAULT '0'",
+            'file'       => Schema::TYPE_STRING.'(512)',
+            'line'       => Schema::TYPE_INTEGER,
             'trace'      => 'BLOB',
         ], $tableOptions);