bedezign / yii2-audit

src/components/panels/PanelTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
     }
 
     /**
-     * @return array|bool
+     * @return boolean
      */
     public function getIndexUrl()
     {
@@ -48,7 +48,7 @@ 
     }
 
     /**
-     * @return string|bool
+     * @return boolean
      */
     public function getChart()
     {

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()
     {
-        return Url::toRoute(['/' . $this->module->id . '/entry/view',
+        return Url::toRoute(['/'.$this->module->id.'/entry/view',
             'panel' => $this->id,
             'id' => $this->tag,
         ]);

Braces +3 added lines, -2 removed lines

@@ -87,8 +87,9 @@
     public function cleanup($maxAge = null)
     {
         $maxAge = $maxAge !== null ? $maxAge : $this->maxAge;
-        if ($maxAge === null)
-            return false;
+        if ($maxAge === null) {
+                    return false;
+        }
         return AuditTrail::deleteAll([
             '<=', 'created', date('Y-m-d 23:59:59', strtotime("-$maxAge days"))
         ]);

src/components/Version.php

Doc Comments +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
      * @param $id
      * @param $attribute
      * @param $version
-     * @return null|integer|float|string
+     * @return string|null
      */
     public static function findAttribute($class, $id, $attribute, $version)
     {

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

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->data) . ')</small>';
+        return $this->getName().' <small>('.count($this->data).')</small>';
     }
 
     /**
@@ -137,7 +137,7 @@ 
         }
 
         if ($this->content)
-            $this->data[$id]['content']= curl_multi_getcontent($handle);
+            $this->data[$id]['content'] = curl_multi_getcontent($handle);
 
         // Cleanup empty things
         $this->data[$id] = array_filter($this->data[$id]);
@@ -208,7 +208,7 @@ 
         if (!is_resource($resource))
             return false;
 
-        $parts = explode('#', (string)$resource);
+        $parts = explode('#', (string) $resource);
         return array_pop($parts);
     }
 

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

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 /**
- * Allows the user to log the performed cURL requests for later viewing.
- * Since these cannot be tracked automatically, the panel provides 2 utility functions: `curlStart()` and `curlEnd()`
- */
+     * Allows the user to log the performed cURL requests for later viewing.
+     * Since these cannot be tracked automatically, the panel provides 2 utility functions: `curlStart()` and `curlEnd()`
+     */
 
 namespace bedezign\yii2\audit\panels;
 

src/panels/ErrorPanel.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
      * @param string     $file
      * @param int        $line
      * @param array      $trace         Stack trace to include. Use `Helper::generateTrace()` to create it.
-     * @return null|static
+     * @return AuditError|null
      */
     public function logMessage($entry_id, $message, $code = 0, $file = '', $line = 0, $trace = [])
     {

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
             return $entry ? $this->log($entry->id, $e) : null;
         });
 
-        $this->module->registerFunction('errorMessage', function ($message, $code = 0, $file = '', $line = 0, $trace = []) {
+        $this->module->registerFunction('errorMessage', function($message, $code = 0, $file = '', $line = 0, $trace = []) {
             $entry = $this->module->getEntry(true);
             return $entry ? $this->logMessage($entry->id, $message, $code, $file, $line, $trace) : null;
         });
@@ -61,7 +61,7 @@ 
         $error->file        = $exception->getFile();
         $error->line        = $exception->getLine();
         $error->trace       = Helper::cleanupTrace($exception->getTrace());
-        $error->hash        = Helper::hash($error->message . $error->file . $error->line);
+        $error->hash        = Helper::hash($error->message.$error->file.$error->line);
 
         $this->_exceptions[] = $exceptionId;
 
@@ -88,7 +88,7 @@ 
         $error->file        = $file;
         $error->line        = $line;
         $error->trace       = Helper::cleanupTrace($trace);
-        $error->hash        = Helper::hash($error->message . $error->file . $error->line);
+        $error->hash        = Helper::hash($error->message.$error->file.$error->line);
         return $error->save(false) ? $error : null;
     }
 
@@ -113,7 +113,7 @@ 
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->_model->linkedErrors) . ')</small>';
+        return $this->getName().' <small>('.count($this->_model->linkedErrors).')</small>';
     }
 
     /**

Braces +9 added lines, -6 removed lines

@@ -47,12 +47,14 @@ 
     {
         // Only log each exception once
         $exceptionId = spl_object_hash($exception);
-        if (in_array($exceptionId, $this->_exceptions))
-            return true;
+        if (in_array($exceptionId, $this->_exceptions)) {
+                    return true;
+        }
 
         // If this is a follow up exception, make sure to log the base exception first
-        if ($exception->getPrevious())
-            $this->log($entry_id, $exception->getPrevious());
+        if ($exception->getPrevious()) {
+                    $this->log($entry_id, $exception->getPrevious());
+        }
 
         $error = new AuditError();
         $error->entry_id    = $entry_id;
@@ -165,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 AuditError::deleteAll([
             '<=', 'created', date('Y-m-d 23:59:59', strtotime("-$maxAge days"))
         ]);

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

@@ -2,10 +2,10 @@
 
 namespace bedezign\yii2\audit\panels;
 
+use Yii;
 use bedezign\yii2\audit\components\panels\Panel;
 use bedezign\yii2\audit\models\AuditTrail;
 use bedezign\yii2\audit\models\AuditTrailSearch;
-use Yii;
 use yii\grid\GridViewAsset;
 
 /**

src/panels/MailPanel.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     /**
      * @param Event $event
-     * @return null|static
+     * @return AuditMail|null
      */
     public function record($event)
     {

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;

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
     public function init()
     {
         parent::init();
-        Event::on(BaseMailer::className(), BaseMailer::EVENT_AFTER_SEND, function ($event) {
+        Event::on(BaseMailer::className(), BaseMailer::EVENT_AFTER_SEND, function($event) {
             $this->record($event);
         });
     }
@@ -115,7 +115,7 @@ 
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->_model->mails) . ')</small>';
+        return $this->getName().' <small>('.count($this->_model->mails).')</small>';
     }
 
     /**

Braces +3 added lines, -2 removed lines

@@ -87,8 +87,9 @@
     public function cleanup($maxAge = null)
     {
         $maxAge = $maxAge !== null ? $maxAge : $this->maxAge;
-        if ($maxAge === null)
-            return false;
+        if ($maxAge === null) {
+                    return false;
+        }
         return AuditTrail::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;
 

Spacing +3 added lines, -3 removed lines

@@ -21,8 +21,8 @@ 
     public function getLabel()
     {
         $memory = sprintf('%.1f MB', $this->data['memory'] / 1048576);
-        $time = number_format($this->data['time'] * 1000) . ' ms';
-        return $this->getName() . ' <small>(' . $memory . ' / ' . $time . ')</small>';
+        $time = number_format($this->data['time'] * 1000).' ms';
+        return $this->getName().' <small>('.$memory.' / '.$time.')</small>';
     }
 
     /**
@@ -38,7 +38,7 @@ 
             'dataProvider' => $dataProvider,
             'searchModel' => $searchModel,
             'memory' => sprintf('%.1f MB', $this->data['memory'] / 1048576),
-            'time' => number_format($this->data['time'] * 1000) . ' ms',
+            'time' => number_format($this->data['time'] * 1000).' ms',
         ]);
     }
 

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;
 

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->data) . ')</small>';
+        return $this->getName().' <small>('.count($this->data).')</small>';
     }
 
     /**

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/panels/TrailPanel.php

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

@@ -2,10 +2,10 @@
 
 namespace bedezign\yii2\audit\panels;
 
+use Yii;
 use bedezign\yii2\audit\components\panels\Panel;
 use bedezign\yii2\audit\models\AuditTrail;
 use bedezign\yii2\audit\models\AuditTrailSearch;
-use Yii;
 use yii\grid\GridViewAsset;
 
 /**

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function getLabel()
     {
-        return $this->getName() . ' <small>(' . count($this->_model->trails) . ')</small>';
+        return $this->getName().' <small>('.count($this->_model->trails).')</small>';
     }
 
     /**

Braces +3 added lines, -2 removed lines

@@ -87,8 +87,9 @@
     public function cleanup($maxAge = null)
     {
         $maxAge = $maxAge !== null ? $maxAge : $this->maxAge;
-        if ($maxAge === null)
-            return false;
+        if ($maxAge === null) {
+                    return false;
+        }
         return AuditTrail::deleteAll([
             '<=', 'created', date('Y-m-d 23:59:59', strtotime("-$maxAge days"))
         ]);

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'];

Spacing +1 added lines, -1 removed lines

@@ -38,6 +38,6 @@
     unset($request['result']);
 }
 
-$tabs[] = [ 'label' => \Yii::t('audit', 'Info'), 'content' => $this->render('info_table', ['request' => $request]), 'active' => true ];
+$tabs[] = ['label' => \Yii::t('audit', 'Info'), 'content' => $this->render('info_table', ['request' => $request]), 'active' => true];
 
 echo Html::tag('h2', \Yii::t('audit', 'Request #{id}', ['id' => $index])), Tabs::widget(['items' => array_reverse($tabs)]);

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