From e3ba7ff6f8698194945a67e2de60803f17ef99b2 Mon Sep 17 00:00:00 2001 From: Emily Lamancusa Date: Tue, 25 Feb 2025 20:25:18 +0000 Subject: [PATCH] Bug 38108: (QA follow-up) Use say_success in dbrev Use say_success instead of say when outputting a line that the system preference was added successfully Signed-off-by: Emily Lamancusa --- installer/data/mysql/atomicupdate/bug_38108.pl | 2 +- installer/data/mysql/mandatory/sysprefs.sql | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/installer/data/mysql/atomicupdate/bug_38108.pl b/installer/data/mysql/atomicupdate/bug_38108.pl index e7bb87df63..930153423a 100755 --- a/installer/data/mysql/atomicupdate/bug_38108.pl +++ b/installer/data/mysql/atomicupdate/bug_38108.pl @@ -16,6 +16,6 @@ return { ); # sysprefs - say $out "Added new system preference 'AlwaysShowHoldingsTableFilters'"; + say_success( $out, "Added new system preference 'AlwaysShowHoldingsTableFilters'" ); }, }; diff --git a/installer/data/mysql/mandatory/sysprefs.sql b/installer/data/mysql/mandatory/sysprefs.sql index 0cd7bfc333..1895866f24 100644 --- a/installer/data/mysql/mandatory/sysprefs.sql +++ b/installer/data/mysql/mandatory/sysprefs.sql @@ -56,7 +56,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` ('AlternateHoldingsField','',NULL,'The MARC field/subfield that contains alternate holdings information for bibs taht do not have items attached (e.g. 852abchi for libraries converting from MARC Magician).','free'), ('AlternateHoldingsSeparator','',NULL,'The string to use to separate subfields in alternate holdings displays.','free'), ('AlwaysLoadCheckoutsTable','0','','Option to always load the checkout table','Yes/No'), -('AlwaysShowHoldingsTableFilters','0','','Option to always show filters when loading the holdings table','Yes/No'), +('AlwaysShowHoldingsTableFilters','0','','Option to always show filters when loading the holdings table','YesNo'), ('AmazonAssocTag','','','See: http://aws.amazon.com','free'), ('AmazonCoverImages','0','','Display Cover Images in staff interface from Amazon Web Services','YesNo'), ('AmazonLocale','US','US|CA|DE|FR|IN|JP|UK','Use to set the Locale of your Amazon.com Web Services','Choice'), -- 2.39.5