Merge pull request #1 from jlaurens/jlaurens-patch-1

Added a test for possibly undefined argument
brython-dev · Jan 19, 2019 · 2a3f18c · 2a3f18c
2 parents f2768f1 + ac07310
commit 2a3f18c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/www/src/js_objects.js b/www/src/js_objects.js
@@ -254,7 +254,8 @@ JSObject.__getattribute__ = function(self,attr){
             var res = function(){
                 var args = []
                 for(var i = 0, len = arguments.length; i < len; i++){
-                    if(arguments[i] !== null && arguments[i].$nat !== undefined){
+                    var arg = arguments[i]
+                    if(arg !== undefined && arg !== null && arg.$nat !== undefined){
                         //
                         // Passing keyword arguments to a Javascript function
                         // raises a TypeError : since we don't know the
@@ -266,7 +267,7 @@ JSObject.__getattribute__ = function(self,attr){
                             "A Javascript function can't take " +
                                 "keyword arguments")
                     }else{
-                        args.push(pyobj2jsobj(arguments[i]))
+                        args.push(pyobj2jsobj(arg))
                     }
                 }
                 // IE workaround