Merge pull request #137 from yajra/analysis-64rk9P

Apply fixes from StyleCI

Author: yajra

src/Contracts/DataTableHtmlBuilder.php

@@ -8,6 +8,7 @@ interface DataTableHtmlBuilder
      * Handle building of dataTables html.
      *
      * @return \Yajra\DataTables\Html\Builder
+     *
      * @throws \Exception
      */
     public function handle();

src/Contracts/DataTableScope.php

@@ -7,7 +7,7 @@ interface DataTableScope
     /**
      * Apply a query scope.
      *
-     * @param \Illuminate\Database\Query\Builder|\Illuminate\Database\Eloquent\Builder $query
+     * @param  \Illuminate\Database\Query\Builder|\Illuminate\Database\Eloquent\Builder  $query
      * @return mixed
      */
     public function apply($query);

src/Exports/DataTablesCollectionExport.php

@@ -17,7 +17,7 @@ abstract class DataTablesCollectionExport implements FromCollection, WithHeading
     protected $collection;
 
     /**
-     * @param Collection $collection
+     * @param  Collection  $collection
      */
     public function __construct(Collection $collection)
     {

src/Generators/DataTablesHtmlCommand.php

@@ -36,7 +36,7 @@ class DataTablesHtmlCommand extends DataTablesMakeCommand
     /**
      * Build the class with the given name.
      *
-     * @param string $name
+     * @param  string  $name
      * @return string
      */
     protected function buildClass($name)
@@ -71,7 +71,7 @@ protected function getStub()
     /**
      * Parse the name and format according to the root namespace.
      *
-     * @param string $name
+     * @param  string  $name
      * @return string
      */
     protected function qualifyClass($name)

src/Generators/DataTablesMakeCommand.php

@@ -65,7 +65,7 @@ public function handle()
     /**
      * Build the class with the given name.
      *
-     * @param string $name
+     * @param  string  $name
      * @return string
      */
     protected function buildClass($name)
@@ -88,7 +88,7 @@ protected function buildClass($name)
     /**
      * Replace the filename.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return $this
      */
     protected function replaceFilename(&$stub)
@@ -103,7 +103,7 @@ protected function replaceFilename(&$stub)
     /**
      * Replace the action.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return \Yajra\DataTables\Generators\DataTablesMakeCommand
      */
     protected function replaceAction(&$stub)
@@ -128,7 +128,7 @@ protected function getAction()
     /**
      * Replace columns.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return $this
      */
     protected function replaceTableId(&$stub)
@@ -143,7 +143,7 @@ protected function replaceTableId(&$stub)
     /**
      * Replace dom.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return $this
      */
     protected function replaceDOM(&$stub)
@@ -160,7 +160,7 @@ protected function replaceDOM(&$stub)
     /**
      * Replace buttons.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return $this
      */
     protected function replaceButtons(&$stub)
@@ -194,8 +194,8 @@ protected function getButtons()
     /**
      * Parse array from definition.
      *
-     * @param string $definition
-     * @param int $indentation
+     * @param  string  $definition
+     * @param  int  $indentation
      * @return string
      */
     protected function parseButtons($definition, $indentation = 24)
@@ -223,7 +223,7 @@ protected function parseButtons($definition, $indentation = 24)
     /**
      * Replace columns.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return $this
      */
     protected function replaceColumns(&$stub)
@@ -261,8 +261,8 @@ protected function getColumns()
     /**
      * Parse array from definition.
      *
-     * @param string $definition
-     * @param int $indentation
+     * @param  string  $definition
+     * @param  int  $indentation
      * @return string
      */
     protected function parseColumns($definition, $indentation = 12)
@@ -283,7 +283,7 @@ protected function parseColumns($definition, $indentation = 12)
     /**
      * Replace builder name.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return \Yajra\DataTables\Generators\DataTablesMakeCommand
      */
     protected function replaceBuilder(&$stub)
@@ -299,7 +299,7 @@ protected function replaceBuilder(&$stub)
     /**
      * Parse the name and format according to the root namespace.
      *
-     * @param string $name
+     * @param  string  $name
      * @return string
      */
     protected function qualifyClass($name)
@@ -324,7 +324,7 @@ protected function qualifyClass($name)
     /**
      * Get the default namespace for the class.
      *
-     * @param string $rootNamespace
+     * @param  string  $rootNamespace
      * @return string
      */
     protected function getDefaultNamespace($rootNamespace)
@@ -335,7 +335,7 @@ protected function getDefaultNamespace($rootNamespace)
     /**
      * Replace model name.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return \Yajra\DataTables\Generators\DataTablesMakeCommand
      */
     protected function replaceModel(&$stub)
@@ -374,7 +374,7 @@ protected function getModel()
     /**
      * Replace model import.
      *
-     * @param string $stub
+     * @param  string  $stub
      * @return $this
      */
     protected function replaceModelImport(&$stub)

src/Generators/DataTablesScopeCommand.php

@@ -30,7 +30,7 @@ class DataTablesScopeCommand extends GeneratorCommand
     /**
      * Get the default namespace for the class.
      *
-     * @param  string $rootNamespace
+     * @param  string  $rootNamespace
      * @return string
      */
     protected function getDefaultNamespace($rootNamespace)

src/Html/DataTableHtml.php

@@ -28,9 +28,10 @@ public static function make()
     }
 
     /**
-     * @param string $name
-     * @param mixed $arguments
+     * @param  string  $name
+     * @param  mixed  $arguments
      * @return mixed
+     *
      * @throws \Exception
      */
     public function __call($name, $arguments)
@@ -55,7 +56,7 @@ protected function getHtmlBuilder()
     }
 
     /**
-     * @param mixed $builder
+     * @param  mixed  $builder
      * @return static
      */
     public function setHtmlBuilder($builder)

src/Services/DataTable.php

@@ -166,9 +166,9 @@ abstract class DataTable implements DataTableButtons
     /**
      * Process dataTables needed render output.
      *
-     * @param string $view
-     * @param array $data
-     * @param array $mergeData
+     * @param  string  $view
+     * @param  array  $data
+     * @param  array  $mergeData
      * @return mixed
      */
     public function render($view, $data = [], $mergeData = [])
@@ -322,8 +322,8 @@ public function builder()
     /**
      * Map ajax response to columns definition.
      *
-     * @param mixed $columns
-     * @param string $type
+     * @param  mixed  $columns
+     * @param  string  $type
      * @return array
      */
     protected function mapResponseToColumns($columns, $type)
@@ -369,7 +369,7 @@ protected function getHtmlBuilder()
     /**
      * Add html builder callback hook.
      *
-     * @param callable $callback
+     * @param  callable  $callback
      * @return $this
      */
     public function withHtml(callable $callback)
@@ -382,7 +382,7 @@ public function withHtml(callable $callback)
     /**
      * Add callback before sending the response.
      *
-     * @param callable $callback
+     * @param  callable  $callback
      * @return $this
      */
     public function before(callable $callback)
@@ -395,7 +395,7 @@ public function before(callable $callback)
     /**
      * Add callback after the response was processed.
      *
-     * @param callable $callback
+     * @param  callable  $callback
      * @return $this
      */
     public function response(callable $callback)
@@ -457,7 +457,7 @@ public function getFilename()
     /**
      * Set export filename.
      *
-     * @param string $filename
+     * @param  string  $filename
      * @return DataTable
      */
     public function setFilename($filename)
@@ -502,7 +502,7 @@ protected function exportColumns()
     /**
      * Convert array to collection of Column class.
      *
-     * @param array $columns
+     * @param  array  $columns
      * @return Collection
      */
     private function toColumnsCollection(array $columns)
@@ -573,7 +573,7 @@ public function snappyPdf()
     /**
      * Add basic array query scopes.
      *
-     * @param \Yajra\DataTables\Contracts\DataTableScope $scope
+     * @param  \Yajra\DataTables\Contracts\DataTableScope  $scope
      * @return $this
      */
     public function addScope(DataTableScope $scope)
@@ -586,7 +586,7 @@ public function addScope(DataTableScope $scope)
     /**
      * Push multiples scopes to array query scopes.
      *
-     * @param array $scopes
+     * @param  array  $scopes
      * @return $this
      */
     public function addScopes(array $scopes)
@@ -599,8 +599,8 @@ public function addScopes(array $scopes)
     /**
      * Set a custom class attribute.
      *
-     * @param mixed $key
-     * @param mixed|null $value
+     * @param  mixed  $key
+     * @param  mixed|null  $value
      * @return $this
      */
     public function with($key, $value = null)
@@ -617,7 +617,7 @@ public function with($key, $value = null)
     /**
      * Dynamically retrieve the value of an attribute.
      *
-     * @param string $key
+     * @param  string  $key
      * @return mixed|null
      */
     public function __get($key)
@@ -630,7 +630,7 @@ public function __get($key)
     /**
      * Apply query scopes.
      *
-     * @param \Illuminate\Database\Query\Builder|\Illuminate\Database\Eloquent\Builder $query
+     * @param  \Illuminate\Database\Query\Builder|\Illuminate\Database\Eloquent\Builder  $query
      * @return mixed
      */
     protected function applyScopes($query)
@@ -645,8 +645,8 @@ protected function applyScopes($query)
     /**
      * Determine if the DataTable has scopes.
      *
-     * @param  array $scopes
-     * @param  bool $validateAll
+     * @param  array  $scopes
+     * @param  bool  $validateAll
      * @return bool
      */
     protected function hasScopes(array $scopes, $validateAll = false)

src/Transformers/DataArrayTransformer.php

@@ -10,9 +10,9 @@ class DataArrayTransformer
     /**
      * Transform row data by columns definition.
      *
-     * @param array  $row
-     * @param mixed  $columns
-     * @param string $type
+     * @param  array  $row
+     * @param  mixed  $columns
+     * @param  string  $type
      * @return array
      */
     public function transform(array $row, $columns, $type = 'printable')
@@ -27,9 +27,9 @@ public function transform(array $row, $columns, $type = 'printable')
     /**
      * Transform row column by collection.
      *
-     * @param array                          $row
-     * @param \Illuminate\Support\Collection $columns
-     * @param string                         $type
+     * @param  array  $row
+     * @param  \Illuminate\Support\Collection  $columns
+     * @param  string  $type
      * @return array
      */
     protected function buildColumnByCollection(array $row, Collection $columns, $type = 'printable')
@@ -56,7 +56,7 @@ protected function buildColumnByCollection(array $row, Collection $columns, $typ
     /**
      * Decode content to a readable text value.
      *
-     * @param string $data
+     * @param  string  $data
      * @return string
      */
     protected function decodeContent($data)