Commons:User scripts/users namespace/reports/User:Galdrad/monobook.js
Jump to navigation
Jump to search
This is the report page for User:Galdrad/monobook.js. It is bot-maintained. All manual changes might be overwritten.
jshint
[edit]- ISSUE:
line 2 character 1
: document.write can be a form of eval. - Evidence:document.write('<script type="text/javascript" src="'
- ISSUE:
line 17 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function query_string_get(paramName) {
- ISSUE:
line 21 character 22
: Expected a conditional expression and instead saw an assignment. - Evidence:if (m=cmdRe.exec(h)) {
- ISSUE:
line 27 character 2
: Unnecessary semicolon. - Evidence:};
- ISSUE:
line 30 character 68
: Script URL. - Evidence:mw.util.addPortletLink('p-cactions', "javascript:prepare_for_autodel()", "s", 'ca-autodellinks', 'Alter all the links on this page to autodel the page when clicked');
- ISSUE:
line 33 character 32
: Use '!==' to compare with 'null'. - Evidence:if(query_string_get('autodel') != null && query_string_get('action') == 'delete') {
- ISSUE:
line 53 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function prepare_for_autodel() {
- ISSUE:
line 55 character 12
: Use '===' to compare with 'null'. - Evidence:if(tag == null) { alert('Action annulée.'); return; }
- ISSUE:
line 56 character 12
: Use '===' to compare with ''. - Evidence:if(tag == "") tag = autodel.default_summary;
- ISSUE:
line 71 character 28
: Use '===' to compare with ''. - Evidence:if(linkpage[1] == '') {
- ISSUE:
line 83 character 69
: Don't make functions within a loop. - Evidence:nli.onmouseup = function() { this.style.color = 'red'; };
- ISSUE:
line 89 character 66
: Don't make functions within a loop. - Evidence:link.onmouseup = function() { this.style.color = 'red'; };
- ISSUE:
line 93 character 40
: 'i' is already defined. - Evidence:if(autodel.delete_talks) for(var i = 0; i < newlinks.length; i++) {
- ISSUE:
line 103 character 53
: Script URL. - Evidence:btn.href = 'javascript:unprepare_for_autodel();';
- ISSUE:
line 108 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function unprepare_for_autodel() {
- ISSUE:
line 112 character 51
: Don't make functions within a loop. - Evidence:link.onclick = function() { return true; };
- ISSUE:
line 113 character 53
: Don't make functions within a loop. - Evidence:link.onmouseup = function() { return true; };
- ISSUE:
line 123 character 51
: Script URL. - Evidence:btn.href = 'javascript:prepare_for_autodel();';
- ISSUE:
line 203 character 33
: Bad or unnecessary escaping. - Evidence:str = str.replace(/ \./g, "\.");
- ISSUE:
line 206 character 39
: Bad or unnecessary escaping. - Evidence:str = str.replace(/\.\.\.\.+/g, "\.\.\.");
- ISSUE:
line 206 character 41
: Bad or unnecessary escaping. - Evidence:str = str.replace(/\.\.\.\.+/g, "\.\.\.");
- ISSUE:
line 206 character 43
: Bad or unnecessary escaping. - Evidence:str = str.replace(/\.\.\.\.+/g, "\.\.\.");
- ISSUE:
line 207 character 41
: Bad or unnecessary escaping. - Evidence:str = str.replace(/etc\.\.+/g, "etc\.");
- ISSUE:
line 241 character 41
: Bad or unnecessary escaping. - Evidence:str = str.replace(/\[\[(.*?) —/g, "\[\[$1 -");
- ISSUE:
line 241 character 43
: Bad or unnecessary escaping. - Evidence:str = str.replace(/\[\[(.*?) —/g, "\[\[$1 -");
- ISSUE:
line 242 character 41
: Bad or unnecessary escaping. - Evidence:str = str.replace(/\|( *)—( *)/g, "\|");
- ISSUE:
line 259 character 24
: Use '===' to compare with '0'. - Evidence:if (b.indexOf(a) == 0 || b.indexOf(TurnFirstToLower(a)) == 0)
- ISSUE:
line 259 character 63
: Use '===' to compare with '0'. - Evidence:if (b.indexOf(a) == 0 || b.indexOf(TurnFirstToLower(a)) == 0)
- ISSUE:
line 280 character 13
: Use '!==' to compare with ''. - Evidence:if (input != "")
- ISSUE:
line 282 character 15
: 'input' is already defined. - Evidence:var input = trim(input);
- ISSUE:
line 292 character 55
: Script URL. - Evidence:mw.util.addPortletLink('p-personal', 'javascript:format()', 'typo', 'ca-format', 'Corrections de la mise en page et de la typographie', '', document.getElementById('ca-edit'));
- ISSUE:
line 301 character 1
: document.write can be a form of eval. - Evidence:document.write('<script type="text/javascript" src="'
- ISSUE:
line 306 character 1
: document.write can be a form of eval. - Evidence:document.write('<script type="text/javascript" src="'
- ISSUE:
line 346 character 1
: document.write can be a form of eval. - Evidence:document.write('<script type="text/javascript" src="'
- ISSUE:
line 579 character 13
: Use '===' to compare with ''. - Evidence:if (token == '' || token != hex_md5(url + vandal + document.cookie)) {
- ISSUE:
line 595 character 11
: Missing semicolon. - Evidence:return
- ISSUE:
line 605 character 11
: Use '===' to compare with '0'. - Evidence:if (i == 0 && name != vandal) {
- ISSUE:
line 614 character 14
: Use '===' to compare with ''. - Evidence:if (editor == '') {
- ISSUE:
line 630 character 11
: Missing semicolon. - Evidence:return
- ISSUE:
line 682 character 29
: Missing semicolon. - Evidence:var t = l[i].innerHTML
- ISSUE:
line 684 character 29
: Use '!==' to compare with ''. - Evidence:if (n >= 0 && article != '' && t.indexOf('oldid=') == -1) {
- ISSUE:
line 700 character 13
: 't' is already defined. - Evidence:var t = l[i].innerHTML
- ISSUE:
line 700 character 29
: Missing semicolon. - Evidence:var t = l[i].innerHTML
- ISSUE:
line 728 character 1
: document.write can be a form of eval. - Evidence:document.write('<script type="text/javascript" src="'
- ISSUE:
line 733 character 1
: document.write can be a form of eval. - Evidence:document.write('<script type="text/javascript" src="'