JSHint modules/views/js/base.js [#1835076]

Follow-up of #1664940: [Policy, patch] Decide on JSHint configuration and part of #1415788: Javascript winter clean-up

Run jshint on the files with the configuration from the parent issue or use jshint.com with the following options:

/*jshint forin:true, noarg:true, eqeqeq:true, undef:true, curly:true, browser:true, expr:true, latedef:true, newcap:true, trailing:true */
/*global drupal, jQuery */

Fix any warnings or errors the tool finds.
Check manually that the fixes did not break any functionality.
Create patch and upload for the testbot.

Files: modules/views/js/base.js

Comment	File	Size	Author
#6	core-js-jshint-views-base-drupal.patch	488 bytes	rballou
#6
#2	core-js-jshint-views-base.patch	2.26 KB	rballou
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rballou CreditAttribution: rballou commented 8 November 2012 at 03:13

Initial pass for jshint:

base.js: line 9, col 3, 'Drupal' is not defined.
base.js: line 14, col 3, 'Drupal' is not defined.
base.js: line 46, col 3, 'Drupal' is not defined.
base.js: line 68, col 3, 'Drupal' is not defined.
base.js: line 70, col 16, 'Drupal' is not defined.
base.js: line 83, col 3, 'Drupal' is not defined.
base.js: line 96, col 3, 'Drupal' is not defined.
base.js: line 97, col 12, 'Drupal' is not defined.
base.js: line 98, col 27, 'Drupal' is not defined.
base.js: line 15, col 30, 'context' is defined but never used.

10 errors