Refactor/video image (#156)

* REFACTOR allow optional image for video slide

* REFACTOR Allow image on video slide

* UPDATE migration task for links

* UPDATE phpcs

.editorconfig

@@ -2,34 +2,26 @@
 # this file, please see the EditorConfig documentation:
 # http://editorconfig.org/
 
-root = true
-
 [*]
 charset = utf-8
 end_of_line = lf
 indent_size = 4
-indent_style = tab
+indent_style = space
 insert_final_newline = true
 trim_trailing_whitespace = true
 
-# Docs say 80 ideally, 100 ok, no more than 120
-# http://doc.silverstripe.org/en/getting_started/coding_conventions/
-max_line_length = 100
-
 [*.md]
 trim_trailing_whitespace = false
 
-[*.yml]
+[*.{yml,js,json,css,scss,eslintrc,feature}]
 indent_size = 2
 indent_style = space
 
-#PSR 2
-[**.php]
-indent_style = space
+[composer.json]
 indent_size = 4
 
-[{.travis.yml,package.json}]
-# The indent size used in the `package.json` file cannot be changed
-# https://github.com/npm/npm/pull/3180#issuecomment-16336516
-indent_size = 2
-indent_style = space
+# Don't perform any clean-up on thirdparty files
+
+[thirdparty/**]
+trim_trailing_whitespace = false
+insert_final_newline = false

src/Model/SlideImage.php

@@ -4,6 +4,7 @@
 
 use function GuzzleHttp\Psr7\parse_request;
 use Sheadawson\Linkable\Forms\EmbeddedObjectField;
+use Sheadawson\Linkable\Forms\LinkField;
 use Sheadawson\Linkable\Models\EmbeddedObject;
 use Sheadawson\Linkable\Models\Link;
 use SilverStripe\AssetAdmin\Forms\UploadField;
@@ -64,7 +65,7 @@ class SlideImage extends DataObject implements PermissionProvider
         'Video' => EmbeddedObject::class,
         'Page' => \Page::class,
         'PageLink' => SiteTree::class,
-        'Link' => Link::class,
+        'SlideLink' => Link::class,
     ];
 
     /**
@@ -141,7 +142,8 @@ public function getCMSFields()
                 'PageID',
                 'Image',
                 'SlideType',
-                'VideoID',
+                'Video',
+                'SlideLinkID',
             ]);
 
             // Name
@@ -174,7 +176,7 @@ public function getCMSFields()
             // Page link
             $fields->replaceField(
                 'PageLinkID',
-                TreeDropdownField::create('PageLinkID', '', SiteTree::class)
+                LinkField::create('SlideLinkID')
                     ->setTitle(
                         _t(__CLASS__ . '.PAGE_LINK', "Choose a page to link to:")
                     )
@@ -196,7 +198,7 @@ public function getCMSFields()
                         ->setTitle('Image or Video'),
                     Wrapper::create(
                         $image
-                    )->displayIf('SlideType')->isEqualTo('Image')->end(),
+                    )->displayIf('SlideType')->isEqualTo('Image')->orIf('SlideType')->isEqualTo('Video')->end(),
                     Wrapper::create(
                         $videoField = EmbeddedObjectField::create('Video')
                             ->setTitle('Video URL')

src/Task/SlideLinkTask.php

@@ -0,0 +1,135 @@
+<?php
+
+namespace Dynamic\Flexslider\Task;
+
+use Dynamic\FlexSlider\Model\SlideImage;
+use Sheadawson\Linkable\Models\Link;
+use SilverStripe\CMS\Model\SiteTree;
+use SilverStripe\Dev\BuildTask;
+use SilverStripe\ORM\DB;
+
+/**
+ * Class SlideLinkTask
+ * @package Dynamic\Flexslider\Task
+ */
+class SlideLinkTask extends BuildTask
+{
+    /**
+     * @var string
+     */
+    protected $title = 'Slide Link Migration Task';
+
+    /**
+     * @var string
+     */
+    private static $segment = 'slide-link-migration-task';
+
+    /**
+     * @var array
+     */
+    private $known_links = [];
+
+    /**
+     * @param \SilverStripe\Control\HTTPRequest $request
+     * @throws \SilverStripe\ORM\ValidationException
+     */
+    public function run($request)
+    {
+        $this->migrateLinks();
+    }
+
+    /**
+     * @throws \SilverStripe\ORM\ValidationException
+     */
+    protected function migrateLinks()
+    {
+        $baseTable = SlideImage::singleton()->baseTable();
+
+        $tables = [
+            $baseTable,
+            "{$baseTable}_Versions",
+            "{$baseTable}_Live",
+        ];
+
+        foreach ($tables as $table) {
+            foreach ($this->yieldSingle(DB::query("SELECT * FROM \"{$table}\"")) as $record) {
+                $linkID = $record['PageLinkID'];
+                $linkLabel = $record['LinkLabel'];
+
+                $slideLink = $this->findOrMakeLink($linkID, $linkLabel);
+
+                if ($slideLink !== false && $slideLink instanceof Link) {
+                    DB::prepared_query(
+                        "UPDATE \"{$table}\" SET \"SlideLinkID\" = ? WHERE \"ID\" = ?",
+                        [$slideLink->ID, $record['ID']]
+                    );
+                }
+            }
+        }
+    }
+
+    /**
+     * @param $list
+     * @return \Generator
+     */
+    private function yieldSingle($list)
+    {
+        foreach ($list as $item) {
+            yield $item;
+        }
+    }
+
+    /**
+     * @param int $linkID
+     * @param string $linkLabel
+     * @return bool|mixed|Link
+     * @throws \SilverStripe\ORM\ValidationException
+     */
+    private function findOrMakeLink($linkID = 0, $linkLabel = '')
+    {
+        if (!$linkID || !($page = SiteTree::get()->byID($linkID))) {
+            return false;
+        }
+
+        if (isset($this->getKnownLinks()[$linkID])) {
+            return $this->getKnownLinks()[$linkID];
+        }
+
+        $link = Link::create();
+        $link->Type = 'SiteTree';
+        $link->SiteTreeID = $linkID;
+        $link->Template = 'button';
+
+        if ($linkLabel !== null && $linkLabel !== '') {
+            $link->Title = $linkLabel;
+        } else {
+            $link->Title = $page->Title;
+        }
+
+        $link->write();
+
+        $this->addKnownLink($linkID, $link);
+
+        return $link;
+    }
+
+    /**
+     * @param $linkID
+     * @param $linkableLinkID
+     * @return $this
+     */
+    private function addKnownLink($linkID, $linkableLinkID)
+    {
+        $this->known_links[$linkID] = $linkableLinkID;
+
+        return $this;
+    }
+
+    /**
+     * @return array
+     */
+    private function getKnownLinks()
+    {
+        return $this->known_links;
+    }
+}