LaravelRUS / SleepingOwlAdmin

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

Unused Use Statements -3 removed lines

@@ -4,13 +4,10 @@
 
 use Closure;
 use Validator;
-use Illuminate\Http\Request;
 use Illuminate\Routing\Router;
-use Illuminate\Http\JsonResponse;
 use Illuminate\Http\UploadedFile;
 use KodiComponents\Support\Upload;
 use SleepingOwl\Admin\Contracts\WithRoutesInterface;
-use SleepingOwl\Admin\Contracts\ModelConfigurationInterface;
 
 class File extends NamedFormElement implements WithRoutesInterface
 {

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
     {
         $routeName = 'admin.form.element.'.static::$route;
 
-        if (! $router->has($routeName)) {
+        if (!$router->has($routeName)) {
             $router->post('{adminModel}/'.static::$route.'/{field}/{id?}', [
                 'as' => $routeName,
                 'uses' => 'SleepingOwl\Admin\Http\Controllers\UploadController@fromField'
@@ -95,7 +95,7 @@ 
      */
     public function getUploadPath(UploadedFile $file)
     {
-        if (! is_callable($this->uploadPath)) {
+        if (!is_callable($this->uploadPath)) {
             return $this->defaultUploadPath($file);
         }
 
@@ -121,7 +121,7 @@ 
      */
     public function getUploadFileName(UploadedFile $file)
     {
-        if (! is_callable($this->uploadFileName)) {
+        if (!is_callable($this->uploadFileName)) {
             return $this->defaultUploadFilename($file);
         }
 

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

@@ -99,12 +99,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');
                 }
             });
@@ -207,7 +207,7 @@ 
      */
     public function setItems($items)
     {
-        if (! is_array($items)) {
+        if (!is_array($items)) {
             $items = func_get_args();
         }
 
@@ -234,7 +234,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);
@@ -301,7 +301,7 @@ 
      */
     public function saveForm(\Illuminate\Http\Request $request, ModelConfigurationInterface $modelConfiguration = null)
     {
-        if (! $this->validModelConfiguration($modelConfiguration)) {
+        if (!$this->validModelConfiguration($modelConfiguration)) {
             return false;
         }
 
@@ -341,7 +341,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);
             }
@@ -356,7 +356,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 {
@@ -374,7 +374,7 @@ 
      */
     public function validateForm(\Illuminate\Http\Request $request, ModelConfigurationInterface $modelConfiguration = null)
     {
-        if (! $this->validModelConfiguration($modelConfiguration)) {
+        if (!$this->validModelConfiguration($modelConfiguration)) {
             return;
         }
 

src/Http/Controllers/FormElementController.php

Spacing +4 added lines, -4 removed lines

@@ -20,9 +20,9 @@ 
      */
     public function dependentSelect(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);
@@ -30,7 +30,7 @@ 
 
             $form = $model->fireEdit($id);
         } else {
-            if (! $model->isCreatable()) {
+            if (!$model->isCreatable()) {
                 return new JsonResponse([
                     'message' => trans('lang.message.access_denied'),
                 ], 403);
@@ -57,7 +57,7 @@ 
         }
 
         return new JsonResponse([
-            'output' => collect($options)->map(function ($value, $key) {
+            'output' => collect($options)->map(function($value, $key) {
                 return ['id' => $key, 'name' => $value];
             }),
             'selected' => $element->getValueFromModel(),

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/DisplayController.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         $display = $model->fireDisplay();
 
         if ($display instanceof DisplayTabbed) {
-            $display->getTabs()->each(function ($tab) use($request) {
+            $display->getTabs()->each(function($tab) use($request) {
                 $content = $tab->getContent();
                 if ($content instanceof DisplayTree) {
                     $content->getRepository()->reorder(

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/Model/ModelConfiguration.php

Spacing +5 added lines, -5 removed lines

@@ -200,7 +200,7 @@ 
      */
     public function isCreatable()
     {
-        if (! is_callable($this->getCreate())) {
+        if (!is_callable($this->getCreate())) {
             return false;
         }
 
@@ -224,7 +224,7 @@ 
      */
     public function isEditable(Model $model)
     {
-        if (! is_callable($this->getEdit())) {
+        if (!is_callable($this->getEdit())) {
             return false;
         }
 
@@ -346,7 +346,7 @@ 
      */
     public function fireDisplay()
     {
-        if (! is_callable($this->getDisplay())) {
+        if (!is_callable($this->getDisplay())) {
             return;
         }
 
@@ -387,7 +387,7 @@ 
      */
     public function fireCreate()
     {
-        if (! is_callable($this->getCreate())) {
+        if (!is_callable($this->getCreate())) {
             return;
         }
 
@@ -434,7 +434,7 @@ 
      */
     public function fireEdit($id)
     {
-        if (! is_callable($this->getEdit())) {
+        if (!is_callable($this->getEdit())) {
             return;
         }
 

src/Form/Element/DependentSelect.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     {
         $routeName = 'admin.form.element.dependent-select';
 
-        if (! $router->has($routeName)) {
+        if (!$router->has($routeName)) {
             $router->post('{adminModel}/dependent-select/{field}/{id?}', [
                 'as' => $routeName,
                 'uses' => 'SleepingOwl\Admin\Http\Controllers\FormElementController@dependentSelect'