Changeset 23427

Timestamp:

2014/05/16 16:15:21 (10 years ago)

Author:

Seasoft

Message:

#2551 (プラグインの処理によりインストール中の PHP エラー画面の表示が阻害される part 2)

Location:

branches/version-2_13-dev/data/class

Files:

• helper/SC_Helper_HandleError.php (modified) (2 diffs)
• helper/SC_Helper_Plugin.php (modified) (1 diff)
• pages/error/LC_Page_Error.php (modified) (1 diff)
• pages/error/LC_Page_Error_DispError.php (modified) (1 diff)

branches/version-2_13-dev/data/class/helper/SC_Helper_HandleError.php

@@ -32 +32 @@
 class SC_Helper_HandleError
 {
+    /** エラー処理中か */
+    static $under_error_handling = false;
+
     /**
      * 処理の読み込みを行う
@@ -194 +197 @@
     public static function displaySystemError($errstr = null)
     {
+        SC_Helper_HandleError_Ex::$under_error_handling = true;
+
         ob_clean();
 

branches/version-2_13-dev/data/class/helper/SC_Helper_Plugin.php

@@ -311 +311 @@
     public static function hook($hook_point, $arrArgs = array(), $plugin_activate_flg = PLUGIN_ACTIVATE_FLAG)
     {
+        // エラー処理中は実行しない
+        if (SC_Helper_HandleError_Ex::$under_error_handling) {
+            return;
+        }
+
         $objPlugin = SC_Helper_Plugin::getSingletonInstance($plugin_activate_flg);
+
+        // 以前、エラー処理中に (オブジェクトではない) false に対し、doAction をコールする不具合があった。(#1971, #2551)
+        // 現在、そういった状況は回避している認識だが、念のため同様の状況が発生した場合、ログを残し、実行しない。
+        if (!is_object($objPlugin)) {
+            // XXX 致命的エラーの処理中だと、この方法ではログが残らない模様。実質的に問題無いと考えている。
+            trigger_error('プラグインの処理で意図しない状況が発生しました。', E_USER_WARNING);
+            return;
+        }
+
         $objPlugin->doAction($hook_point, $arrArgs);
     }

branches/version-2_13-dev/data/class/pages/error/LC_Page_Error.php

@@ -57 +57 @@
     public function init()
     {
+        SC_Helper_HandleError_Ex::$under_error_handling = true;
+
         $this->tpl_mainpage = 'error.tpl';
         $this->tpl_title = 'エラー';

branches/version-2_13-dev/data/class/pages/error/LC_Page_Error_DispError.php

@@ -42 +42 @@
     public function init()
     {
+        SC_Helper_HandleError_Ex::$under_error_handling = true;
+
         $this->template = LOGIN_FRAME;
         $this->tpl_mainpage = 'login_error.tpl';