Bump PHP version requirement to 7.4 (#182)

* Bump PHP version requirement to 7.4

* Fix CS

Author: norkunas

composer.json

@@ -10,7 +10,7 @@
         }
     ],
     "require": {
-        "php": ">=7.3",
+        "php": ">=7.4",
         "ext-json": "*",
         "psr/http-client": "^1.0",
         "psr/http-client-implementation": "^1.0",
@@ -21,9 +21,9 @@
     },
     "require-dev": {
         "nyholm/psr7": "^1.8.1",
-        "php-cs-fixer/shim": "^v3.40.0",
-        "phpstan/phpstan": "^1.10.46",
-        "phpstan/phpstan-phpunit": "^1.3.15",
+        "php-cs-fixer/shim": "^v3.64.0",
+        "phpstan/phpstan": "^1.12.0",
+        "phpstan/phpstan-phpunit": "^1.4.0",
         "symfony/http-client": "^5.0|^6.2|^7.0",
         "symfony/phpunit-bridge": "^5.3|^6.2|^7.0"
     },

src/AbstractApi.php

@@ -36,7 +36,7 @@ protected function createRequest(string $method, string $uri): RequestInterface
      *
      * @phpstan-param int<1, max> $maxDepth
      */
-    protected function createStream($value, int $flags = null, int $maxDepth = 512): StreamInterface
+    protected function createStream($value, ?int $flags = null, int $maxDepth = 512): StreamInterface
     {
         $flags = $flags ?? (JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT | JSON_PRESERVE_ZERO_FRACTION);
 

src/Config.php

@@ -10,7 +10,7 @@ final class Config
     private $applicationAuthKey;
     private $userAuthKey;
 
-    public function __construct(string $applicationId, string $applicationAuthKey, string $userAuthKey = null)
+    public function __construct(string $applicationId, string $applicationAuthKey, ?string $userAuthKey = null)
     {
         $this->applicationId = $applicationId;
         $this->applicationAuthKey = $applicationAuthKey;

src/Devices.php

@@ -53,7 +53,7 @@ public function getOne(string $id): array
      * @param int $limit  How many devices to return. Max is 300. Default is 300
      * @param int $offset Result offset. Default is 0. Results are sorted by id
      */
-    public function getAll(int $limit = null, int $offset = null): array
+    public function getAll(?int $limit = null, ?int $offset = null): array
     {
         $query = ['app_id' => $this->client->getConfig()->getApplicationId()];
 
@@ -184,7 +184,7 @@ public function onFocus(string $id, array $data): array
      *                                Only devices from that segment will make it into the export
      * @param int    $lastActiveSince An epoch to filter results to users active after this time
      */
-    public function csvExport(array $extraFields = [], string $segmentName = null, int $lastActiveSince = null): array
+    public function csvExport(array $extraFields = [], ?string $segmentName = null, ?int $lastActiveSince = null): array
     {
         $request = $this->createRequest('POST', "/players/csv_export?app_id={$this->client->getConfig()->getApplicationId()}");
         $request = $request->withHeader('Authorization', "Basic {$this->client->getConfig()->getApplicationAuthKey()}");

src/Dto/Segment/CreateSegment.php

@@ -23,7 +23,7 @@ class CreateSegment implements AbstractDto
      * @param non-empty-string|null           $name
      * @param array<int, AbstractFilter>|null $filters
      */
-    public function __construct(string $name = null, array $filters = null)
+    public function __construct(?string $name = null, ?array $filters = null)
     {
         $this->name = $name;
         $this->filters = $filters;

src/Dto/Segment/ListSegments.php

@@ -22,7 +22,7 @@ class ListSegments implements AbstractDto
      * @param int<0, 2147483648>|null $limit
      * @param int<0, 2147483648>|null $offset
      */
-    public function __construct(int $limit = null, int $offset = null)
+    public function __construct(?int $limit = null, ?int $offset = null)
     {
         $this->limit = $limit;
         $this->offset = $offset;

src/Notifications.php

@@ -40,12 +40,12 @@ public function getOne(string $id): array
      *
      * Application authentication key and ID must be set.
      *
-     * @param int $limit  How many notifications to return (max 50)
-     * @param int $offset Results offset (results are sorted by ID)
+     * @param int|null $limit  How many notifications to return (max 50)
+     * @param int|null $offset Results offset (results are sorted by ID)
      *
      * @phpstan-param int $kind   Kind of notifications returned. Default (not set) is all notification types
      */
-    public function getAll(int $limit = null, int $offset = null/* , int $kind = null */): array
+    public function getAll(?int $limit = null, ?int $offset = null/* , ?int $kind = null */): array
     {
         if (func_num_args() > 2 && !is_int(func_get_arg(2))) {
             trigger_deprecation('norkunas/onesignal-php-api', '2.1.0', 'Method %s() will have a third `int $kind` argument. Not defining it or passing a non integer value is deprecated.', __METHOD__);