From: Tim Wood <washwithcare@gmail.com>
Date: Tue, 12 Nov 2013 20:19:13 +0000 (-0800)
Subject: events: don't call once twice
X-Git-Tag: v0.10.23~38
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c9d93f34311ce0a9b59ed9f4511a2e3ba69e0f25;p=platform%2Fupstream%2Fnodejs.git

events: don't call once twice

Emitting an event within a `EventEmitter#once` callback of the same
event name will cause subsequent `EventEmitter#once` listeners of the
same name to be called multiple times.

    var emitter = new EventEmitter();

    emitter.once('e', function() {
      emitter.emit('e');
      console.log(1);
    });

    emitter.once('e', function() {
      console.log(2);
    });

    emitter.emit('e');

    // Output
    // 2
    // 1
    // 2

Fix the issue, by calling the listener method only if it was not
already called.
---

diff --git a/lib/events.js b/lib/events.js
index 5188205..8c02a55 100644
--- a/lib/events.js
+++ b/lib/events.js
@@ -170,9 +170,15 @@ EventEmitter.prototype.once = function(type, listener) {
   if (typeof listener !== 'function')
     throw TypeError('listener must be a function');
 
+  var fired = false;
+
   function g() {
     this.removeListener(type, g);
-    listener.apply(this, arguments);
+
+    if (!fired) {
+      fired = true;
+      listener.apply(this, arguments);
+    }
   }
 
   g.listener = listener;
diff --git a/test/simple/test-event-emitter-once.js b/test/simple/test-event-emitter-once.js
index 1f0a207..2eaebcc 100644
--- a/test/simple/test-event-emitter-once.js
+++ b/test/simple/test-event-emitter-once.js
@@ -47,3 +47,19 @@ process.on('exit', function() {
   assert.equal(1, times_hello_emited);
 });
 
+var times_recurse_emitted = 0;
+
+e.once('e', function() {
+	e.emit('e');
+	times_recurse_emitted++;
+});
+
+e.once('e', function() {
+	times_recurse_emitted++;
+});
+
+e.emit('e');
+
+process.on('exit', function() {
+  assert.equal(2, times_recurse_emitted);
+});