Ticket #568 (closed バグ指摘: 修正済)

Opened 11 years ago

Last modified 10 years ago

LC_Page_Admin_Products_Product#lfGetAddSuffix 不自然な default

Reported by: Seasoft Owned by: somebody
Priority: Milestone: EC-CUBE2.4.2
Component: 管理画面 Version: 2.4.1
Keywords: Cc:
修正済み:

Description

        switch($to_key){
            case "main_list_image":
                $dist_name = '_s';
                break;
            case "main_image":
                $dist_name = '_m';
                break;
            default;
                $arrRet = explode('sub_image', $to_key);
                $dist_name = '_sub' .$arrRet[1];
                break;
        }

下記の誤りと思われる。仮に、現実装が正しいなら、非常に紛らわしいので、コメントを付すべき。

【誤】default;
【正】default:

Change History

comment:1 Changed 11 years ago by kajiwara

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

Seasoft様 本件ご報告いただき、ありがとうございます。
毎度、チェックの正確性には感服します。

本件、特に意図はなく誤りだと思われますので、r18411 にて修正いたしました。

comment:2 Changed 10 years ago by Seasoft

コミュニティ版は r18463 でマージしました。

Note: See TracTickets for help on using tickets.