June 2016 - Koha-bugs - lists.koha-community.org

[Bug 15023] Allow patron anonymize/bulk delete tool to be limited by branch
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15023 --- Comment #2 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 52900 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52900&action=ed… Bug 15023: (followup) Remove warnings By setting 'use Modern::Perl' some previously hidden warnings arised. This patch removes them by testing the variable $balance for undef before using it in a comparisson. Sponsored-by: VOKAL -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15023] Allow patron anonymize/bulk delete tool to be limited by branch
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15023 --- Comment #1 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 52899 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52899&action=ed… Bug 15023: Allow patron anonymize/bulk delete tool to be limited by branch This patch makes the bulk patron delete/anonymize functionality be limited by branch. It does so by adding a branch selection dropdown and using the already defined APIs for filtering by branch. It makes use of C4::Branches::onlymine for the IndependentBranches use case and it adds a way to call it from the Branches template plugin. To test: - Apply the patch 1) Have a superlibrarian user - Go to Tools > Batch patron deletion/anonymization => SUCCESS: Verify you can pick a branch (or all of them) - Try doing some operations => SUCCESS: Verify the selection is respected, and carried around all steps 2) Have a user with tools/delete_anonymize_patrons permissions - Set IndependentBranches on - Go to Tools > Batch patron deletion/anonymization => SUCCESS: It picks the librarian's branch and doesn't let us choose another one - Try doing some operations => SUCCESS: Verify the user's branch is respected, and carried around all steps - Sign off :-D Sponsored-by: VOKAL -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13796] Alert in OPAC when renewing an item with a rental charge
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13796 --- Comment #5 from Sally Healey <sally.healey(a)cheshiresharedservices.gov.uk> --- Hi Marc, The language revision of: NOTE: A charge of 5.00 for Books will be added - sounds good. I've just tested again using the sandbox at: http://sandbox1.ptfs-europe.co.uk/ with the MARC21 database. I'm using a DVD with an item type of "DVD Film" and a collection code of "Film media" - but it returns 'fee for Books', rather than 'fee for DVD Film' as I would expect: Black snake moan [dvd] 02/07/2016 39999000016611 Renew Fee for Books 1.00 (5 of 5 renewals remaining) I'm not sure what's causing the discrepancy; when you test with an item other than a book, does it return the correct item type? Thanks for your work on this - it'd be great to get this into Koha. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12599] IndependentBranches Holes
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12599 --- Comment #8 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Owen Leonard from comment #7) > (In reply to M. Tompsett from comment #6) > > (In reply to Kyle M Hall from comment #5) > > > Indeed, ... . It's on my todo list ; ) > > > > TODO lists get lost. How do we get that on a "Doing It Now" list? :) > > Oh that's easy: You put up some time or money. I'd be happy to write up an RFP if anyone is interested in getting started on this! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 0

[Bug 14238] C4::Matcher is mangling data to match on
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14238 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|3.18 |master -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14238] C4::Matcher is mangling data to match on
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14238 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #39352|0 |1 is obsolete| | --- Comment #4 from Kyle M Hall <kyle(a)bywatersolutions.com> --- Created attachment 52897 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52897&action=ed… Bug 14238 - Unit Test to demonstrate issue -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 1088] expand independant branches functionallity
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=1088 --- Comment #5 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Katrin Fischer from comment #4) > I think this bug is too unspecific and could only be an omnibus bug of some > kind. I am adding Kyle as he has been working in this area recently, maybe > there is some inspiration to be found, but closing it in favor of newer bug > reports. Agreed! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13903] Add API routes to list, create, update, delete holds
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13903 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |16815 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16815 [Bug 16815] API routes to list, create, update, delete holds need permissions -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 3534] Patron quick add form
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3534 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #47 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7441] Search results showing wrong branch
by bugzilla-daemon＠bugs.koha-community.org 27 Jun '16

27 Jun '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7441 --- Comment #77 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Jonathan Druart from comment #76) > I have extracted the UNIMARC XSLT changes from the patch (as it was before > the last patchset). > So if I understand everything correctly, we still need a signoff from an > UNIMARC user and the changes to the NORMARC XSLT, that's it? Yes, it would be good to have a unimarc s/o. > I am wondering if it would not be easier to use something like > items.$pref_value instead of the xsl:choose blocks which copy/paste code. I'm not sure what you mean, but if you want to submit a followup to make that change I'd be happy to test and sign off on it! -- You are receiving this mail because: You are watching all bug changes.

1 0