Merge pull request #1236 from lblod/fix/blank-node-no-subject

Ignore instead of fail when rendering BlankNode with no subject
lblod · Dec 18, 2024 · e4772c6 · e4772c6
2 parents 70202f7 + 7895a2c
commit e4772c6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/.changeset/witty-moose-mate.md b/.changeset/witty-moose-mate.md
@@ -0,0 +1,5 @@
+---
+'@lblod/ember-rdfa-editor': patch
+---
+
+If trying to render a BlankNode with no subject, ignore it instead of crashing
diff --git a/addon/core/schema.ts b/addon/core/schema.ts
@@ -289,6 +289,13 @@ export function renderInvisibleRdfa(
       case 'BlankNode': {
         // the triple refers to a URI which does not have a corresponding
         // resource node
+        if (!isSome(nodeOrMark.attrs['subject'])) {
+          console.warn(
+            'Trying to render a BlankNode with no subject, ignoring',
+            nodeOrMark.attrs,
+          );
+          break;
+        }
         const subject: string = unwrap(nodeOrMark.attrs['subject'] as string);
         propElements.push(namedNodeSpan(subject, predicate, object.value));
         break;