[Koha-bugs] [Bug 10541] New: Enable cross-browser AJAX in additem.js

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

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10541

            Bug ID: 10541
           Summary: Enable cross-browser AJAX in additem.js
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: major
          Priority: P5 - low
         Component: Templates
          Assignee: oleonard at myacpl.org
          Reporter: dcook at prosentient.com.au

Currently, the additem.js is using "indexOf" to search for a value in an array.
While this works in Chrome, Firefox, and IE > 9, it fails miserably in IE 8 and
7 (which don't have the indexOf method). This means that users aren't able to
add items using the acquisitions module!

Instead of using "indexOf", we should be using the jQuery function $.inArray.
It was added in jQuery v1.2 (3.8.0 uses v1.3.2 so even our oldest supported
release can use this method). It's perfectly cross-browser compatible...works
in Chrome, Firefox, and every version of IE that I've tried (i.e. 7, 8, 9).

Patch and test plan to follow...

-- 
You are receiving this mail because:
You are watching all bug changes.