From 9eb23b5aec575549229ad9901be6f82233e6564c Mon Sep 17 00:00:00 2001
From: xmujay <xmujay@gmail.com>
Date: Sun, 5 May 2013 20:49:15 +0800
Subject: [PATCH] fix some JSLint error: using dot notation

---
 js/config.js               |  2 +-
 js/makegrid.js             |  4 ++--
 js/navigation.js           |  4 ++--
 js/server_privileges.js    | 12 ++++++------
 js/tbl_zoom_plot_jqplot.js |  4 ++--
 5 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/js/config.js b/js/config.js
index 83482a0e3f..b7115a6e94 100644
--- a/js/config.js
+++ b/js/config.js
@@ -670,7 +670,7 @@ AJAX.registerOnload('config.js', function () {
 
     // detect localStorage state
     var ls_supported = window.localStorage || false;
-    var ls_exists = ls_supported ? (window.localStorage['config'] || false) : false;
+    var ls_exists = ls_supported ? (window.localStorage.config || false) : false;
     $('div.localStorage-' + (ls_supported ? 'un' : '') + 'supported').hide();
     $('div.localStorage-' + (ls_exists ? 'empty' : 'exists')).hide();
     if (ls_exists) {
diff --git a/js/makegrid.js b/js/makegrid.js
index 2b2e882201..3a9ebb619c 100644
--- a/js/makegrid.js
+++ b/js/makegrid.js
@@ -1444,9 +1444,9 @@ function PMA_makegrid(t, enableResize, enableReorder, enableVisib, enableGridEdi
                 .dblclick(function (e) {
                     e.preventDefault();
                     $("<div/>")
-                    .prop("title", PMA_messages["strColNameCopyTitle"])
+                    .prop("title", PMA_messages.strColNameCopyTitle)
                     .addClass("modal-copy")
-                    .text(PMA_messages["strColNameCopyText"])
+                    .text(PMA_messages.strColNameCopyText)
                     .append(
                         $("<input/>")
                         .prop("readonly", true)
diff --git a/js/navigation.js b/js/navigation.js
index 138685bc39..c4f6afdd91 100644
--- a/js/navigation.js
+++ b/js/navigation.js
@@ -134,8 +134,8 @@ $(function () {
         if (this.value !== '') {
             var arr = jQuery.parseJSON(this.value);
             var $form = $(this).closest('form');
-            $form.find('input[name=db]').val(arr['db']);
-            $form.find('input[name=table]').val(arr['table']);
+            $form.find('input[name=db]').val(arr.db);
+            $form.find('input[name=table]').val(arr.table);
             $form.submit();
         }
     });
diff --git a/js/server_privileges.js b/js/server_privileges.js
index 9cc7af1ca6..9da401a922 100644
--- a/js/server_privileges.js
+++ b/js/server_privileges.js
@@ -16,15 +16,15 @@
  */
 function checkAddUser(the_form)
 {
-    if (the_form.elements['pred_hostname'].value == 'userdefined' && the_form.elements['hostname'].value === '') {
+    if (the_form.elements.pred_hostname.value == 'userdefined' && the_form.elements.hostname.value === '') {
         alert(PMA_messages.strHostEmpty);
-        the_form.elements['hostname'].focus();
+        the_form.elements.hostname.focus();
         return false;
     }
 
-    if (the_form.elements['pred_username'].value == 'userdefined' && the_form.elements['username'].value === '') {
+    if (the_form.elements.pred_username.value == 'userdefined' && the_form.elements.username.value === '') {
         alert(PMA_messages.strUserEmpty);
-        the_form.elements['username'].focus();
+        the_form.elements.username.focus();
         return false;
     }
 
@@ -196,7 +196,7 @@ AJAX.registerOnload('server_privileges.js', function () {
                 'username' : username
             };
             $.get(href, params, function(data) {
-                if (data['user_exists']) {
+                if (data.user_exists) {
                     $warning.show();
                 } else {
                     $warning.hide();
@@ -405,7 +405,7 @@ AJAX.registerOnload('server_privileges.js', function () {
 
         // If any option other than 'keep the old one'(option 4) is chosen, we need to remove
         // the old one from the table.
-        var $row_to_remove = null;
+        var $row_to_remove;
         if (curr_submit_name == 'change_copy'
                 && $('input[name=mode]:checked', '#fieldset_mode').val() != '4') {
             var old_username = $t.find('input[name="old_username"]').val();
diff --git a/js/tbl_zoom_plot_jqplot.js b/js/tbl_zoom_plot_jqplot.js
index f6a0532403..369ce6aa48 100644
--- a/js/tbl_zoom_plot_jqplot.js
+++ b/js/tbl_zoom_plot_jqplot.js
@@ -377,7 +377,7 @@ AJAX.registerOnload('tbl_zoom_plot_jqplot.js', function () {
                 }
             }
             sql_query = sql_query.substring(0, sql_query.length - 2);
-            sql_query += ' WHERE ' + PMA_urldecode(searchedData[searchedDataKey]['where_clause']);
+            sql_query += ' WHERE ' + PMA_urldecode(searchedData[searchedDataKey].where_clause);
 
             //Post SQL query to sql.php
             $.post('sql.php', {
@@ -546,7 +546,7 @@ AJAX.registerOnload('tbl_zoom_plot_jqplot.js', function () {
                 // extra Y values
                 value[dataLabel], // for highlighter
                                   // (may set an undefined value)
-                value['where_clause'], // for click on point
+                value.where_clause, // for click on point
                 key               // key from searchedData
             ]);
         });