Skip to content

Commit 172566a

Browse files
pierremoreauepriestley
authored and
epriestley
committedFeb 3, 2015
Aphlict - fix incrementation of _messagesIn
Summary: Ref T7124. The local version of `this` in the handler of 'end' was incremented rather than the global one. Test Plan: Sending test notifications did not increment the `messages.in` value before this patch even if it should have. With the patch sending test notifications does increment `messages.in`. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin, epriestley Maniphest Tasks: T7124 Differential Revision: https://secure.phabricator.com/D11648
1 parent 70cddaa commit 172566a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

‎support/aphlict/server/lib/AphlictAdminServer.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ JX.install('AphlictAdminServer', {
5555

5656
self.getLogger().log(
5757
'Received notification: ' + JSON.stringify(msg));
58-
++this._messagesIn;
58+
++self._messagesIn;
5959

6060
try {
6161
self._transmit(msg);

0 commit comments

Comments
 (0)
Failed to load comments.