Don't trigger changes if nothing was actually changed

[maxpain]

oldDoc not passing

Items.find().observe({
  changed(newDoc, oldDoc) {
    console.log(newDoc, oldDoc) // { foo: "bar" } undefined
  }
})

It's possbile fix it?

[theodorDiaconu]

@Maxpain177 good find. yes it will be fixed.

[theodorDiaconu]

@Maxpain177 a lot of changes are currently required to do such a thing. Basically the current observe() is not a full implementation of the observe() which has support for addedBefore, etc.

Do you really need this functionality ? This is my question. Because if there is no way to work-around this in some scenarios we will implement it.

[maxpain]

@theodorDiaconu Yes, In my project (which contains several microservices, connected to one mongodb and redis) it is using everywhere.

[theodorDiaconu]

@Maxpain177 I understand. Redis-oplog is now ready to go under some restructuring and refactoring, and this will be included :)

[maxpain]

@theodorDiaconu, Currently, i can't use this package, because i need remake most part of application architecture for working without oldDoc.
Thanks 👍

[theodorDiaconu]

@Maxpain177 check it now :)

[maxpain]

@theodorDiaconu Yes, it works. But, if nothing changed in doc, changed function still fires.

[theodorDiaconu]

@Maxpain177 can you give me an example ? What kind of update you do ?

[maxpain]

@theodorDiaconu
Just updating document.

Counts.update({ _id: "users" }, { $set: { count: 0 } })

[theodorDiaconu]

@maxnowack so if "count" was previously ZERO, it should not fire the change event.

[maxpain]

@theodorDiaconu, Reproduction

[theodorDiaconu]

Fixed.