ENH PHP 8.2 support

composer.json

@@ -27,7 +27,7 @@
         "composer/installers": "^2.2",
         "guzzlehttp/guzzle": "^7.5.0",
         "guzzlehttp/psr7": "^2.4.0",
-        "embed/embed": "^4.4.4",
+        "embed/embed": "^4.4.7",
         "league/csv": "^9.8.0",
         "m1/env": "^2.2.0",
         "monolog/monolog": "^3.2.0",

src/Control/HTTPRequest.php

@@ -579,7 +579,7 @@ public function match($pattern, $shiftOnSuccess = false)
                         $shiftCount = sizeof($patternParts ?? []);
                         $remaining = count($this->dirParts ?? []) - $i;
                         for ($j = 1; $j <= $remaining; $j++) {
-                            $arguments["$${j}"] = $this->dirParts[$j + $i - 1];
+                            $arguments['$' . $j] = $this->dirParts[$j + $i - 1];
                         }
                         $patternParts = array_merge($patternParts, array_keys($arguments ?? []));
                         break;

src/Dev/Constraint/SSListContains.php

@@ -20,6 +20,8 @@ class SSListContains extends Constraint implements TestOnly
      */
     protected $matches = [];
 
+    protected SSListExporter $exporter;
+
     /**
      * Check if the list has left over items that don't match
      *

src/Dev/Constraint/SSListContainsOnlyMatchingItems.php

@@ -19,6 +19,8 @@ class SSListContainsOnlyMatchingItems extends Constraint implements TestOnly
      */
     private $match;
 
+    protected SSListExporter $exporter;
+
     /**
      * @var ViewableDataContains
      */

src/Dev/FixtureBlueprint.php

@@ -34,6 +34,8 @@ class FixtureBlueprint
      */
     protected $class;
 
+    private FixtureFactory $factory;
+
     /**
      * @var array
      */
@@ -70,6 +72,18 @@ public function __construct($name, $class = null, $defaults = [])
         $this->defaults = $defaults;
     }
 
+    public function getFactory(): FixtureFactory
+    {
+        return $this->factory;
+    }
+
+    public function setFactory(FixtureFactory $factory): static
+    {
+        $this->factory = $factory;
+        return $this;
+    }
+
+
     /**
      * @param string $identifier Unique identifier for this fixture type
      * @param array $data Map of property names to their values.

src/Dev/SapphireTest.php

@@ -161,6 +161,8 @@ abstract class SapphireTest extends TestCase implements TestOnly
      */
     protected static $tempDB = null;
 
+    protected FixtureFactory|bool $fixtureFactory;
+
     /**
      * @return TempDatabase
      */

src/ORM/Connect/MySQLDatabase.php

@@ -65,6 +65,8 @@ class MySQLDatabase extends Database implements TransactionManager
      */
     private $transactionManager = null;
 
+    private int $transactionNesting = 0;
+
     /**
      * Default collation
      *

src/ORM/Connect/MySQLStatement.php

@@ -73,7 +73,6 @@ public function __construct($statement, $metadata)
     public function __destruct()
     {
         $this->statement->close();
-        $this->currentRecord = false;
     }
 
     /**

src/View/ViewableData.php

@@ -66,6 +66,11 @@ class ViewableData implements IteratorAggregate
      */
     private static $casting_cache = [];
 
+    /**
+     * Acts as a PHP 8.2+ compliant replacement for dynamic properties
+     */
+    private array $data = [];
+
     // -----------------------------------------------------------------------------------------------------------------
 
     /**
@@ -191,7 +196,7 @@ public function getFailover()
      */
     public function hasField($field)
     {
-        return property_exists($this, $field ?? '');
+        return property_exists($this, $field) || isset($this->data[$field]);
     }
 
     /**
@@ -202,7 +207,10 @@ public function hasField($field)
      */
     public function getField($field)
     {
-        return $this->$field;
+        if (property_exists($this, $field)) {
+            return $this->$field;
+        }
+        return $this->data[$field];
     }
 
     /**
@@ -215,7 +223,13 @@ public function getField($field)
     public function setField($field, $value)
     {
         $this->objCacheClear();
-        $this->$field = $value;
+        // prior to PHP 8.2 support ViewableData::setField() simply used `$this->field = $value;`
+        // so the following logic essentially mimics this behaviour, though without the use
+        // of now deprecated dynamic properties
+        if (property_exists($this, $field)) {
+            $this->$field = $value;
+        }
+        $this->data[$field] = $value;
         return $this;
     }
 
@@ -509,14 +523,14 @@ public function obj($fieldName, $arguments = [], $cache = false, $cacheName = nu
      * A simple wrapper around {@link ViewableData::obj()} that automatically caches the result so it can be used again
      * without re-running the method.
      *
-     * @param string $field
+     * @param string $fieldName
      * @param array $arguments
      * @param string $identifier an optional custom cache identifier
      * @return Object|DBField
      */
-    public function cachedCall($field, $arguments = [], $identifier = null)
+    public function cachedCall($fieldName, $arguments = [], $identifier = null)
     {
-        return $this->obj($field, $arguments, true, $identifier);
+        return $this->obj($fieldName, $arguments, true, $identifier);
     }
 
     /**

src/View/ViewableData_Customised.php

@@ -59,19 +59,26 @@ public function hasMethod($method)
         return $this->customised->hasMethod($method) || $this->original->hasMethod($method);
     }
 
-    public function cachedCall($field, $arguments = null, $identifier = null)
+    public function cachedCall($fieldName, $arguments = null, $identifier = null)
     {
-        if ($this->customised->hasMethod($field) || $this->customised->hasField($field)) {
-            return $this->customised->cachedCall($field, $arguments, $identifier);
+        if ($this->customisedHas($fieldName)) {
+            return $this->customised->cachedCall($fieldName, $arguments, $identifier);
         }
-        return $this->original->cachedCall($field, $arguments, $identifier);
+        return $this->original->cachedCall($fieldName, $arguments, $identifier);
     }
 
     public function obj($fieldName, $arguments = null, $cache = false, $cacheName = null)
     {
-        if ($this->customised->hasField($fieldName) || $this->customised->hasMethod($fieldName)) {
+        if ($this->customisedHas($fieldName)) {
             return $this->customised->obj($fieldName, $arguments, $cache, $cacheName);
         }
         return $this->original->obj($fieldName, $arguments, $cache, $cacheName);
     }
+
+    private function customisedHas(string $fieldName): bool
+    {
+        return property_exists($this->customised, $fieldName) ||
+            $this->customised->hasField($fieldName) ||
+            $this->customised->hasMethod($fieldName);
+    }
 }

tests/php/Core/Injector/AopProxyServiceTest/AnotherService.php

@@ -4,5 +4,7 @@
 
 class AnotherService
 {
+    public $config_property;
+
     public $filters = [];
 }

tests/php/Core/Injector/AopProxyServiceTest/SampleService.php

@@ -4,6 +4,7 @@
 
 class SampleService
 {
+    public $auto;
     public $constructorVarOne;
     public $constructorVarTwo;
 

tests/php/Core/Injector/InjectorTest/TestObject.php

@@ -6,6 +6,7 @@
 
 class TestObject implements TestOnly
 {
+    public $auto;
 
     public $sampleService;
 

tests/php/Dev/FixtureBlueprintTest.php

@@ -17,6 +17,8 @@ class FixtureBlueprintTest extends SapphireTest
 
     protected $usesDatabase = true;
 
+    private int $_called = 0;
+
     protected static $extra_dataobjects = [
         TestDataObject::class,
         DataObjectRelation::class,

tests/php/Forms/GridField/GridFieldFilterHeaderTest.php

@@ -138,18 +138,17 @@ public function testHandleActionReset()
     public function testGetSearchForm()
     {
         $searchForm = $this->component->getSearchForm($this->gridField);
-
         $this->assertTrue($searchForm instanceof Form);
-        $this->assertEquals('Search__q', $searchForm->fields[0]->Name);
-        $this->assertEquals('Search__Name', $searchForm->fields[1]->Name);
-        $this->assertEquals('Search__City', $searchForm->fields[2]->Name);
-        $this->assertEquals('Search__Cheerleader__Hat__Colour', $searchForm->fields[3]->Name);
+        $fields = $searchForm->Fields()->toArray();
+        $this->assertEquals('Search__q', $fields[0]->Name);
+        $this->assertEquals('Search__Name', $fields[1]->Name);
+        $this->assertEquals('Search__City', $fields[2]->Name);
+        $this->assertEquals('Search__Cheerleader__Hat__Colour', $fields[3]->Name);
         $this->assertEquals('TeamsSearchForm', $searchForm->Name);
-        $this->assertEquals('cms-search-form', $searchForm->extraClasses['cms-search-form']);
-
-        foreach ($searchForm->fields as $field) {
-            $this->assertEquals('stacked', $field->extraClasses['stacked']);
-            $this->assertEquals('no-change-track', $field->extraClasses['no-change-track']);
+        $this->assertTrue($searchForm->hasExtraClass('cms-search-form'));
+        foreach ($fields as $field) {
+            $this->assertTrue($field->hasExtraClass('stacked'));
+            $this->assertTrue($field->hasExtraClass('no-change-track'));
         }
     }
 

tests/php/View/ArrayDataTest/NonEmptyObject.php

@@ -6,8 +6,9 @@
 
 class NonEmptyObject implements TestOnly
 {
-
-    static $c = "Cucumber";
+    public $a;
+    public $b;
+    public static $c = "Cucumber";
 
     public function __construct()
     {

tests/php/View/Embed/MockResponse.php

@@ -8,7 +8,7 @@
 class MockResponse implements ResponseInterface
 {
     private EmbedUnitTest $unitTest;
-    private string $firstReponse;
+    private string $firstResponse;
     private string $secondResponse;
 
     public function __construct(EmbedUnitTest $unitTest, string $firstResponse, string $secondResponse)

thirdparty/php-peg/Parser.php

@@ -9,6 +9,13 @@
  *  the bracket if a failed match + restore has moved the current position backwards - so we have to check that too.
  */
 class ParserRegexp {
+
+	public $parser;
+	public $rx;
+	public $matches;
+	public $match_pos;
+	public $check_pos;
+
 	function __construct( $parser, $rx ) {
 		$this->parser = $parser ;
 		$this->rx = $rx . 'Sx' ;