From: erickson <erickson@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Thu, 9 Oct 2008 15:56:30 +0000 (+0000)
Subject: don't call ->value if no setting exists, return empty string
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=0e08d615aee3dfc2f65eaed0c667c2b71620a042;p=Evergreen.git

don't call ->value if no setting exists, return empty string

git-svn-id: svn://svn.open-ils.org/ILS/trunk@10807 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm b/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm
index e34e4b24ef..b187291a43 100644
--- a/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm
+++ b/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm
@@ -148,6 +148,7 @@ sub user_settings {
 
     if($setting) {
         my $val = $e->search_actor_user_setting({usr => $user_id, name => $setting})->[0];
+        return '' unless $val;
         return OpenSRF::Utils::JSON->JSON2perl($val->value);
     } else {
         my $s = $e->search_actor_user_setting({usr => $user_id});