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

[Bug 14407] Limit web-based self-checkout to specific IP addresses
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14407 Nicholas van Oudtshoorn <vanoudt(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #56066|0 |1 is obsolete| | --- Comment #36 from Nicholas van Oudtshoorn <vanoudt(a)gmail.com> --- Created attachment 68834 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68834&action=ed… Bug 14407 - Allow restricting SCO to IP or IP range Added some unit tests to this. Not added in the db_dependent directory, given that the new function (in_ipset) doesn't actually touch the database at all. It's meant to be a function that can be reused in the future for other ip based limits in Koha. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68646|0 |1 is obsolete| | --- Comment #127 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 68823 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68823&action=ed… bug_11213: Check for $item->{itype} presence to avoid warning -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62438|0 |1 is obsolete| | --- Comment #126 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 68822 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68822&action=ed… bug_11213: Added XSLTParse4Display() to Items test -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62437|0 |1 is obsolete| | --- Comment #125 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 68821 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68821&action=ed… bug_11213: GetItemsInfo() test -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62436|0 |1 is obsolete| | --- Comment #124 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 68820 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68820&action=ed… bug_11213: Include XSLT processing for searchResults() test * Added template paths to temp test dir, so XSLT templates can be picked up -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62435|0 |1 is obsolete| | --- Comment #123 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 68819 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68819&action=ed… bug_11213: whitespace correction -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 30 Oct '17

30 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62434|0 |1 is obsolete| | --- Comment #122 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 68818 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68818&action=ed… bug_11213: Changed XSLTParse4Display() interface The list of biblio items is passed on now, instead of GetItemsInfo() being called. This is because the callers already have the list ready, so the GetItemsInfo() call is being duplicated unnecessarily. Search::searchResults() builds items list from XML, and that one is passed instead. * XSLT::XSLTParse4Display() - supply the items list as input param - removed hidden items list param - hidden should not be in the items list - changed buildKohaItemsNamespace() accordingly * Items - added sort_by input param to GetItemsInfo() * catalogue/detail.pl, opac/opac-detail.pl, shelfpage() - added items list to the XSLTParse4Display() call * Search::searchResults() - include all available info when building items lists - added combined items list (available, on loan, other) to the XSLTParse4Display() call To test: This change is a noop, so following screens need to be checked against any changes: * Intranet: - catalogue/search.pl (results) - catalogue/detail.pl - virtualshelves/shelves.pl * Opac - opac-search.pl (results, hidelostitems syspref on and off) - opac-detail.pl - opac-shelves.pl The display should stay the same before and after patch. The speed should increase though. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9302] Add ability to merge patron records
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9302 Josef Moravec <josef.moravec(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |josef.moravec(a)gmail.com --- Comment #11 from Josef Moravec <josef.moravec(a)gmail.com> --- Kyle, it's nice feature, and I really would like to have it in Koha. But qa tools are complaining about this: FAIL Koha/Patrons.pm FAIL valid Useless use of anonymous hash ({}) in void context FAIL koha-tmpl/intranet-tmpl/prog/en/modules/members/merge-patrons.tt FAIL forbidden patterns forbidden pattern: tab char (line 30) forbidden pattern: tab char (line 33) forbidden pattern: tab char (line 50) forbidden pattern: tab char (line 39) forbidden pattern: tab char (line 42) forbidden pattern: tab char (line 49) forbidden pattern: tab char (line 36) forbidden pattern: tab char (line 123) forbidden pattern: tab char (line 46) forbidden pattern: tab char (line 51) forbidden pattern: tab char (line 32) forbidden pattern: tab char (line 41) forbidden pattern: tab char (line 40) forbidden pattern: tab char (line 55) forbidden pattern: tab char (line 34) forbidden pattern: tab char (line 122) forbidden pattern: tab char (line 53) forbidden pattern: tab char (line 127) forbidden pattern: tab char (line 38) forbidden pattern: tab char (line 37) forbidden pattern: tab char (line 35) forbidden pattern: tab char (line 126) forbidden pattern: tab char (line 48) forbidden pattern: tab char (line 43) forbidden pattern: tab char (line 29) forbidden pattern: tab char (line 52) forbidden pattern: tab char (line 125) forbidden pattern: tab char (line 124) forbidden pattern: tab char (line 31) forbidden pattern: tab char (line 47) forbidden pattern: tab char (line 54) forbidden pattern: tab char (line 45) forbidden pattern: tab char (line 44) forbidden pattern: tab char (line 121) forbidden pattern: tab char (line 120) FAIL members/merge-patrons.pl FAIL valid Useless use of anonymous hash ({}) in void context FAIL t/db_dependent/Patrons.t FAIL forbidden patterns forbidden pattern: tab char (line 149) forbidden pattern: tab char (line 148) FAIL valid Useless use of anonymous hash ({}) in void context -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11213] GetItemsInfo() called twice
by bugzilla-daemon＠bugs.koha-community.org 29 Oct '17

29 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213 Josef Moravec <josef.moravec(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |josef.moravec(a)gmail.com --- Comment #121 from Josef Moravec <josef.moravec(a)gmail.com> --- Needs adaptation at least because of bug 17248 -- You are receiving this mail because: You are watching all bug changes.

1 0