Fix RangeError when formFiller tries to fill empty select

Closes #47
marmelab · Mar 26, 2014 · 71ff92d · 71ff92d
1 parent d8d0a9d
commit 71ff92d
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/species/formFiller.js b/src/species/formFiller.js
@@ -102,14 +102,14 @@ define(function(require) {
                 }
             }
 
-            var character = config.elementMapTypes[elementType](element);
+            var value = config.elementMapTypes[elementType](element);
 
             if (typeof config.showAction == 'function') {
                 config.showAction(element);
             }
 
             if (typeof config.logger.log == 'function') {
-                config.logger.log('gremlin', 'formFiller', 'input', character, 'in', element);
+                config.logger.log('gremlin', 'formFiller', 'input', value, 'in', element);
             }
         }
 
@@ -129,6 +129,7 @@ define(function(require) {
 
         function fillSelect(element) {
             var options = element.querySelectorAll('option');
+            if (options.length === 0) return;
             var randomOption = config.randomizer.pick(options);
 
             for (var i = 0, c = options.length; i < c; i++) {