bedezign / yii2-audit

src/panels/CurlPanel.php

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

@@ -8,8 +8,8 @@
 
 use Yii;
 use bedezign\yii2\audit\components\panels\DataStoragePanel;
-use yii\grid\GridViewAsset;
 use yii\data\ArrayDataProvider;
+use yii\grid\GridViewAsset;
 
 /**
  * Class CurlPanel

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

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

@@ -2,8 +2,8 @@
 
 namespace bedezign\yii2\audit\panels;
 
-use bedezign\yii2\audit\components\panels\DataStoragePanelTrait;
 use Yii;
+use bedezign\yii2\audit\components\panels\DataStoragePanelTrait;
 use yii\debug\models\search\Db;
 use yii\grid\GridViewAsset;
 

src/panels/LogPanel.php

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

@@ -2,8 +2,8 @@
 
 namespace bedezign\yii2\audit\panels;
 
-use bedezign\yii2\audit\components\panels\DataStoragePanelTrait;
 use Yii;
+use bedezign\yii2\audit\components\panels\DataStoragePanelTrait;
 use yii\debug\models\search\Log;
 use yii\grid\GridViewAsset;
 

src/panels/MailPanel.php

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

@@ -2,14 +2,14 @@
 
 namespace bedezign\yii2\audit\panels;
 
-use bedezign\yii2\audit\Audit;
-use bedezign\yii2\audit\components\panels\Panel;
-use bedezign\yii2\audit\models\AuditMail;
-use bedezign\yii2\audit\models\AuditMailSearch;
 use Swift_Message;
 use Swift_Mime_Attachment;
 use Swift_Mime_MimePart;
 use Yii;
+use bedezign\yii2\audit\Audit;
+use bedezign\yii2\audit\components\panels\Panel;
+use bedezign\yii2\audit\models\AuditMail;
+use bedezign\yii2\audit\models\AuditMailSearch;
 use yii\base\Event;
 use yii\grid\GridViewAsset;
 use yii\mail\BaseMailer;

Braces +3 added lines, -2 removed lines

@@ -167,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 AuditMail::deleteAll([
             '<=', 'created', date('Y-m-d 23:59:59', strtotime("-$maxAge days"))
         ]);

src/panels/ProfilingPanel.php

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

@@ -2,8 +2,8 @@
 
 namespace bedezign\yii2\audit\panels;
 
-use bedezign\yii2\audit\components\panels\DataStoragePanelTrait;
 use Yii;
+use bedezign\yii2\audit\components\panels\DataStoragePanelTrait;
 use yii\debug\models\search\Profile;
 use yii\grid\GridViewAsset;
 

src/panels/SoapPanel.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace bedezign\yii2\audit\panels;
 
 use Yii;
-use bedezign\yii2\audit\models\AuditError;
 use bedezign\yii2\audit\components\panels\DataStoragePanel;
 use yii\data\ArrayDataProvider;
 

Braces +3 added lines, -2 removed lines

@@ -35,8 +35,9 @@
     {
         $this->module->registerPanel($this);
 
-        if (!is_array($this->data))
-            $this->data = [];
+        if (!is_array($this->data)) {
+                    $this->data = [];
+        }
 
         if (isset($data['error'])) {
             $error = $this->module->exception($data['error']);

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

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

@@ -1,10 +1,10 @@
 <?php
 /* @var $panel bedezign\yii2\audit\panels\SoapPanel */
 
-use yii\bootstrap\Tabs;
-use yii\helpers\Html;
 use bedezign\yii2\audit\components\Helper;
+use yii\bootstrap\Tabs;
 use yii\helpers\ArrayHelper;
+use yii\helpers\Html;
 use yii\helpers\VarDumper;
 
 $preformatted = ['class' => 'well', 'style' => 'overflow: auto; white-space: pre'];

Braces +3 added lines, -2 removed lines

@@ -24,11 +24,12 @@
 
 if (isset($request['result'])) {
     $xml = Helper::formatAsXML($request['result']);
-    if ($xml)
-        $tabs[] = [
+    if ($xml) {
+            $tabs[] = [
             'label' => \Yii::t('audit', 'Result - XML'),
             'content' => Html::tag('div', $xml, $preformatted)
         ];
+    }
 
     $tabs[] = [
         'label' => \Yii::t('audit', 'Result'),

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

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 use yii\helpers\Html;
 use yii\helpers\VarDumper;
-use yii\helpers\Url;
 
 
 

Braces +5 added lines, -4 removed lines

@@ -28,10 +28,11 @@
                 <th><?= Html::encode(\yii\helpers\Inflector::humanize($name)) ?></th>
                 <td>
 <?php
-    if (is_array($value) || is_object($value))
-        $value = \yii\helpers\ArrayHelper::toArray($value);
-    elseif ($name == 'duration')
-        $value = number_format($value, 2) . 's';
+    if (is_array($value) || is_object($value)) {
+            $value = \yii\helpers\ArrayHelper::toArray($value);
+    } elseif ($name == 'duration') {
+            $value = number_format($value, 2) . 's';
+    }
     echo $name == 'error' ? $value : $formatter->asText(is_scalar($value) ? $value : VarDumper::dumpAsString($value));
 ?>
                 </td>

src/LogTarget.php

Braces +12 added lines, -9 removed lines

@@ -36,13 +36,15 @@ 
      */
     public function export()
     {
-        if (!\Yii::$app)
-            // Things like this can happen in tests etc, but it is too late for us to do anything about that.
+        if (!\Yii::$app) {
+                    // Things like this can happen in tests etc, but it is too late for us to do anything about that.
             return;
+        }
 
         $module = $this->module;
-        if (!$module->entry || empty($module->panels))
-            return;
+        if (!$module->entry || empty($module->panels)) {
+                    return;
+        }
 
         $entry = $module->entry;
 
@@ -53,11 +55,12 @@ 
         $records = array_filter($records);
 
         if (!empty($records)) {
-            if ($module->batchSave)
-                $entry->addBatchData($records, false);
-            else {
-                foreach ($records as $type => $record)
-                    $entry->addData($type, $record, false);
+            if ($module->batchSave) {
+                            $entry->addBatchData($records, false);
+            } else {
+                foreach ($records as $type => $record) {
+                                    $entry->addData($type, $record, false);
+                }
             }
         }
         $this->messages = [];