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

[Bug 14697] Extend and enhance "Claims returned" lost status
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14697 George Williams (NEKLS) <george(a)nekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |george(a)nekls.org -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12412] Add ability for plugins to convert arbitrary files to MARC from record staging tool
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12412 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle.m.hall(a)gmail.com --- Comment #18 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- *** Bug 9964 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9964] Add plugins management link to the Administration page
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9964 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|Failed QA |RESOLVED --- Comment #5 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Julian Maurice from comment #4) > I believe those changes already have been pushed as part of bug 12412. > Can we close this one ? Confirmed (In reply to Jonathan Druart from comment #3) > The link is displayed even if C4::Context->config("enable_plugins") is false. This is certainly still true *** This bug has been marked as a duplicate of bug 12412 *** -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15339] TestBuilder build parameter warnings
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15339 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart(a)bugs.koha-c | |ommunity.org --- Comment #7 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Patches have been rebased and adapt to the new code. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15339] TestBuilder build parameter warnings
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15339 --- Comment #6 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 66894 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66894&action=ed… Bug 15339: Improve parameter validation Only value and source are allowed Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15339] TestBuilder build parameter warnings
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15339 --- Comment #5 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 66893 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66893&action=ed… Bug 15339: TestBuilder warnings (tests) Makes TestBuilder::build() alert the user when unreognized parameters are passed, which happens when the user supplies the column values directly, forgetting the 'value' hash. This patch contains the tests that doubles as a demonstration of the kind of error the patch is intended to prevent. Sponsored-By: Halland County Library Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15339] TestBuilder build parameter warnings
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15339 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #45527|0 |1 is obsolete| | Attachment #45528|0 |1 is obsolete| | --- Comment #4 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 66892 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66892&action=ed… Bug 15339: TestBuilder warnings (code) Makes TestBuilder::build() alert the user when unreognized parameters are passed, which happens when the user supplies the column values directly, forgetting the 'value' hash. This patch holds the code changes. Examples of the kind of errors that it catches are in the tests (separate patch). Sponsored-By: Halland County Library Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15339] TestBuilder build parameter warnings
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

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

1 0

[Bug 11580] If returnBeforeExpiry is on, holidays are not taken into account to calculate the due date
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580 --- Comment #24 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Alex Arnaud from comment #21) > Created attachment 66886 [details] [review] > Bug 11580 - Fix expiry calculation and rewrite tests (In reply to Jonathan Druart from comment #23) > Created attachment 66888 [details] [review] > Bug 11580: Add one more test and clear the cache > > Make sure we are not just returning J-1 and clear the cache before and > after the tests. These last 2 patches have been pushed to master, thanks Alex! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12691] Use Koha.Preference for calls to SCOUserJS, SCOUserCSS, OPACUserCSS, opacuserjs, etc in Self-Checkout
by bugzilla-daemon＠bugs.koha-community.org 06 Sep '17

06 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12691 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |In Discussion --- Comment #24 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- What are we doing here? If it is just replacing syspref fetch with Koha.Preference, the following is wrong: - [% IF ( display_patron_image ) %] + [% IF ( Koha.Preference('ShowPatronImageInWebBasedSelfCheck') ) %] display_patron_image is set when $borrower is set AND the pref is set. We had too many regressions in this area, I will not take any risks. If we are doing something else, it is not written in the commit messages. Please clarify. -- You are receiving this mail because: You are watching all bug changes.

1 0