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

[Bug 6979] LDAP authentication fails during password comparison
by bugzilla-daemon＠bugs.koha-community.org 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6979 Alex Arnaud <alex.arnaud(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff --- Comment #29 from Alex Arnaud <alex.arnaud(a)biblibre.com> --- Hello Alex, Patches are rebased on master. You can test again. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 6979] LDAP authentication fails during password comparison
by bugzilla-daemon＠bugs.koha-community.org 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6979 Alex Arnaud <alex.arnaud(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57483|0 |1 is obsolete| | --- Comment #28 from Alex Arnaud <alex.arnaud(a)biblibre.com> --- Created attachment 57966 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57966&action=ed… Bug 6979 - Fix Already in a transaction error -- 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 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6979 Alex Arnaud <alex.arnaud(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57482|0 |1 is obsolete| | --- Comment #27 from Alex Arnaud <alex.arnaud(a)biblibre.com> --- Created attachment 57965 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57965&action=ed… Bug 6979 - Handle multiple branches in non-auth_by_bin -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 6979] LDAP authentication fails during password comparison
by bugzilla-daemon＠bugs.koha-community.org 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6979 Alex Arnaud <alex.arnaud(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57481|0 |1 is obsolete| | --- Comment #26 from Alex Arnaud <alex.arnaud(a)biblibre.com> --- Created attachment 57964 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57964&action=ed… Bug 6979 - Update tests -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug.

1 0

[Bug 6979] LDAP authentication fails during password comparison
by bugzilla-daemon＠bugs.koha-community.org 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6979 Alex Arnaud <alex.arnaud(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57480|0 |1 is obsolete| | --- Comment #25 from Alex Arnaud <alex.arnaud(a)biblibre.com> --- Created attachment 57963 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57963&action=ed… Bug #6979 I removed several lines of code in the checkpw_ldap subroutine where LDAP authentication takes place, in the "else" part of the conditional that checks for the auth_by_bind config parameter. I added several lines to check whether the user can log in to LDAP using their DN and the password supplied in the login form. If they are able to bind, login contiues as normal and the LDAP attributes can be harvested as normal if the update options are turned on. The routine that was in place was failing because it was trying to check against a non-existent LDAP entry attribute called 'userpassword'. Instead of checking against a 'userpassword' attribute, the routine really should be checking to make sure the user can actually bind to LDAP. That's what I set up, and it is a safer way to test authentication against LDAP. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 17591] New: Use fully qualified C4::Items function names in C4:: Circulation
by bugzilla-daemon＠bugs.koha-community.org 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17591 Bug ID: 17591 Summary: Use fully qualified C4::Items function names in C4::Circulation Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: tomascohen(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org -- You are receiving this mail because: You are watching all bug changes.

1 15

[Bug 10798] OPAC_SEARCH_LIMIT behaves badly with search groups
by bugzilla-daemon＠bugs.koha-community.org 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10798 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|17683 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17683 [Bug 17683] OPAC_SEARCH_LIMIT no longer accepts group codes as limits -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13219] Add graphs to guided reports
by bugzilla-daemon＠bugs.koha-community.org 05 Dec '16

05 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13219 Eugene Espinoza <eugenegf(a)yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |eugenegf(a)yahoo.com -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 17583] New: Use Koha::Patron->is_expired from circulation.pl
by bugzilla-daemon＠bugs.koha-community.org 04 Dec '16

04 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17583 Bug ID: 17583 Summary: Use Koha::Patron->is_expired from circulation.pl Change sponsored?: --- Product: Koha Version: unspecified 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 Depends on: 17579 Bug 17579 introduces a Koha::Patron->is_expired method. We should use it from the circulation page Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17579 [Bug 17579] Add the Koha::Patron->is_expired method -- You are receiving this mail because: You are watching all bug changes.

1 50

[Bug 17671] New: Remove unused variables in Reserves.pm
by bugzilla-daemon＠bugs.koha-community.org 04 Dec '16

04 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17671 Bug ID: 17671 Summary: Remove unused variables in Reserves.pm Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: boutrosboutrosboutros(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com I was reading through Reserves.pm trying to figure out a bug - found some unused variables instead. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 14