We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | |
| 23 | 23 | public function getNotnull() |
| 24 | 24 | { |
| 25 | - return ! $this->column['nullable']; |
|
| 25 | + return !$this->column['nullable']; |
|
| 26 | 26 | } |
| 27 | 27 | |
| 28 | 28 | public function getDefault() |
@@ -14,8 +14,8 @@ discard block |
||
| 14 | 14 | { |
| 15 | 15 | $this->getDbColumnTypes(); |
| 16 | 16 | |
| 17 | - array_map(function ($field) use ($setFields, $setColumns) { |
|
| 18 | - if ($setFields && ! isset($this->getCleanStateFields()[$field])) { |
|
| 17 | + array_map(function($field) use ($setFields, $setColumns) { |
|
| 18 | + if ($setFields && !isset($this->getCleanStateFields()[$field])) { |
|
| 19 | 19 | $this->addField([ |
| 20 | 20 | 'name' => $field, |
| 21 | 21 | 'label' => $this->makeLabel($field), |
@@ -28,7 +28,7 @@ discard block |
||
| 28 | 28 | ]); |
| 29 | 29 | } |
| 30 | 30 | |
| 31 | - if ($setColumns && ! in_array($field, $this->model->getHidden()) && ! isset($this->columns()[$field])) { |
|
| 31 | + if ($setColumns && !in_array($field, $this->model->getHidden()) && !isset($this->columns()[$field])) { |
|
| 32 | 32 | $this->addColumn([ |
| 33 | 33 | 'name' => $field, |
| 34 | 34 | 'label' => $this->makeLabel($field), |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | { |
| 51 | 51 | $dbColumnTypes = []; |
| 52 | 52 | |
| 53 | - if (! $this->driverIsSql()) { |
|
| 53 | + if (!$this->driverIsSql()) { |
|
| 54 | 54 | return $dbColumnTypes; |
| 55 | 55 | } |
| 56 | 56 | $dbColumns = $this->getDbTableColumns(); |
@@ -115,7 +115,7 @@ discard block |
||
| 115 | 115 | |
| 116 | 116 | $dbColumnTypes = $this->getDbColumnTypes(); |
| 117 | 117 | |
| 118 | - if (! isset($dbColumnTypes[$fieldName])) { |
|
| 118 | + if (!isset($dbColumnTypes[$fieldName])) { |
|
| 119 | 119 | return 'text'; |
| 120 | 120 | } |
| 121 | 121 | |
@@ -210,12 +210,12 @@ discard block |
||
| 210 | 210 | { |
| 211 | 211 | $fillable = $this->model->getFillable(); |
| 212 | 212 | |
| 213 | - if (! $this->driverIsSql()) { |
|
| 213 | + if (!$this->driverIsSql()) { |
|
| 214 | 214 | $columns = $fillable; |
| 215 | 215 | } else { |
| 216 | 216 | // Automatically-set columns should be both in the database, and in the $fillable variable on the Eloquent Model |
| 217 | 217 | $columns = $this->model::getDbTableSchema()->getColumnsNames(); |
| 218 | - if (! empty($fillable)) { |
|
| 218 | + if (!empty($fillable)) { |
|
| 219 | 219 | $columns = array_intersect($columns, $fillable); |
| 220 | 220 | } |
| 221 | 221 | } |
@@ -848,7 +848,7 @@ |
||
| 848 | 848 | |
| 849 | 849 | public function testSetDoctrineTypesMapping() |
| 850 | 850 | { |
| 851 | - if (! method_exists($this->crudPanel->getModel()->getConnection(), 'getDoctrineConnection')) { |
|
| 851 | + if (!method_exists($this->crudPanel->getModel()->getConnection(), 'getDoctrineConnection')) { |
|
| 852 | 852 | $this->markTestSkipped('This test is only for Laravel 10, Laravel 11 does not have dbal as a dependency anymore'); |
| 853 | 853 | } |
| 854 | 854 | |
@@ -22,7 +22,7 @@ discard block |
||
| 22 | 22 | { |
| 23 | 23 | $connection = DB::connection($this->getConnectionName()); |
| 24 | 24 | |
| 25 | - if (! method_exists($connection, 'getDoctrineSchemaManager')) { |
|
| 25 | + if (!method_exists($connection, 'getDoctrineSchemaManager')) { |
|
| 26 | 26 | return $connection; |
| 27 | 27 | } |
| 28 | 28 | |
@@ -35,7 +35,7 @@ discard block |
||
| 35 | 35 | if (self::isSqlConnection()) { |
| 36 | 36 | $platform = $connection->getDoctrineSchemaManager()->getDatabasePlatform(); |
| 37 | 37 | foreach ($types as $type_key => $type_value) { |
| 38 | - if (! $platform->hasDoctrineTypeMappingFor($type_key)) { |
|
| 38 | + if (!$platform->hasDoctrineTypeMappingFor($type_key)) { |
|
| 39 | 39 | $platform->registerDoctrineTypeMapping($type_key, $type_value); |
| 40 | 40 | } |
| 41 | 41 | } |
@@ -65,7 +65,7 @@ discard block |
||
| 65 | 65 | */ |
| 66 | 66 | public function getColumnType($columnName) |
| 67 | 67 | { |
| 68 | - if (! self::isSqlConnection()) { |
|
| 68 | + if (!self::isSqlConnection()) { |
|
| 69 | 69 | return 'text'; |
| 70 | 70 | } |
| 71 | 71 | |
@@ -80,7 +80,7 @@ discard block |
||
| 80 | 80 | */ |
| 81 | 81 | public static function isColumnNullable($columnName) |
| 82 | 82 | { |
| 83 | - if (! self::isSqlConnection()) { |
|
| 83 | + if (!self::isSqlConnection()) { |
|
| 84 | 84 | return true; |
| 85 | 85 | } |
| 86 | 86 | |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | */ |
| 96 | 96 | public static function dbColumnHasDefault($columnName) |
| 97 | 97 | { |
| 98 | - if (! self::isSqlConnection()) { |
|
| 98 | + if (!self::isSqlConnection()) { |
|
| 99 | 99 | return false; |
| 100 | 100 | } |
| 101 | 101 | |
@@ -110,7 +110,7 @@ discard block |
||
| 110 | 110 | */ |
| 111 | 111 | public static function getDbColumnDefault($columnName) |
| 112 | 112 | { |
| 113 | - if (! self::isSqlConnection()) { |
|
| 113 | + if (!self::isSqlConnection()) { |
|
| 114 | 114 | return false; |
| 115 | 115 | } |
| 116 | 116 | |
@@ -20,7 +20,7 @@ discard block |
||
| 20 | 20 | public function getColumnsNames() |
| 21 | 21 | { |
| 22 | 22 | return array_values( |
| 23 | - array_map(function ($item) { |
|
| 23 | + array_map(function($item) { |
|
| 24 | 24 | return $item->getName(); |
| 25 | 25 | }, $this->getColumns()) |
| 26 | 26 | ); |
@@ -34,7 +34,7 @@ discard block |
||
| 34 | 34 | */ |
| 35 | 35 | public function getColumnType(string $columnName) |
| 36 | 36 | { |
| 37 | - if (! $this->schemaExists() || ! $this->schema->hasColumn($columnName)) { |
|
| 37 | + if (!$this->schemaExists() || !$this->schema->hasColumn($columnName)) { |
|
| 38 | 38 | return 'varchar'; |
| 39 | 39 | } |
| 40 | 40 | |
@@ -51,7 +51,7 @@ discard block |
||
| 51 | 51 | */ |
| 52 | 52 | public function hasColumn($columnName) |
| 53 | 53 | { |
| 54 | - if (! $this->schemaExists()) { |
|
| 54 | + if (!$this->schemaExists()) { |
|
| 55 | 55 | return false; |
| 56 | 56 | } |
| 57 | 57 | |
@@ -66,7 +66,7 @@ discard block |
||
| 66 | 66 | */ |
| 67 | 67 | public function columnIsNullable($columnName) |
| 68 | 68 | { |
| 69 | - if (! $this->hasColumn($columnName)) { |
|
| 69 | + if (!$this->hasColumn($columnName)) { |
|
| 70 | 70 | return true; |
| 71 | 71 | } |
| 72 | 72 | |
@@ -83,7 +83,7 @@ discard block |
||
| 83 | 83 | */ |
| 84 | 84 | public function columnHasDefault($columnName) |
| 85 | 85 | { |
| 86 | - if (! $this->hasColumn($columnName)) { |
|
| 86 | + if (!$this->hasColumn($columnName)) { |
|
| 87 | 87 | return false; |
| 88 | 88 | } |
| 89 | 89 | |
@@ -100,7 +100,7 @@ discard block |
||
| 100 | 100 | */ |
| 101 | 101 | public function getColumnDefault($columnName) |
| 102 | 102 | { |
| 103 | - if (! $this->hasColumn($columnName)) { |
|
| 103 | + if (!$this->hasColumn($columnName)) { |
|
| 104 | 104 | return false; |
| 105 | 105 | } |
| 106 | 106 | |
@@ -116,7 +116,7 @@ discard block |
||
| 116 | 116 | */ |
| 117 | 117 | public function getColumns() |
| 118 | 118 | { |
| 119 | - if (! $this->schemaExists()) { |
|
| 119 | + if (!$this->schemaExists()) { |
|
| 120 | 120 | return []; |
| 121 | 121 | } |
| 122 | 122 | |
@@ -130,7 +130,7 @@ discard block |
||
| 130 | 130 | */ |
| 131 | 131 | private function schemaExists() |
| 132 | 132 | { |
| 133 | - if (! empty($this->schema)) { |
|
| 133 | + if (!empty($this->schema)) { |
|
| 134 | 134 | return true; |
| 135 | 135 | } |
| 136 | 136 | |
@@ -10,7 +10,7 @@ discard block |
||
| 10 | 10 | |
| 11 | 11 | class MultipleFiles extends Uploader |
| 12 | 12 | { |
| 13 | - public static function for(array $field, $configuration): UploaderInterface |
|
| 13 | + public static function for (array $field, $configuration): UploaderInterface |
|
| 14 | 14 | { |
| 15 | 15 | return (new self($field, $configuration))->multiple(); |
| 16 | 16 | } |
@@ -29,7 +29,7 @@ discard block |
||
| 29 | 29 | $previousFiles = []; |
| 30 | 30 | } |
| 31 | 31 | |
| 32 | - if (! is_array($previousFiles) && is_string($previousFiles)) { |
|
| 32 | + if (!is_array($previousFiles) && is_string($previousFiles)) { |
|
| 33 | 33 | $previousFiles = json_decode($previousFiles, true); |
| 34 | 34 | } |
| 35 | 35 | |
@@ -38,14 +38,14 @@ discard block |
||
| 38 | 38 | if (in_array($previousFile, $filesToDelete)) { |
| 39 | 39 | Storage::disk($this->getDisk())->delete($previousFile); |
| 40 | 40 | |
| 41 | - $previousFiles = Arr::where($previousFiles, function ($value, $key) use ($previousFile) { |
|
| 41 | + $previousFiles = Arr::where($previousFiles, function($value, $key) use ($previousFile) { |
|
| 42 | 42 | return $value != $previousFile; |
| 43 | 43 | }); |
| 44 | 44 | } |
| 45 | 45 | } |
| 46 | 46 | } |
| 47 | 47 | |
| 48 | - if (! is_array($value)) { |
|
| 48 | + if (!is_array($value)) { |
|
| 49 | 49 | $value = []; |
| 50 | 50 | } |
| 51 | 51 | |
@@ -82,13 +82,13 @@ discard block |
||
| 82 | 82 | // create a temporary variable that we can unset keys |
| 83 | 83 | // everytime one is found. That way we avoid iterating |
| 84 | 84 | // already handled keys (notice we do a deep array copy) |
| 85 | - $tempFileOrder = array_map(function ($item) { |
|
| 85 | + $tempFileOrder = array_map(function($item) { |
|
| 86 | 86 | return $item; |
| 87 | 87 | }, $fileOrder); |
| 88 | 88 | |
| 89 | 89 | foreach ($previousRepeatableValues as $previousRow => $previousFiles) { |
| 90 | 90 | foreach ($previousFiles ?? [] as $key => $file) { |
| 91 | - $previousFileInArray = array_filter($tempFileOrder, function ($items, $key) use ($file, $tempFileOrder) { |
|
| 91 | + $previousFileInArray = array_filter($tempFileOrder, function($items, $key) use ($file, $tempFileOrder) { |
|
| 92 | 92 | $found = array_search($file, $items ?? [], true); |
| 93 | 93 | if ($found !== false) { |
| 94 | 94 | Arr::forget($tempFileOrder, $key.'.'.$found); |
@@ -98,7 +98,7 @@ discard block |
||
| 98 | 98 | |
| 99 | 99 | return false; |
| 100 | 100 | }, ARRAY_FILTER_USE_BOTH); |
| 101 | - if ($file && ! $previousFileInArray) { |
|
| 101 | + if ($file && !$previousFileInArray) { |
|
| 102 | 102 | Storage::disk($this->getDisk())->delete($file); |
| 103 | 103 | } |
| 104 | 104 | } |
@@ -48,7 +48,7 @@ |
||
| 48 | 48 | $app['config']->set('database.default', 'testing'); |
| 49 | 49 | $app['config']->set('backpack.base.route_prefix', 'admin'); |
| 50 | 50 | |
| 51 | - $app->bind('App\Http\Middleware\CheckIfAdmin', function () { |
|
| 51 | + $app->bind('App\Http\Middleware\CheckIfAdmin', function() { |
|
| 52 | 52 | return new class |
| 53 | 53 | { |
| 54 | 54 | public function handle($request, $next) |
@@ -21,8 +21,7 @@ discard block |
||
| 21 | 21 | } |
| 22 | 22 | |
| 23 | 23 | $previousValues = str_contains($attribute, '.') ? |
| 24 | - (Arr::get($entry?->{Str::before($attribute, '.')} ?? [], Str::after($attribute, '.')) ?? []) : |
|
| 25 | - ($entry?->{$attribute} ?? []); |
|
| 24 | + (Arr::get($entry?->{Str::before($attribute, '.')} ?? [], Str::after($attribute, '.')) ?? []) : ($entry?->{$attribute} ?? []); |
|
| 26 | 25 | |
| 27 | 26 | if (is_string($previousValues)) { |
| 28 | 27 | $previousValues = json_decode($previousValues, true) ?? []; |
@@ -39,7 +38,7 @@ discard block |
||
| 39 | 38 | |
| 40 | 39 | // if there is no entry, the values we are going to validate need to be files |
| 41 | 40 | // the request was tampered so we will set the attribute to null |
| 42 | - if (! $entry && ! empty(Arr::get($data, $attribute)) && ! $this->allFiles(Arr::get($data, $attribute))) { |
|
| 41 | + if (!$entry && !empty(Arr::get($data, $attribute)) && !$this->allFiles(Arr::get($data, $attribute))) { |
|
| 43 | 42 | Arr::set($data, $attribute, null); |
| 44 | 43 | } |
| 45 | 44 | |
@@ -50,7 +49,7 @@ discard block |
||
| 50 | 49 | private function allFiles(array $values): bool |
| 51 | 50 | { |
| 52 | 51 | foreach ($values as $value) { |
| 53 | - if (! $value instanceof \Illuminate\Http\UploadedFile) { |
|
| 52 | + if (!$value instanceof \Illuminate\Http\UploadedFile) { |
|
| 54 | 53 | return false; |
| 55 | 54 | } |
| 56 | 55 | } |
@@ -20,7 +20,7 @@ discard block |
||
| 20 | 20 | |
| 21 | 21 | // if the attribute is not set in the request, and an entry exists, |
| 22 | 22 | // we will check if there is a previous value, as this field might not have changed. |
| 23 | - if (! Arr::has($this->data, $attribute) && $entry) { |
|
| 23 | + if (!Arr::has($this->data, $attribute) && $entry) { |
|
| 24 | 24 | if (str_contains($attribute, '.') && get_class($entry) === get_class(CrudPanelFacade::getModel())) { |
| 25 | 25 | $previousValue = Arr::get($this->data, '_order_'.Str::before($attribute, '.')); |
| 26 | 26 | $previousValue = Arr::get($previousValue, Str::after($attribute, '.')); |
@@ -37,25 +37,25 @@ discard block |
||
| 37 | 37 | |
| 38 | 38 | // if the value is an uploaded file, or the attribute is not |
| 39 | 39 | // set in the request, we force fill the data with the value |
| 40 | - if ($value instanceof UploadedFile || ! Arr::has($this->data, $attribute)) { |
|
| 40 | + if ($value instanceof UploadedFile || !Arr::has($this->data, $attribute)) { |
|
| 41 | 41 | Arr::set($this->data, $attribute, $value); |
| 42 | 42 | } |
| 43 | 43 | |
| 44 | 44 | // if there are no entry, and the new value it's not a file ... well we don't want it at all. |
| 45 | - if (! $entry && ! $value instanceof UploadedFile) { |
|
| 45 | + if (!$entry && !$value instanceof UploadedFile) { |
|
| 46 | 46 | Arr::set($this->data, $attribute, null); |
| 47 | 47 | } |
| 48 | 48 | |
| 49 | 49 | $fieldErrors = $this->validateFieldRules($attribute); |
| 50 | 50 | |
| 51 | - if (! empty($value) && ! empty($this->getFileRules())) { |
|
| 51 | + if (!empty($value) && !empty($this->getFileRules())) { |
|
| 52 | 52 | $fileErrors = $this->validateFileRules($attribute, $value); |
| 53 | 53 | } |
| 54 | 54 | |
| 55 | 55 | return array_merge($fieldErrors, $fileErrors ?? []); |
| 56 | 56 | } |
| 57 | 57 | |
| 58 | - public static function field(string|array|ValidationRule|Rule $rules = []): self |
|
| 58 | + public static function field(string | array | ValidationRule | Rule $rules = []): self |
|
| 59 | 59 | { |
| 60 | 60 | return parent::field($rules); |
| 61 | 61 | } |