fix: mediaPicker and contentPicker have error while getValue

Author: cklei-carly

src/Fields/Configs/ContentPicker.php

@@ -9,6 +9,8 @@
 use SolutionForest\FilamentFieldGroup\FieldTypes\Configs\Attributes\FormComponent;
 use SolutionForest\FilamentFieldGroup\FieldTypes\Configs\Contracts\FieldTypeConfig;
 use SolutionForest\FilamentFieldGroup\FieldTypes\Configs\FieldTypeBaseConfig;
+use SolutionForest\InspireCms\Fields\Configs\Attributes\Converter;
+use SolutionForest\InspireCms\Fields\Converters\ContentPickerConverter;
 use SolutionForest\InspireCms\Filament\Forms\Components\ContentPicker as ContentPickerComponent;
 use SolutionForest\InspireCms\Helpers\UIHelper;
 use SolutionForest\InspireCms\InspireCmsConfig;
@@ -18,6 +20,7 @@
 #[FormComponent(ContentPickerComponent::class)]
 #[DbType('mysql', 'varchar')]
 #[DbType('sqlite', 'text')]
+#[Converter(ContentPickerConverter::class)]
 class ContentPicker extends FieldTypeBaseConfig implements FieldTypeConfig
 {
     public ?int $max = null;

src/Fields/Converters/MediaPickerConverter.php

@@ -13,6 +13,10 @@ public function toDisplayValue(mixed $sourceValue, ?string $locale, ?string $fal
         // todo: improve performance
         $result = [];
 
+        if (is_null($sourceValue)) {
+            return $result;
+        }
+
         $keysToFind = collect($sourceValue)->where(fn ($v) => is_string($v));
         $mediaAssets = $keysToFind->isNotEmpty()
             ? inspirecms_asset()->findByKeys($keysToFind->all())->mapWithKeys(fn (Model $record) => [$record->getKey() => $record])

src/Fields/PropertyValueTransformer.php

@@ -14,7 +14,7 @@ class PropertyValueTransformer implements PropertyValueTransformerInterface
 {
     public function transform(PropertyDataDto $propertyDataDto, ?string $locale, ?string $fallbackLocale)
     {
-        $converter = $this->getConverter($propertyDataDto->propertyType?->config);
+        $converter = $this->getConverter($propertyDataDto?->propertyType?->config);
 
         return $converter->toDisplayValue($propertyDataDto->getSourceValue(), $locale, $fallbackLocale);
     }
@@ -24,6 +24,12 @@ public function attemptTransform(PropertyDataDto $propertyDataDto, ?string $loca
         try {
             return $this->transform($propertyDataDto, $locale, $fallbackLocale);
         } catch (\Exception $e) {
+            
+            logger()->warning('Failed to transform property data', [
+                'exception' => $e,
+                'propertyData' => $propertyDataDto,
+            ]);
+
             return null;
         }
     }

stubs/SampleTemplates/manifest/Blogs.stub

@@ -1,4 +1,6 @@
 @php
+    use SolutionForest\InspireCms\Dtos\ContentDto;
+
     $blogPage = request()->query('page') ?? 1;
     $featuredPage = request()->query('featured') ?? 1;
 
@@ -17,13 +19,14 @@
                 </div>
             </section>
             <div class="container ps-0">
-                @propertyArray('featured_blogs', 'blogs')
-                @php
-                    $featuredBlog = collect($featured_blogs_blogs)->skip($featuredPage - 1)->first();
-                    $featuredBlogTemplate = $featuredBlog?->getTemplate('blog-featured-item');
-                @endphp
-                @if ($featuredBlogTemplate)
-                    {!! $featuredBlogTemplate->render(['content' => $featuredBlog, 'locale' => $locale]) !!}
+                @propertyNotEmpty('featured_blogs', 'blogs')
+                    @php
+                        $featuredBlog = collect($featured_blogs_blogs)->skip($featuredPage - 1)->first();
+                        $featuredBlogTemplate = $featuredBlog instanceof ContentDto ? $featuredBlog?->getTemplate('blog-featured-item') : null;
+                    @endphp
+                    @if ($featuredBlogTemplate)
+                        {!! $featuredBlogTemplate->render(['content' => $featuredBlog, 'locale' => $locale]) !!}
+                    @endif
                 @endif
 
                 <div class="categories-wrapper">