From d96919227f38c8f5a1df12a1f6bef61154a4aee3 Mon Sep 17 00:00:00 2001 From: "Michael J. Rubinsky" Date: Thu, 26 Feb 2009 09:06:31 -0500 Subject: [PATCH] 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. --- framework/View/lib/Horde/View/Base.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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 ''; } /** -- 2.11.0