refactor: move condition check into function

Author: DannyvdSluijs

phpstan-baseline.neon

@@ -30,31 +30,11 @@ parameters:
 			count: 1
 			path: src/JsonSchema/Constraints/BaseConstraint.php
 
-		-
-			message: "#^Cannot access property \\$additionalItems on stdClass\\|null\\.$#"
-			count: 3
-			path: src/JsonSchema/Constraints/CollectionConstraint.php
-
-		-
-			message: "#^Cannot access property \\$items on stdClass\\|null\\.$#"
-			count: 6
-			path: src/JsonSchema/Constraints/CollectionConstraint.php
-
 		-
 			message: "#^Method JsonSchema\\\\Constraints\\\\CollectionConstraint\\:\\:validateItems\\(\\) has parameter \\$value with no value type specified in iterable type array\\.$#"
 			count: 1
 			path: src/JsonSchema/Constraints/CollectionConstraint.php
 
-		-
-			message: "#^Parameter \\#1 \\$object_or_class of function property_exists expects object\\|string, stdClass\\|null given\\.$#"
-			count: 1
-			path: src/JsonSchema/Constraints/CollectionConstraint.php
-
-		-
-			message: "#^Parameter \\#2 \\$schema of method JsonSchema\\\\Constraints\\\\CollectionConstraint\\:\\:validateItems\\(\\) expects stdClass\\|null, object given\\.$#"
-			count: 1
-			path: src/JsonSchema/Constraints/CollectionConstraint.php
-
 		-
 			message: "#^Method JsonSchema\\\\Constraints\\\\Constraint\\:\\:checkObject\\(\\) has parameter \\$appliedDefaults with no type specified\\.$#"
 			count: 1

src/JsonSchema/Constraints/CollectionConstraint.php

@@ -50,10 +50,7 @@ public function check(&$value, $schema = null, ?JsonPointer $path = null, $i = n
             }
         }
 
-        // Verify items
-        if (isset($schema->items)) {
-            $this->validateItems($value, $schema, $path, $i);
-        }
+         $this->validateItems($value, $schema, $path, $i);
     }
 
     /**
@@ -65,7 +62,11 @@ public function check(&$value, $schema = null, ?JsonPointer $path = null, $i = n
      */
     protected function validateItems(&$value, $schema = null, ?JsonPointer $path = null, $i = null): void
     {
-        if (is_bool($schema->items) && $schema->items === true) {
+        if (\is_null($schema) || !isset($schema->items)) {
+            return;
+        }
+
+        if ($schema->items === true) {
             return;
         }