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

[Bug 18275] opac-memberentry.pl security vulnerabilities
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

1 0

[Bug 9988] Leave larger authority merges to merge_authorities cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 --- Comment #46 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- (In reply to Jacek Ablewicz from comment #42) > fails (I'm guessing it will fail, for the postponed "real" merge ?), > $MARCfrom will be ignored, no matter what (it's not used anywhere else in > the merge() subroutine) and @record_from will be empty. Good catch. There still is a "subtle" bug. Although I send the MARCfrom record and it is only used for possibly clearing out a subfield, it is ignored indeed. Thinking about an economic way to repair this.. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9988] Leave larger authority merges to merge_authorities cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |BLOCKED --- Comment #45 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Blocking temporarily -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9988] Leave larger authority merges to merge_authorities cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 --- Comment #44 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- (In reply to Jacek Ablewicz from comment #42) > (In reply to Marcel de Rooy from comment #41) > If the search > > my $authfrom = Koha::Authorities->find($mergefrom); > > fails (I'm guessing it will fail, for the postponed "real" merge ?), > $MARCfrom will be ignored, no matter what (it's not used anywhere else in > the merge() subroutine) and @record_from will be empty. Hmm Some doubts starts to arise. Will dig a little bit further.. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6979] LDAP authentication fails during password comparison
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6979 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|koha-bugs(a)lists.koha-commun |martin.renvoize@ptfs-europe |ity.org |.com -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 6979] LDAP authentication fails during password comparison
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6979 --- Comment #34 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- Any chance of a quick rebase Alex? I'm attempting to QA and I have a SHA1 missing error here ;) -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 9988] Leave larger authority merges to merge_authorities cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 --- Comment #43 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- I tested this quite extensively. So I am sure it does not fail :) But I welcome any interest in this patch set.. Please look at the merge call in the updated cron job: + merge({ + mergefrom => $req->authid, + MARCfrom => scalar $req->oldmarc, + mergeto => $req->authid_new, + MARCto => $newmarc, + override_limit => 1, + }); MARCfrom is coming from the adjusted need_merge_authorities table. It contains the report tag (in marcxml). For creation of that record, see Koha::Authority::MergeRequest->new. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8010] Search history can be added to the wrong patron
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8010 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |martin.renvoize@ptfs-europe | |.com --- Comment #21 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- All works as expected and passes qa script checks. Looks good to me. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8010] Search history can be added to the wrong patron
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8010 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #61258|0 |1 is obsolete| | --- Comment #20 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- Created attachment 61290 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61290&action=ed… BUG 8010: Removed Auth2.t Signed-off-by: Marc Véron <veron(a)veron.ch> Signed-off-by: Martin Renvoize <martin.renvoize(a)ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8010] Search history can be added to the wrong patron
by bugzilla-daemon＠bugs.koha-community.org 20 Mar '17

20 Mar '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8010 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #61257|0 |1 is obsolete| | --- Comment #19 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- Created attachment 61289 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61289&action=ed… BUG 8010: Correct a few syntax errors, and moved tests - Tests are now in t/db_dependent/Search/History.t - There were 2 differents sysprefs in sysprefs.sql and in atomicupdate => fixed Signed-off-by: Marc Véron <veron(a)veron.ch> Signed-off-by: Martin Renvoize <martin.renvoize(a)ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.

1 0