From eac9485924f9d3f534efde3903246ec56c90cf71 Mon Sep 17 00:00:00 2001
From: Brett Tasker <brett@silverstripe.com>
Date: Sat, 22 Apr 2017 21:24:33 +1200
Subject: [PATCH] SS 4.0 - Update Variants

---
 code/search/variants/SearchVariant.php                     | 7 ++++++-
 .../variants/SearchVariantSiteTreeSubsitesPolyhome.php     | 4 +++-
 code/search/variants/SearchVariantSubsites.php             | 6 +++++-
 code/search/variants/SearchVariantVersioned.php            | 7 +------
 4 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/code/search/variants/SearchVariant.php b/code/search/variants/SearchVariant.php
index 57f6479..cb03fbb 100644
--- a/code/search/variants/SearchVariant.php
+++ b/code/search/variants/SearchVariant.php
@@ -1,5 +1,7 @@
 <?php
+
 namespace SilverStripe\FullTextSearch\Search\Variants;
+
 use SilverStripe\Core\ClassInfo;
 /**
  * A Search Variant handles decorators and other situations where the items to reindex or search through are modified
@@ -22,7 +24,10 @@ abstract class SearchVariant
      * Return false if there is something missing from the environment (probably a
      * not installed module) that means this variant can't apply to any class
      */
-    abstract public function appliesToEnvironment();
+    public function appliesToEnvironment()
+    {
+        return true;
+    }
 
     /**
      * Return true if this variant applies to the passed class & subclass
diff --git a/code/search/variants/SearchVariantSiteTreeSubsitesPolyhome.php b/code/search/variants/SearchVariantSiteTreeSubsitesPolyhome.php
index 5101dd1..58d5c81 100644
--- a/code/search/variants/SearchVariantSiteTreeSubsitesPolyhome.php
+++ b/code/search/variants/SearchVariantSiteTreeSubsitesPolyhome.php
@@ -1,6 +1,8 @@
 <?php
 namespace SilverStripe\FullTextSearch\Search\Variants;
 
+use SilverStripe\ORM\Queries\SQLSelect;
+
 class SearchVariantSiteTreeSubsitesPolyhome extends SearchVariant
 {
     public function appliesToEnvironment()
@@ -78,7 +80,7 @@ class SearchVariantSiteTreeSubsitesPolyhome extends SearchVariant
             }
 
             if (self::$subsites === null) {
-                $query = new SQLQuery('ID', 'Subsite');
+                $query = new SQLSelect('ID', 'Subsite');
                 self::$subsites = array_merge(array('0'), $query->execute()->column());
             }
 
diff --git a/code/search/variants/SearchVariantSubsites.php b/code/search/variants/SearchVariantSubsites.php
index 1f62f71..f331407 100644
--- a/code/search/variants/SearchVariantSubsites.php
+++ b/code/search/variants/SearchVariantSubsites.php
@@ -1,5 +1,9 @@
 <?php
+
 namespace SilverStripe\FullTextSearch\Search\Variants;
+
+use SilverStripe\ORM\Queries\SQLSelect;
+
 class SearchVariantSubsites extends SearchVariant
 {
     public function appliesToEnvironment()
@@ -74,7 +78,7 @@ class SearchVariantSubsites extends SearchVariant
     public function extractManipulationWriteState(&$writes)
     {
         $self = get_class($this);
-        $query = new SQLQuery('"ID"', '"Subsite"');
+        $query = new SQLSelect('"ID"', '"Subsite"');
         $subsites = array_merge(array('0'), $query->execute()->column());
 
         foreach ($writes as $key => $write) {
diff --git a/code/search/variants/SearchVariantVersioned.php b/code/search/variants/SearchVariantVersioned.php
index 767bdec..e155d96 100644
--- a/code/search/variants/SearchVariantVersioned.php
+++ b/code/search/variants/SearchVariantVersioned.php
@@ -3,14 +3,9 @@ namespace SilverStripe\FullTextSearch\Search\Variants;
 
 class SearchVariantVersioned extends SearchVariant
 {
-    public function appliesToEnvironment()
-    {
-        return class_exists('Versioned');
-    }
-
     public function appliesTo($class, $includeSubclasses)
     {
-        return SearchIntrospection::has_extension($class, 'Versioned', $includeSubclasses);
+        return SearchIntrospection::has_extension($class, Versioned::class, $includeSubclasses);
     }
 
     public function currentState()