define term equals methods as prototype methods instead of arrow func…

…tions

.changeset/lucky-experts-kneel.md

@@ -0,0 +1,11 @@
+---
+"@lblod/ember-rdfa-editor": patch
+---
+
+Define `equals` as a prototype method on RDF term instances instead of as arrow functions.
+
+**Why is this necessary?**
+Defining `equals` as a prototype method ensures that is not part of the RDF term objects themselves. This makes it easier/less error-prone to compare objects containing these terms. (e.g. the object-comparison library used by prosemirror-tools does not support function properties)
+
+**Possible drawbacks of this approach**
+This approach does have the drawback that spreading an RDF term results in losing the `equals` method.

addon/core/say-data-factory/blank-node.ts

@@ -13,9 +13,9 @@ export class SayBlankNode implements RDF.BlankNode {
     this.value = value;
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return (
       !!other && other.termType === 'BlankNode' && other.value === this.value
     );
-  };
+  }
 }

addon/core/say-data-factory/default-graph.ts

@@ -16,7 +16,7 @@ export class SayDefaultGraph implements RDF.DefaultGraph {
     // Private constructor
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return !!other && other.termType === 'DefaultGraph';
-  };
+  }
 }

addon/core/say-data-factory/literal.ts

@@ -37,13 +37,13 @@ export class SayLiteral implements RDF.Literal {
     }
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return (
       !!other &&
       other.termType === 'Literal' &&
       other.value === this.value &&
       other.language === this.language &&
       this.datatype.equals(other.datatype)
     );
-  };
+  }
 }

addon/core/say-data-factory/named-node.ts

@@ -15,9 +15,9 @@ export class SayNamedNode<Iri extends string = string>
     this.value = value;
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return (
       !!other && other.termType === 'NamedNode' && other.value === this.value
     );
-  };
+  }
 }

addon/core/say-data-factory/prosemirror-terms/content-literal.ts

@@ -33,12 +33,12 @@ export class ContentLiteralTerm {
     }
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return (
       !!other &&
       other.termType === 'ContentLiteral' &&
       other.language === this.language &&
       this.datatype.equals(other.datatype)
     );
-  };
+  }
 }

addon/core/say-data-factory/prosemirror-terms/literal-node.ts

@@ -37,13 +37,13 @@ export class LiteralNodeTerm {
     }
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return (
       !!other &&
       other.termType === 'LiteralNode' &&
       other.value === this.value &&
       other.language === this.language &&
       this.datatype.equals(other.datatype)
     );
-  };
+  }
 }

addon/core/say-data-factory/prosemirror-terms/resource-node.ts

@@ -12,9 +12,9 @@ export class ResourceNodeTerm<Iri extends string = string> {
     this.value = value;
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return (
       !!other && other.termType === 'ResourceNode' && other.value === this.value
     );
-  };
+  }
 }

addon/core/say-data-factory/quad.ts

@@ -17,7 +17,7 @@ export class SayQuad implements RDF.BaseQuad {
     public readonly graph: RDF.Term,
   ) {}
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     // `|| !other.termType` is for backwards-compatibility with old factories without RDF* support.
     return (
       !!other &&
@@ -27,5 +27,5 @@ export class SayQuad implements RDF.BaseQuad {
       this.object.equals(other.object) &&
       this.graph.equals(other.graph)
     );
-  };
+  }
 }

addon/core/say-data-factory/variable.ts

@@ -13,9 +13,9 @@ export class SayVariable implements RDF.Variable {
     this.value = value;
   }
 
-  equals = (other?: Option<SayTerm>) => {
+  equals(other?: Option<SayTerm>) {
     return (
       !!other && other.termType === 'Variable' && other.value === this.value
     );
-  };
+  }
 }