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

Opened 8 years ago

Last modified 7 years ago

SC_Utils_Ex::sfGetProductClassIdで、$classcategory_id1 と $classcategory_id2 が使用されていない。

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

Description

コード内に既に記述があるTODOです。

CSVからの商品規格レコード更新時に、このメソッドを利用するようになったので( r22301 / #2027 )、機能的に不足している可能性があります。

Change History

comment:1 Changed 8 years ago by undertree

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

comment:2 Changed 8 years ago by kim

  • Milestone set to EC-CUBE2.12.4

マイルストーンが設定されていなかったため、ひとまず2.12.4とします。

comment:3 Changed 8 years ago by undertree

  • 修正済み set

r22446 としてコミットいたしました。

comment:4 Changed 8 years ago by kim

  • 修正済み unset

undertree様
修正ありがとうございます。
こちらのコミットによる仕様変更で、以下のテストケースが通らなくなっています。
可能であれば、tests/以下のテストケースのメンテナンスもお願いできないでしょうか。
▽該当のテストケース
SC_Utils_sfGetProductClassIdTest.testSfGetProductClassId_存在するIDを指定した場合_対応する製品クラスが取得できる

comment:5 Changed 8 years ago by undertree

  • 修正済み set

テストの修正漏れ失礼致しました。

テストの修正と追加を行い、 r22532 としてコミットいたしました。 よろしくお願い致します。

comment:6 Changed 8 years ago by kim

ありがとうございます。
テストケースが成功していることを確認いたしました。

comment:7 Changed 7 years ago by h_yoshimoto

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

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

Note: See TracTickets for help on using tickets.