@sap-ux/edmx-parser - npm Package Compare versions

Comparing version 0.5.15 to 0.5.16

CHANGELOG.md

 # @sap-ux/edmx-parser
 
+## 0.5.16
+
+### Patch Changes
+
+-   3116cd5: The annotation converter now returns the correct references
+
 ## 0.5.15
@@ -4,0 +10,0 @@

dist/utils.d.ts

@@ -63,4 +63,3 @@ /**
     addParserOutput(parserOutput: RawMetadata): void;
-    private mergeReferences;
 }
 //# sourceMappingURL=utils.d.ts.map

dist/utils.js

@@ -92,3 +92,3 @@ "use strict";
         this._parserOutput.push(parserOutput);
-        this.mergeReferences(parserOutput.references);
+        this._references = this._references.concat(parserOutput.references);
         this._associations = this._associations.concat(parserOutput.schema.associations);
@@ -108,14 +108,4 @@ this._associationSets = this._associationSets.concat(parserOutput.schema.associationSets);
     }
-    mergeReferences(references) {
-        for (const reference of references) {
-            const clash = this._references.find((r) => r.namespace === reference.namespace || r.alias === reference.alias);
-            // Assuming that the parser output passed to addParserOutput() is unaliased, it is safe to ignore
-            // non-unique references here. If it is not, then this is going to be problematic
-            if (!clash) {
-                this._references.push(reference);
-            }
-        }
-    }
 }
 exports.MergedRawMetadata = MergedRawMetadata;
 //# sourceMappingURL=utils.js.map

package.json

 {
   "name": "@sap-ux/edmx-parser",
-  "version": "0.5.15",
+  "version": "0.5.16",
   "description": "SAP Fiori OData - EDMX File parser",
@@ -5,0 +5,0 @@ "repository": {

src/utils.ts

@@ -123,3 +123,3 @@ /**
         this._parserOutput.push(parserOutput);
-        this.mergeReferences(parserOutput.references);
+        this._references = this._references.concat(parserOutput.references);
         this._associations = this._associations.concat(parserOutput.schema.associations);
@@ -139,15 +139,2 @@ this._associationSets = this._associationSets.concat(parserOutput.schema.associationSets);
     }
-
-    private mergeReferences(references: Reference[]) {
-        for (const reference of references) {
-            const clash = this._references.find(
-                (r) => r.namespace === reference.namespace || r.alias === reference.alias
-            );
-            // Assuming that the parser output passed to addParserOutput() is unaliased, it is safe to ignore
-            // non-unique references here. If it is not, then this is going to be problematic
-            if (!clash) {
-                this._references.push(reference);
-            }
-        }
-    }
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

252641

decreased by-0.51%

Lines of code

4580

decreased by-0.5%

No dependency changes