LaravelRUS / SleepingOwlAdmin

src/Display/Column/NamedColumn.php

Doc Comments +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
     /**
      * Get column value from instance.
      *
-     * @param Collection|Model $instance
+     * @param Model $instance
      * @param string           $name
      *
      * @return mixed

Spacing +2 added lines, -2 removed lines

@@ -87,11 +87,11 @@
 
         if ($instance instanceof Collection) {
             $instance = $instance->pluck($part);
-        } elseif (! is_null($instance)) {
+        } elseif (!is_null($instance)) {
             $instance = $instance->getAttribute($part);
         }
 
-        if (! empty($parts) && ! is_null($instance)) {
+        if (!empty($parts) && !is_null($instance)) {
             return $this->getValueFromObject($instance, implode('.', $parts));
         }
 

src/Display/Column/Order.php

Doc Comments +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
 
     /**
      * Get instance move up url.
-     * @return Route
+     * @return string
      */
     protected function moveUpUrl()
     {
@@ -105,7 +105,7 @@ 
 
     /**
      * Get instance move down url.
-     * @return Route
+     * @return string
      */
     protected function moveDownUrl()
     {

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     {
         Route::post('{adminModel}/{adminModelId}/up', [
             'as' => 'admin.model.move-up',
-            function ($model, $id) {
+            function($model, $id) {
                 $instance = $model->getRepository()->find($id);
                 $instance->moveUp();
 
@@ -25,7 +25,7 @@ 
 
         Route::post('{adminModel}/{adminModelId}/down', [
             'as' => 'admin.model.move-down',
-            function ($model, $id) {
+            function($model, $id) {
                 $instance = $model->getRepository()->find($id);
                 $instance->moveDown();
 

src/Display/DisplayDatatablesAsync.php

Doc Comments +4 added lines, -4 removed lines

@@ -122,7 +122,7 @@ 
     }
 
     /**
-     * @param mixed $distinct
+     * @param string|null $distinct
      *
      * @return $this
      */
@@ -165,7 +165,7 @@ 
     /**
      * Apply offset and limit to the query.
      *
-     * @param $query
+     * @param Builder $query
      */
     protected function applyOffset($query)
     {
@@ -182,7 +182,7 @@ 
     /**
      * Apply orders to the query.
      *
-     * @param $query
+     * @param Builder $query
      */
     protected function applyOrders($query)
     {
@@ -247,7 +247,7 @@ 
     /**
      * Convert collection to the datatables structure.
      *
-     * @param array|Collection $collection
+     * @param Collection $collection
      * @param int $totalCount
      * @param int $filteredCount
      *

Spacing +17 added lines, -17 removed lines

@@ -18,8 +18,8 @@ 
      */
     public static function registerRoutes()
     {
-        Route::get('{adminModel}/async/{adminDisplayName?}', ['as' => 'admin.model.async',
-            function (ModelConfiguration $model, $name = null) {
+        Route::get('{adminModel}/async/{adminDisplayName?}', [ 'as' => 'admin.model.async',
+            function(ModelConfiguration $model, $name = null) {
                 $display = $model->fireDisplay();
                 if ($display instanceof DisplayTabbed) {
                     $display = static::findDatatablesAsyncByName($display, $name);
@@ -135,7 +135,7 @@ 
         $totalCount = $query->count();
         $filteredCount = 0;
 
-        if (! is_null($this->distinct)) {
+        if (!is_null($this->distinct)) {
             $filteredCount = $query->distinct()->count($this->getDistinct());
         }
 
@@ -178,11 +178,11 @@ 
      */
     protected function applyOrders($query)
     {
-        $orders = Request::input('order', []);
+        $orders = Request::input('order', [ ]);
 
         foreach ($orders as $order) {
-            $columnIndex = $order['column'];
-            $orderDirection = $order['dir'];
+            $columnIndex = $order[ 'column' ];
+            $orderDirection = $order[ 'dir' ];
             $column = $this->getColumns()->all()->get($columnIndex);
 
             if ($column instanceof NamedColumn && $column->isOrderable()) {
@@ -204,7 +204,7 @@ 
             return;
         }
 
-        $query->where(function ($query) use ($search) {
+        $query->where(function($query) use ($search) {
             $columns = $this->getColumns()->all();
             foreach ($columns as $column) {
                 if ($column instanceof Text) {
@@ -222,7 +222,7 @@ 
      */
     protected function applyColumnSearch(Builder $query)
     {
-        $queryColumns = Request::input('columns', []);
+        $queryColumns = Request::input('columns', [ ]);
 
         foreach ($queryColumns as $index => $queryColumn) {
             $search = array_get($queryColumn, 'search.value');
@@ -230,7 +230,7 @@ 
             $column = $this->getColumns()->all()->get($index);
             $columnFilter = array_get($this->getColumnFilters()->all(), $index);
 
-            if (! is_null($columnFilter) && ! is_null($column)) {
+            if (!is_null($columnFilter) && !is_null($column)) {
                 $columnFilter->apply($this->repository, $column, $query, $search, $fullSearch);
             }
         }
@@ -249,21 +249,21 @@ 
     {
         $columns = $this->getColumns();
 
-        $result = [];
-        $result['draw'] = Request::input('draw', 0);
-        $result['recordsTotal'] = $totalCount;
-        $result['recordsFiltered'] = $filteredCount;
-        $result['data'] = [];
+        $result = [ ];
+        $result[ 'draw' ] = Request::input('draw', 0);
+        $result[ 'recordsTotal' ] = $totalCount;
+        $result[ 'recordsFiltered' ] = $filteredCount;
+        $result[ 'data' ] = [ ];
 
         foreach ($collection as $instance) {
-            $_row = [];
+            $_row = [ ];
 
             foreach ($columns->all() as $column) {
                 $column->setModel($instance);
-                $_row[] = (string) $column;
+                $_row[ ] = (string) $column;
             }
 
-            $result['data'][] = $_row;
+            $result[ 'data' ][ ] = $_row;
         }
 
         return $result;

src/Display/Extension/ColumnFilters.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     }
 
     /**
-     * @return Collection|\SleepingOwl\Admin\Contracts\ActionInterface[]
+     * @return ColumnFilterInterface[]
      */
     public function all()
     {

Spacing +6 added lines, -6 removed lines

@@ -14,7 +14,7 @@ 
     /**
      * @var ColumnFilterInterface[]
      */
-    protected $columnFilters = [];
+    protected $columnFilters = [ ];
 
     /**
      * @var string
@@ -33,11 +33,11 @@ 
      */
     public function set($columnFilters)
     {
-        if (! is_array($columnFilters)) {
+        if (!is_array($columnFilters)) {
             $columnFilters = func_get_args();
         }
 
-        $this->columnFilters = [];
+        $this->columnFilters = [ ];
 
         foreach ($columnFilters as $filter) {
             $this->push($filter);
@@ -61,7 +61,7 @@ 
      */
     public function push(ColumnFilterInterface $filter = null)
     {
-        $this->columnFilters[] = $filter;
+        $this->columnFilters[ ] = $filter;
 
         return $this;
     }
@@ -134,13 +134,13 @@ 
             }
         }
 
-        if (! $this->hasHtmlAttribute('class')) {
+        if (!$this->hasHtmlAttribute('class')) {
             $this->setHtmlAttribute('class', 'panel-footer');
         }
 
         $template = app('sleeping_owl.template')->getViewPath($this->getDisplay()->getView());
 
-        view()->composer($template, function (\Illuminate\View\View $view) {
+        view()->composer($template, function(\Illuminate\View\View $view) {
             $view->getFactory()->inject(
                 $this->getPosition(),
                 app('sleeping_owl.template')->view($this->getView(), $this->toArray())

src/Display/Extension/Scopes.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     /**
      * @param array|string $scopes
      *
-     * @return $this
+     * @return \SleepingOwl\Admin\Contracts\DisplayInterface
      */
     public function set($scopes)
     {

Spacing +5 added lines, -5 removed lines

@@ -9,7 +9,7 @@ 
     /**
      * @var string[]
      */
-    protected $scopes = [];
+    protected $scopes = [ ];
 
     /**
      * @return string[]
@@ -38,11 +38,11 @@ 
      */
     public function push($scope)
     {
-        if (! is_array($scope)) {
+        if (!is_array($scope)) {
             $scope = func_get_args();
         }
 
-        $this->scopes[] = $scope;
+        $this->scopes[ ] = $scope;
 
         return $this;
     }
@@ -65,13 +65,13 @@ 
     public function modifyQuery(Builder $query)
     {
         foreach ($this->scopes as $scope) {
-            if (! is_null($scope)) {
+            if (!is_null($scope)) {
                 if (is_array($scope)) {
                     $method = array_shift($scope);
                     $params = $scope;
                 } else {
                     $method = $scope;
-                    $params = [];
+                    $params = [ ];
                 }
 
                 call_user_func_array([

src/Form/Element/Columns.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     }
 
     /**
-     * @return array
+     * @return Collection
      */
     public function getColumns()
     {

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
     {
         parent::initialize();
 
-        $this->applyCallbackToItems(function ($item) {
+        $this->applyCallbackToItems(function($item) {
             if ($item instanceof Initializable) {
                 $item->initialize();
             }
@@ -65,7 +65,7 @@ 
     {
         parent::setModel($model);
 
-        $this->applyCallbackToItems(function ($item) use ($model) {
+        $this->applyCallbackToItems(function($item) use ($model) {
             if ($item instanceof FormElementInterface) {
                 $item->setModel($model);
             }
@@ -111,7 +111,7 @@ 
      */
     public function getValidationMessages()
     {
-        $messages = [];
+        $messages = [ ];
 
         foreach ($this->getColumns() as $columnItems) {
             foreach ($columnItems as $item) {
@@ -129,7 +129,7 @@ 
      */
     public function getValidationLabels()
     {
-        $labels = [];
+        $labels = [ ];
 
         foreach ($this->getColumns() as $columnItems) {
             foreach ($columnItems as $item) {
@@ -146,7 +146,7 @@ 
     {
         parent::save();
 
-        $this->applyCallbackToItems(function ($item) {
+        $this->applyCallbackToItems(function($item) {
             if ($item instanceof FormElementInterface) {
                 $item->save();
             }
@@ -157,7 +157,7 @@ 
     {
         parent::afterSave();
 
-        $this->applyCallbackToItems(function ($item) {
+        $this->applyCallbackToItems(function($item) {
             if ($item instanceof FormElementInterface) {
                 $item->afterSave();
             }

src/Repository/BaseRepository.php

Doc Comments +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
     }
 
     /**
-     * @return \string[]
+     * @return string[]
      */
     public function getWith()
     {
@@ -81,7 +81,7 @@ 
 
     /**
      * Get base query.
-     * @return mixed
+     * @return \Illuminate\Database\Eloquent\Builder
      */
     public function getQuery()
     {

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
      * Eager loading relations.
      * @var string[]
      */
-    protected $with = [];
+    protected $with = [ ];
 
     /**
      * @param string $class
@@ -72,7 +72,7 @@ 
      */
     public function with($with)
     {
-        if (! is_array($with)) {
+        if (!is_array($with)) {
             $with = func_get_args();
         }
 
@@ -178,7 +178,7 @@ 
     public function hasColumn($column)
     {
         $table = $this->getModel()->getTable();
-        $columns = Cache::remember('admin.columns.'.$table, 60, function () use ($table) {
+        $columns = Cache::remember('admin.columns.'.$table, 60, function() use ($table) {
             return Schema::getColumnListing($table);
         });
 

src/Repository/TreeRepository.php

Doc Comments +8 added lines, -8 removed lines

@@ -202,7 +202,7 @@ 
      *
      * @param $id
      * @param $parentId
-     * @param $left
+     * @param integer $left
      * @param $right
      */
     public function move($id, $parentId, $left, $right)
@@ -291,7 +291,7 @@ 
      * Call several methods and get first result.
      *
      * @param $instance
-     * @param $methods
+     * @param string[] $methods
      *
      * @return mixed
      * @throws Exception
@@ -311,9 +311,9 @@ 
      *
      * @param $root
      * @param $parentId
-     * @param $left
+     * @param integer $left
      *
-     * @return mixed
+     * @return integer
      */
     protected function recursiveReorder($root, $parentId, $left)
     {
@@ -332,7 +332,7 @@ 
      * Recursive reoder simple tree type.
      *
      * @param $data
-     * @param $parentId
+     * @param string|null $parentId
      */
     protected function recursiveReorderSimple($data, $parentId)
     {
@@ -354,9 +354,9 @@ 
      * Get children for simple tree type structure.
      *
      * @param $collection
-     * @param $id
+     * @param string|null $id
      *
-     * @return static
+     * @return Collection
      */
     protected function getChildren($collection, $id)
     {
@@ -376,7 +376,7 @@ 
 
     /**
      * Create simple tree type structure.
-     * @return static
+     * @return Collection
      */
     protected function createSimpleTree()
     {

Spacing +11 added lines, -11 removed lines

@@ -209,9 +209,9 @@ 
     {
         $instance = $this->find($id);
         $attributes = $instance->getAttributes();
-        $attributes[$this->getLeftColumn($instance)] = $left;
-        $attributes[$this->getRightColumn($instance)] = $right;
-        $attributes[$this->getParentColumn($instance)] = $parentId;
+        $attributes[ $this->getLeftColumn($instance) ] = $left;
+        $attributes[ $this->getRightColumn($instance) ] = $right;
+        $attributes[ $this->getParentColumn($instance) ] = $parentId;
         $instance->setRawAttributes($attributes);
         $instance->save();
     }
@@ -318,11 +318,11 @@ 
     protected function recursiveReorder($root, $parentId, $left)
     {
         $right = $left + 1;
-        $children = array_get($root, 'children', []);
+        $children = array_get($root, 'children', [ ]);
         foreach ($children as $child) {
-            $right = $this->recursiveReorder($child, $root['id'], $right);
+            $right = $this->recursiveReorder($child, $root[ 'id' ], $right);
         }
-        $this->move($root['id'], $parentId, $left, $right);
+        $this->move($root[ 'id' ], $parentId, $left, $right);
         $left = $right + 1;
 
         return $left;
@@ -337,15 +337,15 @@ 
     protected function recursiveReorderSimple($data, $parentId)
     {
         foreach ($data as $order => $item) {
-            $id = $item['id'];
+            $id = $item[ 'id' ];
 
             $instance = $this->find($id);
             $instance->{$this->getParentField()} = $parentId;
             $instance->{$this->getOrderField()} = $order;
             $instance->save();
 
-            if (isset($item['children'])) {
-                $this->recursiveReorderSimple($item['children'], $id);
+            if (isset($item[ 'children' ])) {
+                $this->recursiveReorderSimple($item[ 'children' ], $id);
             }
         }
     }
@@ -361,14 +361,14 @@ 
     protected function getChildren($collection, $id)
     {
         $parentField = $this->getParentField();
-        $result = [];
+        $result = [ ];
         foreach ($collection as $instance) {
             if ($instance->$parentField != $id) {
                 continue;
             }
 
             $instance->setRelation('children', $this->getChildren($collection, $instance->getKey()));
-            $result[] = $instance;
+            $result[ ] = $instance;
         }
 
         return Collection::make($result);

src/Traits/OrderableModel.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
     /**
      * Move model in the $destination.
      *
-     * @param $destination -1 (move down) or 1 (move up)
+     * @param integer $destination -1 (move down) or 1 (move up)
      */
     protected function move($destination)
     {

Spacing +2 added lines, -2 removed lines

@@ -9,11 +9,11 @@
      */
     protected static function bootOrderableModel()
     {
-        static::creating(function ($row) {
+        static::creating(function($row) {
             $row->updateOrderFieldOnCreate();
         });
 
-        static::deleted(function ($row) {
+        static::deleted(function($row) {
             $row->updateOrderFieldOnDelete();
         });
     }