LaravelRUS / SleepingOwlAdmin

src/Repository/TreeRepository.php

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/Display/Column/Filter/Date.php

Doc Comments +1 added lines

@@ -209,6 +209,7 @@
     }
 
     /**
+     * @param string $format
      * @return string
      */
     protected function generatePickerFormat($format)

Spacing +1 added lines, -1 removed lines

@@ -182,7 +182,7 @@
             $fieldName = array_pop($parts);
             $relationName = implode('.', $parts);
 
-            $query->whereHas($relationName, function ($q) use ($name, $date) {
+            $query->whereHas($relationName, function($q) use ($name, $date) {
                 $this->buildQuery($q, $name, $date);
             });
         }

src/Display/DisplayDatatablesAsync.php

Doc Comments +3 added lines, -3 removed lines

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

Spacing +15 added lines, -15 removed lines

@@ -26,7 +26,7 @@ 
     {
         $router->get('{adminModel}/async/{adminDisplayName?}', [
             'as' => 'admin.model.async',
-            function (ModelConfigurationInterface $model, $name = null) {
+            function(ModelConfigurationInterface $model, $name = null) {
                 $display = $model->fireDisplay();
                 if ($display instanceof DisplayTabbed) {
                     $display = static::findDatatablesAsyncByName($display, $name);
@@ -41,7 +41,7 @@ 
         ]);
         $router->post('{adminModel}/async/{adminDisplayName?}', [
             'as' => 'admin.model.async.inline',
-            function (ModelConfigurationInterface $model, InlineRequest $request) {
+            function(ModelConfigurationInterface $model, InlineRequest $request) {
                 $field = $request->input('name');
                 $value = $request->input('value');
                 $id = $request->input('pk');
@@ -49,7 +49,7 @@ 
                 $display = $model->fireDisplay();
 
                 /** @var ColumnEditableInterface|null $column */
-                $column = $display->getColumns()->all()->filter(function ($column) use ($field) {
+                $column = $display->getColumns()->all()->filter(function($column) use ($field) {
                     return ($column instanceof ColumnEditableInterface) and $field == $column->getName();
                 })->first();
 
@@ -184,7 +184,7 @@ 
         $totalCount = $query->count();
         $filteredCount = 0;
 
-        if (! is_null($this->distinct)) {
+        if (!is_null($this->distinct)) {
             $filteredCount = $query->distinct()->count($this->getDistinct());
         }
 
@@ -232,7 +232,7 @@ 
             return;
         }
 
-        $query->where(function ($query) use ($search) {
+        $query->where(function($query) use ($search) {
             $columns = $this->getColumns()->all();
             foreach ($columns as $column) {
                 if (in_array(get_class($column), $this->searchableColumns)) {
@@ -250,7 +250,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');
@@ -258,7 +258,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);
             }
         }
@@ -277,14 +277,14 @@ 
     {
         $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);
@@ -293,10 +293,10 @@ 
                     $column->initialize();
                 }
 
-                $_row[] = (string) $column;
+                $_row[ ] = (string) $column;
             }
 
-            $result['data'][] = $_row;
+            $result[ 'data' ][ ] = $_row;
         }
 
         return $result;

src/Display/TableColumn.php

Doc Comments +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
     }
 
     /**
-     * @return int
+     * @return string
      */
     public function getWidth()
     {

Spacing +4 added lines, -4 removed lines

@@ -63,7 +63,7 @@ 
     {
         $this->header = app(TableHeaderColumnInterface::class);
 
-        if (! is_null($label)) {
+        if (!is_null($label)) {
             $this->setLabel($label);
         }
 
@@ -169,7 +169,7 @@ 
         $this->model = $model;
         $append = $this->getAppends();
 
-        if (! is_null($append)) {
+        if (!is_null($append)) {
             $append->setModel($model);
         }
 
@@ -210,7 +210,7 @@ 
             $orderable = new OrderByClause($orderable);
         }
 
-        if ($orderable !== false && ! $orderable instanceof OrderByClauseInterface) {
+        if ($orderable !== false && !$orderable instanceof OrderByClauseInterface) {
             throw new \InvalidArgumentException('Argument must be instance of SleepingOwl\Admin\Contracts\Display\OrderByClauseInterface interface');
         }
 
@@ -235,7 +235,7 @@ 
      */
     public function orderBy(Builder $query, $condition)
     {
-        if (! $this->isOrderable()) {
+        if (!$this->isOrderable()) {
             throw new \InvalidArgumentException('Argument must be instance of SleepingOwl\Admin\Contracts\Display\OrderByClauseInterface interface');
         }
 

src/Model/ModelConfiguration.php

Doc Comments +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
     }
 
     /**
-     * @return string
+     * @return \Illuminate\Support\Collection
      */
     public function getRedirect()
     {

Spacing +11 added lines, -11 removed lines

@@ -69,7 +69,7 @@ 
     /**
      * @var array
      */
-    protected $redirect = ['edit' => 'edit', 'create' => 'edit'];
+    protected $redirect = [ 'edit' => 'edit', 'create' => 'edit' ];
 
     /**
      * @var Closure|null
@@ -368,7 +368,7 @@ 
      */
     public function isCreatable()
     {
-        if (! is_callable($this->getCreate())) {
+        if (!is_callable($this->getCreate())) {
             return false;
         }
 
@@ -392,7 +392,7 @@ 
      */
     public function isEditable(Model $model)
     {
-        if (! is_callable($this->getEdit())) {
+        if (!is_callable($this->getEdit())) {
             return false;
         }
 
@@ -502,7 +502,7 @@ 
      */
     public function fireDisplay()
     {
-        if (! is_callable($this->display)) {
+        if (!is_callable($this->display)) {
             return;
         }
 
@@ -520,7 +520,7 @@ 
      */
     public function fireCreate()
     {
-        if (! is_callable($this->create)) {
+        if (!is_callable($this->create)) {
             return;
         }
 
@@ -547,11 +547,11 @@ 
      */
     public function fireEdit($id)
     {
-        if (! is_callable($this->edit)) {
+        if (!is_callable($this->edit)) {
             return;
         }
 
-        $form = app()->call($this->edit, ['id' => $id]);
+        $form = app()->call($this->edit, [ 'id' => $id ]);
         if ($form instanceof DisplayInterface) {
             $form->setModelClass($this->getClass());
         }
@@ -576,7 +576,7 @@ 
     public function fireDelete($id)
     {
         if (is_callable($this->getDelete())) {
-            return app()->call($this->getDelete(), [$id]);
+            return app()->call($this->getDelete(), [ $id ]);
         }
     }
 
@@ -588,7 +588,7 @@ 
     public function fireDestroy($id)
     {
         if (is_callable($this->getDestroy())) {
-            return app()->call($this->getDestroy(), [$id]);
+            return app()->call($this->getDestroy(), [ $id ]);
         }
     }
 
@@ -600,7 +600,7 @@ 
     public function fireRestore($id)
     {
         if (is_callable($this->getRestore())) {
-            return app()->call($this->getRestore(), [$id]);
+            return app()->call($this->getRestore(), [ $id ]);
         }
 
         return $this->getRestore();
@@ -744,7 +744,7 @@ 
         $repository = $this->getRepository();
         $item = $repository->find($id);
 
-        if (is_null($item) || ! $this->isEditable($item)) {
+        if (is_null($item) || !$this->isEditable($item)) {
             abort(404);
         }
 

src/Model/SectionModelConfiguration.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     }
 
     /**
-     * @return string
+     * @return \Illuminate\Support\Collection
      */
     public function getRedirect()
     {

Spacing +10 added lines, -10 removed lines

@@ -12,7 +12,7 @@ 
     /**
      * @var array
      */
-    protected $redirect = ['edit' => 'edit', 'create' => 'edit'];
+    protected $redirect = [ 'edit' => 'edit', 'create' => 'edit' ];
 
     /**
      * @param string $redirect
@@ -54,11 +54,11 @@ 
      */
     public function fireDisplay()
     {
-        if (! method_exists($this, 'onDisplay')) {
+        if (!method_exists($this, 'onDisplay')) {
             return;
         }
 
-        $display = app()->call([$this, 'onDisplay']);
+        $display = app()->call([ $this, 'onDisplay' ]);
 
         if ($display instanceof DisplayInterface) {
             $display->setModelClass($this->getClass());
@@ -73,11 +73,11 @@ 
      */
     public function fireCreate()
     {
-        if (! method_exists($this, 'onCreate')) {
+        if (!method_exists($this, 'onCreate')) {
             return;
         }
 
-        $form = app()->call([$this, 'onCreate']);
+        $form = app()->call([ $this, 'onCreate' ]);
         if ($form instanceof DisplayInterface) {
             $form->setModelClass($this->getClass());
         }
@@ -100,11 +100,11 @@ 
      */
     public function fireEdit($id)
     {
-        if (! method_exists($this, 'onEdit')) {
+        if (!method_exists($this, 'onEdit')) {
             return;
         }
 
-        $form = app()->call([$this, 'onEdit'], ['id' => $id]);
+        $form = app()->call([ $this, 'onEdit' ], [ 'id' => $id ]);
         if ($form instanceof DisplayInterface) {
             $form->setModelClass($this->getClass());
         }
@@ -129,7 +129,7 @@ 
     public function fireDelete($id)
     {
         if (method_exists($this, 'onDelete')) {
-            return app()->call([$this, 'onDelete'], ['id' => $id]);
+            return app()->call([ $this, 'onDelete' ], [ 'id' => $id ]);
         }
     }
 
@@ -141,7 +141,7 @@ 
     public function fireDestroy($id)
     {
         if (method_exists($this, 'onDestroy')) {
-            return app()->call([$this, 'onDestroy'], ['id' => $id]);
+            return app()->call([ $this, 'onDestroy' ], [ 'id' => $id ]);
         }
     }
 
@@ -153,7 +153,7 @@ 
     public function fireRestore($id)
     {
         if (method_exists($this, 'onRestore')) {
-            return app()->call([$this, 'onRestore'], ['id' => $id]);
+            return app()->call([ $this, 'onRestore' ], [ 'id' => $id ]);
         }
     }
 }

src/Widgets/WidgetsRegistry.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     }
 
     /**
-     * @param $widget
+     * @param string $widget
      *
      * @return \Closure
      */

Spacing +5 added lines, -5 removed lines

@@ -54,22 +54,22 @@
         }
 
         $groupedBlocks = $this->widgets
-            ->map(function ($class) {
+            ->map(function($class) {
                 return $this->makeWidget($class);
             })
-            ->filter(function (WidgetInterface $block) {
+            ->filter(function(WidgetInterface $block) {
                 return $block->active();
             })
-            ->groupBy(function (WidgetInterface $block) {
+            ->groupBy(function(WidgetInterface $block) {
                 return $block->template();
             });
 
         foreach ($groupedBlocks as $template => $widgets) {
-            $factory->composer($template, function (View $view) use ($widgets) {
+            $factory->composer($template, function(View $view) use ($widgets) {
                 $factory = $view->getFactory();
 
                 /** @var Collection|WidgetInterface[] $widgets */
-                $widgets = $widgets->sortBy(function (WidgetInterface $block) {
+                $widgets = $widgets->sortBy(function(WidgetInterface $block) {
                     return $block->position();
                 });
 

src/Form/Element/DateRange.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
      */
     public function getDefaultFrom()
     {
-        if (! $this->defaultFrom) {
+        if (!$this->defaultFrom) {
             $this->defaultFrom = Carbon::now();
         }
 
@@ -53,7 +53,7 @@ 
      */
     public function getDefaultTo()
     {
-        if (! $this->defaultTo) {
+        if (!$this->defaultTo) {
             $this->defaultTo = Carbon::now();
         }
 
@@ -81,7 +81,7 @@ 
      */
     public function setValue(Model $model, $attribute, $value)
     {
-        $value = ! empty($value) ? array_map(function ($date) {
+        $value = !empty($value) ? array_map(function($date) {
             return Carbon::createFromFormat($this->getPickerFormat(), $date);
         }, explode('::', $value)) : null;
 

src/Form/Element/DependentSelect.php

Spacing +13 added lines, -13 removed lines

@@ -18,17 +18,17 @@ 
     {
         $routeName = 'admin.form.element.dependent-select';
 
-        if (! $router->has($routeName)) {
-            $router->post('{adminModel}/dependent-select/{field}/{id?}', ['as' => $routeName, function (
+        if (!$router->has($routeName)) {
+            $router->post('{adminModel}/dependent-select/{field}/{id?}', [ 'as' => $routeName, function(
                 Request $request,
                 ModelConfigurationInterface $model,
                 $field,
                 $id = null
             ) {
 
-                if (! is_null($id)) {
+                if (!is_null($id)) {
                     $item = $model->getRepository()->find($id);
-                    if (is_null($item) || ! $model->isEditable($item)) {
+                    if (is_null($item) || !$model->isEditable($item)) {
                         return new JsonResponse([
                             'message' => trans('lang.message.access_denied'),
                         ], 403);
@@ -36,7 +36,7 @@ 
 
                     $form = $model->fireEdit($id);
                 } else {
-                    if (! $model->isCreatable()) {
+                    if (!$model->isCreatable()) {
                         return new JsonResponse([
                             'message' => trans('lang.message.access_denied'),
                         ], 403);
@@ -52,24 +52,24 @@ 
                 }
 
                 $element->setAjaxParameters(
-                    $request->input('depdrop_all_params', [])
+                    $request->input('depdrop_all_params', [ ])
                 );
 
                 $options = $element->getOptions();
 
                 if ($element->isNullable()) {
-                    $options = [null => trans('sleeping_owl::lang.select.nothing')] + $options;
+                    $options = [ null => trans('sleeping_owl::lang.select.nothing') ] + $options;
                 }
 
                 $options = array_except($options, $element->exclude);
 
                 return new JsonResponse([
                     'output' => collect($options)->map(function($value, $key) {
-                        return ['id' => $key, 'name' => $value];
+                        return [ 'id' => $key, 'name' => $value ];
                     }),
                     'selected' => $element->getValue()
                 ]);
-            }]);
+            } ]);
         }
     }
 
@@ -81,7 +81,7 @@ 
     /**
      * @var array
      */
-    protected $dataDepends = [];
+    protected $dataDepends = [ ];
 
     /**
      * @var array
@@ -95,9 +95,9 @@ 
      * @param null $label
      * @param array $depends
      */
-    public function __construct($path, $label = null, array $depends = [])
+    public function __construct($path, $label = null, array $depends = [ ])
     {
-        parent::__construct($path, $label, []);
+        parent::__construct($path, $label, [ ]);
 
         $this->setDataDepends($depends);
     }
@@ -183,7 +183,7 @@ 
         ];
 
         if ($this->isReadonly()) {
-            $attributes['disabled'] = 'disabled';
+            $attributes[ 'disabled' ] = 'disabled';
         }
 
         return [