Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 34584: (QA follow-up) Update system preference SocialNetworks

Browse source 
SocialNetworks was set to be YesNo, but is actual choice with
3 options: linkedin, facebook, email.

This updates the database and the sysprefs.sql file.

It doesn't correct the value, but if it still contains twitter, this
will be fixed by the next change of the pref and has no negative
consequences.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Katrin Fischer 2023-08-24 20:14:13 +00:00 committed by Tomas Cohen Arazi
parent 1162629cf0
commit 0ad83f95ec
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F
2 changed files with 18 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
installer/data/mysql/atomicupdate/bug_34584_remove_twitter.pl Executable file
View file

@ -0,0 +1,17 @@
use Modern::Perl;
return {
bug_number => "34584",
description => "Update SocialNetworks system preference to Choice",
up => sub {
my ($args) = @_;
my ( $dbh, $out ) = @$args{qw(dbh out)};
# Do you stuffs here
$dbh->do(
q{UPDATE systempreferences SET type = "Choice", options = "facebook|linkedin|email" WHERE variable = "SocialNetworks"}
);
say $out "Updated system preference 'SocialNetworks'";
},
};

2
installer/data/mysql/mandatory/sysprefs.sql
View file

@ -680,7 +680,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `
('SMSSendDriver','','','Sets which SMS::Send driver is used to send SMS messages.','free'),
('SMSSendPassword', '', '', 'Password used to send SMS messages', 'free'),
('SMSSendUsername', '', '', 'Username/Login used to send SMS messages', 'free'),
('SocialNetworks','0','','Enable/Disable social networks links in opac detail pages','YesNo'),
('SocialNetworks','','facebook|linkedin|email','Enable/Disable social networks links in opac detail pages','Choice'),
('SpecifyDueDate','1','','Define whether to display \"Specify Due Date\" form in Circulation','YesNo'),
('SpecifyReturnDate',1,'','Define whether to display \"Specify Return Date\" form in Circulation','YesNo'),
('SpineLabelAutoPrint','0','','If this setting is turned on, a print dialog will automatically pop up for the quick spine label printer.','YesNo'),