dijit.TooltipDialog broken after r11841

[guest]

Vladimir Solomenchuk/cla

due to changes in dijit._base.place not orient function in widget call with 3(node, aroundCorner, corner) parameters, instead of 1 (corner)

Index: Dialog.js
===================================================================
--- Dialog.js	(revision 11889)
+++ Dialog.js	(working copy)
@@ -351,14 +351,14 @@
 			this.containerNode.title=this.title;
 		},
 
-		orient: function(/*Object*/ corner){
+		orient: function(node, aroundCorner, /*Object*/ corner){
 			// summary: configure widget to be displayed in given position relative to the button
 			this.domNode.className="dijitTooltipDialog " +" dijitTooltipAB"+(corner.charAt(1)=='L'?"Left":"Right")+" dijitTooltip"+(corner.charAt(0)=='T' ? "Below" : "Above");
 		},
 
 		onOpen: function(/*Object*/ pos){
 			// summary: called when dialog is displayed
-			this.orient(pos.corner);
+			this.orient(null,pos.aroundCorner, pos.corner);
 			this._loadCheck(); // lazy load trigger
 			this.containerNode.focus();
 		},

[bill]

Oops, sorry about that, will fix.

[bill]

PS: but, can you give a more specific description of what problem you are having? All you wrote is that "it's broken". Also, why do you think that Dialog.orient() needs the same signature as dijit._place()'s third argument?

[guest]

Vladimir Solomenchuk/cla Problem reproduced in test_Toolbar.html. Click Login button and you will receive error "corner.charAt not a function". TooltipDialog?.orient called twice - once from dijit._base.place and second TooltipDialog?.onOpen. Problem caused by call from dijit._base.place at line no 115 with code

if(layoutNode){
   layoutNode(node, choices[i].aroundCorner, corner);
}

[bill]

Oh I see. OK, thanks!

[bill]

Ugh, fixed by [11890] but I wrote the wrong version number in the checkin.