jerromo / managediframe

Automatically exported from code.google.com/p/managediframe
0 stars 0 forks source link

ManagedIFrame v2.1.6 calls getDom() with wrong signature #12

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Ext.ux.ManagedIFrame.Element in version 2.1.6 has a method getDom(el) which 
delegates the call to Ext.getDom(el,d). This method needs to be called like in 
other places in code with a second parameter indicating if the check of the 
existence should be made in strict mode. This method passes a second argument 
the document so the result, if this element exists, always evaluates to null. 
This error occurs, if one is working with DDProxies and passing elem.id instead 
of the dom reference.

Original code : 

 /**
   * Return the dom node for the passed string (id), dom node, or
   * Ext.Element relative to the embedded frame document context.
   *
   * @param {Mixed} el
   * @return HTMLElement
   */
getDom : function(el) {
   var d;
   if (!el || !(d = this.getFrameDocument())) {
     return (d=null);
   }
   return Ext.getDom(el, d);
}

Greetings Oliver Fischer

Original issue reported on code.google.com by ofisc...@v-welten.de on 12 Jun 2012 at 7:45

GoogleCodeExporter commented 8 years ago
Method getDom(el,strict,doc) is called at line 377 in multidom.js

Original comment by ofisc...@v-welten.de on 12 Jun 2012 at 7:47