Bug #11944 fixed utf8 encoding koha wide, however as part of it, it updates all database table to utf8 charset and utf8_unicode_ci collation. This is a good thing for anyone who left the default coallition to being with, but is a step backwards for those working on Wheezy and Ubuntu who may have chosen the utf8_mb4 pragma. The update should first check for such cases, and skip if found.
Created attachment 35391 [details] [review] BUG 13596: Prevent utf8mb4 -> utf8 convertion utf8mb4 is a superset of utf8 and thus is compatible. We should not force an entire db change upon people who have proactively set utf8mb4 encoding before now. This patch aslo removed the deprecated use of ->tables stanza in favour of the table_info stanze.
Created attachment 35392 [details] [review] BUG 13596: Prevent utf8mb4 -> utf8 convertion utf8mb4 is a superset of utf8 and thus is compatible. We should not force an entire db change upon people who have proactively set utf8mb4 encoding before now. This patch aslo removed the deprecated use of ->tables stanza in favour of the table_info stanze.
Created attachment 35393 [details] [review] BUG 13596: Prevent utf8mb4 -> utf8 convertion utf8mb4 is a superset of utf8 and thus is compatible. We should not force an entire db change upon people who have proactively set utf8mb4 encoding before now. This patch aslo removed the deprecated use of ->tables stanza in favour of the table_info stanze.
Created attachment 35395 [details] [review] BUG 13596: Prevent utf8mb4 -> utf8 convertion utf8mb4 is a superset of utf8 and thus is compatible. We should not force an entire db change upon people who have proactively set utf8mb4 encoding before now. This patch aslo removed the deprecated use of ->tables stanza in favour of the table_info stanze.
Created attachment 35432 [details] [review] BUG 13596: Prevent utf8mb4 -> utf8 convertion utf8mb4 is a superset of utf8 and thus is compatible. We should not force an entire db change upon people who have proactively set utf8mb4 encoding before now. This patch also removed the deprecated use of ->tables stanza in favour of the table_info stanza.
Created attachment 35931 [details] [review] BUG 13596: Prevent utf8mb4 -> utf8 convertion utf8mb4 is a superset of utf8 and thus is compatible. We should not force an entire db change upon people who have proactively set utf8mb4 encoding before now. This patch also removed the deprecated use of ->tables stanza in favour of the table_info stanza. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 36071 [details] [review] [PASSED QA] BUG 13596: Prevent utf8mb4 -> utf8 convertion utf8mb4 is a superset of utf8 and thus is compatible. We should not force an entire db change upon people who have proactively set utf8mb4 encoding before now. This patch also removed the deprecated use of ->tables stanza in favour of the table_info stanza. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Patch pushed to master. Thanks Martin!