fernando/luos
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 6006: Has been required a replacement of the functionality of the rendering of dependent dropdown fields, the remove function is no longer required.

Browse Source
This commit is contained in:
Gustavo Adolfo Cruz Laura
2011-02-21 22:52:11 +00:00
parent 8257290a65
commit 6de947d96e
1 changed files with 18 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
gulliver/js/form/core/form.js
View File

@@ -56,7 +56,6 @@ function G_Form ( element, id )
function G_Field ( form, element, name ) function G_Field ( form, element, name )
{ {
var dependentGridRow;
var me=this; var me=this;
this.form=form; this.form=form;
this.element=element; this.element=element;
@@ -159,7 +158,7 @@ function G_Field ( form, element, name )
oAux2.setValue(newcont[i].value); oAux2.setValue(newcont[i].value);
} }
// this line is also needed to trigger the onchange event to trigger the calculation of // this line is also needed to trigger the onchange event to trigger the calculation of
// sumatory or average funct5ions in text fields // sumatory or average functions in text fields
if (i == (newcont.length-1)){ if (i == (newcont.length-1)){
if (oAux2.element.fireEvent) { if (oAux2.element.fireEvent) {
oAux2.element.fireEvent("onchange"); oAux2.element.fireEvent("onchange");
@@ -278,17 +277,20 @@ function G_DropDown( form, element, name )
this.parent = G_Field; this.parent = G_Field;
this.parent( form, element, name ); this.parent( form, element, name );
this.setContent=function(content) { this.setContent=function(content) {
var dd=me.element; var dd=me.element;
while(dd.options.length>0) dd.remove(0); for (var key in dd.options){
for(var o=0;o<content.options.length;o++) { dd.options[key] = null;
var optn = $dce("OPTION"); }
optn.text = content.options[o].value; // the remove function is no longer reliable
optn.value = content.options[o].key; // while(dd.options.length>0) dd.remove(0);
dd.options[o]=optn; for(var o=0;o<content.options.length;o++) {
} var optn = $dce("OPTION");
optn.text = content.options[o].value;
optn.value = content.options[o].key;
dd.options[o]=optn;
}
} }
if (!element) return; if (!element) return;
//alert("hello");
leimnud.event.add(this.element,'change',this.updateDepententFields); leimnud.event.add(this.element,'change',this.updateDepententFields);
} }
G_DropDown.prototype=new G_Field(); G_DropDown.prototype=new G_Field();
@@ -1596,10 +1598,12 @@ var validateGridForms = function(invalidFields){
} }
/** /**
* Note added by Gustavo Cruz gustavo-at-colosa.com *
* This function seems that validate via javascript * This function validates via javascript
* the required fields in the Json array in the form tag of a dynaform * the required fields in the Json array in the form tag of a dynaform
* now the required fields in a grid have a "required" atribute set in 1 * now the required fields in a grid have a "required" atribute set in 1
* @param String sRequiredFields
*
**/ **/
var validateForm = function(sRequiredFields) { var validateForm = function(sRequiredFields) {
@@ -1780,7 +1784,7 @@ var validateForm = function(sRequiredFields) {
sMessage += (j > 0)? ', ': ''; sMessage += (j > 0)? ', ': '';
sMessage += invalid_fields[j]; sMessage += invalid_fields[j];
} }
new leimnud.module.app.alert().make({ new leimnud.module.app.alert().make({
label:G_STRINGS.ID_REQUIRED_FIELDS + ": <br/><br/>[ " + sMessage + " ]", label:G_STRINGS.ID_REQUIRED_FIELDS + ": <br/><br/>[ " + sMessage + " ]",
width:450, width:450,