Attachment 166221 Details for Bug 36790 – Bug 36790: List table columns for 230600052 queries

[patch] Bug 36790: List table columns for 230600052 queries

Bug-36790-List-table-columns-for-230600052-queries.patch (text/plain), 3.75 KB, created by Kyle M Hall (khall) on 2024-05-06 12:46:25 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2024-05-06 12:46:25 UTC

Size: 3.75 KB

patch

obsolete

>From a22644aa31a3e50cc0e503b8947c321272a04302 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 6 May 2024 12:58:33 +0200
>Subject: [PATCH] Bug 36790: List table columns for 230600052 queries
>
>We must list the columns, or the db rev will fail when a new column is
>added. It happened here when 33478 added 'style'
>
>Also remove id and dates
>
>To test:
>1. On current main, run:
>   $ ktd --shell
>  k$ perl /kohadevbox/misc4dev/run_tests.pl --koha-dir=. --run-db-upgrade-only
>=> FAIL: Tests explode for DB query issues
>2. Apply this patch
>3. Repeat 1
>=> SUCCESS: Tests pass!
>4. Sign off :-D
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> installer/data/mysql/db_revs/230600052.pl | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
>diff --git a/installer/data/mysql/db_revs/230600052.pl b/installer/data/mysql/db_revs/230600052.pl
>index 416e30d42ec..5687d9040b4 100755
>--- a/installer/data/mysql/db_revs/230600052.pl
>+++ b/installer/data/mysql/db_revs/230600052.pl
>@@ -9,7 +9,8 @@ return {
> 
>         $dbh->do(
>             q{
>-            INSERT IGNORE INTO `letter` VALUES (NULL,'reserves','HOLDDGST','','Hold available for pickup (digest)',0,'Hold(s) available for pickup','You have one or more holds available for pickup:\r\n\r\n----\r\nTitle: [% hold.biblio.title %]\r\nAuthor: [% hold.biblio.author %]\r\nCopy: [% hold.item.copynumber %]\r\nLocation: [% hold.branch.branchname %]\r\nWaiting since: [% hold.waitingdate %]\r\nWaiting at: [%hold.branch.branchname%]\r\n[% hold.branch.branchaddress1 %]\r\n[% hold.branch.branchaddress2 %]\r\n[% hold.branch.branchaddress3 %]\r\n[% hold.branch.branchcity %] [% hold.branch.branchzip %]\r\n----','email','default','2023-08-29 18:42:15');
>+            INSERT IGNORE INTO `letter` (`module`, `code`, `branchcode`, `name`, `is_html`, `title`, `content`, `message_transport_type`, `lang`)
>+            VALUES ('reserves','HOLDDGST','','Hold available for pickup (digest)',0,'Hold(s) available for pickup','You have one or more holds available for pickup:\r\n\r\n----\r\nTitle: [% hold.biblio.title %]\r\nAuthor: [% hold.biblio.author %]\r\nCopy: [% hold.item.copynumber %]\r\nLocation: [% hold.branch.branchname %]\r\nWaiting since: [% hold.waitingdate %]\r\nWaiting at: [%hold.branch.branchname%]\r\n[% hold.branch.branchaddress1 %]\r\n[% hold.branch.branchaddress2 %]\r\n[% hold.branch.branchaddress3 %]\r\n[% hold.branch.branchcity %] [% hold.branch.branchzip %]\r\n----','email','default');
>         }
>         );
> 
>@@ -17,7 +18,8 @@ return {
> 
>         $dbh->do(
>             q{
>-            INSERT IGNORE INTO `letter` VALUES (NULL,'reserves','HOLDDGST','','Hold available for pickup (digest)',0,'Hold(s) available for pickup','You have one or more holds available for pickup:\r\n----\r\n[% hold.biblio.title %]\r\n----','sms','default','2023-08-29 18:42:15');
>+            INSERT IGNORE INTO `letter` (`module`, `code`, `branchcode`, `name`, `is_html`, `title`, `content`, `message_transport_type`, `lang`)
>+            VALUES ('reserves','HOLDDGST','','Hold available for pickup (digest)',0,'Hold(s) available for pickup','You have one or more holds available for pickup:\r\n----\r\n[% hold.biblio.title %]\r\n----','sms','default');
>         }
>         );
> 
>@@ -25,7 +27,9 @@ return {
> 
>         $dbh->do(
>             q{
>-            INSERT IGNORE INTO message_transports VALUES
>+            INSERT IGNORE INTO message_transports
>+            (`message_attribute_id`, `message_transport_type`, `is_digest`, `letter_module`, `letter_code`, `branchcode`)
>+            VALUES
>             ( 4, "email", 1, "reserves", "HOLDDGST", "" ),
>             ( 4, "sms", 1, "reserves", "HOLDDGST", "" ),
>             ( 4, "phone", 1, "reserves", "HOLDDGST", "");
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 36790: 166204 | 166218 | 166220 | 166221