Fixing missing requires for dojo.declare(), refs #13035 !strict. Also…

… removed module return values for deprecated modules, and did various other cleanup. git-svn-id: http://svn.dojotoolkit.org/src/dijit/trunk@25823 560b804f-0ae3-0310-86f3-f6aa0a117693
dojo · Jul 19, 2011 · aa4c588 · aa4c588
1 parent 19bf5af
commit aa4c588
Show file tree

Hide file tree

Showing 61 changed files with 65 additions and 31 deletions.
diff --git a/InlineEditBox.js b/InlineEditBox.js
@@ -12,9 +12,10 @@ define([
 	"./focus",
 	"dojo/i18n!./nls/common",
 	"dojo/_base/array", // dojo.forEach
-	"dojo/_base/connect", // dojo.keys.ENTER dojo.keys.ESCAPE
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.addClass dojo.attr dojo.create dojo.destroy dojo.getComputedStyle dojo.removeClass dojo.style dojo.toggleClass
+	"dojo/keys", // dojo.keys.ENTER dojo.keys.ESCAPE
 	"dojo/_base/lang", // dojo.delegate dojo.hitch dojo.trim
 	"dojo/_base/sniff" // dojo.isIE
 ], function(dojo, dijit, template){

diff --git a/Menu.js b/Menu.js
@@ -7,6 +7,7 @@ define([
 	"./DropDownMenu",
 	"dojo/_base/array", // dojo.forEach
 	"dojo/_base/connect", // dojo.connect dojo.disconnect dojo.keys.F10
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.attr dojo.byId dojo.getComputedStyle dojo.hasAttr dojo.isDescendant dojo.position dojo.removeAttr
 	"dojo/_base/lang", // dojo.hitch

diff --git a/MenuBar.js b/MenuBar.js
@@ -4,6 +4,7 @@ define([
 	"dojo/text!./templates/MenuBar.html",
 	"./Menu",
 	"dojo/_base/connect", // dojo.keys dojo.keys.DOWN_ARROW
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event" // dojo.stopEvent
 ], function(dojo, dijit, template){
 

diff --git a/ProgressBar.js b/ProgressBar.js
@@ -6,6 +6,7 @@ define([
 	"dojo/number", // dojo.number.format
 	"./_Widget",
 	"./_TemplatedMixin",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html" // dojo.toggleClass
 ], function(require, dojo, dijit, template){
 

diff --git a/TitlePane.js b/TitlePane.js
@@ -8,6 +8,7 @@ define([
 	"./_CssStateMixin",
 	"dojo/_base/array", // dojo.forEach
 	"dojo/_base/connect", // dojo.keys.DOWN_ARROW dojo.keys.ENTER
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html" // dojo.attr dojo.marginBox dojo.removeAttr dojo.replaceClass dojo.setSelectable
 ], function(dojo, dijit, template){

diff --git a/Tree.js b/Tree.js
@@ -18,6 +18,7 @@ define([
 	"dojo/_base/Deferred", // dojo.Deferred
 	"dojo/_base/array", // dojo.filter dojo.forEach dojo.map
 	"dojo/_base/connect", // dojo.isCopyKey dojo.keys dojo.publish
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.addClass dojo.isDescendant dojo.marginBox dojo.removeClass dojo.replaceClass dojo.style dojo.toggleClass
 	"dojo/_base/lang" // dojo.hitch dojo.isString

diff --git a/_Calendar.js b/_Calendar.js
@@ -15,6 +15,5 @@ define([
 	// not satisfy dijit's a11y policy.
 	dijit._Calendar = dijit.Calendar;
 
-
 	return dijit._Calendar;
 });
diff --git a/_CssStateMixin.js b/_CssStateMixin.js
@@ -3,6 +3,7 @@ define([
 	".",
 	"dojo/touch",
 	"dojo/_base/array", // dojo.forEach dojo.map
+	"dojo/_base/declare",	// dojo.declare
 	"dojo/_base/html", // dojo.toggleClass
 	"dojo/_base/lang", // dojo.hitch
 	"dojo/_base/window" // dojo.body

diff --git a/_MenuBase.js b/_MenuBase.js
@@ -7,6 +7,7 @@ define([
 	"./_FocusMixin",
 	"./_KeyNavContainer",
 	"./_TemplatedMixin",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.isDescendant dojo.replaceClass
 	"dojo/_base/lang", // dojo.hitch
 	"dojo/_base/array"	// dojo.indexOf

diff --git a/_PaletteMixin.js b/_PaletteMixin.js
@@ -3,6 +3,7 @@ define([
 	".",
 	"./_CssStateMixin",
 	"dojo/_base/connect", // dojo.keys
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.addClass dojo.attr dojo.create dojo.place dojo.removeClass
 	"dojo/_base/lang", // dojo.hitch

diff --git a/_Widget.js b/_Widget.js
@@ -5,6 +5,7 @@ define([
 	"./_WidgetBase",
 	"./_OnDijitClickMixin",
 	"./_FocusMixin",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/lang", // dojo.hitch
 	"dojo/_base/connect",	// dojo.connect
 	"dojo/uacss",		// brower sniffing

diff --git a/_base.js b/_base.js
@@ -1,5 +1,4 @@
 define([
-	"dojo/_base/kernel",
 	".",
 	"./_base/focus",
 	"./_base/manager",
@@ -10,7 +9,7 @@ define([
 	"./_base/typematic",
 	"./_base/wai",
 	"./_base/window"
-], function(dojo, dijit){
+], function(dijit){
 
 	// module:
 	//		dijit/_base

diff --git a/_base/sniff.js b/_base/sniff.js
@@ -1,12 +1,6 @@
-define([
-	"dojo/_base/kernel",
-	"..",
-	"dojo/uacss"
-], function(dojo, dijit){
+define([ "dojo/uacss" ], function(){
 	// module:
 	//		dijit/_base/sniff
 	// summary:
 	//		Back compatibility module, new code should require dojo/uacss directly instead of this module.
-
-	return dijit;
 });
diff --git a/_base/window.js b/_base/window.js
@@ -1,16 +1,13 @@
 define([
-	"dojo/_base/kernel",
 	"..",
 	"dojo/window" // dojo.window.get
-], function(dojo, dijit){
+], function(dijit, window){
 	// module:
 	//		dijit/_base/window
 	// summary:
 	//		Back compatibility module, new code should use dojo/window directly instead of using this module.
 
 	dijit.getDocumentWindow = function(doc){
-		return dojo.window.get(doc);
+		return window.get(doc);
 	};
-
-	return dijit;
 });
diff --git a/_editor/RichText.js b/_editor/RichText.js
@@ -7,9 +7,10 @@ define([
 	"./range",
 	"./html",
 	"../focus",
-	"dojo/_base/Deferred", // dojo.Deferred
 	"dojo/_base/array", // dojo.forEach dojo.indexOf dojo.some
 	"dojo/_base/connect", // dojo.connect dojo.keys.BACKSPACE dojo.keys.TAB dojo.publish
+	"dojo/_base/declare", // dojo.declare
+	"dojo/_base/Deferred", // dojo.Deferred
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.addClass dojo.attr dojo.byId dojo.contentBox dojo.create dojo.destroy dojo.getComputedStyle dojo.place dojo.position dojo.removeClass dojo.style
 	"dojo/_base/lang", // dojo.clone dojo.hitch dojo.isArray dojo.isFunction dojo.isString dojo.trim

diff --git a/_editor/_Plugin.js b/_editor/_Plugin.js
@@ -5,6 +5,7 @@ define([
 	"../form/Button",
 	"dojo/_base/array", // dojo.forEach
 	"dojo/_base/connect", // dojo.connect dojo.disconnect
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/lang" // dojo.hitch
 ], function(dojo, dijit){
 
@@ -290,4 +291,4 @@ dojo.declare("dijit._editor._Plugin", null, {
 });
 
 return dijit._editor._Plugin;
-});
+});
diff --git a/_editor/plugins/AlwaysShowToolbar.js b/_editor/plugins/AlwaysShowToolbar.js
@@ -4,6 +4,7 @@ define([
 	"../_Plugin",
 	"dojo/_base/array", // dojo.forEach
 	"dojo/_base/connect", // dojo.disconnect
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.addClass dojo.marginBox dojo.place dojo.position dojo.removeClass
 	"dojo/_base/lang", // dojo.hitch
 	"dojo/_base/sniff", // dojo.isIE dojo.isOpera

diff --git a/_editor/plugins/EnterKeyHandling.js b/_editor/plugins/EnterKeyHandling.js
@@ -5,6 +5,7 @@ define([
 	"../_Plugin",
 	"../range",
 	"dojo/_base/connect", // dojo.keys.ENTER
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.destroy dojo.place
 	"dojo/_base/lang", // dojo.hitch

diff --git a/_editor/plugins/FontChoice.js b/_editor/plugins/FontChoice.js
@@ -12,6 +12,7 @@ define([
 	"dojo/i18n", // dojo.i18n.getLocalization
 	"dojo/i18n!../nls/FontChoice",
 	"dojo/_base/array", // dojo.indexOf dojo.map
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.place
 	"dojo/_base/lang", // dojo.delegate dojo.hitch dojo.isString
 	"dojo/_base/window" // dojo.withGlobal

diff --git a/_editor/plugins/FullScreen.js b/_editor/plugins/FullScreen.js
@@ -9,6 +9,7 @@ define([
 	"../../form/ToggleButton",
 	"dojo/i18n!../nls/commands",
 	"dojo/_base/connect", // dojo.connect dojo.disconnect dojo.keys.F11 dojo.keys.TAB
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.addClass dojo.marginBox dojo.removeClass dojo.style
 	"dojo/_base/lang", // dojo.hitch

diff --git a/_editor/plugins/LinkDialog.js b/_editor/plugins/LinkDialog.js
@@ -13,6 +13,7 @@ define([
 	"dojo/i18n!../../nls/common",
 	"dojo/i18n!../nls/LinkDialog",
 	"dojo/_base/connect", // dojo.keys.ENTER
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.attr
 	"dojo/_base/lang", // dojo.delegate dojo.hitch dojo.trim
 	"dojo/_base/sniff", // dojo.isIE

diff --git a/_editor/plugins/NewPage.js b/_editor/plugins/NewPage.js
@@ -5,6 +5,7 @@ define([
 	"../../form/Button",
 	"dojo/i18n", // dojo.i18n.getLocalization
 	"dojo/i18n!../nls/commands",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/lang" // dojo.hitch
 ], function(dojo, dijit){
 

diff --git a/_editor/plugins/Print.js b/_editor/plugins/Print.js
@@ -6,6 +6,7 @@ define([
 	"../../form/Button",
 	"dojo/i18n", // dojo.i18n.getLocalization
 	"dojo/i18n!../nls/commands",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/lang", // dojo.hitch
 	"dojo/_base/sniff" // dojo.isChrome dojo.isOpera
 ], function(dojo, dijit){

diff --git a/_editor/plugins/TextColor.js b/_editor/plugins/TextColor.js
@@ -4,6 +4,8 @@ define([
 	"../_Plugin",
 	"../../form/DropDownButton",
 	"../../ColorPalette",
+	"dojo/_base/connect", // dojo.subscribe
+	"dojo/_base/declare", // dojo.declare
 	"dojo/colors" // dojo.colorFromRgb
 ], function(dojo, dijit){
 

diff --git a/_editor/plugins/ViewSource.js b/_editor/plugins/ViewSource.js
@@ -9,6 +9,7 @@ define([
 	"dojo/i18n!../nls/commands",
 	"dojo/_base/array", // dojo.forEach
 	"dojo/_base/connect", // dojo.connect dojo.disconnect dojo.keys.F12
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.attr dojo.create dojo.marginBox dojo.place dojo.position dojo.style
 	"dojo/_base/lang", // dojo.hitch

diff --git a/_editor/range.js b/_editor/range.js
@@ -2,6 +2,7 @@ define([
 	"dojo/_base/kernel",
 	"..",
 	"dojo/_base/array", // dojo.every
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/lang", // dojo.isArray
 	"dojo/_base/window" // dojo.global
 ], function(dojo, dijit){

diff --git a/_tree/dndSource.js b/_tree/dndSource.js
@@ -14,7 +14,4 @@ define([
 	dojo.getObject("_tree", true, dijit);
 
 	dijit._tree.dndSource = dijit.tree.dndSource;
-
-
-	return dijit._tree.dndSource;
 });
diff --git a/dijit-all.js b/dijit-all.js
@@ -1,5 +1,4 @@
 define([
-	"dojo/_base/kernel",
 	".",
 	"./dijit",
 	"./ColorPalette",
@@ -58,7 +57,7 @@ define([
 	"./layout/SplitContainer",
 	"./layout/StackContainer",
 	"./layout/TabContainer"
-], function(dojo, dijit){
+], function(dijit){
 
 	// module:
 	//		dijit/dijit-all

diff --git a/dijit.js b/dijit.js
@@ -1,5 +1,4 @@
 define([
-	"dojo/_base/kernel",
 	".",
 	"./_base",
 	"dojo/parser",
@@ -9,7 +8,7 @@ define([
 	"./layout/_LayoutWidget",
 	"./form/_FormWidget",
 	"./form/_FormValueWidget"
-], function(dojo, dijit){
+], function(dijit){
 
 	// module:
 	//		dijit/dijit

diff --git a/form/Button.js b/form/Button.js
@@ -7,6 +7,7 @@ define([
 	"./_ButtonMixin",
 	"../_Container",
 	"../_HasDropDown",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.toggleClass
 	"dojo/_base/lang" // dojo.trim
 ], function(dojo, dijit, template, require){

diff --git a/form/ComboButton.js b/form/ComboButton.js
@@ -5,6 +5,7 @@ define([
 	"../focus",		// dijit.focus()
 	"./DropDownButton",
 	"dojo/_base/connect", // dojo.keys
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event" // dojo.stopEvent
 ], function(dojo, dijit, template){
 

diff --git a/form/DropDownButton.js b/form/DropDownButton.js
@@ -7,6 +7,7 @@ define([
 	"../_Container",
 	"../_HasDropDown",
 	"dojo/_base/connect", // dojo.connect dojo.disconnect
+	"dojo/_base/declare", // dojo.declare
 	"dojo/query" // dojo.query
 ], function(dojo, dijit, template, popup){
 

diff --git a/form/HorizontalRule.js b/form/HorizontalRule.js
@@ -1,6 +1,7 @@
 define([
 	"dojo/_base/kernel",
 	"..",
+	"dojo/_base/declare",	// dojo.declare
 	"../_Widget",
 	"../_TemplatedMixin"
 ], function(dojo, dijit){

diff --git a/form/HorizontalRuleLabels.js b/form/HorizontalRuleLabels.js
@@ -2,6 +2,7 @@ define([
 	"dojo/_base/kernel",
 	"..",
 	"./HorizontalRule",
+	"dojo/_base/declare",	// dojo.declare
 	"dojo/number", // dojo.number.format
 	"dojo/query" // dojo.query
 ], function(dojo, dijit){

diff --git a/form/MultiSelect.js b/form/MultiSelect.js
@@ -3,6 +3,7 @@ define([
 	"..",
 	"./_FormValueWidget",
 	"dojo/_base/array", // dojo.indexOf
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.marginBox
 	"dojo/query" // dojo.query
 ], function(dojo, dijit){

diff --git a/form/NumberSpinner.js b/form/NumberSpinner.js
@@ -3,7 +3,8 @@ define([
 	"..",
 	"./_Spinner",
 	"./NumberTextBox",
-	"dojo/_base/connect", // dojo.keys.END dojo.keys.HOME
+	"dojo/keys", // dojo.keys.END dojo.keys.HOME
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event" // dojo.stopEvent
 ], function(dojo, dijit){
 

diff --git a/form/Select.js b/form/Select.js
@@ -10,6 +10,7 @@ define([
 	"../Tooltip",
 	"dojo/i18n!./nls/validate",
 	"dojo/_base/array", // dojo.forEach
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html", // dojo.addClass dojo.attr dojo.create dojo.marginBox dojo.removeClass dojo.toggleClass
 	"dojo/_base/lang", // dojo.hitch

diff --git a/form/SimpleTextarea.js b/form/SimpleTextarea.js
@@ -2,6 +2,7 @@ define([
 	"dojo/_base/kernel",
 	"..",
 	"./TextBox",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.addClass
 	"dojo/_base/sniff", // dojo.isIE dojo.isOpera
 	"dojo/_base/window" // dojo.doc.selection dojo.doc.selection.createRange

diff --git a/form/Slider.js b/form/Slider.js
@@ -16,6 +16,4 @@ define([
 	//		For back-compat, remove for 2.0
 
 	dojo.deprecated("Call require() for HorizontalSlider / VerticalRule, explicitly rather than 'dijit.form.Slider' itself", "", "2.0");
-
-	return dijit.form.HorizontalSlider;
 });
diff --git a/form/Textarea.js b/form/Textarea.js
@@ -3,6 +3,7 @@ define([
 	"..",
 	"./SimpleTextarea",
 	"./_ExpandingTextAreaMixin",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html" // dojo.style
 ], function(dojo, dijit){
 

diff --git a/form/_ButtonMixin.js b/form/_ButtonMixin.js
@@ -1,6 +1,7 @@
 define([
 	"dojo/_base/kernel",
 	"..",
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/event", // dojo.stopEvent
 	"dojo/_base/html" // dojo.setSelectable
 ], function(dojo, dijit){

diff --git a/form/_ComboBoxMenuMixin.js b/form/_ComboBoxMenuMixin.js
@@ -2,6 +2,7 @@ define([
 	"dojo/_base/kernel",
 	"..",
 	"dojo/_base/array", // dojo.forEach
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.attr
 	"dojo/_base/window", // dojo.doc.createTextNode
 	"dojo/i18n" // dojo.i18n.getLocalization

diff --git a/form/_FormSelectWidget.js b/form/_FormSelectWidget.js
@@ -5,6 +5,7 @@ define([
 	"dojo/data/util/sorter", // dojo.data.util.sorter.createSortFunction
 	"dojo/_base/array", // dojo.filter dojo.forEach dojo.map dojo.some
 	"dojo/_base/connect", // dojo.connect dojo.disconnect
+	"dojo/_base/declare", // dojo.declare
 	"dojo/_base/html", // dojo.setSelectable dojo.toggleClass
 	"dojo/_base/lang", // dojo.delegate dojo.isArray dojo.isObject
 	"dojo/data/api/Identity", // dojo.data.api.Identity