isCommandRejected: do not go to revisionStore if evtPayload.reason.ag…

…gregateId is not defined
thenativeweb · Mar 22, 2019 · 96ef47d · 96ef47d
1 parent c85961d
commit 96ef47d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lib/denormalizer.js b/lib/denormalizer.js
@@ -467,7 +467,7 @@ _.extend(Denormalizer.prototype, {
         context: !!this.definitions.event.context ? dotty.get(evt, this.definitions.event.context) : undefined
       };
 
-      if (!this.definitions.event.revision || !dotty.exists(evt, this.definitions.event.revision)) {
+      if (!this.definitions.event.revision || !dotty.exists(evt, this.definitions.event.revision) || !evtPayload.reason.aggregateId || (typeof evtPayload.reason.aggregateId !== 'string' && typeof evtPayload.reason.aggregateId !== 'number')) {
         this.onEventMissingHandle(info, evt);
         if (callback) {
           callback(null, evt, []);

diff --git a/releasenotes.md b/releasenotes.md
@@ -1,3 +1,6 @@
+## [v1.16.52](https://github.com/adrai/node-cqrs-eventdenormalizer/compare/v1.16.51...v1.16.52)
+- isCommandRejected: do not go to revisionStore if evtPayload.reason.aggregateId is not defined
+
 ## [v1.16.51](https://github.com/adrai/node-cqrs-eventdenormalizer/compare/v1.16.5...v1.16.51)
 - update viewmodel