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

[Bug 14930] Leaving OpacFineNoRenewals blank blocks renewals, but should disable feature
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14930 --- Comment #8 from Marc Véron <veron(a)veron.ch> --- (In reply to Kyle M Hall from comment #7) > (In reply to Jonathan Druart from comment #6) > > Should not we take 0 into account? > > I agree. Allowing 0 would mean a patron must have no fines in order to renew > items. I think allowing 0 makes sense. It would also be good to note in the > system preference description that leaving it empty will disable the feature > ( i.e. patrons will always be able to renew items, no matter how much they > owe in fines ). ...and maybe the wording should reflect the >0 and 0 cases (with and without displaying the limit), and in a translatable way (sentence not split by tags). -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 0

[Bug 15288] Error pages: Code duplication removal and better translatability
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15288 --- Comment #40 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Please fix t_db_dependent_Auth_t.The file errors/500.tt should be accessible (contains integers) (734) http://jenkins.koha-community.org/job/Koha_Master_D8/lastBuild/testReport/j… -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15577] Link in OPAC doesn't redirect anywhere
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15577 Julian Maurice <julian.maurice(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice(a)biblibre.com Status|Pushed to Master |Pushed to Stable --- Comment #9 from Julian Maurice <julian.maurice(a)biblibre.com> --- Patch pushed to 3.22.x, will be in 3.22.3 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15703] New: Time::Piece is only used in Review.t and can be replaced
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15703 Bug ID: 15703 Summary: Time::Piece is only used in Review.t and can be replaced 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 1

[Bug 14930] Leaving OpacFineNoRenewals blank blocks renewals, but should disable feature
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14930 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |kyle(a)bywatersolutions.com --- Comment #7 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Jonathan Druart from comment #6) > Should not we take 0 into account? I agree. Allowing 0 would mean a patron must have no fines in order to renew items. I think allowing 0 makes sense. It would also be good to note in the system preference description that leaving it empty will disable the feature ( i.e. patrons will always be able to renew items, no matter how much they owe in fines ). -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 15355] Translatability: Fix issues on check in page
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15355 Julian Maurice <julian.maurice(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice(a)biblibre.com Status|Pushed to Master |Pushed to Stable --- Comment #6 from Julian Maurice <julian.maurice(a)biblibre.com> --- Patch pushed to 3.22.x, will be in 3.22.3 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 4912] After editing private list, user should be redirect to private lists
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4912 Julian Maurice <julian.maurice(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |julian.maurice(a)biblibre.com --- Comment #31 from Julian Maurice <julian.maurice(a)biblibre.com> --- Patch pushed to 3.22.x, will be in 3.22.3 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15599] Unit tests output warnings
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15599 --- Comment #9 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to M. Tompsett from comment #8) > (In reply to Jonathan Druart from comment #7) > > Kyle, the warnings appear for many other tests, see > > t/db_dependent/Reserves.t for instance. > > I don't think this is a right way to fix it. > > Perhaps not overall, however, taking "string $foo->bar" and turning it to > "string $foobar" is a start. Catching warnings is a good test. Perhaps > "$debug and" prepended to the {warn mess} would be fitting, and still allow > for the testing added? I'm open to whatever solution may be proposed. This is starting to go out of scope. The bug is for just two unit tests. I think the $debug and warn idea sounds good! We could have it set to 1 by default and set it to 0 in our unit tests. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15576] Link in OPAC redirects to the wrong page
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15576 Julian Maurice <julian.maurice(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |julian.maurice(a)biblibre.com --- Comment #5 from Julian Maurice <julian.maurice(a)biblibre.com> --- Patch pushed to 3.22.x, will be in 3.22.3 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14598] itemtype is not set on statistics by C4::Circulation::AddReturn
by bugzilla-daemon＠bugs.koha-community.org 29 Jan '16

29 Jan '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14598 --- Comment #67 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Missed your last comment, sorry! We usually use deleteditems for statistics too - so it makes sense to include it I think. -- You are receiving this mail because: You are watching all bug changes.

1 0