mediawiki-extensions-WikiEd.../.jshintrc

{
	/* Common */

	// Enforcing
	"camelcase": true,
	"curly": true,
	"eqeqeq": true,
	"immed": true,
	"latedef": true,
	"newcap": true,
	"noarg": true,
	"noempty": true,
	"nonew": true,
	"quotmark": "single",
	"trailing": true,
	"undef": true,
	"unused": true,
	// Legacy
	"onevar": true,

	/* Local */

	// Relaxing
	"loopfunc": true,
	"multistr": true,
	// Environment
	"browser": true,

	"globals": {
		"mediaWiki": false,
		"jQuery": false,
		"QUnit": false
	}
}
WikiEditor: Linting and conventions * Code clean up in preparation for enabling linting in the future * Update code to use latest code conventions and best practices: - Make use of jQuery.Event (e.g. no need to check both e.keyCode and e.which) - jQuery: .size() -> .length - jQuery: (where appropiate) .attr() -> .prop() Setting properties like 'checked' via attr() has been deprecated in jQuery. - Whitespace - Single quotes instead of double quotes - Use literal keys in object literals instead of strings - Pass mediaWiki to closure, use mw. locally instead of "mediaWiki" global directly. - Fix indentation - Brackets around if, else and for bodies - Strict comparison to 0, null, false, true etc. - Fix missing radix parameter in parseInt - Use local $ instead of global $ - Use `foo \|\| bar` instead of `foo ? foo : bar` - Variable scope hoisting - Double/redundant variable declarations - ['foo'] is better written in dot notation - New line at EOF - Consistency in jQuery construction: Tag name for element creation $( '<div>' ) Valid html for html parsing $( '<div foo="bar"></div>' ) - Fix regex escape warnings per JSLint/JSHint. Do escape ][, don't escape >< - .. * Add .jshintrc / .jshintignore * Updated most files, but not all. Too much at once. Change-Id: I445639b25a9688b3cdf9e5449e3d31cbcfa9c7ae 2012-07-03 05:52:27 +00:00			`{`
Kranitor: Make JSHint pass eqeqeq: * Change loose comparisons to strict comparisons where it seems safe to use a strict comparision instead. Mostly comparisons to strings or objects, and comparisons to numbers where the other value is known to be a number, too. E.g. foo == 'string', bar == node, indexOf() != -1. * Add eqeqeq:false to files where there are non-obvious usages left. onevar, quotmark: * Disabled in files with lots of style violations. unused: * Remove unused variables that have no side-effects in their assigned expression. Coding style cleanups on affected lines where trivial. Change-Id: I5db155a632740e24cb52dba2177c7fc35d5aebd5 2013-11-07 00:38:43 +00:00			`/* Common */`

			`// Enforcing`
			`"camelcase": true,`
			`"curly": true,`
			`"eqeqeq": true,`
			`"immed": true,`
			`"latedef": true,`
			`"newcap": true,`
			`"noarg": true,`
			`"noempty": true,`
			`"nonew": true,`
			`"quotmark": "single",`
			`"trailing": true,`
			`"undef": true,`
			`"unused": true,`
			`// Legacy`
			`"onevar": true,`

			`/* Local */`

			`// Relaxing`
			`"loopfunc": true,`
			`"multistr": true,`
			`// Environment`
			`"browser": true,`

Add QUnit test for ext.wikiEditor.toolbar This patch replaces the manually operated test module for ext.wikiEditor.toolbar with one based on QUnit. The new test suite uncovered a bug with the removeFromToolbar API function in jquery.wikiEditor.toolbar, which prevented the removal of select buttons from the toolbar. This issue has also been fixed in this commit. .jshintrc was updated to ignore the new QUnit global. Bug: T39485 Change-Id: Icef3debcffa484a8d78628bcd9da0892b750bb40 2014-09-01 14:07:28 +00:00			`"globals": {`
			`"mediaWiki": false,`
			`"jQuery": false,`
			`"QUnit": false`
			`}`
WikiEditor: Linting and conventions * Code clean up in preparation for enabling linting in the future * Update code to use latest code conventions and best practices: - Make use of jQuery.Event (e.g. no need to check both e.keyCode and e.which) - jQuery: .size() -> .length - jQuery: (where appropiate) .attr() -> .prop() Setting properties like 'checked' via attr() has been deprecated in jQuery. - Whitespace - Single quotes instead of double quotes - Use literal keys in object literals instead of strings - Pass mediaWiki to closure, use mw. locally instead of "mediaWiki" global directly. - Fix indentation - Brackets around if, else and for bodies - Strict comparison to 0, null, false, true etc. - Fix missing radix parameter in parseInt - Use local $ instead of global $ - Use `foo \|\| bar` instead of `foo ? foo : bar` - Variable scope hoisting - Double/redundant variable declarations - ['foo'] is better written in dot notation - New line at EOF - Consistency in jQuery construction: Tag name for element creation $( '<div>' ) Valid html for html parsing $( '<div foo="bar"></div>' ) - Fix regex escape warnings per JSLint/JSHint. Do escape ][, don't escape >< - .. * Add .jshintrc / .jshintignore * Updated most files, but not all. Too much at once. Change-Id: I445639b25a9688b3cdf9e5449e3d31cbcfa9c7ae 2012-07-03 05:52:27 +00:00			`}`