Remove JShint references

README.md

@@ -24,6 +24,9 @@ Optional for development a pre-commit hook can be used, which runs JSHint.
 
 # Running static analysis
 
+
+ESLint is used for JavaScript linting.
+
 ```
 make lint
 ```

client/zarafa/Zarafa.js

@@ -720,7 +720,6 @@ Ext.apply(Zarafa, {
 
 			// Process data that was passed as URL data
 			Zarafa.core.URLActionMgr.execute(urlActionData);
-			/*jshint -W051 */
 			delete urlActionData;
 
 			// Start the keepalive to make sure we stay logged into the zarafa-server,

client/zarafa/core/ui/ContextContainer.js

@@ -186,7 +186,6 @@ Zarafa.core.ui.ContextContainer = Ext.extend(Zarafa.core.ui.SwitchViewContentCon
 		// views often.
 		if (this.autoClean === true && oldItem && oldItem != newItem) {
 			this.remove(oldItem);
-			/*jshint -W051 */
 			delete oldItem;
 		}
 		var model = newContext.getModel();

client/zarafa/core/ui/SwitchViewContentContainer.js

@@ -114,7 +114,6 @@ Zarafa.core.ui.SwitchViewContentContainer = Ext.extend(Ext.Container, {
 		// views often.
 		if (this.autoClean === true && oldView && oldView != this.getActiveItem()) {
 			this.remove(oldView);
-			/*jshint -W051 */
 			delete oldView;
 		}
 	},

client/zarafa/hierarchy/data/HierarchyTreeLoader.js

@@ -466,7 +466,6 @@ Zarafa.hierarchy.data.HierarchyTreeLoader = Ext.extend(Ext.tree.TreeLoader, {
 			// for 1 ms. This helps with large hierarchies where the JS would normally
 			// try to load the entire tree in a single thread which might take so long
 			// that the browser will kill it.
-			/*jshint unused:false*/
 			/*eslint no-unused-vars: ["error", { "varsIgnorePattern": "defer" }]*/
 			var defer = function(node, fn) {
 				data = this.getFilteredChildNodes(treeNode.getFolder(), 'folder');

package.json

@@ -7,6 +7,7 @@
 	"scripts": {
 		"phplint": "phpmd server text .phpmd.xml || true",
 		"phplintci": "phpmd server xml .phpmd.xml --reportfile phpmd.xml || true",
+		"lint": "eslint client/zarafa",
 		"jsunit": "karma start test/js/unittest.conf.js"
 	},
 	"pre-commit": {

test/js/unittest/RecipientStore.js

@@ -147,7 +147,6 @@ describe("RecipientStore", function() {
 		var recipientStore;
 
 		beforeEach(function() {
-			/*jshint -W020 */
 			container = new Zarafa.core.Container();
 			ipmStore = new Zarafa.core.data.ListModuleStore();
 			ipmRecord = ipmStore.reader.readRecords(data).records[0];