April 2018 - Koha-bugs - lists.koha-community.org

[Bug 20100] Should a non-superlibrarian be able to add superlibrarian privileges?
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20100 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #72747|0 |1 is obsolete| | Attachment #72749|0 |1 is obsolete| | Attachment #72750|0 |1 is obsolete| | --- Comment #21 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 74618 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74618&action=ed… Bug 20100: Introduce preference ProtectSuperlibPrivs This pref allows you to block changes to superlibrarian privileges at client and server side for non-superlibrarians. This patch only contains the db rev and the preference description. Test plan: Run dbrev or new install. Check pref value. Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> Signed-off-by: JM Broust <jean-manuel.broust(a)univ-lyon2.fr> Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 20100] Should a non-superlibrarian be able to add superlibrarian privileges?
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20100 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia(a)bugs.koha-communit |nick(a)bywatersolutions.com |y.org | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13937] Add a Z39.50 daemon that can inject item status MARC subfields
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13937 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|BLOCKED |Failed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13937] Add a Z39.50 daemon that can inject item status MARC subfields
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13937 --- Comment #43 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- === Details Net::Z3950::SimpleServer Just wondering why we need a new dep. SIPServer uses Net::Server::PreFork ? t/Koha/Z3950responder.t This test fails with me. Please check. not ok 3 - We didn't start server , should fail # Failed test 'We didn't start server , should fail' # at t/Koha/Z3950responder.t line 24. # got: undef # expected: '2' not ok 4 - We didn't start server, should fail because it cannot connect # Failed test 'We didn't start server, should fail because it cannot connect' # at t/Koha/Z3950responder.t line 25. # got: undef # expected: 'Cannot connect to upstream server' not ok 5 - There is no handler as we aren't connected # Failed test 'There is no handler as we aren't connected' # at t/Koha/Z3950responder.t line 27. # got: '13' # expected: '30' not ok 6 - We don't have a handler, should fail because we don't # Failed test 'We don't have a handler, should fail because we don't' # at t/Koha/Z3950responder.t line 28. # got: 'Fetch request out of range' # expected: 'No such resultset' Use of uninitialized value $num_to_prefetch in division (/) at /usr/share/koha/devclone/Koha/Z3950Responder/Session.pm line 158. Illegal division by zero at /usr/share/koha/devclone/Koha/Z3950Responder/Session.pm line 158. t/db_dependent/Koha/Z3950Responder/Session.t t/db_dependent/Koha/Z3950Responder/Session2.t The test coverage only focuses on the add item status routine. We are not testing the server itself. Would it be too hard to try? Commit message stuff from qa tools * Commit title does not contain 'follow-up' correctly spelt - 8e0da10 * Commit title does not start with 'Bug XXXXX: ' - 31c0f7b * Commit title does not start with 'Bug XXXXX: ' - cf4f62b * Commit title does not contain 'follow-up' correctly spelt - cf4f62b * Commit title does not start with 'Bug XXXXX: ' - f592650 * Commit title does not contain 'follow-up' correctly spelt - f592650 * Commit title does not start with 'Bug XXXXX: ' - c24c232 Path::Tiny Why is this dependency added? It is not used somewhere. Log::Log4perl::Appender Might be a new enhancement on itself?? my ($item_tag, $itemnumber_subfield) = GetMarcFromKohaField( "items.itemnumber", '' ); Does not need the default framework parameter anymore. See changed API. Testing misc/z3950_responder.pl Crashes on show 0 Can't call method "raw" on an undefined value at /usr/share/koha/devclone/Koha/Z3950Responder/Session.pm line 169. Target closed connection + warn "name: $opt_name and value: $opt_value"; Debug leftover? Configuration Defaults to port 9999 (yaz default). Note that we may use port 9999 too for Zebra authorities.. Passing a listener-address worked though. Should we add our own default? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13937] Add a Z39.50 daemon that can inject item status MARC subfields
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13937 --- Comment #42 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- === General The idea is interesting! It works but I have the impression that it still needs some fine-tuning and further testing. Crashes on a show 0 for instance. Tests need attention. As an optional feature, it will not harm people not using it. On the other hand, it adds 600 lines of code for an additional item status via a Z39.50 daemon (including overlap with C4/Search Zebra code). We need to maintain that. Food for discussion? The feature does not support showing this new status via SRU/SRW in analogy with Zebra? (Only see YAZ responding with 404 on 9999 now in the browser.) In terms of new development, I wonder if it would not be simpler to provide an item status via REST API, and not invest in Zebra/Z39.50 (not looking at specific sponsor specs here)? Further thoughts ? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12310] Accept different monetary formats as input
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12310 Victor Grousset/tuxayo <victor.grousset(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff --- Comment #76 from Victor Grousset/tuxayo <victor.grousset(a)biblibre.com> --- As it's a sensitive issue, more testing would be better to ensure that there are no regressions. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13937] Add a Z39.50 daemon that can inject item status MARC subfields
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13937 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |BLOCKED CC| |m.de.rooy(a)rijksmuseum.nl --- Comment #41 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- QA: Looking here -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15562] Make Koha more suitable for shared hosting with plack
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 Andreas Jonsson <andreas.jonsson(a)kreablo.se> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andreas.jonsson(a)kreablo.se -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 4850] CheckReserves does not respect holds circulation policies
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4850 Hugh Rundle <hughr(a)brimbank.vic.gov.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hughr(a)brimbank.vic.gov.au --- Comment #9 from Hugh Rundle <hughr(a)brimbank.vic.gov.au> --- I disagree that this isn't a problem. Comment 7 is true, but only if the only thing you care about is item types. Imagine you have a circ rule for borrower category A plus item type Z, and another rule for borrower category B plus item type Z. You want Category A to be allowed to place holds on Z items, and you don't want category B to be able to place holds on Z items. "Default holds policy by item type" is a blanket rule based on the item type only. So it's impossible to apply different hold rules on a combination of item type and borrower category. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 11909] Fix hidelostitems, OpacHiddenItems total count, prog them facet display
by bugzilla-daemon＠bugs.koha-community.org 20 Apr '18

20 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11909 Hugh Rundle <hughr(a)brimbank.vic.gov.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hughr(a)brimbank.vic.gov.au --- Comment #12 from Hugh Rundle <hughr(a)brimbank.vic.gov.au> --- Can confirm this is still a problem: titles where all items are hidden still appear in search results, then when you click on it you get a 404 (of course). If you can't reach the title description it shouldn't appear in results. Anyone on this? -- You are receiving this mail because: You are watching all bug changes.

1 0