Merge pull request #65 from rappasoft/analysis-64gnYM

Apply fixes from StyleCI

Author: rappasoft

src/LaravelLivewireTablesServiceProvider.php

@@ -5,9 +5,7 @@
 use Illuminate\Support\ServiceProvider;
 
 /**
- * Class LaravelLivewireTablesServiceProvider
- *
- * @package Rappasoft\LaravelLivewireTables
+ * Class LaravelLivewireTablesServiceProvider.
  */
 class LaravelLivewireTablesServiceProvider extends ServiceProvider
 {

src/TableComponent.php

@@ -19,7 +19,6 @@
  */
 abstract class TableComponent extends Component
 {
-
     use Loading,
         Pagination,
         Search,
@@ -29,7 +28,7 @@ abstract class TableComponent extends Component
         Yajra;
 
     /**
-     * The default pagination theme
+     * The default pagination theme.
      *
      * @var string
      */

src/Traits/CanBeHidden.php

@@ -47,6 +47,6 @@ public function isVisible(): bool
      */
     public function isHidden(): bool
     {
-        return !$this->isVisible();
+        return ! $this->isVisible();
     }
 }

src/Traits/HtmlComponents.php

@@ -2,17 +2,14 @@
 
 namespace Rappasoft\LaravelLivewireTables\Traits;
 
-use Illuminate\Support\HtmlString;
 use Illuminate\Contracts\Routing\UrlGenerator;
+use Illuminate\Support\HtmlString;
 
 /**
- * Trait HtmlBuilder
- *
- * @package Rappasoft\LaravelLivewireTables\Helpers
+ * Trait HtmlBuilder.
  */
 trait HtmlComponents
 {
-
     /**
      * Convert an HTML string to entities.
      *
@@ -39,7 +36,7 @@ public function image($url, $alt = null, $attributes = [], $secure = null): Html
     {
         $attributes['alt'] = $alt;
 
-        return $this->html('<img src="' . resolve(UrlGenerator::class)->asset($url, $secure) . '"' . $this->attributes($attributes) . '>');
+        return $this->html('<img src="'.resolve(UrlGenerator::class)->asset($url, $secure).'"'.$this->attributes($attributes).'>');
     }
 
     /**
@@ -65,7 +62,7 @@ public function link($url, $title = null, $attributes = [], $secure = null, $esc
             $title = $this->entities($title);
         }
 
-        return $this->html('<a href="' . $this->entities($url) . '"' . $this->attributes($attributes) . '>' . $title . '</a>');
+        return $this->html('<a href="'.$this->entities($url).'"'.$this->attributes($attributes).'>'.$title.'</a>');
     }
 
     /**
@@ -170,9 +167,9 @@ public function mailto($email, $title = null, $attributes = [], $escape = true):
             $title = $this->entities($title);
         }
 
-        $email = $this->obfuscate('mailto:') . $email;
+        $email = $this->obfuscate('mailto:').$email;
 
-        return $this->html('<a href="' . $email . '"' . $this->attributes($attributes) . '>' . $title . '</a>');
+        return $this->html('<a href="'.$email.'"'.$this->attributes($attributes).'>'.$title.'</a>');
     }
 
     /**
@@ -206,7 +203,7 @@ public function attributes($attributes): string
             }
         }
 
-        return count($html) > 0 ? ' ' . implode(' ', $html) : '';
+        return count($html) > 0 ? ' '.implode(' ', $html) : '';
     }
 
     /**
@@ -234,11 +231,11 @@ protected function attributeElement($key, $value): string
         }
 
         if (is_array($value) && $key === 'class') {
-            return 'class="' . implode(' ', $value) . '"';
+            return 'class="'.implode(' ', $value).'"';
         }
 
         if (! is_null($value)) {
-            return $key . '="' . e($value, false) . '"';
+            return $key.'="'.e($value, false).'"';
         }
     }
 
@@ -263,11 +260,11 @@ public function obfuscate($value): string
             // the randomly obfuscated letters out of the string on the responses.
             switch (rand(1, 3)) {
                 case 1:
-                    $safe .= '&#' . ord($letter) . ';';
+                    $safe .= '&#'.ord($letter).';';
                     break;
 
                 case 2:
-                    $safe .= '&#x' . dechex(ord($letter)) . ';';
+                    $safe .= '&#x'.dechex(ord($letter)).';';
                     break;
 
                 case 3:
@@ -279,7 +276,7 @@ public function obfuscate($value): string
     }
 
     /**
-     * Transform the string to an Html serializable object
+     * Transform the string to an Html serializable object.
      *
      * @param $html
      *

src/Traits/Loading.php

@@ -7,7 +7,6 @@
  */
 trait Loading
 {
-
     /**
      * Whether or not to show a loading indicator when searching.
      *
@@ -24,7 +23,7 @@ trait Loading
 
     /**
      * When the table is loading, hide all data but the loading row
-     * Otherwise the loading row gets shown above the data (makes the page less jumpy)
+     * Otherwise the loading row gets shown above the data (makes the page less jumpy).
      *
      * @var bool
      */

src/Traits/Table.php

@@ -7,7 +7,6 @@
  */
 trait Table
 {
-
     /**
      * Whether or not to display the table header.
      *

src/Traits/Yajra.php

@@ -85,7 +85,7 @@ public function attribute(Builder $query, $relationships, $attribute): string
     /**
      * @param $attribute
      *
-     * @return mixed|boolean
+     * @return mixed|bool
      */
     protected function getColumnByAttribute($attribute)
     {

src/Views/Column.php

@@ -10,7 +10,6 @@
  */
 class Column
 {
-
     use CanBeHidden;
 
     /**
@@ -145,7 +144,8 @@ public function isRaw(): bool
      *
      * @return $this
      */
-    public function format(callable $callable): Column {
+    public function format(callable $callable): Column
+    {
         $this->formatCallback = $callable;
 
         return $this;

tests/ExampleTest.php

@@ -7,7 +7,6 @@
 
 class ExampleTest extends TestCase
 {
-
     protected function getPackageProviders($app)
     {
         return [LaravelLivewireTablesServiceProvider::class];