Merge pull request #74 from netresearch/cache-schemas

Cache schemas

Author: justinrainbow

src/JsonSchema/Uri/UriRetriever.php

@@ -134,6 +134,29 @@ public function retrieve($uri, $baseUri = null)
             $fetchUri = $resolver->generate($arParts);
         }
 
+        $jsonSchema = $this->loadSchema($fetchUri);
+
+        // Use the JSON pointer if specified
+        $jsonSchema = $this->resolvePointer($jsonSchema, $resolvedUri);
+        $jsonSchema->id = $resolvedUri;
+
+        return $jsonSchema;
+    }
+
+    /**
+     * Fetch a schema from the given URI, json-decode it and return it.
+     * Caches schema objects.
+     *
+     * @param string $fetchUri Absolute URI
+     *
+     * @return object JSON schema object
+     */
+    protected function loadSchema($fetchUri)
+    {
+        if (isset($this->schemaCache[$fetchUri])) {
+            return $this->schemaCache[$fetchUri];
+        }
+
         $uriRetriever = $this->getUriRetriever();
         $contents = $this->uriRetriever->retrieve($fetchUri);
         $this->confirmMediaType($uriRetriever, $fetchUri);
@@ -143,10 +166,7 @@ public function retrieve($uri, $baseUri = null)
             throw new JsonDecodingException($error);
         }
 
-        // Use the JSON pointer if specified
-        $jsonSchema = $this->resolvePointer($jsonSchema, $resolvedUri);
-        $jsonSchema->id = $resolvedUri;
-
+        $this->schemaCache[$fetchUri] = $jsonSchema;
         return $jsonSchema;
     }