JSHint modules/views/js/ajax-views.js [#1835074]

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/ajax_view.js

Comment	File	Size	Author
#7	core-js-jshint-views-ajax-view-drupal.patch	1.22 KB	rballou
#7
#1	core-js-jshint-views-ajax-view.patch	4.89 KB	rballou
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rballou CreditAttribution: rballou commented 8 November 2012 at 02:50

File	Size
core-js-jshint-views-ajax-view.patch	4.89 KB

Original set of errors:

ajax_view.js: line 43, col 21, 'queryString' is already defined.
ajax_view.js: line 12, col 1, 'Drupal' is not defined.
ajax_view.js: line 13, col 1, 'Drupal' is not defined.
ajax_view.js: line 14, col 7, 'Drupal' is not defined.
ajax_view.js: line 14, col 26, 'Drupal' is not defined.
ajax_view.js: line 14, col 51, 'Drupal' is not defined.
ajax_view.js: line 15, col 12, 'Drupal' is not defined.
ajax_view.js: line 16, col 7, 'Drupal' is not defined.
ajax_view.js: line 16, col 39, 'Drupal' is not defined.
ajax_view.js: line 21, col 1, 'Drupal' is not defined.
ajax_view.js: line 22, col 1, 'Drupal' is not defined.
ajax_view.js: line 27, col 1, 'Drupal' is not defined.
ajax_view.js: line 32, col 19, 'Drupal' is not defined.
ajax_view.js: line 73, col 1, 'Drupal' is not defined.
ajax_view.js: line 77, col 30, 'Drupal' is not defined.
ajax_view.js: line 80, col 1, 'Drupal' is not defined.
ajax_view.js: line 89, col 1, 'Drupal' is not defined.
ajax_view.js: line 97, col 1, 'Drupal' is not defined.
ajax_view.js: line 106, col 5, 'Drupal' is not defined.
ajax_view.js: line 108, col 5, 'Drupal' is not defined.
ajax_view.js: line 113, col 22, 'Drupal' is not defined.
ajax_view.js: line 116, col 24, 'Drupal' is not defined.
ajax_view.js: line 119, col 1, 'Drupal' is not defined.
ajax_view.js: line 119, col 81, 'status' is defined but never used.

24 errors

The attached patch cleans up all of the issues listed.