Commons:User scripts/users namespace/reports/User:Tgr/catinsert.js
Jump to navigation
Jump to search
This is the report page for User:Tgr/catinsert.js. It is bot-maintained. All manual changes might be overwritten.
jshint
[edit]- ISSUE:
line 31 character 7
: document.write can be a form of eval. - Evidence:document.write('<link rel="stylesheet" type="text/css" href="' + ciStyleSheet + '">');
- ISSUE:
line 34 character 4
: 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 ciGetDialog() {
- ISSUE:
line 50 character 11
: Missing semicolon. - Evidence:}
- ISSUE:
line 52 character 68
: Use '===' to compare with 'true'. - Evidence:if (typeof(wikEdUseWikEd) != 'undefined' && wikEdUseWikEd == true) {
- ISSUE:
line 55 character 21
: 'box' is already defined. - Evidence:var box = document.getElementById('wpTextbox1');
- ISSUE:
line 57 character 10
: 'box' used out of scope. - Evidence:box.parentNode.insertBefore(ciFrame, box);
- ISSUE:
line 57 character 47
: 'box' used out of scope. - Evidence:box.parentNode.insertBefore(ciFrame, box);
- ISSUE:
line 89 character 4
: 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 ciCloseDialog() {
- ISSUE:
line 91 character 65
: Use '===' to compare with 'true'. - Evidence:if (typeof(wikEdUseWikEd) != 'undefined' && wikEdUseWikEd == true) {
- ISSUE:
line 98 character 4
: 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 ciNextDiv(e) {
- ISSUE:
line 108 character 4
: 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 ciExpandNode(cat, mode, lnk) {
- ISSUE:
line 114 character 65
: Missing semicolon. - Evidence:lnk.onclick= function() { ciCollapseNode(cat, mode, lnk) }
- ISSUE:
line 121 character 4
: 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 ciCollapseNode(cat, mode, lnk) {
- ISSUE:
line 127 character 63
: Missing semicolon. - Evidence:lnk.onclick= function() { ciExpandNode(cat, mode, lnk) }
- ISSUE:
line 130 character 4
: 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 ciLoadNode(cat, mode, lnk, div) {
- ISSUE:
line 135 character 66
: Missing semicolon. - Evidence:lnk.onclick= function() { ciCollapseNode(cat, mode, lnk); }
- ISSUE:
line 140 character 4
: 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 ciLoadChildren(cat, mode, div) {
- ISSUE:
line 147 character 24
: Use '===' to compare with 'false'. - Evidence:if(ciError == false) { // show default list on error
- ISSUE:
line 156 character 22
: Use '===' to compare with ''. - Evidence:if ( result == '' ) {
- ISSUE:
line 159 character 23
: Use '===' to compare with '0'. - Evidence:if ( mode == 0 ) result= categoryTreeNoSubcategoriesMsg;
- ISSUE:
line 176 character 4
: 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 ciInsertCategory(cat) {
- ISSUE:
line 178 character 65
: Use '===' to compare with 'true'. - Evidence:if (typeof(wikEdUseWikEd) != 'undefined' && wikEdUseWikEd == true) {
- ISSUE:
line 183 character 35
: Use '===' to compare with 'true'. - Evidence:if (wikEdHighlightSyntax == true) {
- ISSUE:
line 193 character 35
: Use '===' to compare with 'true'. - Evidence:if (wikEdHighlightSyntax == true) {
- ISSUE:
line 201 character 66
: Missing semicolon. - Evidence:var winScroll = document.documentElement.scrollTop
- ISSUE:
line 203 character 30
: 'winScroll' is already defined. - Evidence:var winScroll = document.body.scrollTop;
- ISSUE:
line 207 character 53
: 'winScroll' used out of scope. - Evidence:document.documentElement.scrollTop = winScroll;
- ISSUE:
line 209 character 42
: 'winScroll' used out of scope. - Evidence:document.body.scrollTop = winScroll;
- ISSUE:
line 224 character 4
: 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 ciButtonClicked() {
- ISSUE:
line 228 character 18
: 'e' is already defined. - Evidence:else var e = event;
- ISSUE:
line 229 character 18
: 'e' used out of scope. - Evidence:if (typeof(e.shiftKey) != "undefined") var shift = e.shiftKey;
- ISSUE:
line 229 character 58
: 'e' used out of scope. - Evidence:if (typeof(e.shiftKey) != "undefined") var shift = e.shiftKey;
- ISSUE:
line 230 character 22
: 'shift' is already defined. - Evidence:else var shift = e.modifiers & Event.SHIFT_MASK;
- ISSUE:
line 230 character 24
: 'e' used out of scope. - Evidence:else var shift = e.modifiers & Event.SHIFT_MASK;
- ISSUE:
line 231 character 11
: 'shift' used out of scope. - Evidence:if (shift) { // set default starting category
- ISSUE:
line 242 character 65
: Use '===' to compare with 'true'. - Evidence:if (typeof(wikEdUseWikEd) != 'undefined' && wikEdUseWikEd == true) {
- ISSUE:
line 256 character 60
: Use '!==' to compare with '0'. - Evidence:while(ciTextRange.text.length == l && count != 0) {
- ISSUE:
line 262 character 27
: 'selection' is already defined. - Evidence:var selection = ciTextRange.text;
- ISSUE:
line 264 character 27
: 'selection' is already defined. - Evidence:var selection = textarea.value.substring(textarea.selectionStart, textarea.selectionEnd);
- ISSUE:
line 268 character 11
: 'selection' used out of scope. - Evidence:if(!selection) { // use ciStartingCategory as default
- ISSUE:
line 269 character 10
: 'selection' used out of scope. - Evidence:selection = ciStartingCategory;
- ISSUE:
line 271 character 22
: 'selection' used out of scope. - Evidence:var match = selection.match(/(\s+)$/);
- ISSUE:
line 273 character 13
: 'selection' used out of scope. - Evidence:selection = selection.substring(0, selection.length - (match[1].length));
- ISSUE:
line 273 character 25
: 'selection' used out of scope. - Evidence:selection = selection.substring(0, selection.length - (match[1].length));
- ISSUE:
line 273 character 48
: 'selection' used out of scope. - Evidence:selection = selection.substring(0, selection.length - (match[1].length));
- ISSUE:
line 276 character 18
: 'selection' used out of scope. - Evidence:match = selection.match(/\[\[.*(\|.*)]]$/);
- ISSUE:
line 280 character 10
: 'selection' used out of scope. - Evidence:selection = selection.replace(/^\[\[[Cc]ategory:/, '');
- ISSUE:
line 280 character 22
: 'selection' used out of scope. - Evidence:selection = selection.replace(/^\[\[[Cc]ategory:/, '');
- ISSUE:
line 281 character 10
: 'selection' used out of scope. - Evidence:selection = selection.replace(/(\|.*)?]]$/, '');
- ISSUE:
line 281 character 22
: 'selection' used out of scope. - Evidence:selection = selection.replace(/(\|.*)?]]$/, '');
- ISSUE:
line 281 character 22
: Too many errors. (92% scanned). - Evidence:undefined