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

Opened 8 years ago

Last modified 8 years ago

別お届け先関連のロジックをページクラスから分離

Reported by: pineray Owned by: pineray
Priority: Milestone: EC-CUBE2.12.3
Component: フロント Version: 2.12.2
Keywords: Cc:
修正済み: yes

Description

会員登録住所以外のお届け先に関するロジックは、ページクラスに直接書かれており、複数の箇所で同じような記述が散見されるため、これをページクラスから分離してまとめる。

Change History

comment:1 Changed 8 years ago by pineray

  • Owner changed from somebody to pineray
  • 修正済み set
  • Type changed from バグ指摘 to 改善提案

r22065 にて実施。

comment:2 Changed 8 years ago by pineray

  • Status changed from new to assigned

r22069 r22073 にて変更を加えています。

comment:3 Changed 8 years ago by h_yoshimoto

関数名についてですが、今後関数が増える事を想定すると
もう少し具体的な役割がわかる形の方が良いかと思います。

修正前のregistAddressの様な形でも
問題ないかと思うのですが如何でしょうか。

comment:4 Changed 8 years ago by kim

pineray様

h_yoshimotoがコメントしております、関数名について修正をお願いできますでしょうか。

メモとして追記 r22075 r22079 でも関連コミット有り

comment:5 Changed 8 years ago by h_yoshimoto

r22497 にて関数名を修正

comment:6 Changed 8 years ago by h_yoshimoto

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

ご対応ありがとうございました。クローズさせて頂きます。

comment:7 Changed 8 years ago by h_yoshimoto

r22497 の内容をdevへマージ(r22546)

Note: See TracTickets for help on using tickets.