From ed336633b6343ac637d6ba1cacf002318b30dcd4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Demians?= Date: Mon, 11 Dec 2017 08:41:05 +0100 Subject: [PATCH] Bug 19788 Local preference with uppercase Fix a bug tranforming new local preference variable name in lowercase. TO TEST: 1. Admin > System preferences > Local use 2. Button New preference. Fill variable field with 'TESTfoo'. Click Save. 3. The new variable is named: 'testfoo'. 4. Apply the patch. 5. Button New preference. Fill variable field with 'TESTfoo2'. Click Save. 6. The new variable is properly named: 'TESTfoo2'. Signed-off-by: Claire Gravely --- C4/Context.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/C4/Context.pm b/C4/Context.pm index 1aa08be..1eb5f88 100644 --- a/C4/Context.pm +++ b/C4/Context.pm @@ -496,6 +496,7 @@ preference. sub set_preference { my ( $self, $variable, $value, $explanation, $type, $options ) = @_; + my $variable_case = $variable; $variable = lc $variable; my $syspref = Koha::Config::SysPrefs->find($variable); @@ -521,7 +522,7 @@ sub set_preference { )->store; } else { $syspref = Koha::Config::SysPref->new( - { variable => $variable, + { variable => $variable_case, value => $value, explanation => $explanation || undef, type => $type, -- 2.1.4