@@ -46,7 +46,7 @@ discard block |
||
46 | 46 | |
47 | 47 | // If no view has been created for this page collection, we try once again to fetch the content value if any. This will silently fail |
48 | 48 | // if no content value is present. We don't consider the 'content' attribute to be a default as we do for module. |
49 | - return $this->map(function ($item) { |
|
49 | + return $this->map(function($item) { |
|
50 | 50 | return ($this->viewParent && $item instanceof ViewableContract) |
51 | 51 | ? $item->setViewParent($this->viewParent)->renderView() |
52 | 52 | : ($item->content ?? ''); |
@@ -79,7 +79,7 @@ discard block |
||
79 | 79 | { |
80 | 80 | $currentPage = $currentPage ?? request()->get('page', 1); |
81 | 81 | $path = request()->path(); |
82 | - $items = array_slice($this->all(), ($currentPage - 1) * $perPage); |
|
82 | + $items = array_slice($this->all(), ($currentPage-1) * $perPage); |
|
83 | 83 | |
84 | 84 | return (new \Illuminate\Pagination\Paginator($items, $perPage, $currentPage))->setPath($path); |
85 | 85 | } |
@@ -96,7 +96,7 @@ discard block |
||
96 | 96 | { |
97 | 97 | $currentPage = $currentPage ?? request()->get('page', 1); |
98 | 98 | $path = request()->path(); |
99 | - $items = array_slice($this->all(), ($currentPage - 1) * $perPage, $perPage); |
|
99 | + $items = array_slice($this->all(), ($currentPage-1) * $perPage, $perPage); |
|
100 | 100 | |
101 | 101 | return (new \Illuminate\Pagination\LengthAwarePaginator($items, $this->count(), $perPage, $currentPage))->setPath($path); |
102 | 102 | } |
@@ -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 | } |
@@ -39,11 +39,11 @@ discard block |
||
39 | 39 | return static::createRedirect($model->url($locale)); |
40 | 40 | } |
41 | 41 | |
42 | - if (method_exists($model, 'isPublished') && ! $model->isPublished()) { |
|
42 | + if (method_exists($model, 'isPublished') && !$model->isPublished()) { |
|
43 | 43 | |
44 | 44 | /** When admin is logged in and this request is in preview mode, we allow the view */ |
45 | - if (! PreviewMode::fromRequest()->check()) { |
|
46 | - throw new NotFoundHttpException('Model found for request ['. $slug .'] but it is not published.'); |
|
45 | + if (!PreviewMode::fromRequest()->check()) { |
|
46 | + throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.'); |
|
47 | 47 | } |
48 | 48 | } |
49 | 49 | |
@@ -54,7 +54,7 @@ discard block |
||
54 | 54 | } |
55 | 55 | } |
56 | 56 | |
57 | - throw new NotFoundHttpException('No url or model found for request ['. $slug .'] for locale ['.$locale.'].'); |
|
57 | + throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].'); |
|
58 | 58 | } |
59 | 59 | |
60 | 60 | private static function createRedirect(string $url) |
@@ -1,6 +1,6 @@ discard block |
||
1 | 1 | <?php |
2 | 2 | |
3 | -declare(strict_types = 1); |
|
3 | +declare(strict_types=1); |
|
4 | 4 | |
5 | 5 | namespace Thinktomorrow\Chief\Fields\Types; |
6 | 6 | |
@@ -142,7 +142,7 @@ discard block |
||
142 | 142 | } |
143 | 143 | |
144 | 144 | if (!in_array($name, ['label', 'key', 'description', 'column', 'name', 'prepend', 'append'])) { |
145 | - throw new \InvalidArgumentException('Cannot set value by ['. $name .'].'); |
|
145 | + throw new \InvalidArgumentException('Cannot set value by ['.$name.'].'); |
|
146 | 146 | } |
147 | 147 | |
148 | 148 | $this->values[$name] = $arguments[0]; |
@@ -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 |