anavel / crud

src/Abstractor/Eloquent/Field.php

Doc Comments +4 added lines, -1 removed lines

@@ -123,7 +123,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getValue()
     {
@@ -162,6 +162,7 @@ 
 
     /**
      *
+     * @param boolean $value
      */
     public function hideValue($value = null)
     {
@@ -174,6 +175,7 @@ 
 
     /**
      *
+     * @param boolean $value
      */
     public function saveIfEmpty($value = null)
     {
@@ -186,6 +188,7 @@ 
 
     /**
      *
+     * @param boolean $value
      */
     public function noValidate($value = null)
     {

Spacing +7 added lines, -7 removed lines

@@ -106,7 +106,7 @@ 
 
     public function setFunctions($functions)
     {
-        if (! is_array($functions)) {
+        if (!is_array($functions)) {
             $functions = array($functions);
         }
 
@@ -116,7 +116,7 @@ 
     public function applyFunctions($value)
     {
         foreach ($this->functions as $function) {
-            if (! function_exists($function)) {
+            if (!function_exists($function)) {
                 throw new \Exception("Function ".$function." does not exist");
             }
 
@@ -134,7 +134,7 @@ 
     {
         $this->value = $value;
 
-        if (! $this->hideValue()) {
+        if (!$this->hideValue()) {
             $this->formField->val($this->value);
         }
     }
@@ -168,7 +168,7 @@ 
 
     public function getFormField()
     {
-        if (! $this->hideValue()) {
+        if (!$this->hideValue()) {
             if (Request::old($this->name)) {
                 $this->formField->val(Request::old($this->name));
             }
@@ -182,7 +182,7 @@ 
      */
     public function hideValue($value = null)
     {
-        if (! is_null($value)) {
+        if (!is_null($value)) {
             $this->hideValue = $value;
         }
 
@@ -194,7 +194,7 @@ 
      */
     public function saveIfEmpty($value = null)
     {
-        if (! is_null($value)) {
+        if (!is_null($value)) {
             $this->saveIfEmpty = $value;
         }
 
@@ -206,7 +206,7 @@ 
      */
     public function noValidate($value = null)
     {
-        if (! is_null($value)) {
+        if (!is_null($value)) {
             $this->noValidate = $value;
         }
 

src/Abstractor/ConfigurationReader.php

Spacing +2 added lines, -2 removed lines

@@ -5,7 +5,7 @@ 
 {
     public function getConfigValue()
     {
-        if (! property_exists($this, 'config') || ! is_array($this->config) || empty($this->config)) {
+        if (!property_exists($this, 'config') || !is_array($this->config) || empty($this->config)) {
             return null;
         }
 
@@ -17,7 +17,7 @@ 
 
         if (is_array($params) && count($params) > 0) {
             foreach ($params as $configKey) {
-                if (! array_key_exists($configKey, $nestedConfig)) {
+                if (!array_key_exists($configKey, $nestedConfig)) {
                     $nestedConfig = array();
                     return null;
                 }

src/Abstractor/Eloquent/Relation/Select.php

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
     {
         $results = $this->eloquentRelation->getResults();
 
-        if (! empty($results)) {
+        if (!empty($results)) {
             $field->setValue($results->getKey());
         }
         return $field;

src/Abstractor/Eloquent/Relation/Traits/CheckRelationCompatibility.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 {
     public function checkRelationCompatibility()
     {
-        if (! in_array(get_class($this->eloquentRelation), $this->compatibleEloquentRelations)) {
+        if (!in_array(get_class($this->eloquentRelation), $this->compatibleEloquentRelations)) {
             throw new RelationException(get_class($this->eloquentRelation)." eloquent relation is not compatible with ".$this->getType()." type");
         }
     }

src/Abstractor/Eloquent/RelationFactory.php

Spacing +7 added lines, -7 removed lines

@@ -72,11 +72,11 @@ 
 
     public function get($name)
     {
-        if (! empty($this->config) && ! empty($this->config['name'])) {
+        if (!empty($this->config) && !empty($this->config['name'])) {
             $name = $this->config['name'];
         }
-        if (! method_exists($this->model, $name)) {
-            throw new FactoryException("Relation " . $name . " does not exist on " . get_class($this->model));
+        if (!method_exists($this->model, $name)) {
+            throw new FactoryException("Relation ".$name." does not exist on ".get_class($this->model));
         }
 
         $relationInstance = $this->model->$name();
@@ -84,8 +84,8 @@ 
 
 
         if (empty($this->config['type'])) {
-            if (! array_key_exists($relationEloquentType, $this->eloquentTypeToRelationType)) {
-                throw new FactoryException($relationEloquentType . " relation not supported");
+            if (!array_key_exists($relationEloquentType, $this->eloquentTypeToRelationType)) {
+                throw new FactoryException($relationEloquentType." relation not supported");
             }
 
             $type = $this->eloquentTypeToRelationType[$relationEloquentType];
@@ -93,8 +93,8 @@ 
             $type = $this->config['type'];
         }
 
-        if (! array_key_exists($type, $this->typesMap)) {
-            throw new FactoryException("Unexpected relation type: " . $type);
+        if (!array_key_exists($type, $this->typesMap)) {
+            throw new FactoryException("Unexpected relation type: ".$type);
         }
 
         $this->config['name'] = $name;

src/CrudModuleProvider.php

Spacing +4 added lines, -4 removed lines

@@ -47,14 +47,14 @@ 
 
         $this->app->bind(
             'Anavel\Crud\Contracts\Abstractor\FieldFactory',
-            function () {
+            function() {
                 return new FieldAbstractorFactory(new FormFactory);
             }
         );
 
         $this->app->bind(
             'Anavel\Crud\Contracts\Abstractor\RelationFactory',
-            function () {
+            function() {
                 return new RelationAbstractorFactory(
                     $this->app['ANavallaSuiza\Laravel\Database\Contracts\Manager\ModelManager'],
                     $this->app['Anavel\Crud\Contracts\Abstractor\FieldFactory']
@@ -64,7 +64,7 @@ 
 
         $this->app->bind(
             'Anavel\Crud\Contracts\Abstractor\ModelFactory',
-            function () {
+            function() {
                 return new ModelAbstractorFactory(
                     config('anavel-crud.models'),
                     $this->app['ANavallaSuiza\Laravel\Database\Contracts\Manager\ModelManager'],
@@ -77,7 +77,7 @@ 
 
         $this->app->bind(
             'Anavel\Crud\Contracts\Form\Generator',
-            function () {
+            function() {
                 return new FormGenerator(new FormFactory);
             }
         );

src/Http/routes.php

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
         'prefix' => 'crud',
         'namespace' => 'Anavel\Crud\Http\Controllers'
     ],
-    function () {
+    function() {
         Route::get('/', [
             'as'   => 'anavel-crud.home',
             'uses' => 'HomeController@index'

src/helpers.php

Spacing +4 added lines, -4 removed lines

@@ -2,7 +2,7 @@ 
 
 use EasySlugger\Slugger;
 
-if (! function_exists('slugify')) {
+if (!function_exists('slugify')) {
     /**
      * Generate slug
      *
@@ -15,7 +15,7 @@ 
     }
 }
 
-if (! function_exists('uniqueSlugify')) {
+if (!function_exists('uniqueSlugify')) {
     /**
      * Generate unique slug
      *
@@ -28,7 +28,7 @@ 
     }
 }
 
-if (! function_exists('transcrud')) {
+if (!function_exists('transcrud')) {
     /**
      * Translate string but remove file key if translation not found
      *
@@ -37,7 +37,7 @@ 
      */
     function transcrud($text)
     {
-        $translation = trans('anavel-crud::models.' . $text);
+        $translation = trans('anavel-crud::models.'.$text);
 
         return str_replace('anavel-crud::models.', '', $translation);
     }

src/Abstractor/Eloquent/Model.php

Doc Comments +4 added lines, -1 removed lines

@@ -85,6 +85,9 @@ 
         return transcrud($this->name);
     }
 
+    /**
+     * @return string
+     */
     public function getModel()
     {
         return $this->model;
@@ -334,7 +337,7 @@ 
     }
 
     /**
-     * @param array $requestForm
+     * @param array $request
      * @return mixed
      */
     public function persist(Request $request)

Spacing +25 added lines, -25 removed lines

@@ -128,32 +128,32 @@ 
         }
 
         $customDisplayedColumns = $this->getConfigValue($action, 'display');
-        $customHiddenColumns = $this->getConfigValue($action, 'hide') ? : [];
+        $customHiddenColumns = $this->getConfigValue($action, 'hide') ?: [];
 
         $relations = $this->getRelations();
 
         $columns = array();
-        if (! empty($customDisplayedColumns) && is_array($customDisplayedColumns)) {
+        if (!empty($customDisplayedColumns) && is_array($customDisplayedColumns)) {
             foreach ($customDisplayedColumns as $customColumn) {
                 if (strpos($customColumn, '.')) {
                     $customColumnRelation = explode('.', $customColumn);
 
-                    if (! $relations->has($customColumnRelation[0])) {
-                        throw new AbstractorException("Relation " . $customColumnRelation[0] . " not configured on " . $this->getModel());
+                    if (!$relations->has($customColumnRelation[0])) {
+                        throw new AbstractorException("Relation ".$customColumnRelation[0]." not configured on ".$this->getModel());
                     }
 
                     $relation = $relations->get($customColumnRelation[0]);
 
                     $relationColumns = $relation->getModelAbstractor()->getColumns($action);
 
-                    if (! array_key_exists($customColumnRelation[1], $relationColumns)) {
-                        throw new AbstractorException("Column " . $customColumnRelation[1] . " does not exist on relation ".$customColumnRelation[0]. " of model " . $this->getModel());
+                    if (!array_key_exists($customColumnRelation[1], $relationColumns)) {
+                        throw new AbstractorException("Column ".$customColumnRelation[1]." does not exist on relation ".$customColumnRelation[0]." of model ".$this->getModel());
                     }
 
                     $columns[$customColumn] = $relationColumns[$customColumnRelation[1]];
                 } else {
-                    if (! array_key_exists($customColumn, $tableColumns)) {
-                        throw new AbstractorException("Column " . $customColumn . " does not exist on " . $this->getModel());
+                    if (!array_key_exists($customColumn, $tableColumns)) {
+                        throw new AbstractorException("Column ".$customColumn." does not exist on ".$this->getModel());
                     }
 
                     $columns[$customColumn] = $tableColumns[$customColumn];
@@ -185,7 +185,7 @@ 
 
         $relations = collect();
 
-        if (! empty($configRelations)) {
+        if (!empty($configRelations)) {
             foreach ($configRelations as $relationName => $configRelation) {
                 if (is_int($relationName)) {
                     $relationName = $configRelation;
@@ -193,7 +193,7 @@ 
 
                 $config = [];
                 if ($configRelation !== $relationName) {
-                    if (! is_array($configRelation)) {
+                    if (!is_array($configRelation)) {
                         $config['type'] = $configRelation;
                     } else {
                         $config = $configRelation;
@@ -208,7 +208,7 @@ 
                 $secondaryRelations = $relation->getSecondaryRelations();
 
 
-                if (! $secondaryRelations->isEmpty()) {
+                if (!$secondaryRelations->isEmpty()) {
                     $relations->put(
                         $relationName,
                         collect(['relation' => $relation, 'secondaryRelations' => $secondaryRelations])
@@ -232,7 +232,7 @@ 
     {
         $columns = $this->getColumns('list');
 
-        $fieldsPresentation = $this->getConfigValue('fields_presentation') ? : [];
+        $fieldsPresentation = $this->getConfigValue('fields_presentation') ?: [];
 
         $fields = array();
         foreach ($columns as $name => $column) {
@@ -267,7 +267,7 @@ 
     {
         $columns = $this->getColumns('detail');
 
-        $fieldsPresentation = $this->getConfigValue('fields_presentation') ? : [];
+        $fieldsPresentation = $this->getConfigValue('fields_presentation') ?: [];
 
         $fields = array();
         foreach ($columns as $name => $column) {
@@ -307,7 +307,7 @@ 
 
         $fields = array();
         foreach ($columns as $name => $column) {
-            if (! in_array($name, $this->getReadOnlyColumns())) {
+            if (!in_array($name, $this->getReadOnlyColumns())) {
                 $presentation = null;
                 if (array_key_exists($name, $this->fieldsPresentation)) {
                     $presentation = $this->fieldsPresentation[$name];
@@ -328,24 +328,24 @@ 
                     ->setConfig($config)
                     ->get();
 
-                if (! empty($this->instance) && ! empty($this->instance->getAttribute($name))) {
+                if (!empty($this->instance) && !empty($this->instance->getAttribute($name))) {
                     $field->setValue($this->instance->getAttribute($name));
                 }
 
                 $fields[$arrayKey][$name] = $field;
 
-                if (! empty($config['form_type']) && $config['form_type'] === 'file') {
+                if (!empty($config['form_type']) && $config['form_type'] === 'file') {
                     $field = $this->fieldFactory
                         ->setColumn($column)
                         ->setConfig([
-                            'name'         => $name . '__delete',
+                            'name'         => $name.'__delete',
                             'presentation' => null,
                             'form_type'    => 'checkbox',
                             'no_validate'  => true,
                             'functions'    => null
                         ])
                         ->get();
-                    $fields[$arrayKey][$name . '__delete'] = $field;
+                    $fields[$arrayKey][$name.'__delete'] = $field;
                 }
             }
         }
@@ -382,7 +382,7 @@ 
     {
         /** @var \ANavallaSuiza\Laravel\Database\Contracts\Manager\ModelManager $modelManager */
         $modelManager = App::make('ANavallaSuiza\Laravel\Database\Contracts\Manager\ModelManager');
-        if (! empty($this->instance)) {
+        if (!empty($this->instance)) {
             $item = $this->instance;
         } else {
             $item = $modelManager->getModelInstance($this->getModel());
@@ -394,7 +394,7 @@ 
             return;
         }
 
-        if (! empty($fields['main'])) {
+        if (!empty($fields['main'])) {
             $skip = null;
             foreach ($fields['main'] as $key => $field) {
                 /** @var FieldContract $field */
@@ -419,20 +419,20 @@ 
 
                 if (get_class($field->getFormField()) === \FormManager\Fields\File::class) {
                     $handleResult = $this->handleField($request, $item, $fields['main'], 'main', $fieldName);
-                    if (! empty($handleResult['skip'])) {
+                    if (!empty($handleResult['skip'])) {
                         $skip = $handleResult['skip'];
                     }
-                    if (! empty($handleResult['requestValue'])) {
+                    if (!empty($handleResult['requestValue'])) {
                         $requestValue = $handleResult['requestValue'];
                     }
                 }
 
 
-                if (! $field->saveIfEmpty() && empty($requestValue)) {
+                if (!$field->saveIfEmpty() && empty($requestValue)) {
                     continue;
                 }
 
-                if (! empty($requestValue) || (empty($requestValue) && ! empty($item->getAttribute($fieldName)))) {
+                if (!empty($requestValue) || (empty($requestValue) && !empty($item->getAttribute($fieldName)))) {
                     $item->setAttribute(
                         $fieldName,
                         $field->applyFunctions($requestValue)
@@ -446,7 +446,7 @@ 
         $this->setInstance($item);
 
 
-        if (! empty($relations = $this->getRelations())) {
+        if (!empty($relations = $this->getRelations())) {
             foreach ($relations as $relationKey => $relation) {
                 if ($relation instanceof Collection) {
                     $input = $request->input($relationKey);