LaravelRUS / SleepingOwlAdmin

src/Http/routes.php

Spacing +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-$router->group(['as' => 'admin.', 'namespace' => 'SleepingOwl\Admin\Http\Controllers'], function ($router) {
-    if (! $router->has('admin.dashboard')) {
+$router->group(['as' => 'admin.', 'namespace' => 'SleepingOwl\Admin\Http\Controllers'], function($router) {
+    if (!$router->has('admin.dashboard')) {
         $router->get('', ['as' => 'dashboard', 'uses' => 'AdminController@getDashboard']);
     }
 

src/Traits/Renderable.php

Doc Comments +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     protected $viewPath;
 
     /**
-     * @param string|View $view
+     * @param string $view
      *
      * @return $this
      */

src/Admin.php

Doc Comments +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
     }
 
     /**
-     * @param string|Model $class
+     * @param string|null $class
      * @return ModelConfigurationInterface
      */
     public function getModel($class)

Spacing +1 added lines, -1 removed lines

@@ -121,7 +121,7 @@
             $class = get_class($class);
         }
 
-        if (! $this->hasModel($class)) {
+        if (!$this->hasModel($class)) {
             $this->registerModel($class);
         }
 

src/Form/FormDefault.php

Doc Comments +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
     /**
      * FormDefault constructor.
      *
-     * @param array $elements
+     * @param Element\Checkbox[] $elements
      */
     public function __construct(array $elements = [])
     {
@@ -190,7 +190,7 @@ 
      * @deprecated 4.5.0
      * @see getElements()
      *
-     * @return Collection[]
+     * @return FormElementsCollection
      */
     public function getItems()
     {

Spacing +9 added lines, -9 removed lines

@@ -100,12 +100,12 @@ 
 
         parent::initialize();
 
-        if (! $this->hasHtmlAttribute('enctype')) {
+        if (!$this->hasHtmlAttribute('enctype')) {
             // Try to find upload element
-            $this->getElements()->each(function ($element) {
+            $this->getElements()->each(function($element) {
 
                 // TODO: this not works withs nested elements
-                if ($element instanceof Upload and ! $this->hasHtmlAttribute('enctype')) {
+                if ($element instanceof Upload and !$this->hasHtmlAttribute('enctype')) {
                     $this->setHtmlAttribute('enctype', 'multipart/form-data');
                 }
             });
@@ -210,7 +210,7 @@ 
      */
     public function setItems($items)
     {
-        if (! is_array($items)) {
+        if (!is_array($items)) {
             $items = func_get_args();
         }
 
@@ -237,7 +237,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;
 
             parent::setModel($model);
@@ -298,7 +298,7 @@ 
      */
     public function saveForm(\Illuminate\Http\Request $request, ModelConfigurationInterface $modelConfiguration = null)
     {
-        if (! $this->validModelConfiguration($modelConfiguration)) {
+        if (!$this->validModelConfiguration($modelConfiguration)) {
             return false;
         }
 
@@ -338,7 +338,7 @@ 
     protected function saveBelongsToRelations(Model $model)
     {
         foreach ($model->getRelations() as $name => $relation) {
-            if ($model->{$name}() instanceof BelongsTo && ! is_null($relation)) {
+            if ($model->{$name}() instanceof BelongsTo && !is_null($relation)) {
                 $relation->save();
                 $model->{$name}()->associate($relation);
             }
@@ -353,7 +353,7 @@ 
     protected function saveHasOneRelations(Model $model)
     {
         foreach ($model->getRelations() as $name => $relation) {
-            if ($model->{$name}() instanceof HasOneOrMany && ! is_null($relation)) {
+            if ($model->{$name}() instanceof HasOneOrMany && !is_null($relation)) {
                 if (is_array($relation) || $relation instanceof \Traversable) {
                     $model->{$name}()->saveMany($relation);
                 } else {
@@ -371,7 +371,7 @@ 
      */
     public function validateForm(\Illuminate\Http\Request $request, ModelConfigurationInterface $modelConfiguration = null)
     {
-        if (! $this->validModelConfiguration($modelConfiguration)) {
+        if (!$this->validModelConfiguration($modelConfiguration)) {
             return;
         }
 

src/Http/Controllers/UploadController.php

Spacing +3 added lines, -3 removed lines

@@ -22,9 +22,9 @@ 
      */
     public function fromField(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);
@@ -32,7 +32,7 @@ 
 
             $form = $model->fireEdit($id);
         } else {
-            if (! $model->isCreatable()) {
+            if (!$model->isCreatable()) {
                 return new JsonResponse([
                     'message' => trans('lang.message.access_denied'),
                 ], 403);

src/Http/Controllers/AdminController.php

Spacing +18 added lines, -18 removed lines

@@ -50,8 +50,8 @@ 
 
         $admin->navigation()->setCurrentUrl($request->url());
 
-        if (! $this->breadcrumbs->exists('home')) {
-            $this->breadcrumbs->register('home', function ($breadcrumbs) {
+        if (!$this->breadcrumbs->exists('home')) {
+            $this->breadcrumbs->register('home', function($breadcrumbs) {
                 $breadcrumbs->push(trans('sleeping_owl::lang.dashboard'), route('admin.dashboard'));
             });
         }
@@ -72,8 +72,8 @@ 
         }
 
         foreach ($breadcrumbs as  $breadcrumb) {
-            if (! $this->breadcrumbs->exists($breadcrumb['id'])) {
-                $this->breadcrumbs->register($breadcrumb['id'], function ($breadcrumbs) use ($breadcrumb) {
+            if (!$this->breadcrumbs->exists($breadcrumb['id'])) {
+                $this->breadcrumbs->register($breadcrumb['id'], function($breadcrumbs) use ($breadcrumb) {
                     $breadcrumbs->parent($breadcrumb['parent']);
                     $breadcrumbs->push($breadcrumb['title'], $breadcrumb['url']);
                 });
@@ -117,7 +117,7 @@ 
      */
     public function getDisplay(ModelConfigurationInterface $model)
     {
-        if (! $model->isDisplayable()) {
+        if (!$model->isDisplayable()) {
             abort(404);
         }
 
@@ -133,7 +133,7 @@ 
      */
     public function getCreate(ModelConfigurationInterface $model)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -152,7 +152,7 @@ 
      */
     public function postStore(ModelConfigurationInterface $model, Request $request)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -188,7 +188,7 @@ 
             $redirectPolicy = $model->getRedirect();
 
             /* Make redirect when use in model config && Fix editable redirect */
-            if ($redirectPolicy->get('create') == 'display' || ! $model->isEditable($newModel)) {
+            if ($redirectPolicy->get('create') == 'display' || !$model->isEditable($newModel)) {
                 $redirectUrl = $model->getDisplayUrl();
             }
 
@@ -225,7 +225,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -249,7 +249,7 @@ 
         $editForm = $model->fireEdit($id);
         $item = $editForm->getModel();
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -322,7 +322,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();
 
@@ -333,7 +333,7 @@ 
         $repository = $model->getRepository();
         $item = $repository->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -360,7 +360,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isDeletable($item)) {
+        if (is_null($item) || !$model->isDeletable($item)) {
             abort(404);
         }
 
@@ -389,13 +389,13 @@ 
      */
     public function deleteDestroy(ModelConfigurationInterface $model, Request $request, $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);
         }
 
@@ -424,13 +424,13 @@ 
      */
     public function postRestore(ModelConfigurationInterface $model, Request $request, $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);
         }
 
@@ -556,7 +556,7 @@ 
      */
     protected function registerBreadcrumb($title, $parent)
     {
-        $this->breadcrumbs->register('render', function ($breadcrumbs) use ($title, $parent) {
+        $this->breadcrumbs->register('render', function($breadcrumbs) use ($title, $parent) {
             $breadcrumbs->parent($parent);
             $breadcrumbs->push($title);
         });

Doc Comments +2 added lines, -2 removed lines

@@ -311,7 +311,7 @@ 
      * @param ModelConfigurationInterface $model
      * @param Request $request
      *
-     * @return bool
+     * @return boolean|null
      *
      * @throws \Symfony\Component\HttpKernel\Exception\NotFoundHttpException
      */
@@ -534,7 +534,7 @@ 
     /**
      * @param Request $request
      *
-     * @return null|string
+     * @return string
      */
     protected function getBackUrl(Request $request)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace SleepingOwl\Admin\Http\Controllers;
 
 use Illuminate\Http\Request;
-use Illuminate\Http\Response;
 use Illuminate\Routing\Controller;
 use Illuminate\Contracts\Support\Renderable;
 use Illuminate\Validation\ValidationException;

src/Model/ModelCollection.php

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
      */
     public function aliases()
     {
-        return $this->map(function (ModelConfigurationInterface $model) {
+        return $this->map(function(ModelConfigurationInterface $model) {
             return $model->getAlias();
         });
     }
@@ -22,7 +22,7 @@ 
      */
     public function keyByAlias()
     {
-        return $this->keyBy(function (ModelConfigurationInterface $model) {
+        return $this->keyBy(function(ModelConfigurationInterface $model) {
             return $model->getAlias();
         });
     }

src/Form/Element/Select.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
      */
     public function getOptions()
     {
-        if (! is_null($this->getModelForOptions()) && ! is_null($this->getDisplay())) {
+        if (!is_null($this->getModelForOptions()) && !is_null($this->getDisplay())) {
             $this->setOptions(
                 $this->loadOptions()
             );
@@ -135,7 +135,7 @@ 
      */
     public function exclude($keys)
     {
-        if (! is_array($keys)) {
+        if (!is_array($keys)) {
             $keys = func_get_args();
         }
 

src/Form/Element/DateTime.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
     public function getValueFromModel()
     {
         $value = parent::getValueFromModel();
-        if (! empty($value)) {
+        if (!empty($value)) {
             return $this->parseValue($value);
         }
     }
@@ -82,7 +82,7 @@ 
      */
     public function setModelAttribute($value)
     {
-        $value = ! empty($value)
+        $value = !empty($value)
             ? Carbon::createFromFormat($this->getPickerFormat(), $value)->format($this->getFormat())
             : null;