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 f479c06..1e56613 100644 (file)
--- 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 '';
     }
 
     /**