refactor: simplify set contentType()

Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
CommunitySolidServer · Mar 7, 2022 · 242bd8a · 242bd8a
1 parent ef6a52c
commit 242bd8a
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/http/representation/RepresentationMetadata.ts b/src/http/representation/RepresentationMetadata.ts
@@ -307,17 +307,17 @@ export class RepresentationMetadata {
   }
 
   private setContentTypeParams(input: ContentType | string | undefined): void {
-    if (input === undefined) {
-      this.removeContentTypeParameters();
+    // Make sure complete Content-Type RDF structure is gone
+    this.removeContentTypeParameters();
+
+    if (!input) {
       return;
     }
+
     if (typeof input === 'string') {
       input = parseContentTypeWithParameters(input);
     }
 
-    // Make sure complete Content-Type RDF structure is gone
-    this.removeContentTypeParameters();
-
     Object.entries(input.parameters ?? []).forEach(([ paramKey, paramValue ], idx): void => {
       const paramNode = DataFactory.blankNode(`parameter${idx + 1}`);
       this.addQuad(this.id, SOLID_META.terms.ContentTypeParameter, paramNode);