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

Opened 10 years ago

Last modified 9 years ago

LC_Page_Admin_Products_Category をカスタマイズしやすくする

Reported by: Seasoft Owned by: Seasoft
Priority: Milestone: EC-CUBE2.12.0
Component: 管理画面 Version: 2.11.1
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

  • 既存の category_name 以外に項目を追加しようとすると、大変過ぎる。
    • → 項目に依存する記述を極力抑えた。これで、ロジックに関しては initParam に1行を追加するのみで、項目追加を実現できると思う。
  • 登録(INSERT)・編集(UPDATE)のデータ生成が別々となっており、冗長な記述が必要となる。
    • → action メソッドからの呼び先を1メソッドに統合した。
    • → エラーチェックは(独立した)1メソッドに統合した。
    • → 登録・編集処理を行うメソッドは、各々で固有の処理が大半を占めるため、統合しなかった。
  • 独自のエラーチェックを挟む隙が (無いわけではないが) 狭い。
    • → エラーチェックメソッドを設けた。

r21356 r21357

Change History

comment:1 follow-up: ↓ 3 Changed 9 years ago by kajiwara

Seasoftさん

ご提案ありがとうございます。
案の全てに対応する場合は、マイナーバージョンUPでは対応できないかもしれませんね。

ただ、そこそこクリティカルな部分でもあると思うので、ソース凍結期限あたりでもう一度確認させていただき、具体的な方法が見えなければ、マイルストーンの変更をしたいと思います。

comment:2 Changed 9 years ago by kajiwara

  • Milestone changed from EC-CUBE2.11.2 to EC-CUBE2.11.3(仮)

スミマセン。具体的解決策が現状見えないままですので、一旦次期開発に移行します。

comment:3 in reply to: ↑ 1 ; follow-up: ↓ 5 Changed 9 years ago by Seasoft

案の全てに対応する場合は、マイナーバージョンUPでは対応できないかもしれませんね。

具体的には、どの部分が「対応できないかも」と感じましたか?

comment:4 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.3 to EC-CUBE2.11.4(仮)

comment:5 in reply to: ↑ 3 Changed 9 years ago by Seasoft

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

特段のコメントが付きませんでしたので進めます。

comment:6 Changed 9 years ago by Seasoft

  • 修正済み set
  • Description modified (diff)

comment:7 follow-up: ↓ 8 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.5 to EC-CUBE2.12.0alpha

2_11-devブランチのコミットキャンセルが必要

comment:8 in reply to: ↑ 7 Changed 9 years ago by Seasoft

kotani への返信

2_11-devブランチのコミットキャンセルが必要

r21405 で対応しました。

comment:9 Changed 9 years ago by kajiwara

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.