NGSTACK-1000 upgrade bundle to ibexa v5

.github/workflows/coding_standards.yml

@@ -14,6 +14,6 @@ jobs:
 
     steps:
       - uses: actions/checkout@v2
-      - uses: docker://oskarstark/php-cs-fixer-ga
+      - uses: docker://oskarstark/php-cs-fixer-ga:3.26.0
         with:
           args: --diff --dry-run

.github/workflows/tests.yml

@@ -15,15 +15,10 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: ['8.1']
-        symfony: ['~5.4.0']
+        php: ['8.3']
+        symfony: ['~7.3.0']
         phpunit: ['phpunit.xml']
         deps: ['normal']
-        include:
-          - php: '8.1'
-            symfony: '~5.4.0'
-            phpunit: 'phpunit.xml'
-            deps: 'low'
 
     steps:
       - uses: actions/checkout@v2

.gitignore

@@ -4,3 +4,4 @@ vendor
 composer.lock
 .idea
 build
+.phpunit.cache

.php-cs-fixer.php

@@ -29,6 +29,7 @@
         'single_line_comment_style' => false,
         'trailing_comma_in_multiline' => ['elements' => ['arrays', 'arguments', 'match', 'parameters']],
         'yoda_style' => false,
+        'fully_qualified_strict_types' => false,
 
         // Additional rules
         'date_time_immutable' => true,

bundle/Handler/FieldType/Handler.php

@@ -15,12 +15,9 @@
 
 abstract class Handler extends BaseHandler
 {
-    protected FieldHelper $fieldHelper;
-
-    public function __construct(FieldHelper $fieldHelper)
-    {
-        $this->fieldHelper = $fieldHelper;
-    }
+    public function __construct(
+        protected FieldHelper $fieldHelper,
+    ) {}
 
     public function getMetaTags(string $tagName, array $params = []): array
     {

bundle/Handler/FieldType/Image.php

@@ -17,27 +17,19 @@
 use Symfony\Component\HttpFoundation\RequestStack;
 
 use function ltrim;
-use function mb_strpos;
+use function str_starts_with;
 
 final class Image extends Handler
 {
-    private VariationHandler $imageVariationService;
-
-    private RequestStack $requestStack;
-
-    private LoggerInterface $logger;
-
     public function __construct(
         FieldHelper $fieldHelper,
-        VariationHandler $imageVariationService,
-        RequestStack $requestStack,
-        ?LoggerInterface $logger = null,
+        private readonly VariationHandler $imageVariationService,
+        private readonly RequestStack $requestStack,
+        private ?LoggerInterface $logger = null,
     ) {
         parent::__construct($fieldHelper);
 
-        $this->imageVariationService = $imageVariationService;
-        $this->requestStack = $requestStack;
-        $this->logger = $logger ?? new NullLogger();
+        $this->logger ??= new NullLogger();
     }
 
     protected function getFieldValue(Field $field, string $tagName, array $params = []): string
@@ -48,7 +40,7 @@ protected function getFieldValue(Field $field, string $tagName, array $params =
             try {
                 $variationUri = $this->imageVariationService->getVariation($field, $this->content->getVersionInfo(), $variationName)->uri;
 
-                if (mb_strpos($variationUri, '/') === 0 && ($request = $this->requestStack->getCurrentRequest()) !== null) {
+                if (str_starts_with($variationUri, '/') && ($request = $this->requestStack->getCurrentRequest()) !== null) {
                     $variationUri = $request->getUriForPath('/' . ltrim($variationUri, '/'));
                 }
 

bundle/Handler/Literal/CanonicalUrl.php

@@ -15,15 +15,10 @@
 
 final class CanonicalUrl implements HandlerInterface
 {
-    private RequestStack $requestStack;
-
-    private UrlGeneratorInterface $urlGenerator;
-
-    public function __construct(RequestStack $requestStack, UrlGeneratorInterface $urlGenerator)
-    {
-        $this->requestStack = $requestStack;
-        $this->urlGenerator = $urlGenerator;
-    }
+    public function __construct(
+        private readonly RequestStack $requestStack,
+        private readonly UrlGeneratorInterface $urlGenerator,
+    ) {}
 
     public function getMetaTags($tagName, array $params = []): array
     {

bundle/Handler/Literal/Url.php

@@ -15,12 +15,9 @@
 
 final class Url implements HandlerInterface
 {
-    private RequestStack $requestStack;
-
-    public function __construct(RequestStack $requestStack)
-    {
-        $this->requestStack = $requestStack;
-    }
+    public function __construct(
+        private readonly RequestStack $requestStack,
+    ) {}
 
     public function getMetaTags(string $tagName, array $params = []): array
     {

bundle/MetaTag/Collector.php

@@ -16,31 +16,24 @@
 
 final class Collector implements CollectorInterface
 {
-    private Registry $metaTagHandlers;
-
-    private ContentTypeService $contentTypeService;
-
-    private ConfigResolverInterface $configResolver;
-
-    public function __construct(Registry $metaTagHandlers, ContentTypeService $contentTypeService, ConfigResolverInterface $configResolver)
-    {
-        $this->metaTagHandlers = $metaTagHandlers;
-        $this->contentTypeService = $contentTypeService;
-        $this->configResolver = $configResolver;
-    }
+    public function __construct(
+        private readonly Registry $metaTagHandlers,
+        private readonly ContentTypeService $contentTypeService,
+        private readonly ConfigResolverInterface $configResolver,
+    ) {}
 
     public function collect(Content $content): array
     {
         $metaTags = [];
 
-        $allHandlers = $this->configResolver->hasParameter('global_handlers', 'netgen_open_graph') ?
-            $this->configResolver->getParameter('global_handlers', 'netgen_open_graph') :
-            [];
+        $allHandlers = $this->configResolver->hasParameter('global_handlers', 'netgen_open_graph')
+            ? $this->configResolver->getParameter('global_handlers', 'netgen_open_graph')
+            : [];
 
         $contentType = $this->contentTypeService->loadContentType($content->contentInfo->contentTypeId);
-        $contentTypeHandlers = $this->configResolver->hasParameter('content_type_handlers', 'netgen_open_graph') ?
-            $this->configResolver->getParameter('content_type_handlers', 'netgen_open_graph') :
-            [];
+        $contentTypeHandlers = $this->configResolver->hasParameter('content_type_handlers', 'netgen_open_graph')
+            ? $this->configResolver->getParameter('content_type_handlers', 'netgen_open_graph')
+            : [];
 
         if (isset($contentTypeHandlers[$contentType->identifier])) {
             $allHandlers = array_merge(

bundle/MetaTag/Item.php

@@ -6,15 +6,10 @@
 
 final class Item
 {
-    private string $tagName;
-
-    private string $tagValue;
-
-    public function __construct(string $tagName, string $tagValue)
-    {
-        $this->tagName = $tagName;
-        $this->tagValue = $tagValue;
-    }
+    public function __construct(
+        private readonly string $tagName,
+        private readonly string $tagValue,
+    ) {}
 
     /**
      * Returns tag name.

bundle/Templating/Twig/Extension/NetgenOpenGraphRuntime.php

@@ -13,26 +13,18 @@
 
 final class NetgenOpenGraphRuntime
 {
-    private CollectorInterface $tagCollector;
-
-    private RendererInterface $tagRenderer;
-
-    private LoggerInterface $logger;
-
     private bool $throwExceptions = true;
 
     public function __construct(
-        CollectorInterface $tagCollector,
-        RendererInterface $tagRenderer,
-        ?LoggerInterface $logger = null,
+        private readonly CollectorInterface $tagCollector,
+        private readonly RendererInterface $tagRenderer,
+        private ?LoggerInterface $logger = null,
     ) {
-        $this->tagCollector = $tagCollector;
-        $this->tagRenderer = $tagRenderer;
-        $this->logger = $logger ?? new NullLogger();
+        $this->logger ??= new NullLogger();
     }
 
     /**
-     * Sets the flag that determines if the exceptions will thrown instead of logged.
+     * Sets the flag that determines if the exceptions will be thrown instead of logged.
      */
     public function setThrowExceptions(bool $throwExceptions = true): void
     {

composer.json

@@ -18,15 +18,15 @@
         }
     ],
     "require": {
-        "php": "^8.1",
-        "ibexa/core": "^4.4",
+        "php": "^8.3",
+        "ibexa/core": "^5.0",
         "twig/twig": "^3.0"
     },
     "require-dev": {
-        "ibexa/fieldtype-richtext": "^4.4",
-        "phpunit/phpunit": "^9.6",
-        "matthiasnoback/symfony-config-test": "^5.0",
-        "matthiasnoback/symfony-dependency-injection-test": "^5.0"
+        "ibexa/fieldtype-richtext": "^5.0",
+        "phpunit/phpunit": "^12.0",
+        "matthiasnoback/symfony-config-test": "^6.0",
+        "matthiasnoback/symfony-dependency-injection-test": "^6.0"
     },
     "minimum-stability": "dev",
     "prefer-stable": true,

phpunit.xml

@@ -1,12 +1,10 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit bootstrap="vendor/autoload.php"
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/12.4/phpunit.xsd"
+         bootstrap="vendor/autoload.php"
+         cacheDirectory=".phpunit.cache"
          backupGlobals="false"
-         backupStaticAttributes="false"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
          colors="true"
-         verbose="true"
          processIsolation="false"
          stopOnFailure="false"
 >
@@ -15,18 +13,17 @@
             <directory>tests</directory>
         </testsuite>
     </testsuites>
-    <filter>
-        <whitelist>
+
+    <source>
+        <include>
             <directory suffix=".php">bundle</directory>
-            <exclude>
-                <directory>bundle/Resources</directory>
-            </exclude>
-        </whitelist>
-    </filter>
+        </include>
+        <exclude>
+            <directory>bundle/Resources</directory>
+        </exclude>
+    </source>
+
     <logging>
-        <log type="junit" target="build/report.junit.xml"/>
-        <log type="coverage-html" target="build/coverage"/>
-        <log type="coverage-text" target="build/coverage.txt"/>
-        <log type="coverage-clover" target="build/logs/clover.xml"/>
+        <junit outputFile="build/report.junit.xml"/>
     </logging>
 </phpunit>

tests/DependencyInjection/NetgenOpenGraphExtensionTest.php

@@ -6,12 +6,11 @@
 
 use Matthias\SymfonyDependencyInjectionTest\PhpUnit\AbstractExtensionTestCase;
 use Netgen\Bundle\OpenGraphBundle\DependencyInjection\NetgenOpenGraphExtension;
+use PHPUnit\Framework\Attributes\DoesNotPerformAssertions;
 
 final class NetgenOpenGraphExtensionTest extends AbstractExtensionTestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
+    #[DoesNotPerformAssertions]
     public function testItSetsValidContainerParameters(): void
     {
         $this->container->setParameter('ibexa.site_access.list', []);

tests/Handler/Literal/UrlTest.php

@@ -8,6 +8,7 @@
 use Netgen\Bundle\OpenGraphBundle\Handler\HandlerInterface;
 use Netgen\Bundle\OpenGraphBundle\Handler\Literal\Url;
 use Netgen\Bundle\OpenGraphBundle\MetaTag\Item;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
@@ -37,9 +38,7 @@ public function testGettingTagsWithEmptyParams(): void
         $this->url->getMetaTags('some_tag');
     }
 
-    /**
-     * @dataProvider validResultProvider
-     */
+    #[DataProvider('validResultProvider')]
     public function testGettingTagsWithValidResult(string $input, string $output): void
     {
         $request = Request::create('https://domain.com');
@@ -54,7 +53,7 @@ public function testGettingTagsWithValidResult(string $input, string $output): v
         self::assertSame($output, $result[0]->getTagValue());
     }
 
-    public function validResultProvider(): iterable
+    public static function validResultProvider(): iterable
     {
         return [
             ['https://other.domain.com/some/path', 'https://other.domain.com/some/path'],
@@ -66,9 +65,7 @@ public function validResultProvider(): iterable
         ];
     }
 
-    /**
-     * @dataProvider validResultProviderWithoutRequest
-     */
+    #[DataProvider('validResultProviderWithoutRequest')]
     public function testGettingTagsWithValidResultAndWithoutRequest(string $input, string $output): void
     {
         $result = $this->url->getMetaTags('some_tag', [$input]);
@@ -79,7 +76,7 @@ public function testGettingTagsWithValidResultAndWithoutRequest(string $input, s
         self::assertSame($output, $result[0]->getTagValue());
     }
 
-    public function validResultProviderWithoutRequest(): iterable
+    public static function validResultProviderWithoutRequest(): iterable
     {
         return [
             ['https://other.domain.com/some/path', 'https://other.domain.com/some/path'],

tests/Handler/RegistryTest.php

@@ -20,15 +20,15 @@ protected function setUp(): void
 
     public function testAddingHandlers(): void
     {
-        $handler = $this->getMockForAbstractClass(HandlerInterface::class);
+        $handler = $this->createStub(HandlerInterface::class);
         $this->registry->addHandler('some_handler', $handler);
 
         self::assertSame($this->registry->getHandler('some_handler'), $handler);
     }
 
     public function testGettingHandlers(): void
     {
-        $handler = $this->getMockForAbstractClass(HandlerInterface::class);
+        $handler = $this->createStub(HandlerInterface::class);
         $this->registry->addHandler('some_handler', $handler);
 
         $returnedHandler = $this->registry->getHandler('some_handler');