Page MenuHomePhabricator

Configure the no-jquery/no-global-selector rule in CheckUser by setting allowIds to true and remove exceptions
Closed, ResolvedPublic

Description

Problem
In several JavaScript files in CheckUser the no-jquery/no-global-selector rule is being completely disabled.

Proposed Solution
Change the configuration of the rule by setting allowIds to true then the rule can be used properly on all of the JavaScript files in CheckUser.

Event Timeline

dbarratt created this task.Jul 15 2020, 8:49 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 15 2020, 8:49 PM

Change 612951 had a related patch set uploaded (by Dbarratt; owner: Dbarratt):
[mediawiki/extensions/CheckUser@master] Configure the no-jquery/no-global-selector rule & remove exceptions

https://gerrit.wikimedia.org/r/612951

Just realised that this is scoped to CheckUser. Apologies for the spam...

Tchanders renamed this task from Configure the no-jquery/no-global-selector rule by setting allowIds to true and remove exceptions to Configure the no-jquery/no-global-selector rule in CheckUser by setting allowIds to true and remove exceptions.Jul 15 2020, 11:25 PM

Change 612951 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Configure the no-jquery/no-global-selector rule & remove exceptions

https://gerrit.wikimedia.org/r/612951

Tchanders closed this task as Resolved.Jul 17 2020, 10:46 PM
Tchanders claimed this task.