AMD don't list dependency on jQuery UI

[crystalfp]

My application relies on AMD and already loads jquery and jquery-ui modules. Now I'm trying to add fancytree. But loading fancytree-all.min.js raises the "Fancytree requires jQuery UI (http://jqueryui.com)" assertion.
The only solution found is to add a RequireJS shim to add the dependency:

requirejs.config({
	paths: {
		"jquery": "jquery-3.1.1.min",
		"jquery-ui": "jquery-ui.min",
		"jquery-fancytree": "jquery.fancytree-all.min"
	},
	shim: {
		"jquery-fancytree": {
			deps: ["jquery-ui"]
		}
	}
});

Seems the AMD support should be corrected to add all required libraries.
Thanks for looking!
mario

[mar10]

Could you test if this header would work for you:

(function( factory ) {
	if ( typeof define === "function" && define.amd ) {
		define( [
			"jquery",
			"jquery-ui/ui/widgets/mouse",
			"jquery-ui/ui/widgets/draggable",
			"jquery-ui/ui/widgets/droppable",
			"jquery-ui/ui/effects/effect-blind",
			"jquery-ui/ui/data",
			"jquery-ui/ui/effect",
			"jquery-ui/ui/focusable",
			"jquery-ui/ui/keycode",
			"jquery-ui/ui/position",
			"jquery-ui/ui/scroll-parent",
			"jquery-ui/ui/tabbable",
			"jquery-ui/ui/unique-id",
			"jquery-ui/ui/widget"
		], factory );
	} else {
		factory( jQuery );
	}
}(function( $ ) {

(See also related #694 for use cases where jQuery UI is required for Fancytree only.)