From 0e6a9eaa4d7332b3acde37656d4547c59cac2c36 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 15 Feb 2017 08:39:23 +0000 Subject: [PATCH] [SIGNED-OFF] Bug 18121: Replace opac_news.new with opac_news.content in notices templates Bug 17960 replaces opac_news.new with opac_news.content, we can replace the content of the notices' templates automatically. Test plan: Checkout a commit prior to bug 17960. Install a new DB with sample data. You should have notices with <> Switch to a branch with this patch, on top of master Execute the updatedatabase.pl sript => The notices with opac_news.new should have been replaced with opac_news.content Signed-off-by: Srdjan --- installer/data/mysql/updatedatabase.pl | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl index 7a80e41..bde4667 100755 --- a/installer/data/mysql/updatedatabase.pl +++ b/installer/data/mysql/updatedatabase.pl @@ -13868,16 +13868,12 @@ if( CheckVersion( $DBversion ) ) { $dbh->do(q|ALTER TABLE opac_news CHANGE COLUMN new content text NOT NULL|); } - my ( $used_in_templates ) = $dbh->selectrow_array(q| - SELECT COUNT(*) FROM letter WHERE content LIKE "%<>%"; + $dbh->do(q| + UPDATE letter SET content = REPLACE(content, "<>", "<>") WHERE content LIKE "%<>%" |); - if ( $used_in_templates ) { - print "WARNING - It seems that you are using the opac_news.new column in your notice templates\n"; - print "Since it has now been renamed with opac_news.content, you should update them.\n"; - } SetVersion( $DBversion ); - print "Upgrade to $DBversion done (Bug 17960 - Rename opac_news with opac_news.content)\n"; + print "Upgrade to $DBversion done (Bug 17960 - Rename opac_news with opac_news.content (template notices have been updated!))\n"; } $DBversion = "16.12.00.008"; -- 2.7.4