]> git.koha-community.org Git - koha.git/commit
Bug 14385: (QA follow-up) Additional changes and fixes
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 12 Oct 2018 05:54:24 +0000 (07:54 +0200)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 2 Nov 2018 10:33:12 +0000 (10:33 +0000)
commit70651422a7517338a68e22321d9918757746ef33
treec16a6de444280add6ef49e01c9a5bb2b7ee91660
parente1b5fa657de843a177fb4bf57947a1376152d021
Bug 14385: (QA follow-up) Additional changes and fixes

[1] searchResults: second my $interface can be removed: unused
[2] call of getitemtypeimagelocation on L2119 needs interface key
[3] ISBDdetail: No need to find patron again (line 182 vs 84)
[4] opac-search: No need to find patron twice (657 and 631)
[5] tabs on line 2220 of C4/Search.pm (qa tools warn)
[6] Ugly hack to overcome "Undefined subroutine &C4::Items::ModZebra"
    by loading C4::Items before C4::Biblio when running tests
    Koha/BiblioUtils/Iterator.t and Labels/t_Label.t.
    This is a more general problem that needs attention somewhere else.
    It seems that Biblio.pm is one of the suspects.
[7] This patch set makes Search.t crash/fail with me. Note that without
    these patches Search.t still passed! Why o why..
    A little debugging pointed me to a missing MPL branch (aarg).
    Adding the simple test on the result of Libraries->find in
    C4::Biblio::GetAuthorisedValueDesc made the test continue.
[8] Resolve: Variable "$borcat" is not available at opac-detail.pl line 246
    Lexical $borcat cannot be used in sub searchAgain in opac-detail.pl
    under Plack. Must be defined with our (or passed as argument).
[9] Resolve crash on TWO serious typos in opac-basket on ONE line:
        Koha::Patron->find({ borrowernumber -> $borrowernumber })
    Yeah: find is in Koha::Patrons and we need => !!
    No need to pass a hash to find method btw for a pk value.
[10] Serious bugfixing here. Add List::Util to opac-basket.
    Can't locate object method "none" via package "1".
    You can't test everything :)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
After this longer list I renamed Final to Additional in the patch title :)

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/Biblio.pm
C4/Search.pm
opac/opac-ISBDdetail.pl
opac/opac-basket.pl
opac/opac-detail.pl
opac/opac-search.pl
t/db_dependent/Koha/BiblioUtils/Iterator.t
t/db_dependent/Labels/t_Label.t