bedezign / yii2-audit

src/commands/AuditController.php

Doc Comments +2 added lines, -2 removed lines

@@ -132,7 +132,7 @@ 
     /**
      * Cleans the AuditEntry data
      *
-     * @param $maxAge
+     * @param integer|null $maxAge
      * @return bool
      */
     protected function cleanupEntry($maxAge)
@@ -160,7 +160,7 @@ 
      * Cleans the Panel data
      *
      * @param $id
-     * @param $maxAge
+     * @param integer|null $maxAge
      * @return bool
      */
     protected function cleanupPanel($id, $maxAge)

Spacing +14 added lines, -14 removed lines

@@ -113,9 +113,9 @@ 
 
         // heading
         $n = count($panels);
-        $this->stdout("Total $n " . ($n === 1 ? 'cleanup' : 'cleanups') . " to be applied:\n", Console::FG_YELLOW);
-        $this->stdout("\t" . 'DATA                      CLEANUP TO DATETIME' . "\n");
-        $this->stdout("\t" . '---------------------------------------------' . "\n");
+        $this->stdout("Total $n ".($n === 1 ? 'cleanup' : 'cleanups')." to be applied:\n", Console::FG_YELLOW);
+        $this->stdout("\t".'DATA                      CLEANUP TO DATETIME'."\n");
+        $this->stdout("\t".'---------------------------------------------'."\n");
 
         // audit panels
         foreach ($panels as $id) {
@@ -125,9 +125,9 @@ 
             $dots = str_repeat('.', 24 - strlen($id));
             if ($age !== null) {
                 $date = date('Y-m-d 23:59:59', strtotime("-$age days"));
-                $this->stdout("\t" . $id . ' ' . $dots . ' ' . $date . "\n");
+                $this->stdout("\t".$id.' '.$dots.' '.$date."\n");
             } else {
-                $this->stdout("\t" . $id . ' ' . $dots . ' no maxAge, skipping' . "\n");
+                $this->stdout("\t".$id.' '.$dots.' no maxAge, skipping'."\n");
             }
         }
 
@@ -135,12 +135,12 @@ 
         if ($entry) {
             $maxAge = $maxAge !== null ? $maxAge : $audit->maxAge;
             $date = $maxAge !== null ? date('Y-m-d 23:59:59', strtotime("-$maxAge days")) : 'no maxAge, skipping';
-            $this->stdout("\t" . 'AuditEntry .............. ' . $date . "\n");
+            $this->stdout("\t".'AuditEntry .............. '.$date."\n");
         }
 
         // audit entry solo
         if ($entrySolo) {
-            $this->stdout("\t" . 'AuditEntry solo ......... ' . date('Y-m-d 23:59:59') . "\n");
+            $this->stdout("\t".'AuditEntry solo ......... '.date('Y-m-d 23:59:59')."\n");
         }
 
         $this->stdout("\n");
@@ -165,11 +165,11 @@ 
         $count = AuditEntry::deleteAll(['<=', 'created', $date]);
         if ($count !== false) {
             $time = microtime(true) - $start;
-            $this->stdout("\n*** cleaned AuditEntry (records: " . $count . ",time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_GREEN);
+            $this->stdout("\n*** cleaned AuditEntry (records: ".$count.",time: ".sprintf("%.3f", $time)."s)\n", Console::FG_GREEN);
             return true;
         }
         $time = microtime(true) - $start;
-        $this->stdout("\n*** failed to clean AuditEntry (time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_RED);
+        $this->stdout("\n*** failed to clean AuditEntry (time: ".sprintf("%.3f", $time)."s)\n", Console::FG_RED);
         return false;
     }
 
@@ -204,7 +204,7 @@ 
             }
         }
         $time = microtime(true) - $start;
-        $this->stdout("\n*** cleaned AuditEntry (records: " . $count . ",time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_GREEN);
+        $this->stdout("\n*** cleaned AuditEntry (records: ".$count.",time: ".sprintf("%.3f", $time)."s)\n", Console::FG_GREEN);
         return true;
     }
 
@@ -229,11 +229,11 @@ 
         $count = $panel->cleanup($maxAge);
         if ($count !== false) {
             $time = microtime(true) - $start;
-            $this->stdout("\n*** cleaned $id (records: " . $count . ", time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_GREEN);
+            $this->stdout("\n*** cleaned $id (records: ".$count.", time: ".sprintf("%.3f", $time)."s)\n", Console::FG_GREEN);
             return true;
         }
         $time = microtime(true) - $start;
-        $this->stdout("\n*** failed to clean $id (time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_RED);
+        $this->stdout("\n*** failed to clean $id (time: ".sprintf("%.3f", $time)."s)\n", Console::FG_RED);
         return false;
     }
 
@@ -265,13 +265,13 @@ 
                 ];
                 $message = [
                     'subject' => Yii::t('audit', 'Audit Error in Audit Entry #{entry_id}', $params),
-                    'text' => Yii::t('audit', '{message}' . "\n" . 'in {file} on line {line}.' . "\n" . '-- {url}', $params),
+                    'text' => Yii::t('audit', '{message}'."\n".'in {file} on line {line}.'."\n".'-- {url}', $params),
                     'html' => Yii::t('audit', '<b>{message}</b><br />in <i>{file}</i> on line <i>{line}</i>.<br/>-- {link}', $params),
                 ];
 
                 // send email
                 Yii::$app->mailer->compose()
-                    ->setFrom([$email => 'Audit :: ' . Yii::$app->name])
+                    ->setFrom([$email => 'Audit :: '.Yii::$app->name])
                     ->setTo($email)
                     ->setSubject($message['subject'])
                     ->setTextBody($message['text'])

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,11 +2,11 @@
 
 namespace bedezign\yii2\audit\commands;
 
+use Yii;
 use bedezign\yii2\audit\Audit;
 use bedezign\yii2\audit\components\panels\Panel;
 use bedezign\yii2\audit\models\AuditEntry;
 use bedezign\yii2\audit\models\AuditError;
-use Yii;
 use yii\base\Exception;
 use yii\console\Controller;
 use yii\console\ExitCode;

src/views/mail/index.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             [
                 'attribute' => 'entry_id',
                 'class' => 'yii\grid\DataColumn',
-                'value' => function ($data) {
+                'value' => function($data) {
                     return $data->entry_id ? Html::a($data->entry_id, ['entry/view', 'id' => $data->entry_id]) : '';
                 },
                 'format' => 'raw',

src/views/entry/panels/soap/index.php

Braces +2 added lines, -1 removed lines

@@ -6,5 +6,6 @@
 
 echo Html::tag('h1', $panel->name);
 
-foreach ($dataProvider->allModels as $index => $request)
+foreach ($dataProvider->allModels as $index => $request) {
     echo $this->render('detail', ['panel' => $panel, 'request' => $request, 'index' => $index + 1]);
+}

src/views/javascript/index.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             [
                 'attribute' => 'entry_id',
                 'class' => 'yii\grid\DataColumn',
-                'value' => function ($data) {
+                'value' => function($data) {
                     return $data->entry_id ? Html::a($data->entry_id, ['entry/view', 'id' => $data->entry_id]) : '';
                 },
                 'format' => 'raw',

src/views/default/panels/trail/chart.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 $defaults = [];
 $startDate = strtotime('-6 days');
 foreach (range(-6, 0) as $day) {
-    $defaults[date('D: Y-m-d', strtotime($day . 'days'))] = 0;
+    $defaults[date('D: Y-m-d', strtotime($day.'days'))] = 0;
 }
 
 $results = AuditTrail::find()

src/components/panels/RendersSummaryChartTrait.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $defaults = [];
         $startDate = strtotime('-6 days');
         foreach (range(-6, 0) as $day) {
-            $defaults[date('D: Y-m-d', strtotime($day . 'days'))] = 0;
+            $defaults[date('D: Y-m-d', strtotime($day.'days'))] = 0;
         }
 
         $panelModel = $this->getChartModel();

src/views/mail/view-html.php

Spacing +2 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
 $this->title                   = Yii::t('audit', 'Mail #{id}', ['id' => $model->id]);
 $this->params['breadcrumbs'][] = ['label' => Yii::t('audit', 'Audit'), 'url' => ['default/index']];
 $this->params['breadcrumbs'][] = ['label' => Yii::t('audit', 'Mails'), 'url' => ['index']];
-$this->params['breadcrumbs'][] = '#' . $model->id;
+$this->params['breadcrumbs'][] = '#'.$model->id;
 
 
 //echo Yii::$app->formatter->asHtml($model->html);
@@ -20,5 +20,5 @@ 
     $parser->setText($model->data);
     echo $parser->getMessageBody('htmlEmbedded');
 } else {
-    echo Html::tag('pre', $model->data) . Html::tag('br') . 'Please install php-mime-mail-parser for better functionality';
+    echo Html::tag('pre', $model->data).Html::tag('br').'Please install php-mime-mail-parser for better functionality';
 }

src/views/error/view.php

Spacing +4 added lines, -5 removed lines

@@ -11,7 +11,7 @@ 
 $this->title = Yii::t('audit', 'Error #{id}', ['id' => $model->id]);
 $this->params['breadcrumbs'][] = ['label' => Yii::t('audit', 'Audit'), 'url' => ['default/index']];
 $this->params['breadcrumbs'][] = ['label' => Yii::t('audit', 'Errors'), 'url' => ['index']];
-$this->params['breadcrumbs'][] = '#' . $model->id;
+$this->params['breadcrumbs'][] = '#'.$model->id;
 
 echo Html::tag('h1', $this->title);
 
@@ -44,16 +44,15 @@ 
         'line',
         [
             'header' => Yii::t('audit', 'Called'),
-            'value' => function ($data) {
+            'value' => function($data) {
                 return
                     isset($data['type']) ?
-                        (( isset($data['class']) ? $data['class'] : '[unknown]') . $data['type'] . $data['function']) :
-                        $data['function'];
+                        ((isset($data['class']) ? $data['class'] : '[unknown]').$data['type'].$data['function']) : $data['function'];
             }
         ],
         [
             'header' => Yii::t('audit', 'Args'),
-            'value' => function ($data) {
+            'value' => function($data) {
                 $out = '<a class="args-toggle glyphicon glyphicon-plus" href="javascript:void(0);"></a>';
                 $out .= '<pre style="display:none;">';
                 $out .= !empty($data['args']) ? VarDumper::dumpAsString($data['args']) : '';