dynamic / silverstripe-flexslider

src/Model/SlideImage.php

Indentation +305 added lines, -305 removed lines

@@ -37,315 +37,315 @@
  */
 class SlideImage extends DataObject implements PermissionProvider
 {
-    /**
-     * @var string
-     */
-    private static $singular_name = 'Slide';
-
-    /**
-     * @var string
-     */
-    private static $plural_name = 'Slides';
-
-    /**
-     * @var array
-     */
-    private static $db = [
-        'Name' => 'Varchar(255)',
-        'Headline' => 'Varchar(255)',
-        'Description' => 'Text',
-        'SortOrder' => 'Int',
-        'SlideType' => 'Varchar',
-    ];
-
-    /**
-     * @var array
-     */
-    private static $has_one = [
-        'Image' => Image::class,
-        //'Video' => Embed::class,
-        'Page' => \Page::class,
-        'PageLink' => SiteTree::class,
-        'SlideLink' => Link::class,
-    ];
-
-    /**
-     * @var array
-     */
-    private static $owns = [
-        'Image',
-    ];
-
-    /**
-     * @var string
-     */
-    private static $table_name = 'SlideImage';
-
-    /**
-     * @var string
-     */
-    private static $default_sort = 'SortOrder';
-
-    /**
-     * Adds Publish button to SlideImage record
-     *
-     * @var bool
-     */
-    private static $versioned_gridfield_extensions = true;
-
-    /**
-     * @var array
-     */
-    private static $defaults = [
-        'SlideType' => 'Image',
-    ];
-
-    /**
-     * @var array
-     */
-    private static $summary_fields = [
-        'Image.CMSThumbnail' => 'Image',
-        'Name' => 'Name',
-    ];
-
-    /**
-     * @var array
-     */
-    private static $searchable_fields = [
-        'Name',
-        'Headline',
-        'Description',
-    ];
-
-    /**
-     * @var int
-     */
-    private static $image_size_limit = 512000;
-
-    /**
-     * @var array
-     */
-    private static $slide_types = [
-        'Image',
-        'Video',
-        'Text',
-    ];
-
-    /**
-     * @param bool $includerelations
-     * @return array
-     */
-    public function fieldLabels($includerelations = true)
-    {
-        $labels = parent::fieldLabels($includerelations);
-
-        $labels['Name'] = _t(__CLASS__ . '.NAME', 'Name');
-        $labels['Headline'] = _t(__CLASS__ . '.HEADLINE', 'Headline');
-        $labels['Description'] = _t(__CLASS__ . '.DESCRIPTION', 'Description');
-        $labels['SlideLinkID'] =  _t(__CLASS__ . '.PAGE_LINK', "Call to action link");
-        $labels['Image'] = _t(__CLASS__ . '.IMAGE', 'Image');
-        $labels['SlideType'] = _t(__CLASS__ . '.SlideType', 'Image or Video');
-        $labels['Video'] = _t(__CLASS__ . '.VideoLabel', 'Video URL');
-
-        return $labels;
-    }
-
-    /**
-     * @return \SilverStripe\Forms\FieldList
-     */
-    public function getCMSFields()
-    {
-        $this->beforeUpdateCMSFields(function ($fields) {
-            $fields->removeByName([
-                'ShowSlide',
-                'SortOrder',
-                'PageID',
-                'Image',
-                'SlideType',
-                'Video',
-                'VideoID',
-                'SlideLinkID',
-            ]);
-
-            // Name
-            $fields->dataFieldByName('Name')
-                ->setDescription(
-                    _t(__CLASS__ . '.INTERNAL_USE', 'for internal reference only')
-                );
-
-            // Headline
-            $fields->dataFieldByName('Headline')
-                ->setDescription(
-                    _t(__CLASS__ . '.USED_IN_TEMPLATE', 'optional, used in template')
-                );
-
-            // Description
-            $fields->dataFieldByName('Description')
-                ->setDescription(
-                    _t(__CLASS__ . '.USED_IN_TEMPLATE', 'optional, used in template')
-                );
-
-            // Page link
-            $fields->replaceField(
-                'PageLinkID',
-                LinkField::create('SlideLink', $this->fieldLabel('SlideLinkID'), $this)
-            );
-
-            // Image
-            $image = UploadField::create('Image', $this->fieldLabel('Image'))
-                ->setFolderName('Uploads/SlideImages');
-
-            $image->getValidator()->setAllowedExtensions(['jpg', 'jpeg', 'png', 'gif']);
-
-            $fields->addFieldToTab(
-                'Root.Main',
-                CompositeField::create(FieldList::create(
-                    DropdownField::create('SlideType', $this->fieldLabel('SlideType'))
-                        ->setSource($this->getTypeSource()),
-                    Wrapper::create(
-                        $image
-                    )->displayIf('SlideType')->isEqualTo('Image')->orIf('SlideType')->isEqualTo('Video')->end(),
-                    /*Wrapper::create(
+	/**
+	 * @var string
+	 */
+	private static $singular_name = 'Slide';
+
+	/**
+	 * @var string
+	 */
+	private static $plural_name = 'Slides';
+
+	/**
+	 * @var array
+	 */
+	private static $db = [
+		'Name' => 'Varchar(255)',
+		'Headline' => 'Varchar(255)',
+		'Description' => 'Text',
+		'SortOrder' => 'Int',
+		'SlideType' => 'Varchar',
+	];
+
+	/**
+	 * @var array
+	 */
+	private static $has_one = [
+		'Image' => Image::class,
+		//'Video' => Embed::class,
+		'Page' => \Page::class,
+		'PageLink' => SiteTree::class,
+		'SlideLink' => Link::class,
+	];
+
+	/**
+	 * @var array
+	 */
+	private static $owns = [
+		'Image',
+	];
+
+	/**
+	 * @var string
+	 */
+	private static $table_name = 'SlideImage';
+
+	/**
+	 * @var string
+	 */
+	private static $default_sort = 'SortOrder';
+
+	/**
+	 * Adds Publish button to SlideImage record
+	 *
+	 * @var bool
+	 */
+	private static $versioned_gridfield_extensions = true;
+
+	/**
+	 * @var array
+	 */
+	private static $defaults = [
+		'SlideType' => 'Image',
+	];
+
+	/**
+	 * @var array
+	 */
+	private static $summary_fields = [
+		'Image.CMSThumbnail' => 'Image',
+		'Name' => 'Name',
+	];
+
+	/**
+	 * @var array
+	 */
+	private static $searchable_fields = [
+		'Name',
+		'Headline',
+		'Description',
+	];
+
+	/**
+	 * @var int
+	 */
+	private static $image_size_limit = 512000;
+
+	/**
+	 * @var array
+	 */
+	private static $slide_types = [
+		'Image',
+		'Video',
+		'Text',
+	];
+
+	/**
+	 * @param bool $includerelations
+	 * @return array
+	 */
+	public function fieldLabels($includerelations = true)
+	{
+		$labels = parent::fieldLabels($includerelations);
+
+		$labels['Name'] = _t(__CLASS__ . '.NAME', 'Name');
+		$labels['Headline'] = _t(__CLASS__ . '.HEADLINE', 'Headline');
+		$labels['Description'] = _t(__CLASS__ . '.DESCRIPTION', 'Description');
+		$labels['SlideLinkID'] =  _t(__CLASS__ . '.PAGE_LINK', "Call to action link");
+		$labels['Image'] = _t(__CLASS__ . '.IMAGE', 'Image');
+		$labels['SlideType'] = _t(__CLASS__ . '.SlideType', 'Image or Video');
+		$labels['Video'] = _t(__CLASS__ . '.VideoLabel', 'Video URL');
+
+		return $labels;
+	}
+
+	/**
+	 * @return \SilverStripe\Forms\FieldList
+	 */
+	public function getCMSFields()
+	{
+		$this->beforeUpdateCMSFields(function ($fields) {
+			$fields->removeByName([
+				'ShowSlide',
+				'SortOrder',
+				'PageID',
+				'Image',
+				'SlideType',
+				'Video',
+				'VideoID',
+				'SlideLinkID',
+			]);
+
+			// Name
+			$fields->dataFieldByName('Name')
+				->setDescription(
+					_t(__CLASS__ . '.INTERNAL_USE', 'for internal reference only')
+				);
+
+			// Headline
+			$fields->dataFieldByName('Headline')
+				->setDescription(
+					_t(__CLASS__ . '.USED_IN_TEMPLATE', 'optional, used in template')
+				);
+
+			// Description
+			$fields->dataFieldByName('Description')
+				->setDescription(
+					_t(__CLASS__ . '.USED_IN_TEMPLATE', 'optional, used in template')
+				);
+
+			// Page link
+			$fields->replaceField(
+				'PageLinkID',
+				LinkField::create('SlideLink', $this->fieldLabel('SlideLinkID'), $this)
+			);
+
+			// Image
+			$image = UploadField::create('Image', $this->fieldLabel('Image'))
+				->setFolderName('Uploads/SlideImages');
+
+			$image->getValidator()->setAllowedExtensions(['jpg', 'jpeg', 'png', 'gif']);
+
+			$fields->addFieldToTab(
+				'Root.Main',
+				CompositeField::create(FieldList::create(
+					DropdownField::create('SlideType', $this->fieldLabel('SlideType'))
+						->setSource($this->getTypeSource()),
+					Wrapper::create(
+						$image
+					)->displayIf('SlideType')->isEqualTo('Image')->orIf('SlideType')->isEqualTo('Video')->end(),
+					/*Wrapper::create(
                         $videoField = HasOneButtonField::create(
                             'Video',
                             'Video',
                             $this
                         )
                     )->displayIf('SlideType')->isEqualTo('Video')->end()*/
-                ))->setName('MediaFields'),
-                'Description'
-            );
-        });
-
-        $fields = parent::getCMSFields();
-
-        $this->extend('updateSlideImageFields', $fields);
-
-        return $fields;
-    }
-
-    /**
-     * @return \SilverStripe\ORM\ValidationResult
-     */
-    public function validate()
-    {
-        $result = parent::validate();
-
-        if (!$this->Name) {
-            $result->addError(
-                _t(__CLASS__ . '.NAME_REQUIRED', 'A Name is required before you can save')
-            );
-        }
-
-        $types = $this->getTypeSource();
-
-        if (isset($types['Video']) && $this->SlideType == 'Video' && !$this->VideoID) {
-            $result->addError(
-                _t(__CLASS__ . '.VIDEO_REQUIRED', 'An Video Link is required before you can save')
-            );
-        }
-
-        if (isset($types['Image']) && $this->SlideType == 'Image' && !$this->ImageID) {
-            $result->addError(
-                _t(__CLASS__ . '.IMAGE_REQUIRED', 'An Image is required before you can save')
-            );
-        }
-
-        if (isset($types['Text']) && $this->SlideType == 'Text' && !$this->Description) {
-            $result->addError(
-                _t(__CLASS__ . '.DESCRIPTION_REQUIRED', 'A Description is required before you can save')
-            );
-        }
-
-        return $result;
-    }
-
-    /**
-     * @return array
-     */
-    public function providePermissions()
-    {
-        return [
-            'Slide_EDIT' => 'Slide Edit',
-            'Slide_DELETE' => 'Slide Delete',
-            'Slide_CREATE' => 'Slide Create',
-        ];
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     *
-     * @return bool|int
-     */
-    public function canCreate($member = null, $context = [])
-    {
-        return Permission::check('Slide_CREATE', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     *
-     * @return bool|int
-     */
-    public function canEdit($member = null, $context = [])
-    {
-        return Permission::check('Slide_EDIT', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     *
-     * @return bool|int
-     */
-    public function canDelete($member = null, $context = [])
-    {
-        return Permission::check('Slide_DELETE', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     *
-     * @return bool
-     */
-    public function canView($member = null, $context = [])
-    {
-        return true;
-    }
-
-    /**
-     * @return array
-     */
-    public function getTypeSource()
-    {
-        $types = $this->config()->get('slide_types');
-        asort($types);
-        return array_combine($types, $types);
-    }
-
-    /**
-     * @param null $template
-     * @param null $customFields
-     * @return \SilverStripe\ORM\FieldType\DBHTMLText
-     */
-    public function renderWith($template = null, $customFields = null)
-    {
-        if ($template === null) {
-            $template = static::class;
-            $template = ($this->SlideType) ? $template . "_{$this->SlideType}" : '';
-        }
-
-        return parent::renderWith($template);
-    }
-
-    /**
-     * @return \SilverStripe\ORM\FieldType\DBHTMLText
-     */
-    public function forTemplate()
-    {
-        return $this->renderWith();
-    }
+				))->setName('MediaFields'),
+				'Description'
+			);
+		});
+
+		$fields = parent::getCMSFields();
+
+		$this->extend('updateSlideImageFields', $fields);
+
+		return $fields;
+	}
+
+	/**
+	 * @return \SilverStripe\ORM\ValidationResult
+	 */
+	public function validate()
+	{
+		$result = parent::validate();
+
+		if (!$this->Name) {
+			$result->addError(
+				_t(__CLASS__ . '.NAME_REQUIRED', 'A Name is required before you can save')
+			);
+		}
+
+		$types = $this->getTypeSource();
+
+		if (isset($types['Video']) && $this->SlideType == 'Video' && !$this->VideoID) {
+			$result->addError(
+				_t(__CLASS__ . '.VIDEO_REQUIRED', 'An Video Link is required before you can save')
+			);
+		}
+
+		if (isset($types['Image']) && $this->SlideType == 'Image' && !$this->ImageID) {
+			$result->addError(
+				_t(__CLASS__ . '.IMAGE_REQUIRED', 'An Image is required before you can save')
+			);
+		}
+
+		if (isset($types['Text']) && $this->SlideType == 'Text' && !$this->Description) {
+			$result->addError(
+				_t(__CLASS__ . '.DESCRIPTION_REQUIRED', 'A Description is required before you can save')
+			);
+		}
+
+		return $result;
+	}
+
+	/**
+	 * @return array
+	 */
+	public function providePermissions()
+	{
+		return [
+			'Slide_EDIT' => 'Slide Edit',
+			'Slide_DELETE' => 'Slide Delete',
+			'Slide_CREATE' => 'Slide Create',
+		];
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 *
+	 * @return bool|int
+	 */
+	public function canCreate($member = null, $context = [])
+	{
+		return Permission::check('Slide_CREATE', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 *
+	 * @return bool|int
+	 */
+	public function canEdit($member = null, $context = [])
+	{
+		return Permission::check('Slide_EDIT', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 *
+	 * @return bool|int
+	 */
+	public function canDelete($member = null, $context = [])
+	{
+		return Permission::check('Slide_DELETE', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 *
+	 * @return bool
+	 */
+	public function canView($member = null, $context = [])
+	{
+		return true;
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getTypeSource()
+	{
+		$types = $this->config()->get('slide_types');
+		asort($types);
+		return array_combine($types, $types);
+	}
+
+	/**
+	 * @param null $template
+	 * @param null $customFields
+	 * @return \SilverStripe\ORM\FieldType\DBHTMLText
+	 */
+	public function renderWith($template = null, $customFields = null)
+	{
+		if ($template === null) {
+			$template = static::class;
+			$template = ($this->SlideType) ? $template . "_{$this->SlideType}" : '';
+		}
+
+		return parent::renderWith($template);
+	}
+
+	/**
+	 * @return \SilverStripe\ORM\FieldType\DBHTMLText
+	 */
+	public function forTemplate()
+	{
+		return $this->renderWith();
+	}
 }

Spacing +20 added lines, -20 removed lines

@@ -139,13 +139,13 @@ 
     {
         $labels = parent::fieldLabels($includerelations);
 
-        $labels['Name'] = _t(__CLASS__ . '.NAME', 'Name');
-        $labels['Headline'] = _t(__CLASS__ . '.HEADLINE', 'Headline');
-        $labels['Description'] = _t(__CLASS__ . '.DESCRIPTION', 'Description');
-        $labels['SlideLinkID'] =  _t(__CLASS__ . '.PAGE_LINK', "Call to action link");
-        $labels['Image'] = _t(__CLASS__ . '.IMAGE', 'Image');
-        $labels['SlideType'] = _t(__CLASS__ . '.SlideType', 'Image or Video');
-        $labels['Video'] = _t(__CLASS__ . '.VideoLabel', 'Video URL');
+        $labels['Name'] = _t(__CLASS__.'.NAME', 'Name');
+        $labels['Headline'] = _t(__CLASS__.'.HEADLINE', 'Headline');
+        $labels['Description'] = _t(__CLASS__.'.DESCRIPTION', 'Description');
+        $labels['SlideLinkID'] = _t(__CLASS__.'.PAGE_LINK', "Call to action link");
+        $labels['Image'] = _t(__CLASS__.'.IMAGE', 'Image');
+        $labels['SlideType'] = _t(__CLASS__.'.SlideType', 'Image or Video');
+        $labels['Video'] = _t(__CLASS__.'.VideoLabel', 'Video URL');
 
         return $labels;
     }
@@ -155,7 +155,7 @@ 
      */
     public function getCMSFields()
     {
-        $this->beforeUpdateCMSFields(function ($fields) {
+        $this->beforeUpdateCMSFields(function($fields) {
             $fields->removeByName([
                 'ShowSlide',
                 'SortOrder',
@@ -170,19 +170,19 @@ 
             // Name
             $fields->dataFieldByName('Name')
                 ->setDescription(
-                    _t(__CLASS__ . '.INTERNAL_USE', 'for internal reference only')
+                    _t(__CLASS__.'.INTERNAL_USE', 'for internal reference only')
                 );
 
             // Headline
             $fields->dataFieldByName('Headline')
                 ->setDescription(
-                    _t(__CLASS__ . '.USED_IN_TEMPLATE', 'optional, used in template')
+                    _t(__CLASS__.'.USED_IN_TEMPLATE', 'optional, used in template')
                 );
 
             // Description
             $fields->dataFieldByName('Description')
                 ->setDescription(
-                    _t(__CLASS__ . '.USED_IN_TEMPLATE', 'optional, used in template')
+                    _t(__CLASS__.'.USED_IN_TEMPLATE', 'optional, used in template')
                 );
 
             // Page link
@@ -233,27 +233,27 @@ 
 
         if (!$this->Name) {
             $result->addError(
-                _t(__CLASS__ . '.NAME_REQUIRED', 'A Name is required before you can save')
+                _t(__CLASS__.'.NAME_REQUIRED', 'A Name is required before you can save')
             );
         }
 
         $types = $this->getTypeSource();
 
-        if (isset($types['Video']) && $this->SlideType == 'Video' && !$this->VideoID) {
+        if (isset($types['Video']) && $this->SlideType=='Video' && !$this->VideoID) {
             $result->addError(
-                _t(__CLASS__ . '.VIDEO_REQUIRED', 'An Video Link is required before you can save')
+                _t(__CLASS__.'.VIDEO_REQUIRED', 'An Video Link is required before you can save')
             );
         }
 
-        if (isset($types['Image']) && $this->SlideType == 'Image' && !$this->ImageID) {
+        if (isset($types['Image']) && $this->SlideType=='Image' && !$this->ImageID) {
             $result->addError(
-                _t(__CLASS__ . '.IMAGE_REQUIRED', 'An Image is required before you can save')
+                _t(__CLASS__.'.IMAGE_REQUIRED', 'An Image is required before you can save')
             );
         }
 
-        if (isset($types['Text']) && $this->SlideType == 'Text' && !$this->Description) {
+        if (isset($types['Text']) && $this->SlideType=='Text' && !$this->Description) {
             $result->addError(
-                _t(__CLASS__ . '.DESCRIPTION_REQUIRED', 'A Description is required before you can save')
+                _t(__CLASS__.'.DESCRIPTION_REQUIRED', 'A Description is required before you can save')
             );
         }
 
@@ -333,9 +333,9 @@ 
      */
     public function renderWith($template = null, $customFields = null)
     {
-        if ($template === null) {
+        if ($template===null) {
             $template = static::class;
-            $template = ($this->SlideType) ? $template . "_{$this->SlideType}" : '';
+            $template = ($this->SlideType) ? $template."_{$this->SlideType}" : '';
         }
 
         return parent::renderWith($template);

src/ORM/FlexSlider.php

Indentation +195 added lines, -195 removed lines

@@ -37,189 +37,189 @@ 
  */
 class FlexSlider extends DataExtension
 {
-    use Configurable;
+	use Configurable;
 
-    /**
-     * @var array
-     */
-    private static $db = [
-        'Animation' => "Enum('slide, fade', 'slide')",
-        'Loop' => 'Boolean',
-        'Animate' => 'Boolean',
-        'ThumbnailNav' => 'Boolean',
-        'SliderControlNav' => 'Boolean',
-        'SliderDirectionNav' => 'Boolean',
-        'CarouselControlNav' => 'Boolean',
-        'CarouselDirectionNav' => 'Boolean',
-        'CarouselThumbnailCt' => 'Int',
-        'FlexSliderSpeed' => 'Double',
-    ];
+	/**
+	 * @var array
+	 */
+	private static $db = [
+		'Animation' => "Enum('slide, fade', 'slide')",
+		'Loop' => 'Boolean',
+		'Animate' => 'Boolean',
+		'ThumbnailNav' => 'Boolean',
+		'SliderControlNav' => 'Boolean',
+		'SliderDirectionNav' => 'Boolean',
+		'CarouselControlNav' => 'Boolean',
+		'CarouselDirectionNav' => 'Boolean',
+		'CarouselThumbnailCt' => 'Int',
+		'FlexSliderSpeed' => 'Double',
+	];
 
-    /**
-     * @var array
-     */
-    private static $has_many = [
-        'Slides' => SlideImage::class,
-    ];
+	/**
+	 * @var array
+	 */
+	private static $has_many = [
+		'Slides' => SlideImage::class,
+	];
 
-    /**
-     *
-     */
-    public function populateDefaults()
-    {
-        $this->owner->Loop = 1;
-        $this->owner->Animate = 1;
-        $this->owner->SliderControlNav = 0;
-        $this->owner->SliderDirectionNav = 1;
-        $this->owner->CarouselControlNav = 0;
-        $this->owner->CarouselDirectionNav = 1;
-        $this->owner->CarouselThumbnailCt = 6;
-        $this->owner->FlexSliderSpeed = $this->getDefaultSpeed();
+	/**
+	 *
+	 */
+	public function populateDefaults()
+	{
+		$this->owner->Loop = 1;
+		$this->owner->Animate = 1;
+		$this->owner->SliderControlNav = 0;
+		$this->owner->SliderDirectionNav = 1;
+		$this->owner->CarouselControlNav = 0;
+		$this->owner->CarouselDirectionNav = 1;
+		$this->owner->CarouselThumbnailCt = 6;
+		$this->owner->FlexSliderSpeed = $this->getDefaultSpeed();
 
-        return parent::populateDefaults();
-    }
+		return parent::populateDefaults();
+	}
 
-    /**
-     * @param FieldList $fields
-     */
-    public function updateCMSFields(FieldList $fields)
-    {
-        $fields->removeByName([
-            'Animation',
-            'Loop',
-            'Animate',
-            'ThumbnailNav',
-            'SliderControlNav',
-            'SliderDirectionNav',
-            'CarouselControlNav',
-            'CarouselDirectionNav',
-            'CarouselThumbnailCt',
-            'FlexSliderSpeed',
-            'Slides',
-        ]);
+	/**
+	 * @param FieldList $fields
+	 */
+	public function updateCMSFields(FieldList $fields)
+	{
+		$fields->removeByName([
+			'Animation',
+			'Loop',
+			'Animate',
+			'ThumbnailNav',
+			'SliderControlNav',
+			'SliderDirectionNav',
+			'CarouselControlNav',
+			'CarouselDirectionNav',
+			'CarouselThumbnailCt',
+			'FlexSliderSpeed',
+			'Slides',
+		]);
 
-        // Slides
-        if ($this->owner->ID) {
-            $config = GridFieldConfig_RecordEditor::create();
-            $config->addComponent(new GridFieldOrderableRows('SortOrder'));
-            $config->removeComponentsByType([
-                GridFieldAddExistingAutocompleter::class,
-                GridFieldDeleteAction::class,
-            ]);
-            $SlidesField = GridField::create(
-                'Slides',
-                _t(__CLASS__ . '.SLIDES', 'Slides'),
-                $this->owner->Slides()->sort('SortOrder'),
-                $config
-            );
+		// Slides
+		if ($this->owner->ID) {
+			$config = GridFieldConfig_RecordEditor::create();
+			$config->addComponent(new GridFieldOrderableRows('SortOrder'));
+			$config->removeComponentsByType([
+				GridFieldAddExistingAutocompleter::class,
+				GridFieldDeleteAction::class,
+			]);
+			$SlidesField = GridField::create(
+				'Slides',
+				_t(__CLASS__ . '.SLIDES', 'Slides'),
+				$this->owner->Slides()->sort('SortOrder'),
+				$config
+			);
 
-            $slideTitle = $this->owner->slide_tab_title ?: _t(__CLASS__ . '.SLIDES', 'Slides');
+			$slideTitle = $this->owner->slide_tab_title ?: _t(__CLASS__ . '.SLIDES', 'Slides');
 
-            $animations = [];
-            $animationOptions = $this->owner->dbObject('Animation')->getEnum();
-            foreach ($animationOptions as $value) {
-                $animations[$value] = _t(__CLASS__ . ".$value", $value);
-            }
+			$animations = [];
+			$animationOptions = $this->owner->dbObject('Animation')->getEnum();
+			foreach ($animationOptions as $value) {
+				$animations[$value] = _t(__CLASS__ . ".$value", $value);
+			}
 
-            $fields->addFieldsToTab("Root.{$slideTitle}", [
-                $SlidesField,
-                ToggleCompositeField::create('ConfigHD', _t(__CLASS__ . '.SettingsLabel', 'Slider Settings'), [
-                    DropdownField::create(
-                        'Animation',
-                        _t(__CLASS__ . '.ANIMATION_OPTION', 'Animation option'),
-                        $animations
-                    ),
-                    CheckboxField::create(
-                        'Animate',
-                        _t(__CLASS__ . '.ANIMATE', 'Animate automatically')
-                    ),
-                    CheckboxField::create(
-                        'Loop',
-                        _t(__CLASS__ . '.LOOP', 'Loop the carousel')
-                    ),
-                    CheckboxField::create(
-                        'SliderControlNav',
-                        _t(__CLASS__ . '.CONTROL_NAV', 'Show ControlNav')
-                    ),
-                    CheckboxField::create(
-                        'SliderDirectionNav',
-                        _t(__CLASS__ . '.DIRECTION_NAV', 'Show DirectionNav')
-                    ),
-                    CheckboxField::create(
-                        'ThumbnailNav',
-                        _t(__CLASS__ . '.THUMBNAIL_NAV', 'Thumbnail Navigation')
-                    ),
-                    //DisplayLogicWrapper::create(
-                    CheckboxField::create(
-                        'CarouselControlNav',
-                        _t(__CLASS__ . '.CAROUSEL_CONTROL_NAV', 'Show Carousel ControlNav')
-                    ),
-                    CheckboxField::create(
-                        'CarouselDirectionNav',
-                        _t(__CLASS__ . '.CAROUSEL_DIRECTION_NAV', 'Show Carousel DirectionNav')
-                    ),
-                    NumericField::create(
-                        'CarouselThumbnailCt',
-                        _t(__CLASS__ . '.CAROUSEL_THUMBNAIL_COUNT', 'Number of thumbnails')
-                    ),
-                    NumericField::create(
-                        'FlexSliderSpeed',
-                        _t(__CLASS__ . '.SLIDER_SPEED', 'Slider Speed')
-                    )
-                        ->setDescription('In Seconds')
-                        ->setScale(2),
-                ]),
-            ]);
-        }
-    }
+			$fields->addFieldsToTab("Root.{$slideTitle}", [
+				$SlidesField,
+				ToggleCompositeField::create('ConfigHD', _t(__CLASS__ . '.SettingsLabel', 'Slider Settings'), [
+					DropdownField::create(
+						'Animation',
+						_t(__CLASS__ . '.ANIMATION_OPTION', 'Animation option'),
+						$animations
+					),
+					CheckboxField::create(
+						'Animate',
+						_t(__CLASS__ . '.ANIMATE', 'Animate automatically')
+					),
+					CheckboxField::create(
+						'Loop',
+						_t(__CLASS__ . '.LOOP', 'Loop the carousel')
+					),
+					CheckboxField::create(
+						'SliderControlNav',
+						_t(__CLASS__ . '.CONTROL_NAV', 'Show ControlNav')
+					),
+					CheckboxField::create(
+						'SliderDirectionNav',
+						_t(__CLASS__ . '.DIRECTION_NAV', 'Show DirectionNav')
+					),
+					CheckboxField::create(
+						'ThumbnailNav',
+						_t(__CLASS__ . '.THUMBNAIL_NAV', 'Thumbnail Navigation')
+					),
+					//DisplayLogicWrapper::create(
+					CheckboxField::create(
+						'CarouselControlNav',
+						_t(__CLASS__ . '.CAROUSEL_CONTROL_NAV', 'Show Carousel ControlNav')
+					),
+					CheckboxField::create(
+						'CarouselDirectionNav',
+						_t(__CLASS__ . '.CAROUSEL_DIRECTION_NAV', 'Show Carousel DirectionNav')
+					),
+					NumericField::create(
+						'CarouselThumbnailCt',
+						_t(__CLASS__ . '.CAROUSEL_THUMBNAIL_COUNT', 'Number of thumbnails')
+					),
+					NumericField::create(
+						'FlexSliderSpeed',
+						_t(__CLASS__ . '.SLIDER_SPEED', 'Slider Speed')
+					)
+						->setDescription('In Seconds')
+						->setScale(2),
+				]),
+			]);
+		}
+	}
 
-    /**
-     * @return mixed
-     */
-    public function getSlideShow()
-    {
-        $owner = $this->owner;
+	/**
+	 * @return mixed
+	 */
+	public function getSlideShow()
+	{
+		$owner = $this->owner;
 
-        if (!($owner instanceof SiteTree)) {
-            $this->getCustomScript();
-        }
+		if (!($owner instanceof SiteTree)) {
+			$this->getCustomScript();
+		}
 
-        return $this->owner->Slides()->sort('SortOrder');
-    }
+		return $this->owner->Slides()->sort('SortOrder');
+	}
 
-    /**
-     * add requirements to PageController init()
-     */
-    public function contentcontrollerInit()
-    {
-        // only call custom script if page has Slides and DataExtension
-        if (DataObject::has_extension($this->owner->Classname, FlexSlider::class)) {
-            if ($this->owner->getSlideShow()->exists()) {
-                $this->getCustomScript();
-            }
-        }
-    }
+	/**
+	 * add requirements to PageController init()
+	 */
+	public function contentcontrollerInit()
+	{
+		// only call custom script if page has Slides and DataExtension
+		if (DataObject::has_extension($this->owner->Classname, FlexSlider::class)) {
+			if ($this->owner->getSlideShow()->exists()) {
+				$this->getCustomScript();
+			}
+		}
+	}
 
-    /**
-     *
-     */
-    public function getCustomScript()
-    {
-        // Flexslider options
-        $sync = ($this->owner->ThumbnailNav == true) ? "sync: '.fs-carousel:eq('+index+')'," : '';
+	/**
+	 *
+	 */
+	public function getCustomScript()
+	{
+		// Flexslider options
+		$sync = ($this->owner->ThumbnailNav == true) ? "sync: '.fs-carousel:eq('+index+')'," : '';
 
-        $before = $this->owner->hasMethod('flexSliderBeforeAction')
-            ? $this->owner->flexSliderBeforeAction()
-            : 'function(){}';
+		$before = $this->owner->hasMethod('flexSliderBeforeAction')
+			? $this->owner->flexSliderBeforeAction()
+			: 'function(){}';
 
-        $after = $this->owner->hasMethod('flexSliderAfterAction')
-            ? $this->owner->flexSliderAfterAction()
-            : 'function(){}';
+		$after = $this->owner->hasMethod('flexSliderAfterAction')
+			? $this->owner->flexSliderAfterAction()
+			: 'function(){}';
 
-        $speed = $this->getSlideshowSpeed();
+		$speed = $this->getSlideshowSpeed();
 
-        Requirements::customScript(
-            "(function($) {
+		Requirements::customScript(
+			"(function($) {
                 $(document).ready(function(){
                     jQuery('.flexslider').each(function(index){
 
@@ -265,39 +265,39 @@ 
                     })
                 });
             }(jQuery));'
-        );
-    }
+		);
+	}
 
-    /**
-     * @return int
-     */
-    public function getSlideshowSpeed()
-    {
-        $speed = $this->owner->FlexSliderSpeed > 0
-            ? $this->owner->FlexSliderSpeed
-            : $this->getDefaultSpeed();
+	/**
+	 * @return int
+	 */
+	public function getSlideshowSpeed()
+	{
+		$speed = $this->owner->FlexSliderSpeed > 0
+			? $this->owner->FlexSliderSpeed
+			: $this->getDefaultSpeed();
 
-        return $speed * 1000;
-    }
+		return $speed * 1000;
+	}
 
-    /**
-     * @return mixed
-     */
-    protected function getDefaultSpeed()
-    {
-        return $this->owner->config()->get('flex_slider_speed')
-            ?: Config::inst()->get(FlexSlider::class, 'flex_slider_speed');
-    }
+	/**
+	 * @return mixed
+	 */
+	protected function getDefaultSpeed()
+	{
+		return $this->owner->config()->get('flex_slider_speed')
+			?: Config::inst()->get(FlexSlider::class, 'flex_slider_speed');
+	}
 
-    /**
-     *
-     */
-    public function onBeforeWrite()
-    {
-        parent::onBeforeWrite();
+	/**
+	 *
+	 */
+	public function onBeforeWrite()
+	{
+		parent::onBeforeWrite();
 
-        if (!$this->owner->CarouselThumbnailCt) {
-            $this->owner->CarouselThumbnailCt = 6;
-        }
-    }
+		if (!$this->owner->CarouselThumbnailCt) {
+			$this->owner->CarouselThumbnailCt = 6;
+		}
+	}
 }

Spacing +31 added lines, -31 removed lines

@@ -108,63 +108,63 @@ 
             ]);
             $SlidesField = GridField::create(
                 'Slides',
-                _t(__CLASS__ . '.SLIDES', 'Slides'),
+                _t(__CLASS__.'.SLIDES', 'Slides'),
                 $this->owner->Slides()->sort('SortOrder'),
                 $config
             );
 
-            $slideTitle = $this->owner->slide_tab_title ?: _t(__CLASS__ . '.SLIDES', 'Slides');
+            $slideTitle = $this->owner->slide_tab_title ?: _t(__CLASS__.'.SLIDES', 'Slides');
 
             $animations = [];
             $animationOptions = $this->owner->dbObject('Animation')->getEnum();
             foreach ($animationOptions as $value) {
-                $animations[$value] = _t(__CLASS__ . ".$value", $value);
+                $animations[$value] = _t(__CLASS__.".$value", $value);
             }
 
             $fields->addFieldsToTab("Root.{$slideTitle}", [
                 $SlidesField,
-                ToggleCompositeField::create('ConfigHD', _t(__CLASS__ . '.SettingsLabel', 'Slider Settings'), [
+                ToggleCompositeField::create('ConfigHD', _t(__CLASS__.'.SettingsLabel', 'Slider Settings'), [
                     DropdownField::create(
                         'Animation',
-                        _t(__CLASS__ . '.ANIMATION_OPTION', 'Animation option'),
+                        _t(__CLASS__.'.ANIMATION_OPTION', 'Animation option'),
                         $animations
                     ),
                     CheckboxField::create(
                         'Animate',
-                        _t(__CLASS__ . '.ANIMATE', 'Animate automatically')
+                        _t(__CLASS__.'.ANIMATE', 'Animate automatically')
                     ),
                     CheckboxField::create(
                         'Loop',
-                        _t(__CLASS__ . '.LOOP', 'Loop the carousel')
+                        _t(__CLASS__.'.LOOP', 'Loop the carousel')
                     ),
                     CheckboxField::create(
                         'SliderControlNav',
-                        _t(__CLASS__ . '.CONTROL_NAV', 'Show ControlNav')
+                        _t(__CLASS__.'.CONTROL_NAV', 'Show ControlNav')
                     ),
                     CheckboxField::create(
                         'SliderDirectionNav',
-                        _t(__CLASS__ . '.DIRECTION_NAV', 'Show DirectionNav')
+                        _t(__CLASS__.'.DIRECTION_NAV', 'Show DirectionNav')
                     ),
                     CheckboxField::create(
                         'ThumbnailNav',
-                        _t(__CLASS__ . '.THUMBNAIL_NAV', 'Thumbnail Navigation')
+                        _t(__CLASS__.'.THUMBNAIL_NAV', 'Thumbnail Navigation')
                     ),
                     //DisplayLogicWrapper::create(
                     CheckboxField::create(
                         'CarouselControlNav',
-                        _t(__CLASS__ . '.CAROUSEL_CONTROL_NAV', 'Show Carousel ControlNav')
+                        _t(__CLASS__.'.CAROUSEL_CONTROL_NAV', 'Show Carousel ControlNav')
                     ),
                     CheckboxField::create(
                         'CarouselDirectionNav',
-                        _t(__CLASS__ . '.CAROUSEL_DIRECTION_NAV', 'Show Carousel DirectionNav')
+                        _t(__CLASS__.'.CAROUSEL_DIRECTION_NAV', 'Show Carousel DirectionNav')
                     ),
                     NumericField::create(
                         'CarouselThumbnailCt',
-                        _t(__CLASS__ . '.CAROUSEL_THUMBNAIL_COUNT', 'Number of thumbnails')
+                        _t(__CLASS__.'.CAROUSEL_THUMBNAIL_COUNT', 'Number of thumbnails')
                     ),
                     NumericField::create(
                         'FlexSliderSpeed',
-                        _t(__CLASS__ . '.SLIDER_SPEED', 'Slider Speed')
+                        _t(__CLASS__.'.SLIDER_SPEED', 'Slider Speed')
                     )
                         ->setDescription('In Seconds')
                         ->setScale(2),
@@ -206,7 +206,7 @@ 
     public function getCustomScript()
     {
         // Flexslider options
-        $sync = ($this->owner->ThumbnailNav == true) ? "sync: '.fs-carousel:eq('+index+')'," : '';
+        $sync = ($this->owner->ThumbnailNav==true) ? "sync: '.fs-carousel:eq('+index+')'," : '';
 
         $before = $this->owner->hasMethod('flexSliderBeforeAction')
             ? $this->owner->flexSliderBeforeAction()
@@ -225,18 +225,18 @@ 
 
                          if(jQuery('.fs-carousel').eq(index).length) {
                              jQuery('.fs-carousel').eq(index).flexslider({
-                                slideshow: " . $this->owner->obj('Animate')->NiceAsBoolean() . ",
+                                slideshow: " . $this->owner->obj('Animate')->NiceAsBoolean().",
                                 animation: 'slide',
-                                animationLoop: " . $this->owner->obj('Loop')->NiceAsBoolean() . ",
-                                controlNav: " . $this->owner->obj('CarouselControlNav')->NiceAsBoolean() . ",
-                                directionNav: " . $this->owner->obj('CarouselDirectionNav')->NiceAsBoolean() . ",
+                                animationLoop: " . $this->owner->obj('Loop')->NiceAsBoolean().",
+                                controlNav: " . $this->owner->obj('CarouselControlNav')->NiceAsBoolean().",
+                                directionNav: " . $this->owner->obj('CarouselDirectionNav')->NiceAsBoolean().",
                                 prevText: '',
                                 nextText: '',
                                 pausePlay: false,
                                 asNavFor: '.flexslider:eq('+index+')',
-                                minItems: " . $this->owner->obj('CarouselThumbnailCt') . ",
-                                maxItems: " . $this->owner->obj('CarouselThumbnailCt') . ",
-                                move: " . $this->owner->obj('CarouselThumbnailCt') . ",
+                                minItems: " . $this->owner->obj('CarouselThumbnailCt').",
+                                maxItems: " . $this->owner->obj('CarouselThumbnailCt').",
+                                move: " . $this->owner->obj('CarouselThumbnailCt').",
                                 itemWidth: 100,
                                 itemMargin: 10
                               });
@@ -244,22 +244,22 @@ 
 
                         if(jQuery('.flexslider').eq(index).length){
                             jQuery('.flexslider').eq(index).flexslider({
-                                slideshow: " . $this->owner->obj('Animate')->NiceAsBoolean() . ",
-                                animation: '" . $this->owner->Animation . "',
-                                animationLoop: " . $this->owner->obj('Loop')->NiceAsBoolean() . ",
-                                controlNav: " . $this->owner->obj('SliderControlNav')->NiceAsBoolean() . ",
-                                directionNav: " . $this->owner->obj('SliderDirectionNav')->NiceAsBoolean() . ",
+                                slideshow: " . $this->owner->obj('Animate')->NiceAsBoolean().",
+                                animation: '" . $this->owner->Animation."',
+                                animationLoop: " . $this->owner->obj('Loop')->NiceAsBoolean().",
+                                controlNav: " . $this->owner->obj('SliderControlNav')->NiceAsBoolean().",
+                                directionNav: " . $this->owner->obj('SliderDirectionNav')->NiceAsBoolean().",
                                 prevText: '',
                                 nextText: '',
                                 pauseOnAction: true,
                                 pauseOnHover: true,
-                                " . $sync . "
+                                " . $sync."
                                 start: function(slider){
                                   $('body').removeClass('loading');
                                 },
-                                before: " . $before . ',
-                                after: ' . $after . ',
-                                slideshowSpeed: ' . $speed . '
+                                before: " . $before.',
+                                after: ' . $after.',
+                                slideshowSpeed: ' . $speed.'
                             });
                         }
                     })