[Koha-bugs] [Bug 16583] Strangeness in the staff client on ubuntu 16.04 and MySQL 5.7

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16583

Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bgkriegel at gmail.com

--- Comment #1 from Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> ---
Hi Liz,
we'll have plenty of trouble with (In reply to Liz Rea from comment #0)
> Have noticed some strangeness with a clean install of Koha on MySQL 5.7:

we'll have plenty of trouble with 5.7

> 
> When adding a rule at /cgi-bin/koha/admin/smart-rules.pl, if any box is left
> empty the first empty field pops an error similar to this:
> 
> DBIx::Class::Storage::DBI::_dbh_execute(): Incorrect integer value: '' for
> column 'chargeperiod' at row 1 at
> /usr/share/koha/intranet/cgi-bin/admin/smart-rules.pl line 170

In this case the problem is that we depended on a type conversion done by
MySQL, empty '' string -> 0. In smart-rules.pl for example, every empty box is
sent as an empty string to the database, which used to transform that to 0.
With 5.7+ version this is not the default behavior :(

> 
> When attempting to add a borrower category, it thinks a duplicate has been
> added with the resultant error message
> 
> "An error occurred when inserting this patron category. The patron category
> might already exist."
> 

There are some cases where the strange use of 'GROUP BY' not really needed
triggers the error. This is one of them.

Thoughts: 
a) try to support the 'strict' mode of MySQL 5.7+
b) suggest that anyone interested reconfigure server to a less 'strict' mode
c) ignore the problem because is not in Debian, yet.

I've patches for these problems, and others not yet reported.
Just need to know what are we going to do.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.