Upgrade to PHP 8.1 and upgrade deps

Author: loevgaard

.gitattributes

@@ -1,5 +1,3 @@
-* text=auto
-
 /.github                        export-ignore
 /tests                          export-ignore
 /.editorconfig                  export-ignore

.github/FUNDING.yml

@@ -0,0 +1 @@
+github: Setono

.github/dependabot.yml

@@ -0,0 +1,30 @@
+# See https://github.com/Roave/BackwardCompatibilityCheck
+
+name: "Backwards Compatibility Check"
+
+on:
+    pull_request: ~
+
+jobs:
+    backwards-compatibility-check:
+        name: "Backwards Compatibility Check"
+
+        runs-on: "ubuntu-latest"
+
+        steps:
+            -   name: "Checkout"
+                uses: "actions/checkout@v4"
+                with:
+                    fetch-depth: 0
+
+            -   name: "Setup PHP, with composer and extensions"
+                uses: "shivammathur/setup-php@v2"
+                with:
+                    php-version: "8.1"
+                    coverage: "none"
+
+            -   name: "Install tool"
+                run: "composer global require roave/backward-compatibility-check"
+
+            -   name: "Check for BC breaks"
+                run: "~/.composer/vendor/bin/roave-backward-compatibility-check --from=origin/${{ github.event.pull_request.base.ref }} --format=github-actions"

.github/workflows/backwards-compatibility-check.yaml

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2022 Setono
+Copyright (c) 2025 Setono
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

LICENSE

@@ -0,0 +1,7 @@
+<?php
+
+use ShipMonk\ComposerDependencyAnalyser\Config\Configuration;
+
+return (new Configuration())
+    ->addPathToExclude(__DIR__ . '/tests')
+;

composer-dependency-analyser.php

@@ -10,7 +10,7 @@
         }
     ],
     "require": {
-        "php": ">=7.4",
+        "php": ">=8.1",
         "ext-json": "*",
         "composer-runtime-api": "^2.0",
         "facebook/php-business-sdk": "^14.0",
@@ -26,7 +26,8 @@
         "nyholm/psr7": "^1.5",
         "phpunit/phpunit": "^9.5",
         "psalm/plugin-phpunit": "^0.17",
-        "setono/code-quality-pack": "^2.2"
+        "setono/code-quality-pack": "^2.2",
+        "shipmonk/composer-dependency-analyser": "^1.8"
     },
     "prefer-stable": true,
     "autoload": {

composer-require-checker.json

@@ -2,12 +2,12 @@
 
 declare(strict_types=1);
 
-use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
-use Symplify\EasyCodingStandard\ValueObject\Option;
+use Symplify\EasyCodingStandard\Config\ECSConfig;
 
-return static function (ContainerConfigurator $containerConfigurator): void {
-    $containerConfigurator->import('vendor/sylius-labs/coding-standard/ecs.php');
-    $containerConfigurator->parameters()->set(Option::PATHS, [
-        'src', 'tests'
+return static function (ECSConfig $config): void {
+    $config->import('vendor/sylius-labs/coding-standard/ecs.php');
+    $config->paths([
+        'src',
+        'tests',
     ]);
 };

composer.json

@@ -60,7 +60,7 @@ public function sendEvent(Event $event): void
 
             $request = $requestFactory->createRequest(
                 'POST',
-                sprintf('https://graph.facebook.com/%s/%s/events', self::getEndpointVersion(), $pixel->id)
+                sprintf('https://graph.facebook.com/%s/%s/events', self::getEndpointVersion(), $pixel->id),
             )
             ->withHeader('Content-Type', 'application/x-www-form-urlencoded')
             ->withHeader('Accept', 'application/json')
@@ -80,7 +80,7 @@ private function getHttpClient(): HttpClientInterface
             if (!class_exists(Curl::class)) {
                 throw ClientException::missingDependency(
                     Curl::class,
-                    sprintf('Either set the http client with %s or run composer require kriswallsmith/buzz', self::class . '::setHttpClient()')
+                    sprintf('Either set the http client with %s or run composer require kriswallsmith/buzz', self::class . '::setHttpClient()'),
                 );
             }
 
@@ -101,7 +101,7 @@ private function getRequestFactory(): RequestFactoryInterface
             if (!class_exists(Psr17Factory::class)) {
                 throw ClientException::missingDependency(
                     Psr17Factory::class,
-                    sprintf('Either set the request factory with %s or run composer require nyholm/psr7', self::class . '::setRequestFactory()')
+                    sprintf('Either set the request factory with %s or run composer require nyholm/psr7', self::class . '::setRequestFactory()'),
                 );
             }
 
@@ -122,7 +122,7 @@ private function getResponseFactory(): ResponseFactoryInterface
             if (!class_exists(Psr17Factory::class)) {
                 throw ClientException::missingDependency(
                     Psr17Factory::class,
-                    sprintf('Either set the response factory with %s or run composer require nyholm/psr7', self::class . '::setResponseFactory()')
+                    sprintf('Either set the response factory with %s or run composer require nyholm/psr7', self::class . '::setResponseFactory()'),
                 );
             }
 
@@ -143,7 +143,7 @@ private function getStreamFactory(): StreamFactoryInterface
             if (!class_exists(Psr17Factory::class)) {
                 throw ClientException::missingDependency(
                     Psr17Factory::class,
-                    sprintf('Either set the stream factory with %s or run composer require nyholm/psr7', self::class . '::setStreamFactory()')
+                    sprintf('Either set the stream factory with %s or run composer require nyholm/psr7', self::class . '::setStreamFactory()'),
                 );
             }
 

ecs.php

@@ -18,7 +18,7 @@ public function __construct(
         string $id = null,
         int $quantity = null,
         float $itemPrice = null,
-        string $deliveryCategory = null
+        string $deliveryCategory = null,
     ) {
         $this->id = $id;
         $this->quantity = $quantity;

src/Client/Client.php

@@ -14,7 +14,7 @@ public static function invalidJson(JsonException $jsonException, string $json):
         $message = sprintf(
             'The response from Meta/Facebook was not valid JSON. Given input: %s. Error was: %s',
             $json,
-            $jsonException->getMessage()
+            $jsonException->getMessage(),
         );
 
         return new self($message, 0, $jsonException);
@@ -25,7 +25,7 @@ public static function invalidResponseFormat(string $json): self
         return new self(sprintf(
             'Expected a JSON response like %s, but got %s',
             '{"error":{"message":"string","type":"string","code":int,"fbtrace_id":"string"}}',
-            $json
+            $json,
         ));
     }
 
@@ -37,7 +37,7 @@ public static function fromErrorResponse(ErrorResponse $errorResponse): self
             $errorResponse->code,
             $errorResponse->type,
             $errorResponse->traceId,
-            $errorResponse->json
+            $errorResponse->json,
         );
 
         return new self($message);
@@ -51,7 +51,7 @@ public static function missingDependency(string $class, string $help): self
         return new self(sprintf(
             'You have a missing dependency: %s. %s',
             $class,
-            $help
+            $help,
         ));
     }
 }

src/Event/Content.php

@@ -23,7 +23,7 @@ public function generateInit(
         array $pixels,
         array $userData = [],
         bool $includePageView = true,
-        bool $includeScriptTag = true
+        bool $includeScriptTag = true,
     ): string {
         try {
             $json = [] !== $userData ? json_encode($userData, \JSON_THROW_ON_ERROR) : null;
@@ -57,7 +57,7 @@ public function generateTrack(Event $event, bool $includeScriptTag = true): stri
             $event->isCustom() ? 'trackCustom' : 'track',
             $event->eventName,
             json_encode($event->customData->getPayload(Parameters::PAYLOAD_CONTEXT_BROWSER), \JSON_THROW_ON_ERROR),
-            $event->eventId
+            $event->eventId,
         );
 
         if ($includeScriptTag) {

src/Exception/ClientException.php

@@ -18,7 +18,7 @@ public function generateInit(
         array $pixels,
         array $userData = [],
         bool $includePageView = true,
-        bool $includeScriptTag = true
+        bool $includeScriptTag = true,
     ): string;
 
     /**

src/Generator/FbqGenerator.php

@@ -27,7 +27,7 @@ public static function fromString(string $value): self
             throw new \InvalidArgumentException(sprintf(
                 'The value "%s" didn\'t match the expected pattern for fbc: "%s"',
                 $value,
-                self::REGEXP_FBC
+                self::REGEXP_FBC,
             ));
         }
 

src/Generator/FbqGeneratorInterface.php

@@ -40,7 +40,7 @@ public function value(): string
             'fb.%d.%d.%d',
             $this->getSubdomainIndex(),
             $this->getCreationTime(),
-            $this->getRandomNumber()
+            $this->getRandomNumber(),
         );
     }
 

src/ValueObject/Fbc.php

@@ -42,6 +42,7 @@ public function it_sends_event(): void
 
     /**
      * @return array{pixelId: non-empty-string, testEventCode: non-empty-string, accessToken: non-empty-string, url: non-empty-string, email: non-empty-string}
+     *
      * @psalm-suppress InvalidReturnType,MoreSpecificReturnType
      */
     private function getTestValues(): array

src/ValueObject/Fbp.php

@@ -52,6 +52,7 @@ public function it_has_immutable_setters(): void
 
     /**
      * @test
+     *
      * @dataProvider wrongInputs
      */
     public function it_handles_wrong_input(string $input): void

tests/Client/LiveClientTest.php

@@ -45,6 +45,7 @@ public function it_has_immutable_setters(): void
 
     /**
      * @test
+     *
      * @dataProvider wrongInputs
      */
     public function it_handles_wrong_input(string $input): void