Use `events.one` vs `events.on` inside of `CodeCell.execute`

9 years ago · 60a02df266
parent 32a2932b86
commit 60a02df266
1 changed files with 6 additions and 4 deletions
--- a/notebook/static/notebook/js/codecell.js
+++ b/notebook/static/notebook/js/codecell.js
@ -335,11 +335,13 @@ define([
        this.render();
        this.events.trigger('execute.CodeCell', {cell: this});
        var that = this;
-        this.events.on('finished_iopub.Kernel', function (evt, data) {
+        function handleFinished(evt, data) {
            if (that.kernel.id === data.kernel.id && that.last_msg_id === data.msg_id) {
-		that.events.trigger('finished_execute.CodeCell', {cell: that});
-	    }
-        });
+            		that.events.trigger('finished_execute.CodeCell', {cell: that});
+                that.events.off('finished_iopub.Kernel', handleFinished);
+      	    }
+        }
+        this.events.on('finished_iopub.Kernel', handleFinished);
    };
    
    /**