Ticket #2590 (closed 改善提案: 修正済)

Opened 6 years ago

Last modified 6 years ago

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

Reported by: snitta Owned by: snitta
Priority: Milestone: EC-CUBE2.13.3
Component: その他 Version: 2.13.2
Keywords: Cc:
修正済み: no

Description (last modified by snitta) (diff)

要望

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

改善案

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

回避策

とりあえず今は 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;
    }
}

参考

Attachments

change_visibility.patch Download (391 bytes) - added by snitta 6 years ago.
アクセス修飾子を変更するパッチ

Change History

Changed 6 years ago by snitta

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

comment:1 Changed 6 years ago by snitta

  • Description modified (diff)

comment:2 Changed 6 years ago by kim

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

comment:3 Changed 6 years ago by snitta

  • Owner changed from somebody to snitta
  • Status changed from new to assigned

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

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

comment:4 Changed 6 years ago by kim

  • Status changed from assigned to closed
  • Resolution set to 修正済

shutta 様

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

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

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

Note: See TracTickets for help on using tickets.