Merge pull request #41 from Informatievlaanderen/SDTT-264-oslo-conver…

…ter-uml-ea-does-not-set-scope-correctly Fix addScope function
Informatievlaanderen · Feb 8, 2024 · d05b57f · d05b57f
2 parents de0e841 + b956603
commit d05b57f
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/packages/oslo-converter-uml-ea/lib/interfaces/ConverterHandler.ts b/packages/oslo-converter-uml-ea/lib/interfaces/ConverterHandler.ts
@@ -190,14 +190,15 @@ export abstract class ConverterHandler<T extends EaObject> {
         `[ConverterHandler]: Unable to find the URI for object with path ${object.path}. Setting scope to "Undefined".`,
       );
       scope = Scope.Undefined;
+      return;
     }
 
-    if (uri?.toString().startsWith(packageBaseUri)) {
-      scope = Scope.InPackage;
+    if (uri.toString().startsWith(this.config.publicationEnvironment)) {
+      scope = Scope.InPublicationEnvironment;
     }
 
-    if (uri?.toString().startsWith(this.config.publicationEnvironment)) {
-      scope = Scope.InPublicationEnvironment;
+    if (uri.toString().startsWith(packageBaseUri)) {
+      scope = Scope.InPackage;
     }
 
     quads.push(