View GitHub Repository chief

Schedule Inspection
Subscribe
Passed
Push — ft/fragments ( b30041...2bbc06 )
by Ben
07:30
created
Status
Category
Toggle Patches
src/Fragments/FragmentField.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -63,7 +63,9 @@  discard block
 
 block discarded – undo
63 63 
                                  ->localizedFormat(':name.:locale')
64 64 
                                  ->valueResolver(function($model = null, $locale = null, $field) use($fragment){
65 65
66 
-                                     if(isset($field->value)) return $field->value;
66 
+                                     if(isset($field->value)) {
67 
+                                         return $field->value;
68 
+                                     }
67 69
68 70 
                                      if($field instanceof MediaField){
69 71 
                                          if(!$fragment->hasModelId()){
 
@@ -83,7 +85,9 @@  discard block
 
 block discarded – undo
83 85
84 86 
     public function getDuplicatableFields(): array
85 87 
     {
86 
-        if(count($this->getFragments()) < 1) return [];
88 
+        if(count($this->getFragments()) < 1) {
89 
+            return [];
90 
+        }
87 91
88 92 
         // Take the fields from the first fragment as a starting point for duplication
89 93 
         return array_map(function(\Thinktomorrow\Chief\Fields\Types\Field $field){
Please login to merge, or discard this patch.