From 285c61cc19aaa66c51d1f8b614d5cb1b37012ca3 Mon Sep 17 00:00:00 2001
From: Will Rossiter <will@fullscreen.io>
Date: Mon, 23 Oct 2017 22:36:58 +1300
Subject: [PATCH] =?UTF-8?q?FIX=20Don=E2=80=99t=20override=20ItemRequest=20?=
 =?UTF-8?q?if=20already=20versioned?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

If a developer customises the ItemRequest don’t override it as long as it is already a versioned item hander.
---
 src/VersionedGridFieldDetailForm.php | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/VersionedGridFieldDetailForm.php b/src/VersionedGridFieldDetailForm.php
index 8a9ef468..fa593217 100644
--- a/src/VersionedGridFieldDetailForm.php
+++ b/src/VersionedGridFieldDetailForm.php
@@ -20,12 +20,21 @@ class VersionedGridFieldDetailForm extends Extension
      */
     public function updateItemRequestClass(&$class, $gridField, $record, $requestHandler)
     {
-        // Conditionally use a versioned item handler
+        // Conditionally use a versioned item handler if it doesn't already have one.
         if ($record
             && $record->has_extension(Versioned::class)
             && $record->config()->get('versioned_gridfield_extensions')
         ) {
-            $class = VersionedGridFieldItemRequest::class;
+            // don't override custom classes if they already subclass this
+            if (!$class) {
+                $class = VersionedGridFieldItemRequest::class;
+
+                return;
+            }
+
+            if (!is_subclass_of($class, VersionedGridFieldItemRequest::class)) {
+                $class = VersionedGridFieldItemRequest::class;
+            }
         }
     }
 }