Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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/jquery.ui.dialog.patch.js
Comments
Comment #1
rballou CreditAttribution: rballou commentedJSHint passes as-is.
Comment #2
nod_Nice :) can you update the summary of the clean-up issue with links to all the new views issues?
Comment #3
rballou CreditAttribution: rballou commentedI have added the issues so far. I'm going to try to get the remaining JS in there soon too.