From 6dd17df0074b94d4c29d3849fa5f5ea1c8fb75af Mon Sep 17 00:00:00 2001 From: erickson Date: Fri, 11 Dec 2009 15:40:17 +0000 Subject: [PATCH] return null as the setting value when a user setting is not applied, instead of returning an empty string git-svn-id: svn://svn.open-ils.org/ILS/trunk@15141 dcc99617-32d9-48b4-a31d-7c20da2025e4 --- Open-ILS/src/perlmods/OpenILS/Application/Actor.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm b/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm index d1059c7cb8..07ac6ddc01 100644 --- a/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm +++ b/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm @@ -190,7 +190,7 @@ sub user_settings { sub get_setting { my($e, $user_id, $setting) = @_; my $val = $e->search_actor_user_setting({usr => $user_id, name => $setting})->[0]; - return '' unless $val; # XXX this should really return undef, but needs testing + return undef unless $val; # XXX this should really return undef, but needs testing return OpenSRF::Utils::JSON->JSON2perl($val->value); } -- 2.11.0