Use a FROM NAMED clause to query types per vocabs installed only

NatLibFi · Jun 5, 2022 · 6fec308 · 6fec308
1 parent 5d193c2
commit 6fec308
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 29 deletions.
diff --git a/model/Model.php b/model/Model.php
@@ -124,7 +124,7 @@ public function getVocabularyList($categories = true, $shortname = false)
      * Return all types (RDFS/OWL classes) present in the specified vocabulary or all vocabularies.
      * @return array Array with URIs (string) as key and array of (label, superclassURI) as value
      */
-    public function getTypes($vocid = null, $lang = null)
+    public function getTypes(string $vocid = null, string $lang = null): array
     {
         $sparql = (isset($vocid)) ? $this->getVocabulary($vocid)->getSparql() : $this->getDefaultSparql();
         $result = $sparql->queryTypes($lang);

diff --git a/model/sparql/GenericSparql.php b/model/sparql/GenericSparql.php
@@ -109,10 +109,10 @@ protected function query($query) {
      * @return string
      */
     protected function generateFromClause($vocabs=null) {
-        $clause = '';
         if (!$vocabs) {
             return $this->graph !== '?graph' && $this->graph !== NULL ? "FROM <$this->graph>" : '';
         }
+        $clause = '';
         $graphs = $this->getVocabGraphs($vocabs);
         foreach ($graphs as $graph) {
             $clause .= "FROM NAMED <$graph> ";
@@ -542,34 +542,37 @@ public function queryConceptInfo($uris, $arrayClass = null, $vocabs = array(), $
      * @param string $lang
      * @return string sparql query
      */
-    private function generateQueryTypesQuery($lang) {
-        $fcl = $this->generateFromClause();
+    private function generateQueryTypesQuery(string $lang): string {
+        $vocabs = $this->model->getVocabularies();
+        $fcl = $this->generateFromClause($vocabs);
         $query = <<<EOQ
 SELECT DISTINCT ?type ?label ?superclass $fcl
 WHERE {
-  {
-    { BIND( skos:Concept as ?type ) }
-    UNION
-    { BIND( skos:Collection as ?type ) }
-    UNION
-    { BIND( isothes:ConceptGroup as ?type ) }
-    UNION
-    { BIND( isothes:ThesaurusArray as ?type ) }
-    UNION
-    { ?type rdfs:subClassOf/rdfs:subClassOf* skos:Concept . }
-    UNION
-    { ?type rdfs:subClassOf/rdfs:subClassOf* skos:Collection . }
-  }
-  OPTIONAL {
-    ?type rdfs:label ?label .
-    FILTER(langMatches(lang(?label), '$lang'))
-  }
-  OPTIONAL {
-    ?type rdfs:subClassOf ?superclass .
-  }
-  FILTER EXISTS {
-    ?s a ?type .
-    ?s skos:prefLabel ?prefLabel .
+  GRAPH ?g {
+    {
+      { BIND( skos:Concept as ?type ) }
+      UNION
+      { BIND( skos:Collection as ?type ) }
+      UNION
+      { BIND( isothes:ConceptGroup as ?type ) }
+      UNION
+      { BIND( isothes:ThesaurusArray as ?type ) }
+      UNION
+      { ?type rdfs:subClassOf/rdfs:subClassOf* skos:Concept . }
+      UNION
+      { ?type rdfs:subClassOf/rdfs:subClassOf* skos:Collection . }
+    }
+    OPTIONAL {
+      ?type rdfs:label ?label .
+      FILTER(langMatches(lang(?label), '$lang'))
+    }
+    OPTIONAL {
+      ?type rdfs:subClassOf ?superclass .
+    }
+    FILTER EXISTS {
+      ?s a ?type .
+      ?s skos:prefLabel ?prefLabel .
+    }
   }
 }
 EOQ;
@@ -581,7 +584,7 @@ private function generateQueryTypesQuery($lang) {
      * @param EasyRdf\Sparql\Result $result
      * @return array Array with URIs (string) as key and array of (label, superclassURI) as value
      */
-    private function transformQueryTypesResults($result) {
+    private function transformQueryTypesResults(\EasyRdf\Sparql\Result $result): array {
         $ret = array();
         foreach ($result as $row) {
             $type = array();
@@ -603,7 +606,7 @@ private function transformQueryTypesResults($result) {
      * @param string $lang
      * @return array Array with URIs (string) as key and array of (label, superclassURI) as value
      */
-    public function queryTypes($lang) {
+    public function queryTypes(string $lang): array {
         $query = $this->generateQueryTypesQuery($lang);
         $result = $this->query($query);
         return $this->transformQueryTypesResults($result);