From: Scott González <scott.gonzalez@gmail.com>
Date: Fri, 5 Oct 2012 15:26:49 +0000 (-0400)
Subject: Tooltip: Check if event exists before checking properties in open(). Fixes #8626... 
X-Git-Tag: 1.9.0~2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=96f2aa4c72b5f29782b0736ed95468e50d338d6e;p=jquery-ui.git

Tooltip: Check if event exists before checking properties in open(). Fixes #8626 - Programatically opening a tooltip with out giving an event results in a javascript error.
---

diff --git a/tests/unit/tooltip/tooltip_methods.js b/tests/unit/tooltip/tooltip_methods.js
index 0af1e06e9..896e910c6 100644
--- a/tests/unit/tooltip/tooltip_methods.js
+++ b/tests/unit/tooltip/tooltip_methods.js
@@ -29,6 +29,23 @@ test( "open/close", function() {
 	$.fx.off = false;
 });
 
+// #8626 - Calling open() without an event
+test( "open/close with tracking", function() {
+	expect( 3 );
+	$.fx.off = true;
+	var tooltip,
+		element = $( "#tooltipped1" ).tooltip({ track: true });
+	equal( $( ".ui-tooltip" ).length, 0, "no tooltip on init" );
+
+	element.tooltip( "open" );
+	tooltip = $( "#" + element.data( "ui-tooltip-id" ) );
+	ok( tooltip.is( ":visible" ) );
+
+	element.tooltip( "close" );
+	ok( tooltip.is( ":hidden" ) );
+	$.fx.off = false;
+});
+
 test( "enable/disable", function() {
 	expect( 7 );
 	$.fx.off = true;
diff --git a/ui/jquery.ui.tooltip.js b/ui/jquery.ui.tooltip.js
index a97d698e3..32aad6487 100644
--- a/ui/jquery.ui.tooltip.js
+++ b/ui/jquery.ui.tooltip.js
@@ -216,7 +216,7 @@ $.widget( "ui.tooltip", {
 			positionOption.of = event;
 			tooltip.position( positionOption );
 		}
-		if ( this.options.track && /^mouse/.test( event.originalEvent.type ) ) {
+		if ( this.options.track && event && /^mouse/.test( event.originalEvent.type ) ) {
 			positionOption = $.extend( {}, this.options.position );
 			this._on( this.document, {
 				mousemove: position