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

Opened 7 years ago

Last modified 7 years ago

フックポイントLC_Page_Products_Detail_action_add_favoriteの呼び出し間違い

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

Description

LC_Page_Products_Detail_action_add_favoriteに関するフックポイントの呼び出し条件が間違っている

ローカルフック
LC_Page_Products_Detail_action_add_favorite
LC_Page_Products_Detail_action_add_favorite_sphone
を利用する際に記載されている箇所
LC_Page_Products_Detail.phpですが
518行目と540行目の条件が異なっております。
一度ご確認いただけますでしょうか

Change History

comment:1 Changed 7 years ago by dkazuto

確認しましたが、動作に問題はないと思います。 寧ろ、PC側の処理の「!」を外すと画面がホワイトアウトするのを確認しました。

comment:2 follow-up: ↓ 3 Changed 7 years ago by h_yoshimoto

すみません、少し書き方がまずかったです。
フックポイントの記述箇所が間違っているという不具合になります。

お気に入りに追加した場合
現状
お気に入り追加(正常処理):フックポイントは実行されない
エラー発生:フックポイント(LC_Page_Products_Detail_action_add_favorite)が実行される

正しくは
お気に入り追加(正常処理):フックポイント(LC_Page_Products_Detail_action_add_favorite)が実行される
エラー発生:フックポイントは実行されない

comment:3 in reply to: ↑ 2 Changed 7 years ago by dkazuto

  • 修正済み set

こちら、 LC_Page_Products_Detail.php を閲覧したところ、既に修正済みのようです。

ご確認をお願いいたします。

comment:4 Changed 7 years ago by dkazuto

  • Owner changed from h_yoshimoto to dkazuto
  • Status changed from new to assigned

comment:5 Changed 7 years ago by tokuhiro

  • Owner changed from dkazuto to tokuhiro
  • Status changed from assigned to new

comment:6 Changed 7 years ago by tokuhiro

  • Status changed from new to assigned

comment:7 Changed 7 years ago by tokuhiro

r22972 にてcommit

comment:8 Changed 7 years ago by h_yoshimoto

r23159 にて条件を修正

comment:9 Changed 7 years ago by h_yoshimoto

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