@@ -112,12 +112,12 @@ |
||
| 112 | 112 | $builder->where('locale', $locale); |
| 113 | 113 | } |
| 114 | 114 | |
| 115 | - if (! $includeRedirects) { |
|
| 115 | + if (!$includeRedirects) { |
|
| 116 | 116 | $builder->whereNull('redirect_id'); |
| 117 | 117 | } |
| 118 | 118 | |
| 119 | 119 | if ($ignoredModel) { |
| 120 | - $builder->whereNotIn('id', function ($query) use ($ignoredModel) { |
|
| 120 | + $builder->whereNotIn('id', function($query) use ($ignoredModel) { |
|
| 121 | 121 | $query->select('id') |
| 122 | 122 | ->from('chief_urls') |
| 123 | 123 | ->where('model_type', '=', $ignoredModel->getMorphClass()) |
@@ -65,8 +65,8 @@ discard block |
||
| 65 | 65 | public static function getByModel(Model $model) |
| 66 | 66 | { |
| 67 | 67 | return static::where('model_type', $model->getMorphClass()) |
| 68 | - ->where('model_id', $model->id) |
|
| 69 | - ->get(); |
|
| 68 | + ->where('model_id', $model->id) |
|
| 69 | + ->get(); |
|
| 70 | 70 | } |
| 71 | 71 | |
| 72 | 72 | public static function findRecentRedirect(Model $model, string $locale): ?self |
@@ -134,9 +134,9 @@ discard block |
||
| 134 | 134 | if ($ignoredModel) { |
| 135 | 135 | $builder->whereNotIn('id', function ($query) use ($ignoredModel) { |
| 136 | 136 | $query->select('id') |
| 137 | - ->from('chief_urls') |
|
| 138 | - ->where('model_type', '=', $ignoredModel->getMorphClass()) |
|
| 139 | - ->where('model_id', '=', $ignoredModel->id); |
|
| 137 | + ->from('chief_urls') |
|
| 138 | + ->where('model_type', '=', $ignoredModel->getMorphClass()) |
|
| 139 | + ->where('model_id', '=', $ignoredModel->id); |
|
| 140 | 140 | }); |
| 141 | 141 | } |
| 142 | 142 | |
@@ -8,14 +8,14 @@ |
||
| 8 | 8 | { |
| 9 | 9 | public static function findByModel(Model $model, string $locale = null): UrlRecord |
| 10 | 10 | { |
| 11 | - return chiefMemoize('url-records-find-by-model', function ($model, $locale = null) { |
|
| 11 | + return chiefMemoize('url-records-find-by-model', function($model, $locale = null) { |
|
| 12 | 12 | return parent::findByModel($model, $locale); |
| 13 | 13 | }, [$model, $locale]); |
| 14 | 14 | } |
| 15 | 15 | |
| 16 | 16 | public static function getByModel(Model $model) |
| 17 | 17 | { |
| 18 | - return chiefMemoize('url-records-get-by-model', function ($model) { |
|
| 18 | + return chiefMemoize('url-records-get-by-model', function($model) { |
|
| 19 | 19 | return parent::getByModel($model); |
| 20 | 20 | }, [$model]); |
| 21 | 21 | } |
@@ -36,7 +36,7 @@ discard block |
||
| 36 | 36 | |
| 37 | 37 | // If no view has been created for this model, we try once again to fetch the content value if any. This will silently fail |
| 38 | 38 | // if no content value is present. We consider the 'content' attribute to be a default for our copy. |
| 39 | - return isset($this->content) ? (string) $this->content : ''; |
|
| 39 | + return isset($this->content) ? (string)$this->content : ''; |
|
| 40 | 40 | } |
| 41 | 41 | |
| 42 | 42 | public function setViewParent(ActsAsParent $parent): ViewableContract |
@@ -64,7 +64,7 @@ discard block |
||
| 64 | 64 | } |
| 65 | 65 | |
| 66 | 66 | if (config('thinktomorrow.chief.strict')) { |
| 67 | - throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to ' . get_class($this)); |
|
| 67 | + throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to '.get_class($this)); |
|
| 68 | 68 | }; |
| 69 | 69 | |
| 70 | 70 | return ''; |
@@ -17,7 +17,7 @@ discard block |
||
| 17 | 17 | */ |
| 18 | 18 | public function isAssistedBy(string $assistant): bool |
| 19 | 19 | { |
| 20 | - return !! $this->getAssistantClass($assistant); |
|
| 20 | + return !!$this->getAssistantClass($assistant); |
|
| 21 | 21 | } |
| 22 | 22 | |
| 23 | 23 | public function assistants(): array |
@@ -40,8 +40,8 @@ discard block |
||
| 40 | 40 | */ |
| 41 | 41 | public function assistant(string $assistant): Assistant |
| 42 | 42 | { |
| 43 | - if (! $this->isAssistedBy($assistant)) { |
|
| 44 | - throw new MissingAssistant('No assistant [' . $assistant . '] present on manager ' . get_class($this)); |
|
| 43 | + if (!$this->isAssistedBy($assistant)) { |
|
| 44 | + throw new MissingAssistant('No assistant ['.$assistant.'] present on manager '.get_class($this)); |
|
| 45 | 45 | } |
| 46 | 46 | |
| 47 | 47 | $instance = app($this->getAssistantClass($assistant)); |
@@ -83,12 +83,12 @@ |
||
| 83 | 83 | } |
| 84 | 84 | |
| 85 | 85 | $manager = new \ReflectionClass($managerClass); |
| 86 | - if (! $manager->implementsInterface(Manager::class)) { |
|
| 86 | + if (!$manager->implementsInterface(Manager::class)) { |
|
| 87 | 87 | throw new \InvalidArgumentException('Class ['.$managerClass.'] is expected to implement the ['.Manager::class.'] contract.'); |
| 88 | 88 | } |
| 89 | 89 | |
| 90 | 90 | $model = new \ReflectionClass($modelClass); |
| 91 | - if (! $model->implementsInterface(ManagedModel::class)) { |
|
| 91 | + if (!$model->implementsInterface(ManagedModel::class)) { |
|
| 92 | 92 | throw new \InvalidArgumentException('Class ['.$modelClass.'] is expected to implement the ['.ManagedModel::class.'] contract.'); |
| 93 | 93 | } |
| 94 | 94 | } |
@@ -123,8 +123,8 @@ discard block |
||
| 123 | 123 | |
| 124 | 124 | public function offsetSet($offset, $value) |
| 125 | 125 | { |
| 126 | - if (! $value instanceof Field) { |
|
| 127 | - throw new \InvalidArgumentException('Passed value must be of type ' . Field::class); |
|
| 126 | + if (!$value instanceof Field) { |
|
| 127 | + throw new \InvalidArgumentException('Passed value must be of type '.Field::class); |
|
| 128 | 128 | } |
| 129 | 129 | |
| 130 | 130 | $this->fields[$offset] = $value; |
@@ -153,7 +153,7 @@ discard block |
||
| 153 | 153 | |
| 154 | 154 | private function validateFields(array $fields) |
| 155 | 155 | { |
| 156 | - array_map(function (Field $field) { |
|
| 156 | + array_map(function(Field $field) { |
|
| 157 | 157 | }, $fields); |
| 158 | 158 | } |
| 159 | 159 | |
@@ -63,15 +63,15 @@ discard block |
||
| 63 | 63 | { |
| 64 | 64 | $permission = 'update-page'; |
| 65 | 65 | |
| 66 | - if (in_array($verb, ['index','show'])) { |
|
| 66 | + if (in_array($verb, ['index', 'show'])) { |
|
| 67 | 67 | $permission = 'view-page'; |
| 68 | - } elseif (in_array($verb, ['create','store'])) { |
|
| 68 | + } elseif (in_array($verb, ['create', 'store'])) { |
|
| 69 | 69 | $permission = 'create-page'; |
| 70 | 70 | } elseif (in_array($verb, ['delete'])) { |
| 71 | 71 | $permission = 'delete-page'; |
| 72 | 72 | } |
| 73 | 73 | |
| 74 | - if (! auth()->guard('chief')->user()->hasPermissionTo($permission)) { |
|
| 74 | + if (!auth()->guard('chief')->user()->hasPermissionTo($permission)) { |
|
| 75 | 75 | throw NotAllowedManagerRoute::notAllowedPermission($permission, $this); |
| 76 | 76 | } |
| 77 | 77 | } |
@@ -103,7 +103,7 @@ discard block |
||
| 103 | 103 | public function saveFields(Request $request) |
| 104 | 104 | { |
| 105 | 105 | // Store the morph_key upon creation |
| 106 | - if (! $this->model->morph_key) { |
|
| 106 | + if (!$this->model->morph_key) { |
|
| 107 | 107 | $this->model->morph_key = $this->model->morphKey(); |
| 108 | 108 | } |
| 109 | 109 | |
@@ -142,7 +142,7 @@ discard block |
||
| 142 | 142 | if (is_array_empty($translation)) { |
| 143 | 143 | |
| 144 | 144 | // Nullify all values |
| 145 | - $trans[$locale] = array_map(function ($value) { |
|
| 145 | + $trans[$locale] = array_map(function($value) { |
|
| 146 | 146 | return null; |
| 147 | 147 | }, $translation); |
| 148 | 148 | continue; |
@@ -26,7 +26,7 @@ discard block |
||
| 26 | 26 | |
| 27 | 27 | // Media fields are treated separately |
| 28 | 28 | if ($field->ofType(FieldType::MEDIA, FieldType::DOCUMENT)) { |
| 29 | - if (! isset($this->saveMethods['_files'])) { |
|
| 29 | + if (!isset($this->saveMethods['_files'])) { |
|
| 30 | 30 | $this->saveMethods['_files'] = ['field' => new Fields([$field]), 'method' => 'uploadMedia']; |
| 31 | 31 | continue; |
| 32 | 32 | } |
@@ -36,7 +36,7 @@ discard block |
||
| 36 | 36 | } |
| 37 | 37 | |
| 38 | 38 | // Custom set methods - default is the generic setField() method. |
| 39 | - $methodName = 'set'. ucfirst(Str::camel($field->key())) . 'Field'; |
|
| 39 | + $methodName = 'set'.ucfirst(Str::camel($field->key())).'Field'; |
|
| 40 | 40 | (method_exists($this, $methodName)) |
| 41 | 41 | ? $this->$methodName($field, $request) |
| 42 | 42 | : $this->setField($field, $request); |
@@ -53,7 +53,7 @@ discard block |
||
| 53 | 53 | |
| 54 | 54 | protected function detectCustomSaveMethods(Field $field): bool |
| 55 | 55 | { |
| 56 | - $saveMethodName = 'save'. ucfirst(Str::camel($field->key())) . 'Field'; |
|
| 56 | + $saveMethodName = 'save'.ucfirst(Str::camel($field->key())).'Field'; |
|
| 57 | 57 | |
| 58 | 58 | // Custom save method on assistant |
| 59 | 59 | foreach ($this->assistants() as $assistant) { |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | |
| 96 | 96 | // Make our media fields able to be translatable as well... |
| 97 | 97 | if ($field->ofType(FieldType::MEDIA, FieldType::DOCUMENT)) { |
| 98 | - throw new \Exception('Cannot process the ' . $field->key . ' media field. Currently no support for translatable media files. We should fix this!'); |
|
| 98 | + throw new \Exception('Cannot process the '.$field->key.' media field. Currently no support for translatable media files. We should fix this!'); |
|
| 99 | 99 | } |
| 100 | 100 | |
| 101 | 101 | // Okay so this is a bit odd but since all translations are expected to be inside the trans |
@@ -13,7 +13,7 @@ |
||
| 13 | 13 | |
| 14 | 14 | public function register() |
| 15 | 15 | { |
| 16 | - $this->app->singleton(Settings::class, function ($app) { |
|
| 16 | + $this->app->singleton(Settings::class, function($app) { |
|
| 17 | 17 | return new Settings(); |
| 18 | 18 | }); |
| 19 | 19 | } |