FIX Remove infinite loop by reverting #127 (#129)

Revert "FIX Allow extension methods to be used in lumberjack (#127)" This reverts commit fcdfcdf.
silverstripe · Jul 31, 2023 · ecefcdc · ecefcdc
1 parent fcdfcdf
commit ecefcdc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/Forms/GridFieldConfig_Lumberjack.php b/src/Forms/GridFieldConfig_Lumberjack.php
@@ -40,7 +40,5 @@ public function __construct($itemsPerPage = null)
         $this->addComponent(new GridFieldSiteTreeState());
 
         $pagination->setThrowExceptionOnBadDataType(true);
-
-        $this->extend('updateConfig');
     }
 }
diff --git a/src/Model/Lumberjack.php b/src/Model/Lumberjack.php
@@ -175,7 +175,7 @@ public function liveChildren($showAll = false, $onlyDeletedFromStage = false)
      */
     protected function getLumberjackTitle()
     {
-        if ($this->getOwner()->hasMethod('getLumberjackTitle')) {
+        if (method_exists($this->owner, 'getLumberjackTitle')) {
             return $this->owner->getLumberjackTitle();
         }
 
@@ -189,7 +189,7 @@ protected function getLumberjackTitle()
      */
     protected function getLumberjackGridFieldConfig()
     {
-        if ($this->getOwner()->hasMethod('getLumberjackGridFieldConfig')) {
+        if (method_exists($this->owner, 'getLumberjackGridFieldConfig')) {
             return $this->owner->getLumberjackGridFieldConfig();
         }
 
@@ -227,7 +227,7 @@ protected function shouldFilter()
      */
     public function getLumberjackPagesForGridfield($excluded = array())
     {
-        if ($this->getOwner()->hasMethod('getLumberjackPagesForGridfield')) {
+        if (method_exists($this->owner, 'getLumberjackPagesForGridfield')) {
             return $this->owner->getLumberjackPagesForGridfield($excluded);
         }