[Koha-bugs] [Bug 15850] New: Correct JSHint errors in cataloging.js

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15850

            Bug ID: 15850
           Summary: Correct JSHint errors in cataloging.js
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: minor
          Priority: P5 - low
         Component: Cataloging
          Assignee: gmcharlt at gmail.com
          Reporter: oleonard at myacpl.org
        QA Contact: testopia at bugs.koha-community.org
                CC: koha-bugs at lists.koha-community.org,
                    m.de.rooy at rijksmuseum.nl,
                    testopia at bugs.koha-community.org

Following coding guideline JS8: "Follow guidelines set by JSHint," some errors
should be corrected in cataloging.js

koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 43, col 31, The array
literal notation [] is preferable.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 49, col 72, Use '!==' to
compare with ''.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 99, col 13, 'i' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 116, col 17, 'id_input'
is already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 182, col 37, 'olddiv' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 183, col 37,
'oldcontrol' is already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 108, col 18, 'j' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 108, col 26, 'j' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 108, col 46, 'j' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 109, col 27, 'j' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 109, col 59, 'j' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 110, col 28, 'j' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 220, col 21, 'inputs' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 226, col 25, 'anchors'
is already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 227, col 30, 'j' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 176, col 21,
'CloneButtonPlus' used out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 280, col 13, 'i' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 280, col 17, 'len' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 287, col 13, 'i' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 287, col 17, 'len' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 296, col 13,
'oldcontrol' is already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 315, col 21, 'i' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 409, col 15, 'line' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 425, col 34, 'line' used
out of scope.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 465, col 13, 'i' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 465, col 17, 'len' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 471, col 13, 'i' is
already defined.
koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js: line 471, col 17, 'len' is
already defined.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are watching all bug changes.