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

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

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7441 --- Comment #79 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Jonathan, would you be ok with me testing with a UNIMARC sample database (from the sandboxes I think)? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6934] New report Cash Register Statistics
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6934 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #106 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Please take a look at comment#103 - comment#105 for a hopefully last follow-up. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 13895] Add API routes for checkouts retrieval and renewal
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13895 Jiri Kozlovsky <mail(a)jkozlovsky.cz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #53901|0 |1 is obsolete| | --- Comment #30 from Jiri Kozlovsky <mail(a)jkozlovsky.cz> --- Created attachment 53924 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53924&action=ed… [SIGNED-OFF] Bug 13895: Allow user to access their own checkouts and renew them Let user access their own checkouts and if OpacRenewalAllowed system preference is on, also let user to renew their checkouts. Test plan: 1. Open a browser tab on Koha staff and log in (to create CGISESSID cookie) with a Patron that has no permissions. This Patron will be referred as "your patron" or "your borrowernumber" below. 2. Go to http://yourlibrary/api/v1/checkouts?borrowernumber=XXX (replace XXX with your borrowernumber) and check you receive correct data 3. Go to http://yourlibrary/api/v1/checkouts?borrowernumber=XXX (replace XXX with someone else's borrowernumber) and check you get a permission error. 4. Go to http://yourlibrary/api/v1/checkouts/YYY (replace YYY with an existing checkout id of your Patron) and check you receive correct data 5. Go to http://yourlibrary/api/v1/checkouts/YYY (replace YYY with an existing checkout id of some other Patron) and check you get a permission error. 6. Send PUT request to http://yourlibrary/api/v1/checkouts/YYY, replace YYY with existing checkout id of some other Patron. You should get a permission error. 7. Set system preference OpacRenewalAllowed to 0. 8. Send PUT request to http://yourlibrary/api/v1/checkouts/YYY. YYY should be checkout id of checkout for your patron. You should get a permission error. 9. Set system preference OpacRenewalAllowed to 1. 10. Send PUT requests to http://yourlibrary/api/v1/checkouts/YYY until the maximum number of renewals is reached (you should have a 403 error). YYY should be checkout id of checkout for your patron. 11. Run unit tests in t/db_dependent/api/v1/checkouts.t Signed-off-by: Jiri Kozlovsky <mail(a)jkozlovsky.cz> Wow, works like magic! :) Nice job! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13895] Add API routes for checkouts retrieval and renewal
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13895 Jiri Kozlovsky <mail(a)jkozlovsky.cz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- 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 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13903 Lari Taskula <larit(a)student.uef.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |17032 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17032 [Bug 17032] REST API tests: Make sure Swagger object definition is up-to-date with database -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13799] Add base for building RESTful API
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13799 Lari Taskula <larit(a)student.uef.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |17032 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17032 [Bug 17032] REST API tests: Make sure Swagger object definition is up-to-date with database -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6934] New report Cash Register Statistics
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6934 --- Comment #105 from Owen Leonard <oleonard(a)myacpl.org> --- > But you do get that "active transaction" groups all transactions where > patrons give money to the library, right ? I don't think "active transaction" has any inherent meaning. What about something like "All payments to the library" ? Some other things I notice: - The page is looking for datatables.js in the wrong place. It's now at intranet-tmpl/prog/js/datatables.js. - The table header "Patron cardnumber" should be "Patron card number." -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 6934] New report Cash Register Statistics
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6934 --- Comment #104 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- I think that sounds more logical! Also safer with the option of new types being added - not so likely we change the types for paying and credit. Maybe we could also think about a more 'speaking' name. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 12446] Enable an adult to have a guarantor
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446 --- Comment #38 from Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> --- Created attachment 53913 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53913&action=ed… Bug 12446 - QA Follow-up - Added missing icon on add gurantee - Removed category restriction when adding new guarentee - Put back some deleted entries while fixing merging conflicts. - Fixed some typing errors. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6934] New report Cash Register Statistics
by bugzilla-daemon＠bugs.koha-community.org 02 Aug '16

02 Aug '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6934 --- Comment #103 from Francois Charbonnier <francois.charbonnier(a)inlibro.com> --- (In reply to Katrin Fischer from comment #102) > (In reply to Francois Charbonnier from comment #96) > > (In reply to Katrin Fischer from comment #59) > > > > > - What is the definition of an "Active transaction"? (was not clear to me, > > > so asking :) ) > > > > The idea behind "Active transactions" is to help balance cash register > > transactions. In other words, how much goes in or how much goes out the cash > > register. The active transactions group what patrons paid or gave as credit > > to the library, which means what went IN the register. > > > > The code works this way : it takes all transactions where account types are > > NOT "F" (Fine), "FU" (Fine unpaid), "FOR" (Forgiven), "M" (Sundry), "L" > > (Lost). > > Sorry, I still don't get it :( The list of excluded fine types looks a bit > arbitrary to me - what about custom MANUAL_INV values? And what is left by > excluding those? Hey Katrin, But you do get that "active transaction" groups all transactions where patrons give money to the library, right ? The logic should be different. What do you think of grouping the relevant account types, "Pay" and "C" (are there other?) instead of excluding this list? -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0