search

davinkevin / AngularStompDK

Angular service to communicate to Stomp-Websocket
http://davinkevin.github.io/AngularStompDK/
Apache License 2.0
36 stars 12 forks source link

fix(subscription): add missing digest parameter #40

Closed baczus closed 8 years ago

baczus commented 8 years ago

Hi,

I noticed that function subscribe is invoking without digest flag after reconnect.

davinkevin commented 8 years ago

We should also change the function $$connectionEquality to take into account the digest value. Same thing in the unSubscriber...

I've include your modification and those in the commit https://github.com/davinkevin/AngularStompDK/commit/a0dcb8a0fe25803ff80f5ac3403fda9a028f179c

I think I will refactor the code to prevent this kind of error. I have too many {queue : c.queue ...} in the code. It could be integrated in the task of migration to TypeScript or just in simpler task.

It will be integrated to the v0.9.2 😄