Merge pull request risuiowa#77 from AlexKalinin/master

bigtunacan · web-flow · commit 857d14af3ca5 · 2016-12-03T10:28:50.000-06:00
[Bugfix] Empty selector in Mozilla Firefox
diff --git a/.hound.yml b/.hound.yml
@@ -0,0 +1,2 @@
+jshint:
+  ignore_file: .jshintignore
diff --git a/.jshintignore b/.jshintignore
@@ -0,0 +1 @@
+lib/assets/javascripts/autocomplete-rails.js
diff --git a/.travis.yml b/.travis.yml
@@ -1,3 +1,3 @@
 rvm:
   - 1.9.3
-  - 2.0.0
+  - 2.3.1
diff --git a/lib/assets/javascripts/autocomplete-rails-uncompressed.js b/lib/assets/javascripts/autocomplete-rails-uncompressed.js
@@ -24,6 +24,9 @@
       }
     };
     if (jQuery.fn.on !== undefined) {
+      if (!this.selector) {
+        return;
+      }
       return jQuery(document).on('focus',this.selector,handler);
     }
     else {
diff --git a/lib/assets/javascripts/autocomplete-rails.js b/lib/assets/javascripts/autocomplete-rails.js

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+lib/assets/javascripts/autocomplete-rails.js`
-Original file line number
+Diff line change
@@ @@ -1,3 +1,3 @@ @@
 rvm:
   - 1.9.3
 -  - 2.0.0
 +  - 2.3.1
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,9 @@`
`24`	`24`	`}`
`25`	`25`	`};`
`26`	`26`	`if (jQuery.fn.on !== undefined) {`
	`27`	`+ if (!this.selector) {`
	`28`	`+ return;`
	`29`	`+ }`
`27`	`30`	`return jQuery(document).on('focus',this.selector,handler);`
`28`	`31`	`}`
`29`	`32`	`else {`