Removed scrutinizer, fixed cache middleware

Author: TheCelavi

.github/workflows/ci.yaml

@@ -40,11 +40,3 @@ jobs:
                     pipenv run python bin/ci.py --verbose --teardown
                 env:
                     COMPOSER_ROOT_VERSION: ${{ vars.COMPOSER_ROOT_VERSION }}
-
-            -   name: Upload Scrutinizer coverage
-                uses: sudo-bot/action-scrutinizer@latest
-                # Do not run this step on forked versions of the main repository (example: contributor forks)
-                if: github.repository == 'runopencode/query-resources-loader-bundle'
-                with:
-                    cli-args: "--format=php-clover build/phpunit/clover.xml --revision=${{ github.event.pull_request.head.sha || github.sha }}"                    
-

.scrutinizer.yml

@@ -2,9 +2,6 @@ Query resources loader bundle
 =============================
 
 [![Packagist](https://img.shields.io/packagist/v/RunOpenCode/query-resources-loader-bundle.svg)](https://packagist.org/packages/runopencode/query-resources-loader-bundle)
-[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/RunOpenCode/query-resources-loader-bundle/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/RunOpenCode/query-resources-loader-bundle/?branch=master)
-[![Build Status](https://scrutinizer-ci.com/g/RunOpenCode/query-resources-loader-bundle/badges/build.png?b=master)](https://scrutinizer-ci.com/g/RunOpenCode/query-resources-loader-bundle/build-status/master)
-[![Code Coverage](https://scrutinizer-ci.com/g/RunOpenCode/query-resources-loader-bundle/badges/coverage.png?b=master)](https://scrutinizer-ci.com/g/RunOpenCode/query-resources-loader-bundle/?branch=master)
 
 The purpose of query resources loader is to help you manage and organize your big, long, database queries, especially in
 application that deals with reporting.

README.md

@@ -15,7 +15,6 @@
 use Symfony\Component\Cache\Adapter\TagAwareAdapter;
 use Symfony\Contracts\Cache\CacheInterface;
 use Symfony\Contracts\Cache\ItemInterface;
-use Symfony\Contracts\Cache\TagAwareCacheInterface;
 
 /**
  * Cache middleware.
@@ -27,8 +26,8 @@
     public const TAG = 'runopencode_query_resources_loader_cache';
 
     public function __construct(
-        private CacheInterface&TagAwareCacheInterface $cache = new TagAwareAdapter(new ArrayAdapter()),
-        private ?int                                  $defaultTtl = null,
+        private CacheInterface $cache = new TagAwareAdapter(new ArrayAdapter()),
+        private ?int           $defaultTtl = null,
     ) {
         // noop
     }
@@ -71,11 +70,13 @@ public function __invoke(string $query, Parameters $parameters, Options $options
              * @var CacheIdentityInterface $identity
              */
             $item->expiresAfter($identity->getTtl());
-            /** @psalm-suppress UndefinedInterfaceMethod */
-            $item->tag(\array_merge(
-                $identity->getTags(),
-                [self::TAG]
-            ));
+            
+            if (method_exists($item, 'tag')) {
+                $item->tag(\array_merge(
+                    $identity->getTags(),
+                    [self::TAG]
+                ));
+            }
 
             return $result;
         });

src/RunOpenCode/Bundle/QueryResourcesLoader/Cache/CacheMiddleware.php

@@ -195,13 +195,17 @@ final public function timeImmutable(string $name, ?\DateTimeInterface $value): s
      * $parameters->set($name, $value->name, Types::STRING);
      * ```
      *
-     * @param string    $name  Parameter name.
-     * @param \UnitEnum $value Parameter value.
+     * @param string     $name  Parameter name.
+     * @param ?\UnitEnum $value Parameter value.
      *
      * @return self Fluent return.
      */
-    final public function enum(string $name, \UnitEnum $value): self
+    final public function enum(string $name, ?\UnitEnum $value): self
     {
+        if (null === $value) {
+            return $this->set($name, null, Types::STRING);
+        }
+        
         $reflection = new \ReflectionEnum($value::class);
 
         if (!$reflection->isBacked()) {