| @@ -9,7 +9,9 @@ discard block | ||
| 9 | 9 |  { | 
| 10 | 10 | public function canPreviewAssistant(string $action, $model = null): bool | 
| 11 | 11 |      { | 
| 12 | - if($action !== 'preview' || !$model) return false; | |
| 12 | +        if($action !== 'preview' || !$model) { | |
| 13 | + return false; | |
| 14 | + } | |
| 13 | 15 | |
| 14 | 16 | return $model instanceof ProvidesUrl; | 
| 15 | 17 | } | 
| @@ -27,7 +29,9 @@ discard block | ||
| 27 | 29 | */ | 
| 28 | 30 | public function routePreviewAssistant(string $action, $model = null, ...$parameters): ?string | 
| 29 | 31 |      { | 
| 30 | - if(!$this->canPreviewAssistant($action, $model)) return null; | |
| 32 | +        if(!$this->canPreviewAssistant($action, $model)) { | |
| 33 | + return null; | |
| 34 | + } | |
| 31 | 35 | |
| 32 | 36 | return $model->url(); | 
| 33 | 37 | } | 
| @@ -45,7 +45,9 @@ | ||
| 45 | 45 | // Dynamic localized values or standard translated | 
| 46 | 46 | // For standard translations we set value with the colon notation, e.g. title:en | 
| 47 | 47 |              $this->foreachTranslationEntry(data_get($input, 'trans', []), function($locale, $key, $value) use($field){ | 
| 48 | - if($key !== $field->getColumn()) return; | |
| 48 | +                if($key !== $field->getColumn()) { | |
| 49 | + return; | |
| 50 | + } | |
| 49 | 51 | |
| 50 | 52 |                  if($this->isFieldForDynamicValue($field)) { | 
| 51 | 53 | $this->setDynamic($key, $value, $locale); | 
| @@ -31,7 +31,9 @@ | ||
| 31 | 31 | |
| 32 | 32 | public function routeModuleCrudAssistant(string $action, $model = null, ...$parameters): ?string | 
| 33 | 33 |      { | 
| 34 | - if(!in_array($action, ['create','store'])) return null; | |
| 34 | +        if(!in_array($action, ['create','store'])) { | |
| 35 | + return null; | |
| 36 | + } | |
| 35 | 37 | |
| 36 | 38 | return $this->generateRoute($action, $model, ... $parameters); | 
| 37 | 39 | } | 
| @@ -46,7 +46,9 @@ | ||
| 46 | 46 | |
| 47 | 47 | public function anyRenderable(): bool | 
| 48 | 48 |      { | 
| 49 | - if($this->isEmpty()) return false; | |
| 49 | +        if($this->isEmpty()) { | |
| 50 | + return false; | |
| 51 | + } | |
| 50 | 52 | |
| 51 | 53 | // If at least one of the filters has content to be rendered. | 
| 52 | 54 |          foreach($this->filters as $filter) { | 
| @@ -10,14 +10,22 @@ | ||
| 10 | 10 |  { | 
| 11 | 11 | public static function find(array $segments, string $locale = null) | 
| 12 | 12 |      { | 
| 13 | - if(count($segments) < 1) return '/'; | |
| 14 | - if(count($segments) == 1) return reset($segments); | |
| 13 | +        if(count($segments) < 1) { | |
| 14 | + return '/'; | |
| 15 | + } | |
| 16 | +        if(count($segments) == 1) { | |
| 17 | + return reset($segments); | |
| 18 | + } | |
| 15 | 19 | |
| 16 | 20 | // Localized value | 
| 17 | - if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) return $segments[$key]; | |
| 21 | +        if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) { | |
| 22 | + return $segments[$key]; | |
| 23 | + } | |
| 18 | 24 | |
| 19 | 25 | // Fallback localized value | 
| 20 | -        if (($fallback_locale = config('app.fallback_locale')) && isset($segments[$fallback_locale])) return $segments[$fallback_locale]; | |
| 26 | +        if (($fallback_locale = config('app.fallback_locale')) && isset($segments[$fallback_locale])) { | |
| 27 | + return $segments[$fallback_locale]; | |
| 28 | + } | |
| 21 | 29 | |
| 22 | 30 | // Fallback to first entry | 
| 23 | 31 | return reset($segments); | 
| @@ -25,7 +25,9 @@ | ||
| 25 | 25 | |
| 26 | 26 | public function routeFragmentsAssistant(string $action, $model = null, ...$parameters): ?string | 
| 27 | 27 |      { | 
| 28 | - if(!in_array($action, ['fragments-index','fragments-reorder'])) return null; | |
| 28 | +        if(!in_array($action, ['fragments-index','fragments-reorder'])) { | |
| 29 | + return null; | |
| 30 | + } | |
| 29 | 31 | |
| 30 | 32 | return $this->generateRoute($action, $model, ...$parameters); | 
| 31 | 33 | } | 
| @@ -26,7 +26,9 @@ | ||
| 26 | 26 | */ | 
| 27 | 27 | public function getByOwner(FragmentsOwner $owner): Collection | 
| 28 | 28 |      { | 
| 29 | - if(!$context = ContextModel::ownedBy($owner)) return collect(); | |
| 29 | +        if(!$context = ContextModel::ownedBy($owner)) { | |
| 30 | + return collect(); | |
| 31 | + } | |
| 30 | 32 | |
| 31 | 33 | $fragmentModels = $context->fragments()->get(); | 
| 32 | 34 | |
| @@ -107,7 +107,9 @@ | ||
| 107 | 107 | // Astrotomic translatable | 
| 108 | 108 |          if($model instanceof Translatable) { | 
| 109 | 109 |              foreach (config('chief.locales') as $locale) { | 
| 110 | - if(!isset($trans[$locale])) $trans[$locale] = []; | |
| 110 | +                if(!isset($trans[$locale])) { | |
| 111 | + $trans[$locale] = []; | |
| 112 | + } | |
| 111 | 113 | |
| 112 | 114 | $trans[$locale] = array_merge($trans[$locale], $model->getTranslation($locale)->toArray()); | 
| 113 | 115 | } | 
| @@ -63,7 +63,9 @@ | ||
| 63 | 63 | |
| 64 | 64 | private function onlineStatusAsLabel(): string | 
| 65 | 65 |      { | 
| 66 | - if(!$this instanceof StatefulContract) return ''; | |
| 66 | +        if(!$this instanceof StatefulContract) { | |
| 67 | + return ''; | |
| 68 | + } | |
| 67 | 69 | |
| 68 | 70 |          if ($this->stateOf(PageState::KEY) === PageState::PUBLISHED) { | 
| 69 | 71 | return $this instanceof ProvidesUrl |