Neyric WireIt

Comments and changes to this ticket

• You flagged this item as spam.
  

  ingmars (at web) July 5th, 2009 @ 07:26 PM

  Here's a solution for the mentioned bug (maybe not the most elegant one, but at least it works.)

  The problem was, that the wirable sub fields didn't have a back reference to their containers any more. Therefore, they didn't "know" at what point to stop adding up the top / left offsets and just calculated all the way done to the body element.

  It did work for fields on the first level, but not for sub fields.

  Solution: Just set the options.container property for sub fields of group of list fields as well.

  ===============================================================================

  In the file js/util/inputex/FormContainer-beta.js replace the method renderForm with
  the following two methods:

  /**

  * Render the form
  * @method renderForm
  */
  

  renderForm: function() {

    this.setBackReferenceOnFieldOptionsRecursively(this.options.fields);
  
    var groupParams = {parentEl: this.bodyEl, fields: this.options.fields, legend: this.options.legend, collapsible: this.options.collapsible};
    this.form = new YAHOO.inputEx.Group(groupParams);
  

  },

  setBackReferenceOnFieldOptionsRecursively: function(fieldArray) {

    for(var i = 0 ; i < fieldArray.length ; i++) {
        var inputParams = fieldArray[i].inputParams;
        inputParams.container = this;
  
        // Checking for group sub elements
        if(inputParams.fields && typeof inputParams.fields == 'object') {
            this.setBackReferenceOnFieldOptionsRecursively(inputParams.fields);
        }
  
        // Checking for list sub elements
        if(inputParams.elementType) {
            inputParams.elementType.inputParams.container = this;
  
            // Checking for group elements within list elements
            if(inputParams.elementType.inputParams.fields && typeof inputParams.elementType.inputParams.fields == 'object') {
                this.setBackReferenceOnFieldOptionsRecursively(inputParams.elementType.inputParams.fields);
            }
        }
    }
  

  },

  ===============================================================================

• You flagged this item as spam.
  

  ingmars (at web) July 5th, 2009 @ 07:27 PM

  Once again, now nicer to read (hopefully!)

  In the file js/util/inputex/FormContainer-beta.js replace the method renderForm with
  the following two methods:

  
     /**
      * Render the form
      * @method renderForm
      */
     renderForm: function() {
        this.setBackReferenceOnFieldOptionsRecursively(this.options.fields);
        
        var groupParams = {parentEl: this.bodyEl, fields: this.options.fields, legend: this.options.legend, collapsible: this.options.collapsible};
        this.form = new YAHOO.inputEx.Group(groupParams);
     },
     
     setBackReferenceOnFieldOptionsRecursively: function(fieldArray) {
        for(var i = 0 ; i < fieldArray.length ; i++) {
            var inputParams = fieldArray[i].inputParams;
            inputParams.container = this;
  
            // Checking for group sub elements
            if(inputParams.fields && typeof inputParams.fields == 'object') {
                this.setBackReferenceOnFieldOptionsRecursively(inputParams.fields);
            }
  
            // Checking for list sub elements
            if(inputParams.elementType) {
                inputParams.elementType.inputParams.container = this;
  
                // Checking for group elements within list elements
                if(inputParams.elementType.inputParams.fields && typeof inputParams.elementType.inputParams.fields == 'object') {
                    this.setBackReferenceOnFieldOptionsRecursively(inputParams.elementType.inputParams.fields);
                }
            }
        }
     },
  

• 

  eric.abouaf (at gmail) July 5th, 2009 @ 08:11 PM

  • State changed from “new” to “resolved”

  Thanks a lot for this fix ! This works perfectly.
  It even works for more field than group and list.

  It has been commited on the github repository

• You flagged this item as spam.
  

  ingmars (at web) July 5th, 2009 @ 08:37 PM

  Cool, thanks for committing!