dojoAttachEvent problem within ContentPane

[guest]

0.9 nightly 24/06/07 I have a custom templated widget that uses a content pane within the template Something like this:

Test.html

<div>
<div dojoType="dijit.layout.ContentPane" sizeMin="20" sizeShare="20" id="pageViewer">
<button  dojoAttachEvent="onclick:addPage">Test</button>
</div></div>

Test.js

if(!dojo._hasResource["foo.Test"]){
dojo._hasResource["foo.Test"] = true;
dojo.provide("foo.Test");
dojo.require("dijit.layout.ContentPane");
dojo.require("dijit._Widget");
dojo.require("dijit._Templated");
dojo.declare(
	"foo.Test",
	[dijit._Widget, dijit._Templated],
	{
    	
	   templatePath:"dojotoolkit/foo/templates/Test.html",	
        widgetsInTemplate:true,
		
        addPage: function() {
        alert("bar");    
        }
 

	}
);

When clicking on the button displayed nothing happens. However, if I remove the content pane widget clicking on the button brings up the alert box.

It's as if nested dojoAttachEvent attributes are not being processed.

The actual use case for this scenario is when a widget contains a dialog where the controls affect the containee widget.

Affects Firefox and IE 7.

[guest]

I have a couple of suggestions although I appreciate that the behaviour that I am seeing could be by design, in which case please just close this bug:

In dijit._Templated._attachTemplateNodes:

1. The call to this.connect(baseNode, tevt, thisFunc); should check to see if the target function exists on the baseNode. If it doesn't then it should attach to the baseNode.domNode as the node may be an html input of some sort.

   if(baseNode[tevt]) {
   this.connect(baseNode, tevt, thisFunc);
   }
   else {
   this.connect(baseNode.domNode, tevt, thisFunc);
   }
   

Even if the widget does not contain any widgets it still needs to process dojoAttachPoint and dojoAttachEvent attributes

2. All nodes still need to be processed even if a widget is encountered.

   if(this.widgetsInTemplate && getAttrFunc(baseNode,'dojoType')){
   return;
   }
   

currently short circuits the function if a widget is encountered, however that widget could be a container widget and will need its child nodes processed. Therefore the logic can be changed to:

if(!(this.widgetsInTemplate && getAttrFunc(baseNode,'dojoType'))){

    ---Rest of loop contents here---

}

I have attached the fully modified version of this method.

[guest]

Please ignore the following from the above change:

'Even if the widget does not contain any widgets it still needs to process dojoAttachPoint and dojoAttachEvent attributes'

[guest]

I have done some further development and come up with the following for suggestion 1 above.

if(baseNode[tevt] || baseNode["domNode"] == null) {
//All widgets where the target method exists or all non widgets
this.connect(baseNode, tevt, thisFunc);
}
else {
//All widgets where target event does not exist
this.connect(baseNode.domNode, tevt, thisFunc);
}

[bill]

I don't think this is intended to be supported. Cougar, what do you think?

[guest]

Looks like suggestion 2 has been fixed in the 0.9 beta. Suggestion 1 desn't really pertain to this bug, I'll open a separate enhancement ticket with full justification there.