[Phpmyadmin-git] [SCM] phpMyAdmin branch, master, updated. RELEASE_3_3_8-10864-g0132e33

Marc Delisle lem9 at users.sourceforge.net
Tue Nov 16 14:23:04 CET 2010


The branch, master has been updated
       via  0132e3317701e017360ef135001048863a3c6452 (commit)
      from  18b0fa1482b2056855917367f64374492801f1fb (commit)


- Log -----------------------------------------------------------------
commit 0132e3317701e017360ef135001048863a3c6452
Author: Marc Delisle <marc at infomarc.info>
Date:   Tue Nov 16 08:22:55 2010 -0500

    prefix for jQuery objects; unneeded wrapping of objects

-----------------------------------------------------------------------

Summary of changes:
 js/sql.js |   60 ++++++++++++++++++++++++++++++------------------------------
 1 files changed, 30 insertions(+), 30 deletions(-)

diff --git a/js/sql.js b/js/sql.js
index 0c39fbe..8e04323 100644
--- a/js/sql.js
+++ b/js/sql.js
@@ -21,22 +21,22 @@ function PMA_urldecode(str) {
  * Get the field name for the current field.  Required to construct the query
  * for inline editing
  *
- * @param   this_field_obj  jQuery object that points to the current field's tr
- * @param   disp_mode       string
+ * @param   $this_field  jQuery object that points to the current field's tr
+ * @param   disp_mode    string
  */
-function getFieldName(this_field_obj, disp_mode) {
+function getFieldName($this_field, disp_mode) {
 
     if(disp_mode == 'vertical') {
-        var field_name = $(this_field_obj).siblings('th').find('a').text();
+        var field_name = $this_field.siblings('th').find('a').text();
     }
     else {
-        var this_field_index = $(this_field_obj).index();
+        var this_field_index = $this_field.index();
         if(window.parent.text_dir == 'ltr') {
             // 4 columns to account for the checkbox, edit, delete and appended inline edit anchors but index is zero-based so substract 3
-            var field_name = $(this_field_obj).parents('table').find('thead').find('th:nth('+ (this_field_index-3 )+') a').text();
+            var field_name = $this_field.parents('table').find('thead').find('th:nth('+ (this_field_index-3 )+') a').text();
         }
         else {
-            var field_name = $(this_field_obj).parents('table').find('thead').find('th:nth('+ this_field_index+') a').text();
+            var field_name = $this_field.parents('table').find('thead').find('th:nth('+ this_field_index+') a').text();
         }
     }
 
@@ -345,20 +345,20 @@ $(document).ready(function() {
              */
             var this_row_index = $(this).index();
             /**
-             * @var input_siblings  Object referring to all inline editable events from same row
+             * @var $input_siblings  Object referring to all inline editable events from same row
              */
-            var input_siblings = $(this).parents('tbody').find('tr').find('.data_inline_edit:nth('+this_row_index+')');
+            var $input_siblings = $(this).parents('tbody').find('tr').find('.data_inline_edit:nth('+this_row_index+')');
             /**
              * @var where_clause    String containing the WHERE clause to select this row
              */
             var where_clause = $(this).parents('tbody').find('tr').find('.where_clause:nth('+this_row_index+')').val();
         }
         else {
-            var input_siblings = $(this).parent('tr').find('.data_inline_edit');
+            var $input_siblings = $(this).parent('tr').find('.data_inline_edit');
             var where_clause = $(this).parent('tr').find('.where_clause').val();
         }
 
-        $(input_siblings).each(function() {
+        $input_siblings.each(function() {
             /** @lends jQuery */
             /**
              * @var data_value  Current value of this field
@@ -371,23 +371,23 @@ $(document).ready(function() {
             /**
              * @var this_field  Object referring to this field (<td>)
              */
-            var this_field = $(this);
+            var $this_field = $(this);
             /**
              * @var field_name  String containing the name of this field.
              * @see getFieldName()
              */
-            var field_name = getFieldName($(this), disp_mode);
+            var field_name = getFieldName($this_field, disp_mode);
 
             // In each input sibling, wrap the current value in a textarea
             // and store the current value in a hidden span
-            if($(this).is(':not(.truncated, .transformed, .relation, .enum, .null)')) {
+            if($this_field.is(':not(.truncated, .transformed, .relation, .enum, .null)')) {
                 // handle non-truncated, non-transformed, non-relation values
                 // We don't need to get any more data, just wrap the value
-                $(this).html('<textarea>'+data_value+'</textarea>')
+                $this_field.html('<textarea>'+data_value+'</textarea>')
                 .append('<span class="original_data">'+data_value+'</span>');
                 $(".original_data").hide();
             }
-            else if($(this).is('.truncated, .transformed')) {
+            else if($this_field.is('.truncated, .transformed')) {
                 /** @lends jQuery */
                 //handle truncated/transformed values values
 
@@ -405,7 +405,7 @@ $(document).ready(function() {
                     'inline_edit' : true
                 }, function(data) {
                     if(data.success == true) {
-                        $(this_field).html('<textarea>'+data.value+'</textarea>')
+                        $this_field.html('<textarea>'+data.value+'</textarea>')
                         .append('<span class="original_data">'+data_value+'</span>');
                         $(".original_data").hide();
                     }
@@ -414,14 +414,14 @@ $(document).ready(function() {
                     }
                 }) // end $.post()
             }
-            else if($(this).is('.relation')) {
+            else if($this_field.is('.relation')) {
                 /** @lends jQuery */
                 //handle relations
 
                 /**
                  * @var curr_value  String containing the current value of this relational field
                  */
-                var curr_value = $(this).find('a').text();
+                var curr_value = $this_field.find('a').text();
 
                 /**
                  * @var post_params Object containing parameters for the POST request
@@ -437,18 +437,18 @@ $(document).ready(function() {
                 }
 
                 $.post('sql.php', post_params, function(data) {
-                    $(this_field).html(data.dropdown)
+                    $this_field.html(data.dropdown)
                     .append('<span class="original_data">'+data_value+'</span>');
                     $(".original_data").hide();
                 }) // end $.post()
             }
-            else if($(this).is('.enum')) {
+            else if($this_field.is('.enum')) {
                 /** @lends jQuery */
                 //handle enum fields
                 /**
                  * @var curr_value  String containing the current value of this relational field
                  */
-                var curr_value = $(this).text();
+                var curr_value = $this_field.text();
 
                 /**
                  * @var post_params Object containing parameters for the POST request
@@ -464,14 +464,14 @@ $(document).ready(function() {
                 }
 
                 $.post('sql.php', post_params, function(data) {
-                    $(this_field).html(data.dropdown)
+                    $this_field.html(data.dropdown)
                     .append('<span class="original_data">'+data_value+'</span>');
                     $(".original_data").hide();
                 }) // end $.post()
             }
-            else if($(this).is('.null')) {
+            else if($this_field.is('.null')) {
                 //handle null fields
-                $(this_field).html('<textarea></textarea>')
+                $this_field.html('<textarea></textarea>')
                 .append('<span class="original_data">NULL</span>');
                 $(".original_data").hide();
             }
@@ -506,16 +506,16 @@ $(document).ready(function() {
              */
             var this_td_index = $this_td.index();
             /**
-             * @var input_siblings  Object referring to all inline editable events from same row
+             * @var $input_siblings  Object referring to all inline editable events from same row
              */
-            var input_siblings = $this_td.parents('tbody').find('tr').find('.data_inline_edit:nth('+this_td_index+')');
+            var $input_siblings = $this_td.parents('tbody').find('tr').find('.data_inline_edit:nth('+this_td_index+')');
             /**
              * @var where_clause    String containing the WHERE clause to select this row
              */
             var where_clause = $this_td.parents('tbody').find('tr').find('.where_clause:nth('+this_td_index+')').val();
         }
         else {
-            var input_siblings = $this_td.parent('tr').find('.data_inline_edit');
+            var $input_siblings = $this_td.parent('tr').find('.data_inline_edit');
             var where_clause = $this_td.parent('tr').find('.where_clause').val();
         }
 
@@ -547,7 +547,7 @@ $(document).ready(function() {
          */
         var transformation_fields = false;
 
-        $(input_siblings).each(function() {
+        $input_siblings.each(function() {
             /** @lends jQuery */
             /**
              * @var this_field  Object referring to this field (<td>)
@@ -634,7 +634,7 @@ $(document).ready(function() {
                 PMA_ajaxShowMessage(data.message);
                 $this_td.removeClass('inline_edit_active').addClass('inline_edit_anchor');
 
-                $(input_siblings).each(function() {
+                $input_siblings.each(function() {
                     // Inline edit post has been successful.
                     if($(this).is(':not(.relation, .enum)')) {
                         /**


hooks/post-receive
-- 
phpMyAdmin




More information about the Git mailing list