Merge pull request #10 from TomHAnderson/hotfix/contracts

Fix Psalm errors in Laravel app
API-Skeletons · Jan 19, 2022 · b91c940 · b91c940
2 parents 5acde06 + d6ab979
commit b91c940
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 11 deletions.
diff --git a/phpcs.xml.dist b/phpcs.xml.dist
@@ -14,5 +14,7 @@
     <file>test</file>
 
     <!-- Include full Doctrine Coding Standard -->
-    <rule ref="Doctrine"/>
+    <rule ref="Doctrine">
+        <exclude name="SlevomatCodingStandard.Namespaces.UnusedUses.UnusedUse"/>
+    </rule>
 </ruleset>
diff --git a/src/Contracts/HydratorManagerContract.php b/src/Contracts/HydratorManagerContract.php
@@ -12,10 +12,10 @@ interface HydratorManagerContract
     /** @param mixed $value */
     public function canExtract($value): bool;
 
+    public function extract(mixed $class, ?string $overrideHydrator = null): Collection|Resource;
+
     /**
-     * @param mixed $class
-     *
-     * @return Resource|Collection
+     * @param mixed[]|null $state
      */
-    public function extract($class);
+    public function resource(?array $state = null): Resource;
 }
diff --git a/src/HydratorManager.php b/src/HydratorManager.php
@@ -53,12 +53,7 @@ public function canExtract($value): bool
         return isset($this->classHydrators[get_class($value)]);
     }
 
-    /**
-     * @param mixed $class
-     *
-     * @return Resource|Collection
-     */
-    public function extract($class, ?string $overrideHydrator = null)
+    public function extract(mixed $class, ?string $overrideHydrator = null): Collection|Resource
     {
         if (! $class) {
             return (new Resource())->setHydratorManager($this);