Fixed undefined bug

Author: gsingh93

src/Utils.ts

@@ -1,8 +1,8 @@
 /// <reference path="d.ts/DefinitelyTyped/chrome/chrome.d.ts"/>
 
 module Utils {
-    var logging = true;
-    export function log(message: string) {
+    var logging = false;
+    export function log(message: any) {
         if (logging) {
             console.log(message);
         }
@@ -14,16 +14,16 @@ module Utils {
 
     export function withActiveTab(callback: ActiveTabCallback) {
         chrome.tabs.query({ active: true, lastFocusedWindow: true },
-            function (tabs: chrome.tabs.Tab[]) {
+            function(tabs: chrome.tabs.Tab[]) {
 
             console.assert(tabs.length == 1);
             callback(tabs[0]);
         });
     }
 
     export function sendCommand(commandName: string, responseHandler?: any): void {
-        (function (commandName, responseHandler) {
-            withActiveTab(function (tab: chrome.tabs.Tab) {
+        (function(commandName, responseHandler) {
+            withActiveTab(function(tab: chrome.tabs.Tab) {
                 if (typeof responseHandler === "undefined") {
                     responseHandler = null;
                 }

src/background/TabStateManager.ts

@@ -23,9 +23,11 @@ class TabStateManager {
     public set(tabId: number, propName: string, propVal: any);
     public set(tabId: number, stateOrPropName: any, propVal?: any) {
         if (typeof propVal === "undefined") {
+            Utils.log("Set " + tabId + " to:");
+            Utils.log(stateOrPropName);
             this.tabStates[tabId] = stateOrPropName;
-        }
-        else {
+        } else {
+            Utils.log("Set " + stateOrPropName + " for " + tabId + " to " + propVal);
             this.tabStates[tabId][stateOrPropName] = propVal;
         }
     }
@@ -34,9 +36,7 @@ class TabStateManager {
     public get(tabId: number, propName: string): any;
     public get(tabId: number, propName?: string) {
         if (typeof propName === "undefined") {
-            return this.tabStates[tabId];
-        }
-        else {
+        } else {
             return this.tabStates[tabId][propName];
         }
     }

src/popup/popup.ts

@@ -8,7 +8,7 @@ module Popup {
     var queryInput  = <HTMLInputElement> document.getElementById("query");
     var caseInsensitiveCheckbox = <HTMLInputElement> document.getElementById("case-insensitive");
 
-    Utils.withActiveTab(function (tab: chrome.tabs.Tab) {
+    Utils.withActiveTab(function(tab: chrome.tabs.Tab) {
         var id = tab.id;
         var tabStates = BackgroundInterface.getTabStateManager();
 
@@ -18,6 +18,8 @@ module Popup {
         if (!tabStates.exists(id)) {
             Utils.log("ID doesn't exist. Initializing entry.")
             tabStates.resetState(id);
+            var tabState = tabStates.get(id);
+            Utils.log(tabState);
         }
 
         addListeners(id, tabStates);
@@ -44,27 +46,27 @@ module Popup {
             }
         }
 
-        var queryInputInput = () => {
-            tabStates.set(id, "query", this.value);
+        var queryInputInput = function() {
+            tabStates.set(id, "query", queryInput.value);
 
             if (tabStates.isSearching(id)) {
                 tabStates.set(id, "searching", false);
                 Utils.sendCommand("clear");
             }
 
             // Remove the invalid class if it's there
-            this.className = '';
+            queryInput.className = '';
 
-            if (this.value == "") {
+            if (queryInput.value == "") {
                 setEnabled("next", false);
             } else {
                 setEnabled("next", true);
             }
         }
 
         var checkboxClick = function() {
-            Utils.log("Set checkbox state to " + this.checked);
-            tabStates.set(id, "caseInsensitive", this.checked);
+            Utils.log("Set checkbox state to " + caseInsensitiveCheckbox.checked);
+            tabStates.set(id, "caseInsensitive", caseInsensitiveCheckbox.checked);
         }
 
         prevButton.addEventListener("click", prevButtonClick);