Fixing an issue when the bound function is called as a constructor in…

… ES3.
ljharb · May 5, 2014 · e20122d · e20122d
1 parent a2e29c4
commit e20122d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -15,7 +15,7 @@ function bind(that) {
     var binder = function () {
         if (this instanceof bound) {
             var result = target.apply(
-                that,
+                this,
                 args.concat(slice.call(arguments))
             );
             if (Object(result) === result) {

diff --git a/test/index.js b/test/index.js
@@ -191,6 +191,19 @@ test('with a context', function (t) {
         st.end();
     });
 
+    t.test('has the new instance\'s context when called as a constructor', function (st) {
+        var actualContext;
+        var expectedContext = { foo: 'bar' };
+        var namespace = {
+            Func: functionBind.call(function () {
+                actualContext = this;
+            }, expectedContext)
+        };
+        var result = new namespace.Func();
+        st.notEqual(actualContext, expectedContext);
+        st.end();
+    });
+
     t.end();
 });