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|Closure $instance
+     * @param Model $instance
      * @param string           $name
      *
      * @return mixed

Spacing +4 added lines, -4 removed lines

@@ -73,8 +73,8 @@ 
      */
     public function setOrderable($orderable = true)
     {
-        if ($orderable !== false && ! $orderable instanceof OrderByClauseInterface) {
-            if (! is_string($orderable) && ! $orderable instanceof Closure) {
+        if ($orderable !== false && !$orderable instanceof OrderByClauseInterface) {
+            if (!is_string($orderable) && !$orderable instanceof Closure) {
                 $orderable = $this->getName();
             }
         }
@@ -113,11 +113,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/Display.php

Doc Comments +1 added lines, -1 removed lines

@@ -308,7 +308,7 @@
     }
 
     /**
-     * @return \Illuminate\Foundation\Application|mixed
+     * @return RepositoryInterface
      * @throws \Exception
      */
     protected function makeRepository()

Spacing +3 added lines, -3 removed lines

@@ -190,7 +190,7 @@ 
             $this->title,
         ];
 
-        $this->getExtensions()->each(function (DisplayExtensionInterface $extension) use (&$titles) {
+        $this->getExtensions()->each(function(DisplayExtensionInterface $extension) use (&$titles) {
             if (method_exists($extension, $method = 'getTitle')) {
                 $titles[] = call_user_func([$extension, $method]);
             }
@@ -238,7 +238,7 @@ 
 
         foreach ($blocks as $block => $data) {
             foreach ($data as $html) {
-                if (! empty($html)) {
+                if (!empty($html)) {
                     $view->getFactory()->startSection($block);
                     echo $html;
                     $view->getFactory()->yieldSection();
@@ -287,7 +287,7 @@ 
     protected function makeRepository()
     {
         $repository = app($this->repositoryClass);
-        if (! ($repository instanceof RepositoryInterface)) {
+        if (!($repository instanceof RepositoryInterface)) {
             throw new \Exception('Repository class must be instanced of [RepositoryInterface]');
         }
 

src/Model/ModelConfigurationManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -290,7 +290,7 @@
     }
 
     /**
-     * @return null|string
+     * @return boolean
      */
     public function hasCustomControllerClass()
     {

Spacing +5 added lines, -5 removed lines

@@ -114,7 +114,7 @@ 
 
         $this->repository = $app->make(RepositoryInterface::class);
         $this->repository->setClass($class);
-        if (! $this->alias) {
+        if (!$this->alias) {
             $this->setDefaultAlias();
         }
     }
@@ -303,7 +303,7 @@ 
      */
     public function can($action, Model $model)
     {
-        if (! $this->checkAccess) {
+        if (!$this->checkAccess) {
             return true;
         }
 
@@ -335,7 +335,7 @@ 
      */
     public function hasCustomControllerClass()
     {
-        return ! is_null($controller = $this->getControllerClass()) and class_exists($controller);
+        return !is_null($controller = $this->getControllerClass()) and class_exists($controller);
     }
 
     /**
@@ -472,7 +472,7 @@ 
         $page->setPriority($priority);
 
         if ($badge) {
-            if (! ($badge instanceof BadgeInterface)) {
+            if (!($badge instanceof BadgeInterface)) {
                 $badge = new Badge($badge);
             }
 
@@ -515,7 +515,7 @@ 
      */
     public function fireEvent($event, $halt = true, Model $model = null, ...$payload)
     {
-        if (! isset(self::$dispatcher)) {
+        if (!isset(self::$dispatcher)) {
             return true;
         }
 

src/Navigation/Page.php

Doc Comments +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
     }
 
     /**
-     * @return Closure
+     * @return \Closure
      */
     public function getAccessLogic()
     {

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function getModelConfiguration()
     {
-        if (! $this->hasModel()) {
+        if (!$this->hasModel()) {
             return;
         }
 
@@ -43,7 +43,7 @@ 
      */
     public function hasModel()
     {
-        return ! is_null($this->model) and class_exists($this->model);
+        return !is_null($this->model) and class_exists($this->model);
     }
 
     /**
@@ -87,9 +87,9 @@ 
      */
     public function getAccessLogic()
     {
-        if (! is_callable($this->accessLogic)) {
+        if (!is_callable($this->accessLogic)) {
             if ($this->hasModel()) {
-                return function () {
+                return function() {
                     return $this->getModelConfiguration()->isDisplayable();
                 };
             }
@@ -107,7 +107,7 @@ 
     {
         $data = $this->toArray();
 
-        if (! is_null($view)) {
+        if (!is_null($view)) {
             return view($view, $data)->render();
         }
 

src/Repository/TreeRepository.php

Doc Comments +7 added lines, -7 removed lines

@@ -203,7 +203,7 @@ 
      *
      * @param $id
      * @param $parentId
-     * @param $left
+     * @param integer $left
      * @param $right
      */
     protected function move($id, $parentId, $left, $right)
@@ -300,7 +300,7 @@ 
      * Call several methods and get first result.
      *
      * @param $instance
-     * @param $methods
+     * @param string[] $methods
      *
      * @return mixed
      * @throws Exception
@@ -320,9 +320,9 @@ 
      *
      * @param $root
      * @param $parentId
-     * @param $left
+     * @param integer $left
      *
-     * @return mixed
+     * @return integer
      */
     protected function recursiveReorder($root, $parentId, $left)
     {
@@ -341,7 +341,7 @@ 
      * Recursive reoder simple tree type.
      *
      * @param $data
-     * @param $parentId
+     * @param string|null $parentId
      */
     protected function recursiveReorderSimple(array $data, $parentId)
     {
@@ -363,7 +363,7 @@ 
      * Get children for simple tree type structure.
      *
      * @param $collection
-     * @param $id
+     * @param string|null $id
      *
      * @return Collection
      */
@@ -385,7 +385,7 @@ 
 
     /**
      * Create simple tree type structure.
-     * @return static
+     * @return Collection
      */
     protected function createSimpleTree()
     {

src/Display/Filter/FilterField.php

Spacing +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
             $relationName = implode('.', $parts);
         }
 
-        if (! is_null($relationName)) {
-            $query->whereHas($relationName, function ($q) use ($name, $value) {
+        if (!is_null($relationName)) {
+            $query->whereHas($relationName, function($q) use ($name, $value) {
                 $this->buildQuery($q, $name, $value);
             });
         } else {

src/Form/FormButtons.php

Spacing +4 added lines, -4 removed lines

@@ -313,11 +313,11 @@ 
      */
     public function isShowDeleteButton()
     {
-        if (is_null($this->getModel()->getKey()) || ! $this->showDeleteButton) {
+        if (is_null($this->getModel()->getKey()) || !$this->showDeleteButton) {
             return false;
         }
 
-        $this->showDeleteButton = ! $this->isTrashed() && $this->getModelConfiguration()->isDeletable($this->getModel());
+        $this->showDeleteButton = !$this->isTrashed() && $this->getModelConfiguration()->isDeletable($this->getModel());
 
         return $this->showDeleteButton;
     }
@@ -337,7 +337,7 @@ 
      */
     public function isShowDestroyButton()
     {
-        if (is_null($this->getModel()->getKey()) || ! $this->showDestroyButton) {
+        if (is_null($this->getModel()->getKey()) || !$this->showDestroyButton) {
             return false;
         }
 
@@ -362,7 +362,7 @@ 
      */
     public function isShowRestoreButton()
     {
-        if (is_null($this->getModel()->getKey()) || ! $this->showRestoreButton) {
+        if (is_null($this->getModel()->getKey()) || !$this->showRestoreButton) {
             return false;
         }
 

src/Display/TableColumn.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
     {
         $this->header = app(TableHeaderColumnInterface::class);
 
-        if (! is_null($label)) {
+        if (!is_null($label)) {
             $this->setLabel($label);
         }
 
@@ -206,7 +206,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');
         }
 
@@ -233,7 +233,7 @@ 
      */
     public function orderBy(Builder $query, $direction)
     {
-        if (! $this->isOrderable()) {
+        if (!$this->isOrderable()) {
             throw new \InvalidArgumentException('Argument must be instance of SleepingOwl\Admin\Contracts\Display\OrderByClauseInterface interface');
         }
 

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