Merge pull request #5 from trinketapp/4-kwargs

Checks that keyword arg value is defined. Fixes #4.
trinketapp · Jul 7, 2017 · 557a8db · 557a8db
2 parents a15428c + 107db9c
commit 557a8db
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/__init__.js b/__init__.js
@@ -171,11 +171,15 @@ var $builtinmodule = function(name) {
       });
 
       $loc.render = new Sk.builtin.func(function(self) {
-        var i, key;
+        var i, key, val;
 
         for (i = 0; i < KWARGS.length; i++) {
           key = KWARGS[i];
-          self.instance._options[key] = Sk.ffi.remapToJs(self.tp$getattr(key));
+          val = self.tp$getattr(key);
+
+          if (typeof val !== "undefined") {
+            self.instance._options[key] = Sk.ffi.remapToJs(val);
+          }
         }
 
         return self.instance.render(renderer);