[Rt-commit] rt branch, 4.0/multiple-level-cascaded-cf-js-fix, created. rt-4.0.18-130-gaa9e15e

Alex Vandiver alexmv at bestpractical.com
Fri Dec 27 16:06:18 EST 2013


The branch, 4.0/multiple-level-cascaded-cf-js-fix has been created
        at  aa9e15e4d06135a3958473f1a88ef799622d8087 (commit)

- Log -----------------------------------------------------------------
commit aa9e15e4d06135a3958473f1a88ef799622d8087
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Thu Dec 26 18:59:21 2013 +0800

    more strict basedon element select in case cascaded cfs are used as category as well
    
    old select will get the hidden "...-Complete" element, which is wrong.
    see also #28022

diff --git a/share/html/Elements/EditCustomFieldSelect b/share/html/Elements/EditCustomFieldSelect
index b7a3112..374c68c 100644
--- a/share/html/Elements/EditCustomFieldSelect
+++ b/share/html/Elements/EditCustomFieldSelect
@@ -66,7 +66,7 @@
 <script type="text/javascript" src="<%RT->Config->Get('WebPath')%>/NoAuth/js/cascaded.js"></script>
 <script type="text/javascript"><!--
 jQuery(  function () {
-    var basedon = jQuery('[name^=<% $NamePrefix .  $CustomField->BasedOnObj->id %>-Value][type!=hidden]:input');
+    var basedon = jQuery('[name^=<% $NamePrefix .  $CustomField->BasedOnObj->id %>-Value][type!=hidden]:input:not(.hidden)');
     basedon.each( function() {
         var oldchange = jQuery(this).onchange;
         jQuery(this).change( function () {

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


More information about the rt-commit mailing list