ProjectPicker: Strict-Modus & Warnungen von jshint beseitigt
authorMoritz Bunkus <m.bunkus@linet-services.de>
Fri, 20 Jan 2017 11:41:50 +0000 (12:41 +0100)
committerMoritz Bunkus <m.bunkus@linet-services.de>
Fri, 20 Jan 2017 11:44:24 +0000 (12:44 +0100)
js/autocomplete_project.js

index fb2285f..9234340 100644 (file)
@@ -1,4 +1,6 @@
 namespace('kivi', function(k){
+  "use strict";
+
   k.ProjectPicker = function($real, options) {
     // short circuit in case someone double inits us
     if ($real.data("project_picker"))
@@ -71,7 +73,7 @@ namespace('kivi', function(k){
       if (state == STATES.PICKED) {
         annotate_state();
         return true
-      } else if (state == STATES.UNDEFINED && $dummy.val() == '')
+      } else if (state == STATES.UNDEFINED && $dummy.val() === '')
         set_item({})
       else {
         last_unverified_dummy = $dummy.val();
@@ -83,7 +85,7 @@ namespace('kivi', function(k){
     function annotate_state () {
       if (state == STATES.PICKED)
         $dummy.removeClass(STATES.UNDEFINED).addClass(STATES.PICKED);
-      else if (state == STATES.UNDEFINED && $dummy.val() == '')
+      else if (state == STATES.UNDEFINED && $dummy.val() === '')
         $dummy.removeClass(STATES.UNDEFINED).addClass(STATES.PICKED);
       else {
         last_unverified_dummy = $dummy.val();
@@ -99,7 +101,7 @@ namespace('kivi', function(k){
         }, ajax_data(function(){ var val = $('#project_picker_filter').val(); return val === undefined ? '' : val })),
         success: function(data){ $('#project_picker_result').html(data) }
       });
-    };
+    }
 
     function result_timer (event) {
       if (!$('no_paginate').prop('checked')) {
@@ -135,7 +137,7 @@ namespace('kivi', function(k){
           annotate_state();
         }
       });
-    };
+    }
 
     $dummy.autocomplete({
       source: function(req, rsp) {
@@ -165,7 +167,7 @@ namespace('kivi', function(k){
     $dummy.keydown(function(event){
       if (event.which == KEY.ENTER || event.which == KEY.TAB) {
         // if string is empty assume they want to delete
-        if ($dummy.val() == '') {
+        if ($dummy.val() === '') {
           set_item({});
           return true;
         } else if (state == STATES.PICKED) {