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

[Bug 15196] Order patrons on patron lists by name
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15196 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14582] OPAC detail shows an unuseful link to "add tag" when user is not logged in
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14582 Owen Leonard <oleonard(a)myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7677] New area in subscriptions and new function when receiving
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7677 --- Comment #80 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Katrin Fischer from comment #79) > FAIL valid > Name "Cache::RemovalStrategy::FIELDS" used only once: possible typo > Name "Cache::RemovalStrategy::LRU::FIELDS" used only once: possible typo > Name "Tie::Hash::FIELDS" used only once: possible typo See bug 16104. -- 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 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 --- Comment #20 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Srdjan, did you have a look at bug 16044? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13464] Standardize the pagination class
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13464 --- Comment #1 from Owen Leonard <oleonard(a)myacpl.org> --- Created attachment 49385 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49385&action=ed… Bug 13464 - Standardize the pagination class There are a few template which use a different wrapper class for the pagination links. These should be made consistent for a consistent style. To test, apply the patch and go to Administration -> Budgets. - On this page the pagination markup has been removed in favor of DataTables pagination. The budget tables were already sorted using DataTables, so having server-side pagination on top of that doesn't make sense. In Administration -> Class sources: - This page had pagination markup which was unused. In Authorities -> Authority search results: - Perform an authority search which will return more than one page of results. Confirm that the pagination links are correctly styled. In Search -> Item search: - Perform a search which will return more than one page of results. Confirm that the pagination links are correctly styled. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13464] Standardize the pagination class
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13464 Owen Leonard <oleonard(a)myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

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

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13903 Arve Søreide <arve.soreide(a)kul.oslo.kommune.no> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |arve.soreide(a)kul.oslo.kommu | |ne.no -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7677] New area in subscriptions and new function when receiving
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7677 --- Comment #79 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Just starting with this - the QA tools have some complaints. Could you please take a look? FAIL Koha/Serial/Item.pm OK critic OK forbidden patterns OK pod OK spelling 0 FAIL valid Name "Tie::Hash::FIELDS" used only once: possible typo Name "Cache::RemovalStrategy::LRU::FIELDS" used only once: possible typo Name "Cache::RemovalStrategy::FIELDS" used only once: possible typo FAIL Koha/Serial/Items.pm OK critic OK forbidden patterns OK pod OK spelling 0 FAIL valid Name "Cache::RemovalStrategy::FIELDS" used only once: possible typo Name "Cache::RemovalStrategy::LRU::FIELDS" used only once: possible typo Name "Tie::Hash::FIELDS" used only once: possible typo -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14784] Missing checkin message for debarred patrons when issuing rules 'fine days = 0'
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49283|0 |1 is obsolete| | --- Comment #10 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Created attachment 49377 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49377&action=ed… Bug 14784 : Fix checkin message for debarred patrons If there is an overdue but finedays equals 0, then we have to see if the patron is debarred and if he was forever debarred. So we have to go through the same code a if it was a debarred patron with no overdue. Test plan : 1)Apply patch 2)Select an existing patron that doesn't have an oustanding fine 3)Check out a book for this patron 4)Click the "Restriction" tab 5)Click "Add manual restriction" 6)Write any comment and leave expiration date blank 7)Validate text on patron's checkout page: "Cannot check out! Restricted: Patron's account is restricted with the explanation:" 8)Check in the book that you checked out at step 2 9)Validate the warning message: "Check in message 013880: Heart health for Canadians : Reminder: Patron has an indefinite restriction." Then you can redo these steps and change the date at step 6 to see different messages at step 8. Signed-off-by: barton <barton(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14784] Missing checkin message for debarred patrons when issuing rules 'fine days = 0'
by bugzilla-daemon＠bugs.koha-community.org 21 Mar '16

21 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer(a)bsz-bw.de --- Comment #9 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Hi Genevieve, some things to look out for next time: 1) Please don't use tabs, but 4 spaces in your patches. The QA tools can help you to find such problems, instructions on how to install them are on the wiki: FAIL C4/Circulation.pm FAIL forbidden patterns forbidden pattern: tab char (line 2100) forbidden pattern: tab char (line 2092) forbidden pattern: tab char (line 2098) forbidden pattern: tab char (line 2097) forbidden pattern: tab char (line 2094) 2) Please add line breaks to your patch descriptions as that makes them easier to read, see here: https://wiki.koha-community.org/wiki/Commit_messages 3) Please include the test plan in the commit message next time. I will attach an amended patch in a moment :) -- You are receiving this mail because: You are watching all bug changes.

1 0