[Koha-bugs] [Bug 12127] New: Incorrect DataTable types in Holds to Pull list

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Apr 23 07:48:05 CEST 2014


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

            Bug ID: 12127
           Summary: Incorrect DataTable types in Holds to Pull list
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P5 - low
         Component: Templates
          Assignee: oleonard at myacpl.org
          Reporter: dcook at prosentient.com.au
        QA Contact: testopia at bugs.koha-community.org

At the moment, it appears that the branch filtering isn't working correctly in
the Holds to Pull list. That is, it is being too restrictive rather than too
fuzzy (bug 11502). It will only show rows that have a single branch when
filtering. It won't show rows that have multiple available branches.

I would swear that this wasn't happening earlier in the year. I repeatedly
tested the patch on bug 11502 and it went through the QA process without a
problem.

I would also swear that the DataTables JS hasn't changed in the past few
months. I would swear this because I'm using the exact same code on my machine
but the filtering isn't working correctly. The only thing that seems to
remotely make sense to me is perhaps a change in the browsers' internal JS
processing, but I have no idea really...

In any case, the immediate cause appears to be that DataTables is typing the
column data as "html" rather than "string", so it is stripping out the <br>
tags and creating one long string of branchcodes on which to search.  

This is a problem after the implementation of bug 11502 which uses the angle
brackets from the <br> tags as a separator in regex to make the filtering more
precise.

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


More information about the Koha-bugs mailing list