[Koha-bugs] [Bug 23887] Add support for client-specified sort order in Z3950Responder

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

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

David Nind <david at davidnind.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #13 from David Nind <david at davidnind.com> ---
The patch still applies, however the sorting doesn't work for me.

Testing notes using koha-testing-docker:

- Getting the Z3950Responder to work in koha-testing-docker:
  . the Z3950Responder is setup and running by default (but you can't accessing
using your browser)
  . commands:
    ~ koha-z3950-responder --stop kohadev
    ~ koha-z3950-responder --start kohadev
    ~ a restart_all will also reload it
  . it is running on port 2100, however this is not accessible using the
browser
  . to enable access using the browser:
    ~ edit your docker-compose.yml file in koha-testing-docker
    ~ add port 2100 in the koha services section, for example:
        ports:
            - "8080:8080"
            - "8081:8081"
            - "2100:2100" <== this was added
    ~ close koha-testing-docker down and restart (Ctrl-C and kd, then ku)

- Alternative: use the queries with curl from kshell and output to a file to
make it easier to review the results: curl "queries from the next step" >
filename

- I changed the query to search for perl, as "misth" is not in the sample data:
  .
http://localhost:2100/biblios?version=1.2&operation=searchRetrieve&query=perl
sortby title/ascending&maximumRecords=10&recordSchema=marcxml"
  .
http://localhost:2100/biblios?version=1.2&operation=searchRetrieve&query=perl
sortby title/descending&maximumRecords=10&recordSchema=marcxml"

- For me, the order of the results didn't change, and they appeared in this
order before and after the patch was applied, and when using ascending and
descending:

Perl best practices
CGI programming with Perl
Programming the Perl DBI
Perl Hacks
Intermediate Perl
Effective Perl programming
The definitive guide to Catalyst
Higher-order Perl
Perl Better Practices
Perl template toolkit

- The tests passed before and after the patch was applied

- Tip: Firefox returns raw XML, I used Google Chrome so that the page was more
readable

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