Skip to content

Commit ca636ff

Browse files
committed
broker.subscribeParticipantChange(): Add callback
1 parent 3f999ec commit ca636ff

File tree

4 files changed

+26
-6
lines changed

4 files changed

+26
-6
lines changed

src/amqp.coffee

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,12 @@ class MessageBroker extends Client
203203
return callback null
204204

205205
# Participant registration
206-
subscribeParticipantChange: (handler) ->
206+
subscribeParticipantChange: (handler, callback) ->
207+
defaultCallback = (err) ->
208+
if err
209+
console.err "Error in msgflo.amqp.subscribeParticipantChange, and no callback added", err
210+
callback = defaultCallback if not callback
211+
207212
deserialize = (message) =>
208213
debug 'receive on fbp', message.fields.deliveryTag
209214
data = null
@@ -218,6 +223,7 @@ class MessageBroker extends Client
218223

219224
@channel.assertQueue 'fbp'
220225
@channel.consume 'fbp', deserialize
226+
return callback null
221227

222228
exports.Client = Client
223229
exports.MessageBroker = MessageBroker

src/direct.coffee

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,16 @@ class MessageBroker extends interfaces.MessageBroker
121121
nackMessage: (message) ->
122122
return
123123

124-
subscribeParticipantChange: (handler) ->
124+
subscribeParticipantChange: (handler, callback) ->
125+
defaultCallback = (err) ->
126+
if err
127+
console.err "Error in msgflo.direct.subscribeParticipantChange, and no callback added", err
128+
callback = defaultCallback if not callback
129+
125130
@createQueue '', 'fbp', (err) =>
126-
@subscribeToQueue 'fbp', handler, () ->
131+
return callback err if err
132+
@subscribeToQueue 'fbp', handler, (err) ->
133+
return callback err
127134

128135
exports.MessageBroker = MessageBroker
129136
exports.Client = Client

src/interfaces.coffee

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ class MessageBroker extends MessagingSystem
7373
throw new Error 'Not Implemented'
7474

7575
# Participant registration
76-
subscribeParticipantChange: (handler) ->
76+
subscribeParticipantChange: (handler, callback) ->
7777
throw new Error 'Not Implemented'
7878

7979
exports.MessageBroker = MessageBroker

src/mqtt.coffee

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,9 +125,16 @@ class MessageBroker extends Client
125125
routing.binderMixin this
126126

127127
# Participant registration
128-
subscribeParticipantChange: (handler) ->
128+
subscribeParticipantChange: (handler, callback) ->
129+
defaultCallback = (err) ->
130+
if err
131+
console.err "Error in msgflo.mqtt.subscribeParticipantChange, and no callback added", err
132+
callback = defaultCallback if not callback
133+
129134
@createQueue '', 'fbp', (err) =>
130-
@subscribeToQueue 'fbp', handler, () ->
135+
return callback err if err
136+
@subscribeToQueue 'fbp', handler, (err) ->
137+
return callback err
131138

132139
exports.Client = Client
133140
exports.MessageBroker = MessageBroker

0 commit comments

Comments
 (0)