From 72f37d54a27a3011cea4d52a78d87b472530fda3 Mon Sep 17 00:00:00 2001
From: "godzilla80@gmx.net" <>
Date: Sun, 7 Jun 2009 08:51:24 +0000
Subject: Enhancement: TFirePhpLogRoute: bypass to TBrowserLogRoute if headers
 already sent / php.ini (output_buffering=Off, implicit_flush=On)

---
 HISTORY                       |  1 +
 framework/Util/TLogRouter.php | 12 ++++++++++++
 2 files changed, 13 insertions(+)

diff --git a/HISTORY b/HISTORY
index 2e69dc13..9aaa8e29 100644
--- a/HISTORY
+++ b/HISTORY
@@ -5,6 +5,7 @@ BUG: Issue#164 - CultureInfo::validCulture should be declared as a static method
 BUG: Issue#168 - TSqlMapXmlConfiguration: CacheModel properties are not set (Yves)
 ENH: Issue#175 - TBulletedList: Introduce TBulletStyle::None (Yves)
 ENH: TAssetManager: introduce protected property "Published" to allow subclasses access (Yves)
+ENH: TFirePhpLogRoute: bypass to TBrowserLogRoute if headers already sent / php.ini (output_buffering=Off, implicit_flush=On) (Yves)
 
 Version 3.1.5 May 24, 2009
 BUG: Issue#55 - TPropertyAccess.get and has don't recognize magic getter __get (Yves)
diff --git a/framework/Util/TLogRouter.php b/framework/Util/TLogRouter.php
index b7b2f0a3..64a241fb 100644
--- a/framework/Util/TLogRouter.php
+++ b/framework/Util/TLogRouter.php
@@ -1015,6 +1015,18 @@ class TFirePhpLogRoute extends TLogRoute
 	{
 		if(empty($logs) || $this->getApplication()->getMode()==='Performance') return;
 
+		if( headers_sent() ) {
+			echo '
+				<div style="width:100%; background-color:darkred; color:#FFF; padding:2px">
+					TFirePhpLogRoute.GroupLabel "<i>' . $this -> getGroupLabel() . '</i>" -
+					Routing to FirePHP impossible, because headers already sent!
+				</div>
+			';
+			$fallback = new TBrowserLogRoute();
+			$fallback->processLogs($logs);
+			return;
+		}
+
 		require_once Prado::getPathOfNamespace('System.3rdParty.FirePHPCore') . '/FirePHP.class.php';
 		$firephp = FirePHP::getInstance(true);
 		$firephp -> setOptions(array('useNativeJsonEncode' => false));
-- 
cgit v1.2.3