April 2018 - Koha-bugs - lists.koha-community.org

[Bug 13958] Add a suspensionsCalendar syspref
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13958 Niamh <niamh.walker-headon(a)it-tallaght.ie> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |niamh.walker-headon@it-tall | |aght.ie -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14293] Error in the calculation of the suspension of users per day
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14293 Niamh <niamh.walker-headon(a)it-tallaght.ie> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |niamh.walker-headon@it-tall | |aght.ie -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12123] HTML notices can break the notice viewer
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12123 --- Comment #29 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Jonathan Druart from comment #27) > Still working. > > SO is from 2014, Marcel did not confirm it recreates the issues. > > Switch back to NSO. Demonstration of bug with and without patch: https://www.youtube.com/watch?v=IAr-ht3b5kw -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12310] Accept different monetary formats as input
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12310 Victor Grousset/tuxayo <victor.grousset(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |In Discussion --- Comment #68 from Victor Grousset/tuxayo <victor.grousset(a)biblibre.com> --- (In reply to M. Tompsett from comment #67) > Using DecimalSeparator as ',.', we should be able to enter #,###,###.## and > get #######.## back. There is a contradiction between DecimalSeparator=',.' and wishing to input #,###,###.## and get #######.## back. The current aimed functional behavior is to only accept thousands separators when the DecimalSeparator syspref does not contain a thousands separator in it. Because having a character that depending on the context can be either a decimal sep or a thousands sep could easily create regressions in forgotten edge cases. And it will silently accept user mistakes leading to erroneous values being processed. (e.g. input 5,123.12 instead of 5.123.12 or 5,123,12 when wanting to input 512312) We might want to try to be even more clever and flexible. Actually, handling your case could be a nice addition. But if we do, can it be in a follow up bug? ^^" -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 17816] New: Useless call to GetBibioData in bulkmarcimport.pl(?)
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17816 Bug ID: 17816 Summary: Useless call to GetBibioData in bulkmarcimport.pl(?) Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: gmcharlt(a)gmail.com Reporter: glasklas(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: robin(a)catalyst.net.nz This call to GetBibioData seems without purpose: https://github.com/Koha-Community/Koha/blob/2fe2ef41bd8bf2f142597e39f493edb… $biblioitemnumber is either overwritten on the lines below, or update/create failed/was skipped and the later AddItemBatchFromMarc will fail anyways. -- You are receiving this mail because: You are watching all bug changes.

1 2

[Bug 8000] Test mode for notices
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8000 --- Comment #138 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- (In reply to Jonathan Druart from comment #137) > And what about NoticeBcc? > > Failing QA. Sorry, can you explain the problem with NoticeBCC? Should we file a new bug about the problem in SendAlerts? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12123] HTML notices can break the notice viewer
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12123 --- Comment #28 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- (In reply to Jonathan Druart from comment #27) > Still working. > > SO is from 2014, Marcel did not confirm it recreates the issues. Will have another look at this report tomorrow. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14769] Authorities merge: Set correct indicators in biblio field
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14769 --- Comment #78 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 74052 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74052&action=ed… Bug 14769: (QA follow-up) Remove global var $cached_indicators As requested by RM, this patch replaces using the global $cached_indicators by saving state temporarily during the (limited) lifetime of the object. Essentially this affects two places in code: [1] blinddetail-biblio-search.pl (loading auth record in editor) [2] AuthoritiesMarc::merge (merging authority into biblios) Concurrent runs of [1] and/or [2] together with a simultaneous pref change just in between could cause slight (hypothetical) side-effects. The current approach of keeping state in the object makes that a series of controlled_indicators calls during an immediate merge of one specific authority is not affected by a simultaneous pref change. So the same rules are applied to the set of attached biblio record for that authority. Note also that the cron job ignores a simultaneous pref change, since it reads from the unchanged L1 cache (yes, also hypothetical). Test plan: [1] Run t/Koha/Authority/ControlledIndicators.t [2] Run t/db_dependent/Authority/Merge.t [3] Run t/db_dependent/Koha/Authorities.t Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14769] Authorities merge: Set correct indicators in biblio field
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14769 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Passed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11879] Add a new field to patron record: main contact method
by bugzilla-daemon＠bugs.koha-community.org 12 Apr '18

12 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11879 --- Comment #116 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Have the same conflict. -- You are receiving this mail because: You are watching all bug changes.

1 0