Declaration: can't really create/override methods

[bill]

The Declaration code lets you declare a class with methods:

<table dojoType="dijit.Declaration" widgetClass="demo.Table" ...>
	<script type="dojo/method" event="onSort" args="index">
		...
	</script>
</div>

Unfortunately methods declared this way are implemented as connections, rather than as real methods in the declared class' prototype. This leads to many issues including:

• can't return a value
• can't override an existing method
• i suspect this.inherited() doesn't work

It also breaks the deferredConnect code in Widget.js that checks whether an event handler method has been redefined from a sentinel value:

for(attr in this._deferredConnects){
	if(this[attr] !== dijit._connectOnUseEventHandler){
		delete this._deferredConnects[attr];	// redefined, probably dojoAttachEvent exists
	}
}

... last issue is leading to JS errors in mail demo when clicking an item in the inbox, since an unwanted connection is getting setup from this.domNode.onclick --> this.onClick.

[bill]

(In [15238]) Refactor Declaration code so that methods become real methods. Also do dojo/connects on prototype methods rather than using fake mixin trick.

Also updated demo.Table example to declare onClick method so that code in Widget looking for override to onClick works.

Fixes #7650 !strict

[Juho Manninen]

Changeset [15238] breaks preamble - see my test case on Dijit.Declaration

[bill]

Ah, yeah I see your point; looks like the current code in Declaration.js just ignores a <script type="dojo/method" ...> declaration of preamble (when it should be overriding the definition in the base class).

Probably can just remove the line

preambles = dojo.query("> script[type='dojo/method'][event='preamble']", src).orphan(),

from Declaration.js as it doesn't seem necessary to special case preamble.

[bill]

Maine - thanks for the test case.

Fixed in [15603].