- 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 ---------------------------------------------------------------------------- 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

[Bug 17696] New: Two missing periods in opac-suggestions.tt
by bugzilla-daemon＠bugs.koha-community.org 04 Dec '16

04 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17696 Bug ID: 17696 Summary: Two missing periods in opac-suggestions.tt Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: ephetteplace(a)cca.edu QA Contact: testopia(a)bugs.koha-community.org Around line 38 of the template file a sentence is missing a period: Please fill out this form to make a purchase suggestion. You will receive an email when the library processes your suggestion In an attempt to make this patch more worthwhile, I went looking for other typos in the template and found another missing period around line 259 in the conditional text that appears if the user has more pending suggestions than the MaxOpenSuggestions setting allows: You have reached your limit of suggestions you can place at this time.Once the library has processed those suggestions you will be able to place more Testing plan: - configure MaxOpenSuggestions with a low limit (e.g. 1) - visit opac-suggestions.pl?op=add, login if necessary - note that the period is missing from the second sentence of the leading paragraph under the "Enter a new purchase suggestion" heading - continue to submit suggestions until you've reached the MaxOpenSuggestions limit - observe the missing period in the "TooManySuggestionsText" paragraph - apply patch - revisit opac-suggestions.pl?op=add - note that the period is now present in the second sentence of the leading paragraph under the "Enter a new purchase suggestion" heading - note that the period is also present at the end of the "TooManySuggestionsText" paragraph Patch coming shortly. -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 17709] New: Article request broken - 'internal server error'
by bugzilla-daemon＠bugs.koha-community.org 04 Dec '16

04 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17709 Bug ID: 17709 Summary: Article request broken - 'internal server error' Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Article requsts from OPAC and from intranet are showing an 'internal server error'. I am testing in 16.11 with Plack. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 14

[Bug 17395] New: exporting checkouts in CVS generates a file with wrong extension
by bugzilla-daemon＠bugs.koha-community.org 04 Dec '16

04 Dec '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17395 Bug ID: 17395 Summary: exporting checkouts in CVS generates a file with wrong extension Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: fridolin.somers(a)biblibre.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com In checkouts table, the is an export form (when some exports syspref are enabled). Export using a CSV profile will create a file with name koha.mrc (same as ISO2709 export). It would be better with koha.csv. Bug 14647 manages the export page, this but will only manage for export from checkouts table where file name is hard-coded. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 31