developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Ticket #781: #781-trunk-radiobuttons-js-check.patch

File #781-trunk-radiobuttons-js-check.patch, 2.1 KB (added by bibo, 12 years ago)

fix is better located into getValue prototype

  • lib/jelix-www/js/jforms_jquery.js

     
    9595                        if (elt.options[i].selected)
    9696                            values.push(elt.options[i].value);
    9797                    }
    98                     return values;
     98                    if ( values.length > 0 )
     99                        return values;
    99100            }
    100101        } else if(elt.item){
    101102            // this is a NodeList of radio buttons or multiple checkboxes
     
    105106                if (item.checked)
    106107                    values.push(item.value);
    107108            }
    108             return values;
     109            if ( values.length > 0 )
     110                return values;
    109111        }
    110112        return '';
    111113    },
  • lib/jelix-www/js/jforms_light.js

     
    7373            var trimVal = val;
    7474            if (val.replace)
    7575                var trimVal = val.replace(/^\s\s*/, '').replace(/\s\s*$/, '');
    76             if(trimVal == ''){
     76            if(trimVal === ''){
    7777                if(c.required){
    7878                    this.tForm.errorDecorator.addError(c, 1);
    7979                    valid = false;
     
    107107                        if (elt.options[i].selected)
    108108                            values.push(elt.options[i].value);
    109109                    }
    110                     return values;
     110                    if ( values.length > 0 )
     111                        return values;
    111112            }
    112113        } else if(elt.item){
    113114            // this is a NodeList of radio buttons or multiple checkboxes
     
    117118                if (item.checked)
    118119                    values.push(item.value);
    119120            }
    120             return values;
     121            if ( values.length > 0 )
     122                return values;
    121123        }
    122124        return '';
    123125    },