SC_Helper_Category が持つプロパティのアクセス権を緩やかにして欲しい

[snitta]

要望

SC_Helper_Category::$count_check が SC_Helper_Category_Ex から直接参照できず、既存メソッドをオーバーライドする際に不便なため改善を希望します。

改善案

アクセス修飾子を protected へ変更するのはいかがでしょうか？ パッチを添付しておきます。 attachment:change_visibility.patch

回避策

とりあえず今は SC_Helper_Category_Ex のコンストラクタで別途代入しています。

<?php

class SC_Helper_Category_Ex extends SC_Helper_Category
{
    private $count_check;

    public function __construct($count_check = false)
    {
        parent::__construct($count_check);
        $this->count_check = $count_check;
    }
}

参考

• source:branches/version-2_13-dev/data/class/helper/SC_Helper_Category.php@23566#L33

[snitta]

アクセス修飾子を変更するパッチ

[kim]

shutta様 ありがとうございます。 パッチ採用させていただきたく思いますので、2.13.3のブランチにコミットいただいてもよろしいでしょうか。 ソースは凍結しておりますが、軽微な内容のため、10/10までにご対応いただけるなら採用させていただきたく存じます。

[snitta]

r23637 で対応致しました。 source:branches/version-2_13_3 へのマージはまだ行っておりません。

こちらのクライアントが悪いのか、403 Forbidden と返って来てコミットできませんでした;; 代わりにマージ作業をお願いしてもよろしいでしょうか？ よろしくお願い申し上げます。

[kim]

shutta 様

ご対応ありがとうございます。

r23656 でマージいたしました。

リリースブランチには、権限を設定できていなかったようです。失礼いたしました。