Completed
Push3d7113...c00f94
errored — Build
created
src/commands/AuditController.php 3 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
     /**
133 133
      * Cleans the AuditEntry data
134 134
      *
135
-     * @param $maxAge
135
+     * @param integer|null $maxAge
136 136
      * @return bool
137 137
      */
138 138
     protected function cleanupEntry($maxAge)
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
      * Cleans the Panel data
161 161
      *
162 162
      * @param $id
163
-     * @param $maxAge
163
+     * @param integer|null $maxAge
164 164
      * @return bool
165 165
      */
166 166
     protected function cleanupPanel($id, $maxAge)
Please login to merge, or discard this patch.
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2,11 +2,11 @@
 block discarded – undo
2 2
 
3 3
 namespace bedezign\yii2\audit\commands;
4 4
 
5
+use Yii;
5 6
 use bedezign\yii2\audit\Audit;
6 7
 use bedezign\yii2\audit\components\panels\Panel;
7 8
 use bedezign\yii2\audit\models\AuditEntry;
8 9
 use bedezign\yii2\audit\models\AuditError;
9
-use Yii;
10 10
 use yii\base\Exception;
11 11
 use yii\console\Controller;
12 12
 use yii\helpers\Console;
Please login to merge, or discard this patch.
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -113,9 +113,9 @@  discard block
 block discarded – undo
113 113
 
114 114
         // heading
115 115
         $n = count($panels);
116
-        $this->stdout("Total $n " . ($n === 1 ? 'cleanup' : 'cleanups') . " to be applied:\n", Console::FG_YELLOW);
117
-        $this->stdout("\t" . 'DATA                      CLEANUP TO DATETIME' . "\n");
118
-        $this->stdout("\t" . '---------------------------------------------' . "\n");
116
+        $this->stdout("Total $n ".($n === 1 ? 'cleanup' : 'cleanups')." to be applied:\n", Console::FG_YELLOW);
117
+        $this->stdout("\t".'DATA                      CLEANUP TO DATETIME'."\n");
118
+        $this->stdout("\t".'---------------------------------------------'."\n");
119 119
 
120 120
         // audit panels
121 121
         foreach ($panels as $id) {
@@ -125,9 +125,9 @@  discard block
 block discarded – undo
125 125
             $dots = str_repeat('.', 24 - strlen($id));
126 126
             if ($age !== null) {
127 127
                 $date = date('Y-m-d 23:59:59', strtotime("-$age days"));
128
-                $this->stdout("\t" . $id . ' ' . $dots . ' ' . $date . "\n");
128
+                $this->stdout("\t".$id.' '.$dots.' '.$date."\n");
129 129
             } else {
130
-                $this->stdout("\t" . $id . ' ' . $dots . ' no maxAge, skipping' . "\n");
130
+                $this->stdout("\t".$id.' '.$dots.' no maxAge, skipping'."\n");
131 131
             }
132 132
         }
133 133
 
@@ -135,12 +135,12 @@  discard block
 block discarded – undo
135 135
         if ($entry) {
136 136
             $maxAge = $maxAge !== null ? $maxAge : $audit->maxAge;
137 137
             $date = $maxAge !== null ? date('Y-m-d 23:59:59', strtotime("-$maxAge days")) : 'no maxAge, skipping';
138
-            $this->stdout("\t" . 'AuditEntry .............. ' . $date . "\n");
138
+            $this->stdout("\t".'AuditEntry .............. '.$date."\n");
139 139
         }
140 140
 
141 141
         // audit entry solo
142 142
         if ($entrySolo) {
143
-            $this->stdout("\t" . 'AuditEntry solo ......... ' . date('Y-m-d 23:59:59') . "\n");
143
+            $this->stdout("\t".'AuditEntry solo ......... '.date('Y-m-d 23:59:59')."\n");
144 144
         }
145 145
 
146 146
         $this->stdout("\n");
@@ -165,11 +165,11 @@  discard block
 block discarded – undo
165 165
         $count = AuditEntry::deleteAll(['<=', 'created', $date]);
166 166
         if ($count !== false) {
167 167
             $time = microtime(true) - $start;
168
-            $this->stdout("\n*** cleaned AuditEntry (records: " . $count . ",time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_GREEN);
168
+            $this->stdout("\n*** cleaned AuditEntry (records: ".$count.",time: ".sprintf("%.3f", $time)."s)\n", Console::FG_GREEN);
169 169
             return true;
170 170
         }
171 171
         $time = microtime(true) - $start;
172
-        $this->stdout("\n*** failed to clean AuditEntry (time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_RED);
172
+        $this->stdout("\n*** failed to clean AuditEntry (time: ".sprintf("%.3f", $time)."s)\n", Console::FG_RED);
173 173
         return false;
174 174
     }
175 175
 
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
             }
205 205
         }
206 206
         $time = microtime(true) - $start;
207
-        $this->stdout("\n*** cleaned AuditEntry (records: " . $count . ",time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_GREEN);
207
+        $this->stdout("\n*** cleaned AuditEntry (records: ".$count.",time: ".sprintf("%.3f", $time)."s)\n", Console::FG_GREEN);
208 208
         return true;
209 209
     }
210 210
 
@@ -229,11 +229,11 @@  discard block
 block discarded – undo
229 229
         $count = $panel->cleanup($maxAge);
230 230
         if ($count !== false) {
231 231
             $time = microtime(true) - $start;
232
-            $this->stdout("\n*** cleaned $id (records: " . $count . ", time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_GREEN);
232
+            $this->stdout("\n*** cleaned $id (records: ".$count.", time: ".sprintf("%.3f", $time)."s)\n", Console::FG_GREEN);
233 233
             return true;
234 234
         }
235 235
         $time = microtime(true) - $start;
236
-        $this->stdout("\n*** failed to clean $id (time: " . sprintf("%.3f", $time) . "s)\n", Console::FG_RED);
236
+        $this->stdout("\n*** failed to clean $id (time: ".sprintf("%.3f", $time)."s)\n", Console::FG_RED);
237 237
         return false;
238 238
     }
239 239
 
@@ -265,13 +265,13 @@  discard block
 block discarded – undo
265 265
                 ];
266 266
                 $message = [
267 267
                     'subject' => Yii::t('audit', 'Audit Error in Audit Entry #{entry_id}', $params),
268
-                    'text' => Yii::t('audit', '{message}' . "\n" . 'in {file} on line {line}.' . "\n" . '-- {url}', $params),
268
+                    'text' => Yii::t('audit', '{message}'."\n".'in {file} on line {line}.'."\n".'-- {url}', $params),
269 269
                     'html' => Yii::t('audit', '<b>{message}</b><br />in <i>{file}</i> on line <i>{line}</i>.<br/>-- {link}', $params),
270 270
                 ];
271 271
 
272 272
                 // send email
273 273
                 Yii::$app->mailer->compose()
274
-                    ->setFrom([$email => 'Audit :: ' . Yii::$app->name])
274
+                    ->setFrom([$email => 'Audit :: '.Yii::$app->name])
275 275
                     ->setTo($email)
276 276
                     ->setSubject($message['subject'])
277 277
                     ->setTextBody($message['text'])
Please login to merge, or discard this patch.