The branch, master has been updated via 1b0116ba3b320bdeb3c36f7cdcb2c810e68df32b (commit) from 20619dd544b180f2040bb4068a2891e7fb8cc622 (commit)

- Log ----------------------------------------------------------------- commit 1b0116ba3b320bdeb3c36f7cdcb2c810e68df32b Author: Marc Delisle marc@infomarc.info Date: Sat Jan 1 09:38:33 2011 -0500

The filter on textfield did not do the intended result, it was catching other input elements; Unneeded wrapping of jQuery objects Prefix jQuery objects with dollar sign

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

Summary of changes: js/tbl_change.js | 33 ++++++++++++++++++--------------- 1 files changed, 18 insertions(+), 15 deletions(-)

diff --git a/js/tbl_change.js b/js/tbl_change.js index 41aecd0..861ee99 100644 --- a/js/tbl_change.js +++ b/js/tbl_change.js @@ -362,9 +362,9 @@ $(document).ready(function() { while( curr_rows < target_rows ) {

/** - * @var last_row Object referring to the last row + * @var $last_row Object referring to the last row */ - var last_row = $("#insertForm").find(".insertRowTable:last"); + var $last_row = $("#insertForm").find(".insertRowTable:last");

// need to access this at more than one level // (also needs improvement because it should be calculated @@ -372,12 +372,13 @@ $(document).ready(function() { var new_row_index = 0;

//Clone the insert tables - $(last_row) + $last_row .clone() .insertBefore("#actions_panel") .find('input[name*=multi_edit],select[name*=multi_edit]') .each(function() {

+ var $this_element = $(this); /** * Extract the index from the name attribute for all input/select fields and increment it * name is of format funcs[multi_edit][10][<long random string of alphanum chars>] @@ -386,7 +387,7 @@ $(document).ready(function() { /** * @var this_name String containing name of the input/select elements */ - var this_name = $(this).attr('name'); + var this_name = $this_element.attr('name'); /** split {@link this_name} at [10], so we have the parts that can be concatenated later */ var name_parts = this_name.split(/[\d+]/); /** extract the [10] from {@link name_parts} */ @@ -400,9 +401,10 @@ $(document).ready(function() { var new_name = name_parts[0] + '[' + new_row_index + ']' + name_parts[1];

var hashed_field = name_parts[1].match(/[(.+)]/)[1]; - $(this).attr('name', new_name); + $this_element.attr('name', new_name);

- $(this).filter('.textfield') + if ($this_element.is('.textfield')) { + $this_element .attr('value', '') .unbind('change') .attr('onchange', null) @@ -410,22 +412,23 @@ $(document).ready(function() { Validator( hashed_field, new_row_index, - $(this).closest('tr').find('span.column_type').html() + $this_element.closest('tr').find('span.column_type').html() ); - }) - .end(); + }); + }

- $(this).filter('.checkbox_null') + if ($this_element.is('.checkbox_null')) { + $this_element .bind('click', function(e) { nullify( - $(this).siblings('.nullify_code').val(), - $(this).closest('tr').find('input:hidden').first().val(), + $this_element.siblings('.nullify_code').val(), + $this_element.closest('tr').find('input:hidden').first().val(), hashed_field, '[multi_edit][' + new_row_index + ']' ); - }) - .end(); - }) + }); + } + }) // end each .end() .find('.foreign_values_anchor') .each(function() {

hooks/post-receive