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

[Bug 19065] New: Transaction library resets itself on Cash register report
by bugzilla-daemon＠bugs.koha-community.org 31 Oct '17

31 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19065 Bug ID: 19065 Summary: Transaction library resets itself on Cash register report Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Reports Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org When working with the cash register report the input for 'transaction library' resets itself with every submit of the form. It should show the value that was selected. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 4

[Bug 15261] Verify if checkouts or reserves requests periods overlap with existing reserves
by bugzilla-daemon＠bugs.koha-community.org 31 Oct '17

31 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15261 Alex Arnaud <alex.arnaud(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68664|0 |1 is obsolete| | --- Comment #51 from Alex Arnaud <alex.arnaud(a)biblibre.com> --- Created attachment 68853 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68853&action=ed… Bug 15261 - Check reserves while renewing an issue Test plan: - Create an issue (i.e from 15/09/2017 to 04/10/2017), - place a reserve on the same item from 05/10/2017 to 30/10/2017), - enable PreventCheckoutOnSameReservePeriod, - Check that you are not able to renew the issue. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15261] Verify if checkouts or reserves requests periods overlap with existing reserves
by bugzilla-daemon＠bugs.koha-community.org 31 Oct '17

31 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15261 Alex Arnaud <alex.arnaud(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68663|0 |1 is obsolete| | --- Comment #50 from Alex Arnaud <alex.arnaud(a)biblibre.com> --- Created attachment 68852 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68852&action=ed… Bug 15261: Verify if checkouts/reserves requests periods overlap... ... with existing reserves When checking out or placing hold, we should check if an existing reserve whose period overlap exists. A user place an hold from opac whose requested period overlap an existing reserve period => prevent reserve, A librarian place an hold from staff whose requested period overlap an existing reserve period => Warn librarian (Ask for confirmation), A librarian make a checkout from staff whose requested period overlap an existing reserve period => Warn librarian (Ask for confirmation). Test plan: Enable syspref: AllowHoldDateInFuture OPACAllowHoldDateInFuture PreventChechoutOnSameReservePeriod and PreventReservesOnSamePeriod 1 (staff side): Place a hold on title (which has only one items) level with start date and expiration date. Place another hold (also title level) with period overlaping this reserve. Check you are warned about an existing reserve 2 (staff side): Place a hold on title (which has more than one items) level with start date and expiration date. Place another hold (also title level) with period overlaping this reserve. Check you are NOT warned about an existing reserve. Because it remains at least one item not reserved. 3 (staff side): Place a hold on item level with start date and expiration date. Place another hold on item level with period overlaping this reserve. Check you are warned about an existing reserve. 4 (opac side): Do the same than for staff side. Instead of a warn, reserve is prevented. 5: Place a hold on title (which has only one items) level with start date and expiration date. Try to checkout the unique item from this title with period overlaping the reserve period. Check you are warned about an existing reserve 6: Place a hold on title (which has more than one items) level with start date and expiration date. Checkout an item from this title with period overlaping the reserve period. Check you are NOT warned about an existing reserve. 7: Place a hold on item level with start date and expiration date. Checkout this item period overlaping the reserve period. Check you are warned about an existing reserve Rabased on master Rebased on master (2016-06-23) Rebased on master (2017-03-23) Signed-off-by: Séverine QUEUNE <severine.queune(a)bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune(a)bulac.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 19082] New: Release notes contain reference to INSTALL files that have been removed
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19082 Bug ID: 19082 Summary: Release notes contain reference to INSTALL files that have been removed Change sponsored?: --- Product: Project Infrastructure Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Release tools Assignee: jonathan.druart(a)bugs.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de The generated release notes still include a text pointing to the INSTALL files removed by bug 17626. This should be removed. OR in the INSTALL files that come in the tarball -- You are receiving this mail because: You are watching all bug changes.

1 1

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 --- Comment #164 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- 1/ +TODO: + +- Anything invoking the ->status method; annotated with: + + # Old use of ->status ! What do it mean? 2/ I will not make you rewrite everything, but it would have been better to stay consistent and not mix snake_case and camelCase for method names. First steps with the GUI: 3/ This is what I see when I enable the module: https://screenshots.firefox.com/z9dNoRL91NQ5F3cD/pro.kohadev.org Looks wrong. 4/ Hit /cgi-bin/koha/ill/ill-requests.pl and got: Loading failed for the <script> with source “http://pro.kohadev.org/intranet-tmpl/prog/en/lib/jquery/plugins/jquery.tablesorter.min.js”. ill-requests.pl:151 Loading failed for the <script> with source “http://pro.kohadev.org/intranet-tmpl/prog/en/lib/jquery/plugins/jquery.checkboxes.min.js”. ill-requests.pl:152 5/ Clicked "New ILL request" button and got: Can't locate Koha/Illbackends//Base.pm -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #163 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Please fix QA script failures. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 2454] Improve display of credits on pay fines tab
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2454 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Failed QA -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 2454] Improve display of credits on pay fines tab
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2454 --- Comment #16 from M. Tompsett <mtompset(a)hotmail.com> --- Comment on attachment 68470 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68470 Bug 2454 - Amended patch: Removed the display of accountype, Notify ID, Level values for manual credit in pay.tt and replaced the description with string "Credit" Review of attachment 68470: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=245…) ----------------------------------------------------------------- ::: koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt @@ -132,5 @@ > - [% CASE 'HE' %]Hold waiting too long > - [% CASE 'Rent' %]Rental fee > - [% CASE 'FOR' %]Forgiven > - [% CASE 'LR' %]Lost item fee refund > - [% CASE 'PF' %]Processing fee This is why you put white space changes in a different file. I believe this line got removed incorrectly. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 --- Comment #44 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- (In reply to Dominic Pichette from comment #43) > Created attachment 68791 [details] [review] > Bug 18282: operationId must be unique > > operationId has the following documentation: > "Unique string used to identify the operation. The id MUST be unique among > all > operations described in the API." > > This patch modifies operationIds to be unique accross our API operations. > > Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> > > Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> > Signed-off-by: Dominic Pichette <dominic(a)inlibro.com> Dominique, I think you submitted a signed patch to the wrong bug report. I obsoleted it and marked the bug back as Needs Signoff to avoid confusion. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 2454] Improve display of credits on pay fines tab
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2454 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62994|0 |1 is obsolete| | --- Comment #15 from M. Tompsett <mtompset(a)hotmail.com> --- Comment on attachment 62994 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62994 [SIGNED OFF] Bug 2454 - Amended patch: Removed the display of accountype, Notify ID, Level values for manual credit in pay.tt and replaced the description with string "Credit" The second one is the rebase of this. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0