LaravelRUS / SleepingOwlAdmin

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/Form/Element/Select.php

Doc Comments +1 added lines, -1 removed lines

@@ -308,7 +308,7 @@
     }
 
     /**
-     * @return null|string
+     * @return string
      */
     public function getForeignKey()
     {

Spacing +15 added lines, -15 removed lines

@@ -23,7 +23,7 @@ 
     /**
      * @var array
      */
-    protected $options = [];
+    protected $options = [ ];
 
     /**
      * @var bool
@@ -43,7 +43,7 @@ 
     /**
      * @var array
      */
-    protected $exclude = [];
+    protected $exclude = [ ];
 
     /**
      * @var string|null
@@ -53,7 +53,7 @@ 
     /**
      * @var array
      */
-    protected $fetchColumns = [];
+    protected $fetchColumns = [ ];
 
     /**
      * @var function|\Closure|object callable
@@ -65,7 +65,7 @@ 
      * @param string|null $label
      * @param array|Model $options
      */
-    public function __construct($path, $label = null, $options = [])
+    public function __construct($path, $label = null, $options = [ ])
     {
         parent::__construct($path, $label);
 
@@ -96,7 +96,7 @@ 
             $modelForOptions = app($modelForOptions);
         }
 
-        if (! ($modelForOptions instanceof Model)) {
+        if (!($modelForOptions instanceof Model)) {
             throw new SelectException('Class must be instanced of Illuminate\Database\Eloquent\Model');
         }
 
@@ -130,7 +130,7 @@ 
      */
     public function getOptions()
     {
-        if (! is_null($this->getModelForOptions()) && ! is_null($this->getDisplay())) {
+        if (!is_null($this->getModelForOptions()) && !is_null($this->getDisplay())) {
             $this->loadOptions();
         }
 
@@ -237,7 +237,7 @@ 
      */
     public function setFetchColumns($columns)
     {
-        if (! is_array($columns)) {
+        if (!is_array($columns)) {
             $columns = func_get_args();
         }
 
@@ -298,7 +298,7 @@ 
      */
     public function exclude($keys)
     {
-        if (! is_array($keys)) {
+        if (!is_array($keys)) {
             $keys = func_get_args();
         }
 
@@ -344,13 +344,13 @@ 
         ];
 
         if ($this->isNullable()) {
-            $attributes['data-nullable'] = 'true';
+            $attributes[ 'data-nullable' ] = 'true';
         }
 
         $options = $this->getOptions();
 
         if ($this->isNullable()) {
-            $options = [null => trans('sleeping_owl::lang.select.nothing')] + $options;
+            $options = [ null => trans('sleeping_owl::lang.select.nothing') ] + $options;
         }
 
         $options = array_except($options, $this->exclude);
@@ -367,7 +367,7 @@ 
      */
     protected function loadOptions()
     {
-        $repository = app(RepositoryInterface::class, [$this->getModelForOptions()]);
+        $repository = app(RepositoryInterface::class, [ $this->getModelForOptions() ]);
 
         $key = $repository->getModel()->getKeyName();
 
@@ -378,12 +378,12 @@ 
         }
 
         if (count($this->fetchColumns) > 0) {
-            $columns = array_merge([$key], $this->fetchColumns);
+            $columns = array_merge([ $key ], $this->fetchColumns);
             $options->select($columns);
         }
 
         // call the pre load options query preparer if has be set
-        if (! is_null($preparer = $this->getLoadOptionsQueryPreparer())) {
+        if (!is_null($preparer = $this->getLoadOptionsQueryPreparer())) {
             $options = $preparer($this, $options);
         }
 
@@ -400,9 +400,9 @@ 
 
             // iterate for all options and redefine it as
             // list of KEY and TEXT pair
-            $options = array_map(function ($opt) use ($key, $makeDisplay) {
+            $options = array_map(function($opt) use ($key, $makeDisplay) {
                 // get the KEY and make the display text
-                return [data_get($opt, $key), $makeDisplay($opt)];
+                return [ data_get($opt, $key), $makeDisplay($opt) ];
             }, $options);
 
             // take options as array with KEY => VALUE pair

src/Form/FormDefault.php

Doc Comments +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
      * @deprecated 4.5.0
      * @see getElements()
      *
-     * @return Collection[]
+     * @return Collection
      */
     public function getItems()
     {

Spacing +5 added lines, -5 removed lines

@@ -89,14 +89,14 @@ 
         }
 
         $this->initialized = true;
-        $this->repository = app(RepositoryInterface::class, [$this->class]);
+        $this->repository = app(RepositoryInterface::class, [ $this->class ]);
 
         $this->setModel(app($this->class));
 
         parent::initialize();
 
-        $this->getElements()->each(function ($element) {
-            if ($element instanceof Upload and ! $this->hasHtmlAttribute('enctype')) {
+        $this->getElements()->each(function($element) {
+            if ($element instanceof Upload and !$this->hasHtmlAttribute('enctype')) {
                 $this->setHtmlAttribute('enctype', 'multipart/form-data');
             }
         });
@@ -224,7 +224,7 @@ 
      */
     public function setItems($items)
     {
-        if (! is_array($items)) {
+        if (!is_array($items)) {
             $items = func_get_args();
         }
 
@@ -251,7 +251,7 @@ 
      */
     public function setId($id)
     {
-        if (is_null($this->id) and ! is_null($id) and ($model = $this->getRepository()->find($id))) {
+        if (is_null($this->id) and !is_null($id) and ($model = $this->getRepository()->find($id))) {
             $this->id = $id;
             $this->setModel($model);
         }

src/Http/Controllers/AdminController.php

Doc Comments +1 added lines, -1 removed lines

@@ -227,7 +227,7 @@
     /**
      * @param ModelConfigurationInterface $model
      *
-     * @return bool
+     * @return boolean|null
      */
     public function inlineEdit(ModelConfigurationInterface $model)
     {

Spacing +25 added lines, -25 removed lines

@@ -37,29 +37,29 @@ 
 
         $this->navigation->setCurrentUrl($request->url());
 
-        Breadcrumbs::register('home', function ($breadcrumbs) {
+        Breadcrumbs::register('home', function($breadcrumbs) {
             $breadcrumbs->push(trans('sleeping_owl::lang.dashboard'), route('admin.dashboard'));
         });
 
-        $breadcrumbs = [];
+        $breadcrumbs = [ ];
 
         if ($currentPage = $this->navigation->getCurrentPage()) {
             foreach ($currentPage->getPathArray() as $page) {
-                $breadcrumbs[] = [
-                    'id' => $page['id'],
-                    'title' => $page['title'],
-                    'url' => $page['url'],
+                $breadcrumbs[ ] = [
+                    'id' => $page[ 'id' ],
+                    'title' => $page[ 'title' ],
+                    'url' => $page[ 'url' ],
                     'parent' => $this->parentBreadcrumb,
                 ];
 
-                $this->parentBreadcrumb = $page['id'];
+                $this->parentBreadcrumb = $page[ 'id' ];
             }
         }
 
         foreach ($breadcrumbs as  $breadcrumb) {
-            Breadcrumbs::register($breadcrumb['id'], function ($breadcrumbs) use ($breadcrumb) {
-                $breadcrumbs->parent($breadcrumb['parent']);
-                $breadcrumbs->push($breadcrumb['title'], $breadcrumb['url']);
+            Breadcrumbs::register($breadcrumb[ 'id' ], function($breadcrumbs) use ($breadcrumb) {
+                $breadcrumbs->parent($breadcrumb[ 'parent' ]);
+                $breadcrumbs->push($breadcrumb[ 'title' ], $breadcrumb[ 'url' ]);
             });
         }
     }
@@ -71,7 +71,7 @@ 
      */
     public function getDisplay(ModelConfigurationInterface $model)
     {
-        if (! $model->isDisplayable()) {
+        if (!$model->isDisplayable()) {
             abort(404);
         }
 
@@ -85,7 +85,7 @@ 
      */
     public function getCreate(ModelConfigurationInterface $model)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -103,7 +103,7 @@ 
      */
     public function postStore(ModelConfigurationInterface $model)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -163,7 +163,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -182,7 +182,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -238,7 +238,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();
 
@@ -249,7 +249,7 @@ 
         $repository = $model->getRepository();
         $item = $repository->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -274,7 +274,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isDeletable($item)) {
+        if (is_null($item) || !$model->isDeletable($item)) {
             abort(404);
         }
 
@@ -299,13 +299,13 @@ 
      */
     public function deleteDestroy(ModelConfigurationInterface $model, $id)
     {
-        if (! $model->isRestorableModel()) {
+        if (!$model->isRestorableModel()) {
             abort(404);
         }
 
         $item = $model->getRepository()->findOnlyTrashed($id);
 
-        if (is_null($item) || ! $model->isRestorable($item)) {
+        if (is_null($item) || !$model->isRestorable($item)) {
             abort(404);
         }
 
@@ -330,13 +330,13 @@ 
      */
     public function postRestore($model, $id)
     {
-        if (! $model->isRestorableModel()) {
+        if (!$model->isRestorableModel()) {
             abort(404);
         }
 
         $item = $model->getRepository()->findOnlyTrashed($id);
 
-        if (is_null($item) || ! $model->isRestorable($item)) {
+        if (is_null($item) || !$model->isRestorable($item)) {
             abort(404);
         }
 
@@ -403,7 +403,7 @@ 
     {
         $lang = trans('sleeping_owl::lang');
         if ($lang == 'sleeping_owl::lang') {
-            $lang = trans('sleeping_owl::lang', [], 'messages', 'en');
+            $lang = trans('sleeping_owl::lang', [ ], 'messages', 'en');
         }
 
         $data = [
@@ -444,7 +444,7 @@ 
     {
         if (($backUrl = Request::input('_redirectBack')) == \URL::previous()) {
             $backUrl = null;
-            Request::merge(['_redirectBack' => $backUrl]);
+            Request::merge([ '_redirectBack' => $backUrl ]);
         }
 
         return $backUrl;
@@ -461,7 +461,7 @@ 
      */
     protected function registerBreadcrumb($title, $parent)
     {
-        Breadcrumbs::register('render', function ($breadcrumbs) use ($title, $parent) {
+        Breadcrumbs::register('render', function($breadcrumbs) use ($title, $parent) {
             $breadcrumbs->parent($parent);
             $breadcrumbs->push($title);
         });

src/Model/ModelConfiguration.php

Doc Comments +1 added lines, -1 removed lines

@@ -501,7 +501,7 @@
     }
 
     /**
-     * @param $id
+     * @param integer $id
      *
      * @return mixed|void
      */

Spacing +9 added lines, -9 removed lines

@@ -332,7 +332,7 @@ 
      */
     public function isCreatable()
     {
-        if (! is_callable($this->getCreate())) {
+        if (!is_callable($this->getCreate())) {
             return false;
         }
 
@@ -356,7 +356,7 @@ 
      */
     public function isEditable(Model $model)
     {
-        if (! is_callable($this->getEdit())) {
+        if (!is_callable($this->getEdit())) {
             return false;
         }
 
@@ -466,7 +466,7 @@ 
      */
     public function fireDisplay()
     {
-        if (! is_callable($this->display)) {
+        if (!is_callable($this->display)) {
             return;
         }
 
@@ -484,7 +484,7 @@ 
      */
     public function fireCreate()
     {
-        if (! is_callable($this->create)) {
+        if (!is_callable($this->create)) {
             return;
         }
 
@@ -507,11 +507,11 @@ 
      */
     public function fireEdit($id)
     {
-        if (! is_callable($this->edit)) {
+        if (!is_callable($this->edit)) {
             return;
         }
 
-        $edit = app()->call($this->edit, ['id' => $id]);
+        $edit = app()->call($this->edit, [ 'id' => $id ]);
         if ($edit instanceof DisplayInterface) {
             $edit->setModelClass($this->getClass());
             $edit->initialize();
@@ -545,7 +545,7 @@ 
     public function fireDelete($id)
     {
         if (is_callable($this->getDelete())) {
-            return app()->call($this->getDelete(), [$id]);
+            return app()->call($this->getDelete(), [ $id ]);
         }
     }
 
@@ -557,7 +557,7 @@ 
     public function fireDestroy($id)
     {
         if (is_callable($this->getDestroy())) {
-            return app()->call($this->getDestroy(), [$id]);
+            return app()->call($this->getDestroy(), [ $id ]);
         }
     }
 
@@ -569,7 +569,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();

src/Model/ModelConfigurationManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -279,7 +279,7 @@ 
     }
 
     /**
-     * @return null|string
+     * @return boolean
      */
     public function hasCustomControllerClass()
     {
@@ -329,7 +329,7 @@ 
     }
 
     /**
-     * @param string|int $id
+     * @param integer $id
      *
      * @return string
      */

Spacing +17 added lines, -17 removed lines

@@ -103,9 +103,9 @@ 
         $this->class = $class;
         $this->model = app($class);
 
-        $this->repository = app(RepositoryInterface::class, [$class]);
+        $this->repository = app(RepositoryInterface::class, [ $class ]);
 
-        if (! $this->alias) {
+        if (!$this->alias) {
             $this->setDefaultAlias();
         }
     }
@@ -180,7 +180,7 @@ 
      */
     public function getCreateTitle()
     {
-        return trans('sleeping_owl::lang.model.create', ['title' => $this->getTitle()]);
+        return trans('sleeping_owl::lang.model.create', [ 'title' => $this->getTitle() ]);
     }
 
     /**
@@ -188,7 +188,7 @@ 
      */
     public function getEditTitle()
     {
-        return trans('sleeping_owl::lang.model.edit', ['title' => $this->getTitle()]);
+        return trans('sleeping_owl::lang.model.edit', [ 'title' => $this->getTitle() ]);
     }
 
     /**
@@ -263,7 +263,7 @@ 
      */
     public function can($action, Model $model)
     {
-        if (! $this->checkAccess) {
+        if (!$this->checkAccess) {
             return true;
         }
 
@@ -283,7 +283,7 @@ 
      */
     public function hasCustomControllerClass()
     {
-        return ! is_null($controller = $this->getControllerClass()) and class_exists($controller);
+        return !is_null($controller = $this->getControllerClass()) and class_exists($controller);
     }
 
     /**
@@ -291,7 +291,7 @@ 
      *
      * @return string
      */
-    public function getDisplayUrl(array $parameters = [])
+    public function getDisplayUrl(array $parameters = [ ])
     {
         array_unshift($parameters, $this->getAlias());
 
@@ -303,7 +303,7 @@ 
      *
      * @return string
      */
-    public function getCreateUrl(array $parameters = [])
+    public function getCreateUrl(array $parameters = [ ])
     {
         array_unshift($parameters, $this->getAlias());
 
@@ -325,7 +325,7 @@ 
      */
     public function getEditUrl($id)
     {
-        return route('admin.model.edit', [$this->getAlias(), $id]);
+        return route('admin.model.edit', [ $this->getAlias(), $id ]);
     }
 
     /**
@@ -335,7 +335,7 @@ 
      */
     public function getUpdateUrl($id)
     {
-        return route('admin.model.update', [$this->getAlias(), $id]);
+        return route('admin.model.update', [ $this->getAlias(), $id ]);
     }
 
     /**
@@ -345,7 +345,7 @@ 
      */
     public function getDeleteUrl($id)
     {
-        return route('admin.model.delete', [$this->getAlias(), $id]);
+        return route('admin.model.delete', [ $this->getAlias(), $id ]);
     }
 
     /**
@@ -355,7 +355,7 @@ 
      */
     public function getDestroyUrl($id)
     {
-        return route('admin.model.destroy', [$this->getAlias(), $id]);
+        return route('admin.model.destroy', [ $this->getAlias(), $id ]);
     }
 
     /**
@@ -365,7 +365,7 @@ 
      */
     public function getRestoreUrl($id)
     {
-        return route('admin.model.restore', [$this->getAlias(), $id]);
+        return route('admin.model.restore', [ $this->getAlias(), $id ]);
     }
 
     /**
@@ -420,7 +420,7 @@ 
         $page->setPriority($priority);
 
         if ($badge) {
-            if (! ($badge instanceof BadgeInterface)) {
+            if (!($badge instanceof BadgeInterface)) {
                 $badge = new Badge($badge);
             }
 
@@ -443,7 +443,7 @@ 
      */
     public function fireEvent($event, $halt = true, Model $model = null)
     {
-        if (! isset(self::$dispatcher)) {
+        if (!isset(self::$dispatcher)) {
             return true;
         }
 
@@ -458,7 +458,7 @@ 
 
         $method = $halt ? 'until' : 'fire';
 
-        return self::$dispatcher->$method($event, [$this, $model]);
+        return self::$dispatcher->$method($event, [ $this, $model ]);
     }
 
     /**
@@ -477,7 +477,7 @@ 
         ])) {
             array_unshift($arguments, $method);
 
-            return call_user_func_array([$this, 'registerEvent'], $arguments);
+            return call_user_func_array([ $this, 'registerEvent' ], $arguments);
         }
 
         throw new BadMethodCallException($method);

src/Model/SectionModelConfiguration.php

Doc Comments +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
     }
 
     /**
-     * @param $id
+     * @param integer $id
      *
      * @return mixed|void
      */

Spacing +9 added lines, -9 removed lines

@@ -31,11 +31,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());
@@ -50,11 +50,11 @@ 
      */
     public function fireCreate()
     {
-        if (! method_exists($this, 'onCreate')) {
+        if (!method_exists($this, 'onCreate')) {
             return;
         }
 
-        $create = app()->call([$this, 'onCreate']);
+        $create = app()->call([ $this, 'onCreate' ]);
         if ($create instanceof DisplayInterface) {
             $create->setModelClass($this->getClass());
             $create->initialize();
@@ -74,11 +74,11 @@ 
      */
     public function fireEdit($id)
     {
-        if (! method_exists($this, 'onEdit')) {
+        if (!method_exists($this, 'onEdit')) {
             return;
         }
 
-        $edit = app()->call([$this, 'onEdit'], ['id' => $id]);
+        $edit = app()->call([ $this, 'onEdit' ], [ 'id' => $id ]);
         if ($edit instanceof DisplayInterface) {
             $edit->setModelClass($this->getClass());
             $edit->initialize();
@@ -112,7 +112,7 @@ 
     public function fireDelete($id)
     {
         if (method_exists($this, 'onDelete')) {
-            return app()->call([$this, 'onDelete'], ['id' => $id]);
+            return app()->call([ $this, 'onDelete' ], [ 'id' => $id ]);
         }
     }
 
@@ -124,7 +124,7 @@ 
     public function fireDestroy($id)
     {
         if (method_exists($this, 'onDestroy')) {
-            return app()->call([$this, 'onDestroy'], ['id' => $id]);
+            return app()->call([ $this, 'onDestroy' ], [ 'id' => $id ]);
         }
     }
 
@@ -136,7 +136,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/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()
     {

Indentation +3 added lines, -3 removed lines

@@ -381,9 +381,9 @@
     protected function createSimpleTree()
     {
         $collection = $this->getQuery()
-           ->orderBy($this->getParentField(), 'asc')
-           ->orderBy($this->getOrderField(), 'asc')
-           ->get();
+            ->orderBy($this->getParentField(), 'asc')
+            ->orderBy($this->getOrderField(), 'asc')
+            ->get();
 
         $parent = $this->getRootParentId();
 

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