Add PHPCR ODM 2.x compat

BabDev · Apr 10, 2024 · ca6da20 · ca6da20
1 parent 0dda4d5
commit ca6da20
Show file tree

Hide file tree

Showing 8 changed files with 40 additions and 5 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -203,7 +203,7 @@ jobs:
             composer-flags: '--prefer-stable --prefer-dist --prefer-lowest'
             next-php: false
           - php: '8.3'
-            orm: '^3.0@beta'
+            orm: '^3.0'
             composer-flags: '--prefer-stable --prefer-dist'
             next-php: false
           - php: '8.3'
@@ -248,19 +248,26 @@ jobs:
       fail-fast: true
       matrix:
         php: ['8.1', '8.2', '8.3']
+        odm: ['']
         composer-flags: ['--prefer-stable --prefer-dist']
         next-php: [false]
         patch-packages: [false]
         include:
           - php: '8.1'
+            odm: ''
             composer-flags: '--prefer-stable --prefer-dist --prefer-lowest'
             next-php: false
           - php: '8.3'
+            odm: '^2.0'
+            composer-flags: '--prefer-dist --prefer-lowest'
+            next-php: false
+          - php: '8.3'
+            odm: ''
             composer-flags: '--prefer-stable --prefer-dist'
             next-php: false
             patch-packages: true
 
-    name: Doctrine PHPCR ODM - PHP ${{ matrix.php }} - Composer ${{ matrix.composer-flags }}${{ matrix.patch-packages && ' (Patched Packages)' || '' }}
+    name: Doctrine PHPCR ODM${{ matrix.odm != '' && format(' {0}', matrix.odm) || '' }} - PHP ${{ matrix.php }} - Composer ${{ matrix.composer-flags }}${{ matrix.patch-packages && ' (Patched Packages)' || '' }}
 
     steps:
       - name: Checkout code
@@ -278,6 +285,10 @@ jobs:
         if: ${{ matrix.patch-packages }}
         run: php .github/patch-packages.php
 
+      - name: Restrict ODM version
+        if: ${{ matrix.odm }}
+        run: (cd lib/Adapter/Doctrine/PHPCRODM && composer require --no-update doctrine/phpcr-odm:${{ matrix.odm }})
+
       - name: Install dependencies
         run: (cd lib/Adapter/Doctrine/PHPCRODM && composer update ${{ matrix.composer-flags }})
 

diff --git a/composer.json b/composer.json
@@ -13,7 +13,7 @@
         "doctrine/dbal": "^3.5 || ^4.0",
         "doctrine/mongodb-odm": "^2.4",
         "doctrine/orm": "^2.14 || ^3.0",
-        "doctrine/phpcr-odm": "^1.7",
+        "doctrine/phpcr-odm": "^1.7 || ^2.0",
         "dg/bypass-finals": "^1.5.1",
         "jackalope/jackalope-doctrine-dbal": "^1.9",
         "phpstan/extension-installer": "^1.3.1",

diff --git a/lib/Adapter/Doctrine/PHPCRODM/QueryAdapter.php b/lib/Adapter/Doctrine/PHPCRODM/QueryAdapter.php
@@ -5,6 +5,7 @@
 use Doctrine\ODM\PHPCR\Query\Builder\QueryBuilder;
 use Doctrine\ODM\PHPCR\Query\Query;
 use Pagerfanta\Adapter\AdapterInterface;
+use PHPCR\Query\QueryResultInterface;
 
 /**
  * Adapter which calculates pagination from a Doctrine PHPCR ODM QueryBuilder.

diff --git a/lib/Adapter/Doctrine/PHPCRODM/Tests/bootstrap.php b/lib/Adapter/Doctrine/PHPCRODM/Tests/bootstrap.php
@@ -0,0 +1,11 @@
+<?php declare(strict_types=1);
+
+use DG\BypassFinals;
+
+require __DIR__.'/../vendor/autoload.php';
+
+BypassFinals::setWhitelist([
+    '*/Doctrine/ODM/PHPCR/*',
+]);
+
+BypassFinals::enable(bypassReadOnly: false);
diff --git a/lib/Adapter/Doctrine/PHPCRODM/composer.json b/lib/Adapter/Doctrine/PHPCRODM/composer.json
@@ -6,10 +6,11 @@
     "license": "MIT",
     "require": {
         "php": "^8.1",
-        "doctrine/phpcr-odm": "^1.7",
+        "doctrine/phpcr-odm": "^1.7|| ^2.0",
         "pagerfanta/core": "^3.7 || ^4.0"
     },
     "require-dev": {
+        "dg/bypass-finals": "^1.5.1",
         "jackalope/jackalope-doctrine-dbal": "^1.9",
         "phpunit/phpunit": "^10.5"
     },

diff --git a/lib/Adapter/Doctrine/PHPCRODM/phpunit.xml.dist b/lib/Adapter/Doctrine/PHPCRODM/phpunit.xml.dist
@@ -2,7 +2,7 @@
 <phpunit
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         colors="true"
-        bootstrap="vendor/autoload.php"
+        bootstrap="Tests/bootstrap.php"
         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
 >
     <testsuites>

diff --git a/phpstan-baseline.neon b/phpstan-baseline.neon
@@ -65,6 +65,16 @@ parameters:
 			count: 3
 			path: lib/Adapter/Doctrine/ORM/Tests/QueryAdapterTest.php
 
+		-
+			message: "#^Call to an undefined method Doctrine\\\\Common\\\\Collections\\\\Collection\\|PHPCR\\\\Query\\\\QueryResultInterface\\:\\:getRows\\(\\)\\.$#"
+			count: 1
+			path: lib/Adapter/Doctrine/PHPCRODM/QueryAdapter.php
+
+		-
+			message: "#^Call to an undefined method Iterator\\<mixed, PHPCR\\\\Query\\\\RowInterface\\>\\:\\:count\\(\\)\\.$#"
+			count: 1
+			path: lib/Adapter/Doctrine/PHPCRODM/QueryAdapter.php
+
 		-
 			message: "#^Call to static method PHPUnit\\\\Framework\\\\Assert\\:\\:assertSame\\(\\) with ArrayIterator\\<\\*NEVER\\*, \\*NEVER\\*\\> and iterable will always evaluate to false\\.$#"
 			count: 1

diff --git a/tests/bootstrap.php b/tests/bootstrap.php
@@ -6,6 +6,7 @@
 
 BypassFinals::setWhitelist([
     '*/Doctrine/ODM/MongoDB/*',
+    '*/Doctrine/ODM/PHPCR/*',
 ]);
 
 BypassFinals::enable(bypassReadOnly: false);