From: Michael J. Rubinsky <mrubinsk@horde.org>
Date: Thu, 26 Feb 2009 14:06:31 +0000 (-0500)
Subject: On second thought - I don't see the reason for checking the existence
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=d96919227f38c8f5a1df12a1f6bef61154a4aee3;p=horde.git

On second thought - I don't see the reason for checking the existence
of a public member in __get() - if it existed, __get() would never get
called - and checking this would have the effect of making all non-public
members public. Just return an empty string if a non-visible member is
accessed.
---

diff --git a/framework/View/lib/Horde/View/Base.php b/framework/View/lib/Horde/View/Base.php
index f479c06a3..1e5661316 100644
--- a/framework/View/lib/Horde/View/Base.php
+++ b/framework/View/lib/Horde/View/Base.php
@@ -83,13 +83,13 @@ abstract class Horde_View_Base
     }
 
     /**
-     * Return a view variable
+     * Return an empty string if a variable does not exist.
      *
      * @param string $name Variable name to retrieve
      */
     public function __get($name)
     {
-        return isset($this->$name) ? $this->$name : '';
+        return '';
     }
 
     /**