FIX Allow public extension getter methods to work (#10676)

Accidentally broke this in #10670

src/Core/CustomMethods.php

@@ -147,7 +147,15 @@ protected function registerExtraMethodCallback($name, $callback)
      */
     public function hasMethod($method)
     {
-        return method_exists($this, $method ?? '') || $this->getExtraMethodConfig($method);
+        return method_exists($this, $method ?? '') || $this->hasCustomMethod($method);
+    }
+
+    /**
+     * Determines if a custom method with this name is defined.
+     */
+    protected function hasCustomMethod($method): bool
+    {
+        return $this->getExtraMethodConfig($method) !== null;
     }
 
     /**

src/View/ViewableData.php

@@ -247,11 +247,14 @@ public function setField($field, $value)
     private function isAccessibleMethod(string $method): bool
     {
         if (!method_exists($this, $method)) {
-            return false;
+            // Methods added via extensions are accessible
+            return $this->hasCustomMethod($method);
         }
+        // All methods defined on ViewableData are accessible to ViewableData
         if (static::class === self::class) {
             return true;
         }
+        // Private methods defined on subclasses are not accessible to ViewableData
         $reflectionMethod = new ReflectionMethod($this, $method);
         return !$reflectionMethod->isPrivate();
     }

tests/php/View/ViewableDataTest.php

@@ -7,15 +7,21 @@
 use SilverStripe\Dev\SapphireTest;
 use SilverStripe\View\ArrayData;
 use SilverStripe\View\SSViewer;
+use SilverStripe\View\Tests\ViewableDataTest\ViewableDataTestExtension;
+use SilverStripe\View\Tests\ViewableDataTest\ViewableDataTestObject;
 use SilverStripe\View\ViewableData;
-use SilverStripe\View\Tests\ViewableDataTestObject;
 
 /**
  * See {@link SSViewerTest->testCastingHelpers()} for more tests related to casting and ViewableData behaviour,
  * from a template-parsing perspective.
  */
 class ViewableDataTest extends SapphireTest
 {
+    protected static $required_extensions = [
+        ViewableDataTestObject::class => [
+            ViewableDataTestExtension::class,
+        ],
+    ];
 
     public function testCasting()
     {
@@ -213,6 +219,7 @@ public function testIsAccessibleMethod()
         $reflectionMethod = new ReflectionMethod(ViewableData::class, 'isAccessibleMethod');
         $reflectionMethod->setAccessible(true);
         $object = new ViewableDataTestObject();
+        $viewableData = new ViewableData();
 
         $output = $reflectionMethod->invokeArgs($object, ['privateMethod']);
         $this->assertFalse($output, 'Method should not be accessible');
@@ -226,8 +233,17 @@ public function testIsAccessibleMethod()
         $output = $reflectionMethod->invokeArgs($object, ['missingMethod']);
         $this->assertFalse($output, 'Method should not be accessible');
 
-        $output = $reflectionMethod->invokeArgs(new ViewableData(), ['isAccessibleProperty']);
+        $output = $reflectionMethod->invokeArgs($viewableData, ['isAccessibleProperty']);
         $this->assertTrue($output, 'Method should be accessible');
+
+        $output = $reflectionMethod->invokeArgs($object, ['publicMethodFromExtension']);
+        $this->assertTrue($output, 'Method should be accessible');
+
+        $output = $reflectionMethod->invokeArgs($object, ['protectedMethodFromExtension']);
+        $this->assertFalse($output, 'Method should not be accessible');
+
+        $output = $reflectionMethod->invokeArgs($object, ['privateMethodFromExtension']);
+        $this->assertFalse($output, 'Method should not be accessible');
     }
 
     public function testIsAccessibleProperty()

tests/php/View/ViewableDataTestObject.php → tests/php/View/ViewableDataTest/ViewableDataTestObject.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace SilverStripe\View\Tests;
+namespace SilverStripe\View\Tests\ViewableDataTest;
 
 use SilverStripe\Dev\TestOnly;
 use SilverStripe\ORM\DataObject;

tests/php/View/ViewableDataTest/ViewableDataTextExtension.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace SilverStripe\View\Tests\ViewableDataTest;
+
+use SilverStripe\Core\Extension;
+use SilverStripe\Dev\TestOnly;
+
+class ViewableDataTestExtension extends Extension implements TestOnly
+{
+    private function privateMethodFromExtension(): string
+    {
+        return 'Private function';
+    }
+
+    protected function protectedMethodFromExtension(): string
+    {
+        return 'Protected function';
+    }
+
+    public function publicMethodFromExtension(): string
+    {
+        return 'Public function';
+    }
+}