REFACTOR Augment Slide models

src/Model/ImageSlide.php

@@ -4,8 +4,9 @@
 
 use SilverStripe\Assets\Image;
 use SilverStripe\Forms\FieldList;
-use SilverStripe\LinkField\Form\LinkField;
 use SilverStripe\LinkField\Models\Link;
+use SilverStripe\LinkField\Form\LinkField;
+use SilverStripe\AssetAdmin\Forms\UploadField;
 
 /**
  * Class \Dynamic\Carousel\Model\ImageSlide
@@ -40,6 +41,7 @@ class ImageSlide extends Slide
      */
     private static $has_one = [
         'Image' => Image::class,
+        'ImageMobile' => Image::class,
         'ElementLink' => Link::class,
     ];
 
@@ -49,6 +51,7 @@ class ImageSlide extends Slide
      */
     private static $owns = [
         'Image',
+        'ImageMobile',
         'ElementLink',
     ];
 
@@ -82,6 +85,7 @@ public function getCMSFields(): FieldList
                 $link = LinkField::create('ElementLink')
                     ->setTitle($this->fieldLabel('ElementLink'))
             );
+            $link->setDescription('Optional. Leave blank to hide.');
 
             $fields->addFieldsToTab(
                 'Root.Main',
@@ -93,6 +97,21 @@ public function getCMSFields(): FieldList
                 ],
                 'Content'
             );
+
+            // Image for small screens
+            $fields->insertAfter(
+                'Image',
+                UploadField::create('ImageMobile', 'Image (for small screens)')
+                ->setFolderName('Uploads/Carousel/Slides')
+                ->setDescription('Optional.')
+            );
+
+            if (
+                $this->getOwner()->hasField('ParentID') &&
+                $this->getOwner()->Parent() instanceof \SilverStripe\CMS\Model\SiteTree
+            ) {
+                $fields->dataFieldByName('Image')->setDescription('Recommended size: 1920x823');
+            }
         });
 
         return parent::getCMSFields();

src/Model/Slide.php

@@ -133,6 +133,16 @@ public function getCMSFields(): FieldList
         return parent::getCMSFields();
     }
 
+    /**
+     * ShowCaption
+     */
+    public function ShowCaption(): bool
+    {
+        $owner = $this->getOwner();
+
+        return (($owner->Title && $owner->ShowTitle) || $owner->SubTitle || $owner->Content);
+    }
+
     /**
      * Basic permissions, defaults to parent perms where possible.
      *

src/Model/VideoSlide.php

@@ -4,6 +4,10 @@
 
 use SilverStripe\Assets\File;
 use SilverStripe\Forms\FieldList;
+use nathancox\EmbedField\Forms\EmbedField;
+use nathancox\EmbedField\Model\EmbedObject;
+use UncleCheese\DisplayLogic\Forms\Wrapper;
+use SilverStripe\AssetAdmin\Forms\UploadField;
 
 /**
  * Class \Dynamic\Carousel\Model\VideoSlide
@@ -41,6 +45,7 @@ class VideoSlide extends Slide
      */
     private static $has_one = [
         'Video' => File::class,
+        'EmbedVideo' => EmbedObject::class,
     ];
 
     /**
@@ -56,7 +61,33 @@ class VideoSlide extends Slide
     public function getCMSFields(): FieldList
     {
         $this->beforeUpdateCMSFields(function (FieldList $fields) {
-            //$fields->
+            // Remvove fields
+            $fields->removeByName([
+                'EmbedVideoID',
+                'Video',
+            ]);
+
+            // Native video
+            $uploadVideo = UploadField::create('Video', 'Upload video')
+            ->setFolderName('Uploads/Carousel/Videos');
+
+            $fields->insertAfter(
+                'VideoType',
+                $uploadVideo
+            );
+
+            // Embed video
+            // @phpstan-ignore class.notFound
+            $embedVideo = Wrapper::create(EmbedField::create('EmbedVideoID', 'Embed video'));
+
+            $fields->insertAfter(
+                'VideoType',
+                $embedVideo
+            );
+
+            //
+            $uploadVideo->displayIf("VideoType")->isEqualTo("Native");
+            $embedVideo->displayIf("VideoType")->isEqualTo("Embed");
         });
 
         return parent::getCMSFields();