From 4af8915edaa0e25d651b819590c802b18ea1fb67 Mon Sep 17 00:00:00 2001
From: marcus <marcus@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Thu, 7 Aug 2008 16:42:33 +0000
Subject: Refs #174: Introducing the ElggDummy class which returns false for
 all methods and set variables called on it. This is what the logged out
 $_SESSION['user'] is set to and is also what page_owner_entity will return if
 page_owner is invalid.

This means that calls to things like $_SESSION['user']->getGUID() are now safe and will no longer cause a parsing error.

Please report any issues.

git-svn-id: https://code.elgg.org/elgg/trunk@1770 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 engine/lib/dummy.php     | 29 +++++++++++++++++++++++++++++
 engine/lib/pageowner.php |  4 ++--
 engine/lib/sessions.php  | 14 +++++++++-----
 3 files changed, 40 insertions(+), 7 deletions(-)
 create mode 100644 engine/lib/dummy.php

(limited to 'engine')

diff --git a/engine/lib/dummy.php b/engine/lib/dummy.php
new file mode 100644
index 000000000..a7f8760e0
--- /dev/null
+++ b/engine/lib/dummy.php
@@ -0,0 +1,29 @@
+<?php
+	/**
+	 * Dummy class.
+	 * This function contains dummy functions and classes which return safe dummy values. Essentially this is 
+	 * to make sure that things like calling echo page_owner_entity()->name don't cause a WSoD.
+	 * 
+	 * @package Elgg
+	 * @subpackage Core
+	 * @license http://www.gnu.org/licenses/old-licenses/gpl-2.0.html GNU Public License version 2
+	 * @author Marcus Povey 
+	 * @copyright Curverider Ltd 2008
+	 * @link http://elgg.org/
+	 */
+	
+	/**
+	 * Dummy class.
+	 * For every function call, get and set this function returns false.
+	 */
+	class ElggDummy {
+		
+		 public function __call($method, $args) {
+		 	return false;
+		 }
+		 
+		function __get($name) { return false; }
+		
+		function __set($name, $value) { return false; }
+	}
+?>
\ No newline at end of file
diff --git a/engine/lib/pageowner.php b/engine/lib/pageowner.php
index 3f867e750..46394c8c9 100644
--- a/engine/lib/pageowner.php
+++ b/engine/lib/pageowner.php
@@ -67,8 +67,8 @@
 	        if ($page_owner > 0)
 	        	return get_entity($page_owner);
 	        	
-			return false;
-            
+			//return false;
+            return new ElggDummy();
         }
         
     /**
diff --git a/engine/lib/sessions.php b/engine/lib/sessions.php
index 87e2f931d..5b08a6e5b 100644
--- a/engine/lib/sessions.php
+++ b/engine/lib/sessions.php
@@ -184,14 +184,16 @@
 	                $code = $_COOKIE['elggperm'];
 	                $code = md5($code);
 	                $_SESSION['guid'] = 0;
-	                $_SESSION['id'] = 0;
+	                $_SESSION['id'] = 0;
+	                $_SESSION['user'] = new ElggDummy();
 	                if ($user = get_user_by_code($code)) {
                     	$_SESSION['user'] = $user;
                         $_SESSION['id'] = $user->getGUID();
                         $_SESSION['guid'] = $_SESSION['id'];
                         $_SESSION['code'] = $_COOKIE['elggperm'];
 	                }
-	            } else {
+	            } else {
+	            	$_SESSION['user'] = new ElggDummy();
 	                $_SESSION['id'] = 0;
 	                $_SESSION['guid'] = 0;
 	            }
@@ -201,11 +203,13 @@
 	                if ($user = get_user_by_code($code)) {
 	                	$_SESSION['user'] = $user;
 	                } else {
-	                	unset($_SESSION['user']);
+	                	//unset($_SESSION['user']);
+	                	$_SESSION['user'] = new ElggDummy();
 	                	$_SESSION['guid'] = 0;
 	                	$_SESSION['id'] = 0;
 	                }
-	            } else {
+	            } else {
+	            	$_SESSION['user'] = new ElggDummy();
 	            	$_SESSION['guid'] = 0;
 	                $_SESSION['id'] = 0;
 	            }
@@ -213,7 +217,7 @@
 	        if ($_SESSION['id'] > 0) {
 	            set_last_action($_SESSION['id']);
 	        }
-	        
+	        print_r($_SESSION);
 	        register_action("login",true);
     		register_action("logout");
     		
-- 
cgit v1.2.3