support ioArgs argument to ContentPane

[ptwobrussell]

While digging into ContentPane? (as of r12172), I noticed the following lines in _downloadExternalContent:

      if(dojo.isObject(this.ioArgs)){
            dojo.mixin(getArgs, this.ioArgs);
        }     

Well, as it turns out, ioArgs is not defined in this widget, or any that it inherits from. So, if you try to include ioArgs when creating it, it has no effect. For example:

div preload="true" ioArgs="{handleAs:'javascript'}" dojoType="dijit.layout.ContentPane" href="./bar">

does not treat the returned content from href as javascript. Instead it treats it as text, the default for handleAs via ioArgs.

Seems to me that the fix is trivial: define ioArgs as a member and then in constructor actually create it, (because defining it inline since it would be treated as a static since ioArgs is an object).

I marked this as high priority for 1.1, because it seems very important and the fix is trivial. (I checked the logs and this "problem" has existed forever it seems--making me wonder if I'm missing something here since this hasn't been reported yet.)

In case you're wondering, my working assumption for the use case is that it would be really nice to use a ContentPane? to have the server return back an arbitrary dijit that was not originally dojo.required into the parent document. e.g. getting JavaScript? back from the ContentPane? that would dojo.require in the dijit, construct it, and append it to somewhere in the page at will (preferably the ContentPane? itself, I guess).

[ptwobrussell]

I just noticed a weird-ism when experimenting with JavaScript? that is returned from the server: It appears that document.body refers to the ContentPane? versus the entire document's body. For example, if the script returned from the server via bar (see example above) is

var e = document.createElement("div");
e.innerHTML = "foobar";
document.body.appendChild(e); //appends inside the ContentPane
console.log(document.body) //shows the document's body, not the ContentPane

then the <div>foobar</div> gets appended *into* the ContentPane? instead of after it. I can't see how that would be expected, but wanted to follow up and note it so someone could confirm/deny/explain.

[bill]

Hmm, ContentPane is only meant to support text in a href so this isn't a bug but rather an enhancement request, and I'm not sure how important it is given that in theory you could just wrap your text from the server in a <script> tag, and use dojox.layout.ContentPane? with evalScripts=true option.

Basically, this is beyond what I intend dijit.layout.ContentPane? to support, but will consider for 2.0.

[alex]

Milestone 2.0 deleted

[bill]

Sam can you take a look at these? Hopefully these are fixed by your refactor or you can fix them

[bill]

OK, I guess this ticket has nothing in particular to do with handleAs... could be setting timeout etc.

I'll fix as you suggested... note though that this.ioArgs={}; in the constructor is not necessary, since when the parser sets ioArgs it will do this.ioArgs = { ... }; , defining a new Object private to the instance, rather than mucking w/the existing shared object.

[bill]

(In [15958]) Fixes #5686: support ioArgs argument to ContentPane? declared via markup. ioArgs argument to ContentPane? was already supported for programatic creation. !strict