September 2017 - Koha-bugs - lists.koha-community.org

[Bug 19280] New: CanBookBeIssued must take a Koha::Patron in parameter
by bugzilla-daemon＠bugs.koha-community.org 28 Feb '18

28 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19280 Bug ID: 19280 Summary: CanBookBeIssued must take a Koha::Patron in parameter Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org We need to make subroutine from C4 use more Koha::Object objects Seeing bug 19276, starting here is a good start. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 24

[Bug 18975] New: Wrong CSRF token when emailing cart contents
by bugzilla-daemon＠bugs.koha-community.org 26 Feb '18

26 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18975 Bug ID: 18975 Summary: Wrong CSRF token when emailing cart contents Change sponsored?: --- Product: Koha Version: 16.11 Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: bc(a)interleaf.ie QA Contact: testopia(a)bugs.koha-community.org When you are not logged into the OPAC and attempt to email the contents of your cart you get the following error : "Wrong CSRF token at /usr/share/koha/opac/cgi-bin/opac/opac-sendbasket.pl line 55" Some testing shows: If you log in before adding items to your cart then send it, it works. If you're not logged in and you log in as part of sending the basket, the error occurs It must be that authenticating in that window doesn't set the session and id / secret aren't being set before submitting If you aren't logged in, click send the basket and log in, refresh the page before sending, then send, it forces it to pick up the session and works -- You are receiving this mail because: You are watching all bug changes.

1 15

[Bug 17608] New: Fix behaviour of bulkmarcimport.pl -insert
by bugzilla-daemon＠bugs.koha-community.org 26 Feb '18

26 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17608 Bug ID: 17608 Summary: Fix behaviour of bulkmarcimport.pl -insert Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Command-line Utilities Assignee: gmcharlt(a)gmail.com Reporter: magnus(a)libriotech.no QA Contact: testopia(a)bugs.koha-community.org CC: robin(a)catalyst.net.nz According to the POD in bulkmarcimport.pl: -insert if set, only insert when possible This works as expected for bibliographic records, if -insert is specified and a records already exists (based on the criteria specified by the -match option), the record is skipped and no record is inserted. But I do find the behaviour related to items a bit surprising. Even if the record is skipped and not inserted, any items attached to the record in the file that is being imported is attached to the matching record. I would expect items to be ignored if the record exists and is skipped. So the question is: is this behaviour a bug or a feature? Even if it is a bug, I guess we must assume that some libraries out there are relying on the current behaviour of bulkmarcimport.pl, so that the best way to get the behaviour I expect is to add another command line switch, to ignore items for matching records. This way existing users will not get nasty surprises and I will get the behaviour I expect. Opinions? -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 17383] New: Build debian package failed
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '18

23 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17383 Bug ID: 17383 Summary: Build debian package failed Change sponsored?: --- Product: Koha Version: 16.05 Hardware: All OS: Linux Status: NEW Severity: enhancement Priority: P5 - low Component: Packaging Assignee: koha-bugs(a)lists.koha-community.org Reporter: martin.stastny(a)teamlibrary.cz QA Contact: testopia(a)bugs.koha-community.org CC: mirko(a)abunchofthings.net When trying to build a Debian package from git with build-git-snapshot, dependencies fail. Unable to resolve dependencies! Giving up... The following NEW packages will be installed: ca-certificates{a} libalgorithm-checkdigits-perl{a} libanyevent-http-perl{a} libanyevent-perl{a} libarchive-zip-perl{a} libauthen-cas-client-perl{a} libbiblio-endnotestyle-perl{a} libbusiness-isbn-data-perl{a} libbusiness-isbn-perl{a} libcache-fastmmap-perl{a} libcache-memcached-fast-perl{a} libcache-perl{a} libcgi-session-driver-memcached-perl{a} libcgi-session-perl{a} libcgi-session-serialize-yaml-perl{a} libclass-accessor-perl{a} libclass-factory-util-perl{a} libcommon-sense-perl{a} libencode-locale-perl{a} libfile-listing-perl{a} libfile-nfslock-perl{a} libheap-perl{a} libhtml-parser-perl{a} libhtml-tagset-perl{a} libhtml-tree-perl{a} libhttp-cookies-perl{a} libhttp-date-perl{a} libhttp-message-perl{a} libhttp-negotiate-perl{a} libio-socket-ssl-perl{a} libio-string-perl{a} liblwp-mediatypes-perl{a} liblwp-protocol-https-perl{a} libnet-http-perl{a} libnet-ssleay-perl{a} libssl1.0.0{a} libsub-name-perl{a} liburi-perl{a} libwww-perl{a} libwww-robotrules-perl{a} libxml-libxml-perl{a} libxml-namespacesupport-perl{a} libxml-sax-base-perl{a} libxml-sax-perl{a} libxml2{a} libyaml-syck-perl{a} netbase{a} openssl{a} ucf{a} 0 packages upgraded, 49 newly installed, 0 to remove and 0 not upgraded. Need to get 6299 kB of archives. After unpacking 16.0 MB will be used. Abort. E: pbuilder-satisfydepends failed. I: Copying back the cached apt archive contents I: unmounting dev/pts filesystem I: unmounting run/shm filesystem I: unmounting proc filesystem I: cleaning the build env I: removing directory /var/cache/pbuilder/build//24915 and its subdirectories Command 'pdebuild --buildresult /root/debian' returns an error (256) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 10

[Bug 17250] New: Koha::AuthorisedValues - Remove GetAuthValCode
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '18

22 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17250 Bug ID: 17250 Summary: Koha::AuthorisedValues - Remove GetAuthValCode Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: jonathan.druart(a)bugs.koha-community.org Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org -- You are receiving this mail because: You are watching all bug changes.

1 30

[Bug 15879] New: Allow multiple plugin directories to be defined in koha-conf.xml
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '18

22 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15879 Bug ID: 15879 Summary: Allow multiple plugin directories to be defined in koha-conf.xml Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: kyle(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org It would be very useful to be able to define multiple plugin directories in the Koha conf file. This would allow for ease of plugin development so that each plugin installed can live in its own git repository. For compatibility, the first plugindir instance defined should be the one used for uploading plugins via the web interface. -- You are receiving this mail because: You are watching all bug changes.

1 21

[Bug 19018] New: Can we make XSLT_Handler a little bit less verbose ?
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '18

22 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19018 Bug ID: 19018 Summary: Can we make XSLT_Handler a little bit less verbose ? Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: m.de.rooy(a)rijksmuseum.nl QA Contact: testopia(a)bugs.koha-community.org As reported on bug 17807, XSLT_Handler reports that a XSLT file is not found for each record it wants to process (running oai.pl). Additionally, we also have: Can't locate object method "nodeType" via package "<?xml version="1.0" encoding="UTF-8"?> [ And a complete MARC record dump. ] " (perhaps you forgot to load "<?xml version="1.0" encoding="UTF-8"?> [ Same MARC record dump again ] "?) at /usr/share/perl5/HTTP/OAI/Encapsulation.pm line 68. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 3

[Bug 18198] New: MARC21: Further improve handling of 5XX$u in GetMarcNotes
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '18

16 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18198 Bug ID: 18198 Summary: MARC21: Further improve handling of 5XX$u in GetMarcNotes Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: MARC Bibliographic data support Assignee: gmcharlt(a)gmail.com Reporter: m.de.rooy(a)rijksmuseum.nl QA Contact: testopia(a)bugs.koha-community.org This actually is a follow-up of bug 14306. That report only touched field 555 in MARC21. But a lot of other 5XX fields have a $u subfield for URIs. This patch generalizes the change made for 555 and extends it to all 5XX $u. -- You are receiving this mail because: You are watching all bug changes.

1 16

[Bug 16365] New: Selectively introduce GetMarcStructure() "unsafe" variant for better performance
by bugzilla-daemon＠bugs.koha-community.org 13 Feb '18

13 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16365 Bug ID: 16365 Summary: Selectively introduce GetMarcStructure() "unsafe" variant for better performance Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: abl(a)biblos.pk.edu.pl QA Contact: testopia(a)bugs.koha-community.org GetMarcStructure() currently uses Koha::Cache in the "safe" mode (returning deep copy of the result data structure by default), which causes numerous performance issues in many Koha scripts. Switching it to the "unsafe" mode globally (2nd patch from Bug 16140) resolves those issues, but ensuring that it is regression-free (and that it will stay that way in the future) is far from easy. This patch proposes a bit more manageable solution, it introduces a possibility to use "unsafe" variant selectively (only in those places in the code where GetMarcStructure() is called repetively). That way, amount of the code that needs to be audited for possible problems gets vastly reduced, without any performance trade-offs. -- You are receiving this mail because: You are watching all bug changes.

1 29

[Bug 19243] New: Selenium test for testing the administration module functionality
by bugzilla-daemon＠bugs.koha-community.org 13 Feb '18

13 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19243 Bug ID: 19243 Summary: Selenium test for testing the administration module functionality Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: alexbuckley(a)catalyst.net.nz Reporter: alexbuckley(a)catalyst.net.nz QA Contact: testopia(a)bugs.koha-community.org The selenium test for this bug report will test the following Administrative module functionality: * Add item type * Add circulation rule * Modify frameworks * Add library * Add authorised values * Add patron category * Add patron attribute type * Add Z39.50 target server * Add item circulation alert * Add city -- You are receiving this mail because: You are watching all bug changes.

1 59