Changeset 18814 for branches/version-2_5-dev/data/class

Timestamp:

2010/09/20 03:19:39 (16 years ago)

Author:

Seasoft

Message:

#624(軽微な表示乱れを修正)

• #567 に伴いバックトレース切捨ての位置が不自然になるので、handle_error 以前を切り捨てるように変更。
• デバッグ用のメッセージの出力位置を変更。SC_Query#traceError が生成したメッセージの出力を意識する。
• 現状では print_r を利用する必要は無いようなので、echo を利用するように変更。

File:

• branches/version-2_5-dev/data/class/pages/error/LC_Page_Error_SystemError.php (modified) (3 diffs)

branches/version-2_5-dev/data/class/pages/error/LC_Page_Error_SystemError.php

@@ -92 +92 @@
             echo '<div>▼▼▼ デバッグ情報ここから ▼▼▼</div>';
             echo '<pre>';
-            print_r(htmlspecialchars($this->sfGetErrMsg(), ENT_QUOTES, CHAR_CODE));
+            echo htmlspecialchars($this->sfGetErrMsg(), ENT_QUOTES, CHAR_CODE);
             echo '</pre>';
             echo '<div>▲▲▲ デバッグ情報ここまで ▲▲▲</div>';
@@ -117 +117 @@
         $errmsg .= $this->lfGetErrMsgHead();
         $errmsg .= "\n";
-        
-        // デバッグ用のメッセージが指定されている場合
-        if (!empty($this->arrDebugMsg)) {
-            $errmsg .= implode("\n\n", $this->arrDebugMsg) . "\n";
-            $errmsg .= "\n";
-        }
-        
+
         // PEAR エラーを伴う場合
         if (!is_null($this->pearResult)) {
@@ -136 +130 @@
         // (上に該当せず)バックトレースを生成できる環境(一般的には PHP 4 >= 4.3.0, PHP 5)の場合
         else if (function_exists("debug_backtrace")) {
-            $errmsg .= SC_Utils_Ex::sfBacktraceToString(array_slice(debug_backtrace(), 2));
+            $backtrace = debug_backtrace();
+
+            // バックトレースのうち handle_error 以前は通常不要と考えられるので削除
+            $cnt = 0;
+            $offset = 0;
+            foreach ($backtrace as $key => $arrLine) {
+                $cnt ++;
+                if (!isset($arrLine['file']) && $arrLine['function'] === 'handle_error') {
+                    $offset = $cnt;
+                    break;
+                }
+            }
+            if ($offset !== 0) {
+                $backtrace = array_slice($backtrace, $offset);
+            }
+
+            $errmsg .= SC_Utils_Ex::sfBacktraceToString($backtrace);
         }
-        
+
+        // デバッグ用のメッセージが指定されている場合
+        if (!empty($this->arrDebugMsg)) {
+            $errmsg .= implode("\n\n", $this->arrDebugMsg) . "\n";
+        }
+
         return $errmsg;
     }