Merge pull request #56 from ddvlanck/main

Patch for property path quads
TREEcg · Nov 17, 2021 · 58d7949 · 58d7949
2 parents 7dcdf32 + 4b88bfe
commit 58d7949
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -290,8 +290,8 @@ class BucketizerFragmentStrategy {
             const bucketPath = `${config.storage}/${(0, Sanitizer_1.sanitize)(node)}.${fileExtension}`;
             const otherPages = relations.map(relation => relation.nodeId);
             pages = [...new Set([...pages, ...otherPages, node])];
-            const controlQuads = this.createHypermediaControlQuads(relations, node, outputDirPath, fileExtension, treePathQuad, propertyPathQuads);
-            tasks.push(this.writeToBucket(bucketPath, controlQuads));
+            const quads = this.createHypermediaControlQuads(relations, node, outputDirPath, fileExtension, treePathQuad, propertyPathQuads);
+            tasks.push(this.writeToBucket(bucketPath, quads));
         });
         pages.forEach(async (page) => tasks.push(this.addCollectionInformation(page, outputDirPath, config.url, config.storage, fileExtension)));
         await Promise.all(tasks);
@@ -313,7 +313,8 @@ class BucketizerFragmentStrategy {
         const quads = [];
         const bucketPath = `${outputDirPath}/${(0, Sanitizer_1.sanitize)(node)}.${fileExtension}`;
         // This indicates that the property path is a list of predicates instead of just one predicate
-        if (treePathQuad && treePathQuad.object.termType === 'BlankNode') {
+        // And only add property path quads if page contains actual relations
+        if (relations.length > 0 && treePathQuad && treePathQuad.object.termType === 'BlankNode') {
             quads.push(...propertyPathQuads);
         }
         relations.forEach(relation => {

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/fragment-strategy/BucketizerFragmentStrategy.ts b/src/fragment-strategy/BucketizerFragmentStrategy.ts
@@ -143,7 +143,8 @@ class BucketizerFragmentStrategy implements FragmentStrategy {
     const bucketPath = `${outputDirPath}/${sanitize(node)}.${fileExtension}`;
 
     // This indicates that the property path is a list of predicates instead of just one predicate
-    if (treePathQuad && treePathQuad.object.termType === 'BlankNode') {
+    // And only add property path quads if page contains actual relations
+    if (relations.length > 0 && treePathQuad && treePathQuad.object.termType === 'BlankNode') {
       quads.push(...propertyPathQuads);
     }