Merge pull request #782 from lblod/workaround/virtuoso-duplication-bug

fix duplicated SPARQL results
lblod · Dec 2, 2024 · 82d1528 · 82d1528
2 parents d260764 + 28f57ed
commit 82d1528
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/.changeset/ninety-tomatoes-swim.md b/.changeset/ninety-tomatoes-swim.md
@@ -0,0 +1,5 @@
+---
+'frontend-gelinkt-notuleren': patch
+---
+
+Work around virtuoso optional-nested-select duplication bug by replacing `OPTIONAL` statements by `UNION` statements
diff --git a/app/config/mandatee-table-config.js b/app/config/mandatee-table-config.js
@@ -1332,9 +1332,16 @@ export const mandateeTableConfigRMW = (meeting) => {
             ?mandataris a mandaat:Mandataris.
             ?mandataris mandaat:isBestuurlijkeAliasVan ?persoon.
             ?mandataris org:hasMembership/org:organisation ?fractie.
-            OPTIONAL {
+
+            {
               ?mandataris mandaat:einde ?persoon_mandaat_einde.
             }
+            UNION
+            {
+              FILTER NOT EXISTS {
+                ?mandataris mandaat:einde ?persoon_mandaat_einde.
+              }
+            }
 
             ?fractie a mandaat:Fractie.
             ?fractie regorg:legalName ?fractie_naam.
@@ -1438,9 +1445,16 @@ export const mandateeTableConfigRMW = (meeting) => {
             ?mandataris mandaat:isBestuurlijkeAliasVan ?persoon.
             ?mandataris org:hasMembership/org:organisation ?fractie.
             ?mandataris mandaat:start ?mandaat_start.
-            OPTIONAL {
+
+            {
               ?mandataris mandaat:einde ?mandaat_einde.
             }
+            UNION
+            {
+              FILTER NOT EXISTS {
+                ?mandataris mandaat:einde ?mandaat_einde.
+              }
+            }
 
             ?fractie a mandaat:Fractie.
             ?fractie regorg:legalName ?fractie_naam.