chore: Minor improvements and removal of duplicate code;

README.md

@@ -67,7 +67,8 @@ Awesome people who contributed to this package
 
 - [dsentker](https://github.com/dsentker)
 - [schwabwaldemar](https://github.com/schwabwaldemar)
+- [haraldpdl](https://github.com/haraldpdl)
 
 ## Honorable mentions
-thanks for support go to:
+thanks for support goes to:
 - [Robinson-Software-Development](https://github.com/robinson-software-development)

src/CachedServiceGenerator/Dto/MethodCallObject.php

@@ -44,7 +44,7 @@ public function getCachePrefix(): string
         return str_replace(['\\', ':', ' '], ['_', '-', ''], $this->getClass());
     }
 
-    public function getCallable(): ?callable
+    public function getCallable(): callable
     {
         return fn () => call_user_func_array([$this->class, $this->method], $this->arguments);
     }

src/DataCollector/MultiLevelCacheDataCollector.php

@@ -18,7 +18,7 @@
 
 class MultiLevelCacheDataCollector extends DataCollector implements DataCollectorInterface
 {
-    private const LOW_HITRATE_NOTICE_THRESHOLD = 9;
+    private const int LOW_HITRATE_NOTICE_THRESHOLD = 9;
 
     private bool $isInitialized = false;
     private bool $processedAdditionalIssues = false;

src/Service/MultiLevelCacheService.php

@@ -177,14 +177,6 @@ public function getBulk(
                         'mlcCacheableMethodAttribute' => $mlcCacheableMethodAttribute,
                     ],
                 ));
-                $this->raiseIssue(ErrorEnum::ERROR_BULK_CONFIG_MISSING, new DataCollectorIssueOccurrenceDto(
-                    affectedCacheGroup: $this->cacheGroupName,
-                    affectedKeys: $keys,
-                    context: [
-                        'methodCallObject' => $methodCallObject,
-                        'mlcCacheableMethodAttribute' => $mlcCacheableMethodAttribute,
-                    ],
-                ));
 
                 throw new RuntimeException('BulkConfig is required for bulk operations');
             }
@@ -206,9 +198,6 @@ public function getBulk(
 
             // do one bulk call to source for all non-cached requests and cache them individually
             if (!empty($requestsToSource)) {
-                $sourceArguments = $methodCallObject->getArguments();
-                $sourceArguments[0] = $requestsToSource;
-                $methodCallObjectForSource = $methodCallObject->clone(arguments: $sourceArguments);
                 $sourceArguments = $methodCallObject->getArguments();
                 $sourceArguments[0] = $requestsToSource;
                 $methodCallObjectForSource = $methodCallObject->clone(arguments: $sourceArguments);

tests/CachedServiceGenerator/Service/FileOperationServiceTest.php

@@ -114,7 +114,7 @@ public function testFindRootForClassWithCorruptedNamespace(): void
     {
         require_once($this->testFilesRootPath . '/CorruptedNamespace.php');
         $this->expectException(RuntimeException::class);
-        $result = FileOperationService::findRootForClass(CorruptedNamespace::class);
+        FileOperationService::findRootForClass(CorruptedNamespace::class);
     }
 
     private function checkFileSyntax(string $file): void