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

[Bug 10988] Allow login via Google OAuth2 (OpenID Connect)
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10988 Brendan Gallagher <brendan(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |brendan(a)bywatersolutions.co | |m Status|Passed QA |Pushed to Master --- Comment #72 from Brendan Gallagher <brendan(a)bywatersolutions.com> --- Pushed to Master - Should be in the May 2016 release. Thanks! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11807] Add categorycode conversions to LDAP authentication.
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11807 Brendan Gallagher <brendan(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |brendan(a)bywatersolutions.co | |m Status|Passed QA |Pushed to Master --- Comment #16 from Brendan Gallagher <brendan(a)bywatersolutions.com> --- Pushed to Master - Should be in the May 2016 release. Thanks! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 16173] New: db_structure.t shouldn't have a fixed number of tests
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16173 Bug ID: 16173 Summary: db_structure.t shouldn't have a fixed number of tests Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: tomascohen(a)gmail.com QA Contact: gmcharlt(a)gmail.com Right now, this file contains a fixed number of tests. So any structure change on the relevant tables should taouch this file to adjust the number to what it expects. The point of tests is that they reflect problems, not cause them :-D So this specific tests file should not have a fixed number of tests, and it will serve its purpose by comparing the table vs deletedtable cases no matter how many columns they have. -- You are receiving this mail because: You are watching all bug changes.

1 10

[Bug 15113] koha-rebuild-zebra should check USE_INDEXER_DAEMON and skip if enabled
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15113 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |16190 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16190 [Bug 16190] Enable the indexer daemon by default -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13074] C4::Items::_build_default_values_for_mod_marc should use Koha::Cache
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13074 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart(a)bugs.koha-c | |ommunity.org -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15113] koha-rebuild-zebra should check USE_INDEXER_DAEMON and skip if enabled
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15113 --- Comment #1 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 49811 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49811&action=ed… Bug 15113: koha-rebuild-zebra should check USE_INDEXER_DAEMON and skip if enabled This patch changes the behaviour of the koha-rebuild-zebra script in the following way: USE_INDEXER_DAEMON=no - Keeps the current behaviour USE_INDEXER_DAEMON=yes - It skips incremental indexing to avoid races. Caveats: - A --force option is introduced for useing in a specific situtation that might need it (i.e. the administrator knows what he's doing). - If --full is passed, the reindexing is not skipped. The documentation files and messages are adjusted accordingly. This patch should help users that want to use the indexing daemon, in which case they wouldn't need to change their default 5 min cronjob (it will be just skipped). Ultimately, koha-common could have USE_INDEXER_DAEMON = yes by default, but that's subject for another bug report. To test: - Play with the different option switches and USE_INDEXER_DAEMON - Things work as expected - Sign off Regards -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15113] koha-rebuild-zebra should check USE_INDEXER_DAEMON and skip if enabled
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15113 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9021] Add SMS via email as an alternative to SMS services via SMS::Send drivers
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021 Indranil Das Gupta <indradg(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |16188 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16188 [Bug 16188] SMS via email not applicable in India due to telecom regulation -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15599] Unit tests output warnings
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15599 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Patch doesn't apply CC| |tomascohen(a)gmail.com --- Comment #10 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- I don't agree with Jonathan on this patch. I think tests (like Reserves.t) should create well-formed scenarios. If they are raising warnings, then they need to be fixed. And that's not the situation here. because we are explicitly creating this 'bad' scenario, to test the behaviour in that specific scenario. So, worth testing for the warning this time. I might be missing something anyway, so I'm open to discuss this. It doesn't apply BTW, because of the latest rebase of bug 14598. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14598] itemtype is not set on statistics by C4::Circulation::AddReturn
by bugzilla-daemon＠bugs.koha-community.org 01 Apr '16

01 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14598 --- Comment #79 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 49807 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49807&action=ed… Bug 14598: (followup) Remove unused and non-existent C4::ItemType include Signed-off-by: Tomas Cohen Arazi <tomascohen(a)unc.edu.ar> -- You are receiving this mail because: You are watching all bug changes.

1 0