LC_Page_Admin.php でのローカルフックポイント実行処理での条件分岐が曖昧

[nanasess]

何を想定して doAction() を2回コールしているか不明。

ec-cube2/data/class/pages/admin/LC_Page_Admin.php

Lines 179 to 185 in 2df1730

	// ローカルフックポイントを実行
	$parent_class_name = get_parent_class($this);
	$objPlugin->doAction($parent_class_name.'_action_before', [$this]);
	$class_name = static::class;
	if ($class_name != $parent_class_name) {
	$objPlugin->doAction($class_name.'_action_before', [$this]);
	}

以下が正しい？

        // ローカルフックポイントを実行
        $class_name = static::class;
        if (str_ends_with($class_name, '_Ex')) {
            $parent_class_name = get_parent_class($this);
            $objPlugin->doAction($parent_class_name.'_action_before', [$this]);
        } else {
            $objPlugin->doAction($class_name.'_action_before', [$this]);
        }

refs #1213

[nanasess]

以下のようなプラグインを Playwright でインストールして各ページの動作をチェックするのがよさげ。
オリジナルのコードだと _Ex クラスが存在しない場合、 LC_Page_Admin_Order_action_before と LC_Page_Admin_Order_Edit_action_before が多重実行されてしまう？

<?php
// plugin_info.php
class plugin_info
{
    public static $PLUGIN_CODE = 'HookPointCheckerPlugin';
    public static $PLUGIN_NAME = 'HookPointCheckerPlugin';
    public static $CLASS_NAME = 'HookPointCheckerPlugin';
    public static $PLUGIN_VERSION = '0.0.0';
    public static $COMPLIANT_VERSION = '2.25';
    public static $AUTHOR = 'dummy';
    public static $DESCRIPTION = 'dummy';
    public static $HOOK_POINTS = [
        ['LC_Page_Admin_Order_Edit_action_before', 'actionBefore'],
        ['LC_Page_Admin_Order_Edit_action_after', 'actionAfter'],
    ];
}

<?php
// HookPointCheckerPlugin.php
class HookPointCheckerPlugin extends SC_Plugin_Base
{
    public function actionBefore(LC_Page_Ex $objectPage)
    {
        if (isset($objectPage->tpl_onload)) {
            $objectPage->tpl_onload .= '$("body").addClass("call-action-before");';
        } else {
            $objectPage->tpl_onload = '$("body").addClass("call-action-before");';
        }
    }

    public function actionAfter(LC_Page_Ex $objectPage)
    {
        if (isset($objectPage->tpl_onload)) {
            $objectPage->tpl_onload .= '$("body").addClass("call-action-after");';
        } else {
            $objectPage->tpl_onload = '$("body").addClass("call-action-after");';
        }
    }
}