fix listegenerering sparql - nationality in same pattern as creator

digibib · Feb 6, 2014 · 517b044 · 517b044
1 parent 7da1d2f
commit 517b044
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib/sparql.rb b/lib/sparql.rb
@@ -76,8 +76,10 @@ def self.generate(criteria)
       query.distinct
       query.where([:work, RDF::FABIO.hasManifestation, :book],
                   [:book, RDF::REV.hasReview, :review])
-      query.optional([:work, RDF::DC.creator, :creator],
-                  [:creator, RDF::FOAF.name, :author])
+      creatorPattern = [[:work, RDF::DC.creator, :creator],
+                        [:creator, RDF::FOAF.name, :author]]
+      creatorPattern.push([:creator, RDF::XFOAF.nationality, :nationality]) if criteria["nationalities"]
+      query.optional(*creatorPattern)
 
       query.where([:book, RDF::DC.language, :language]) if criteria["languages"]
       if criteria["subjects"]
@@ -95,7 +97,6 @@ def self.generate(criteria)
         query.where([:narrower, RDF::SKOS.broader, :genre])
       end
       query.where([:book, RDF::DEICHMAN.literaryFormat, :format]) if criteria["formats"]
-      query.where([:creator, RDF::XFOAF.nationality, :nationality]) if criteria["nationalities"]
 
       query.filter("?subject = <" + criteria["subjects"].join("> || ?subject = <") +">") if criteria["subjects"]
       query.filter("?person = <" + criteria["persons"].join("> || ?person = <") +">") if criteria["persons"]