From 70a794bbd5e7fa44dd7db3aecb68bdb1a38eadf4 Mon Sep 17 00:00:00 2001
From: Guy Sartorelli <guy.sartorelli@silverstripe.com>
Date: Thu, 16 Feb 2023 12:20:13 +1300
Subject: [PATCH] FIX Always use ?int for subsite state

---
 src/Search/Variants/SearchVariantSubsites.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Search/Variants/SearchVariantSubsites.php b/src/Search/Variants/SearchVariantSubsites.php
index 9c1c186..aea6008 100644
--- a/src/Search/Variants/SearchVariantSubsites.php
+++ b/src/Search/Variants/SearchVariantSubsites.php
@@ -49,7 +49,7 @@ class SearchVariantSubsites extends SearchVariant
 
     public function currentState()
     {
-        return (string) SubsiteState::singleton()->getSubsiteId();
+        return SubsiteState::singleton()->getSubsiteId();
     }
 
     public function reindexStates()
@@ -74,8 +74,8 @@ class SearchVariantSubsites extends SearchVariant
 
         if (is_numeric($state)) {
             $state = (int) $state;
-        } else {
-            throw new InvalidArgumentException("Invalid state ID. State ID should be number.");
+        } elseif ($state !== null) {
+            throw new InvalidArgumentException("Invalid state ID. State ID should be number or null.");
         }
 
         // Note: Setting directly to the SubsiteState because we don't want the subsite ID to be persisted