[#12540] ErrorHandler: Only call $registry->isAdmin() if registry is available
Summary ErrorHandler: Only call $registry->isAdmin() if registry is available
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners slusarz@horde.org
Requester thomas.jarosch@intra2net.com
Created 2013-08-07 (2528 days ago)
Due
Updated 2013-08-07 (2528 days ago)
Assigned
Resolved 2013-08-07 (2528 days ago)
Milestone
Patch Yes

Comments
Thomas Jarosch <thomas.jarosch@intra2net.com> 2013-08-07 12:48:14
Hi,

attached patch fixes the ErrorHandler if the registry is not yet available.

Without the patch, the ErrorHandler crashed like this:
[Wed Aug 07 14:38:42 2013] [error] [client 127.0.0.1] PHP Fatal error: 
  Call to a member function isAdmin() on a non-object in 
/datastore/share/pear/Horde/ErrorHandler.php on line 81

Cheers,
Thomas


Git Commit <commits@lists.horde.org> 2013-08-07 23:18:09
Changes have been made in Git (master):

commit 21bc7a682d0771fa0f55996664aaefdce0a467bb
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Wed Aug 7 17:06:05 2013 -0600

     [mms] Ensure Registry exists in error handler before attempting 
to access it (Bug #12540).

  framework/Core/lib/Horde/ErrorHandler.php |    2 +-
  framework/Core/package.xml                |    4 ++--
  2 files changed, 3 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/21bc7a682d0771fa0f55996664aaefdce0a467bb

Michael Slusarz <slusarz@horde.org> 2013-08-07 23:18:49
Horde_Core 2.6.6.