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 functionalities
Create patch and upload for the testbot.
Files: contextual/contextual.js
Comment | File | Size | Author |
---|---|---|---|
#7 | core-jshint-contextual-1684834-7.patch | 612 bytes | nod_ |
#4 | contextual.jshint.patch | 1.33 KB | droplet |
Comments
Comment #1
droplet CreditAttribution: droplet commentedNo errors
Comment #2
nod_Comment #3
nod_core/modules/contextual/contextual.js: line 106, col 24, Did you mean to return a conditional instead of an assignment?
Comment #4
droplet CreditAttribution: droplet commentedComment #5
nod_The contextual refactor fixed it.
Comment #6
nod_And broke it again. New JSHint config #1995996: Update JSHint configuration.
Comment #7
nod_this one makes me kind of sad actually.
Comment #8
droplet CreditAttribution: droplet commentedOuch!!
Comment #9
alexpottCommitted f01622c and pushed to 8.x. Thanks!