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

Opened 11 years ago

Last modified 11 years ago

仮登録メール再送の不具合

Reported by: shutta Owned by: nanasess
Priority: Milestone: EC-CUBE2.4.4
Component: 管理画面 Version: 2.4.3
Keywords: Cc:
修正済み:

Description

管理画面の顧客管理から仮登録メールを再送した際に、 全てPC用の仮登録メールが再送されてしまう。 なので、携帯サイトから仮登録されたお客様に再送した場合に、 PCサイト用のURLを記述したメールを送ってしまいトラブルになります。

Change History

comment:1 Changed 11 years ago by shutta

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

r18591(2.4-dev),r18592(コミュニティ版。修正は、r18591 と同一の内容) をコミット。

携帯メールアドレスが登録されていれば、携帯サイトからの登録とみなし、その場合には携帯用のテンプレートを呼ぶように修正した。

かなり簡易的な判定なので、将来的には直した方が良いが、現状の実装でデフォルトのまま使う分には、多くの場合この方法で解決するはず。

comment:2 follow-up: ↓ 3 Changed 11 years ago by Seasoft

コミュニティ版のソース査読で気になった点の報告です。

SC_SiteView の第一引数 false を指定した方が良いかも。セッションデータ(前のURL情報)が書き換えられる予感がしますので。(動作未確認)

comment:3 in reply to: ↑ 2 Changed 11 years ago by shutta

SC_SiteView の第一引数 false を指定した方が良いかも。セッションデータ(前のURL情報)が書き換えられる予感がしますので。(動作未確認)

ご指摘有難うございます。

元のソースが何もセットしていなかったのでそのまま流用していたのですが、 確かにfalseを渡しておいた方が安全なような気がしますね。

r18597, r18598 としてコミットしておきました。

また、SC_MobileViewもSC_SiteViewと同じ引数が渡せるべきだと思うので、 r18595, r18596 をコミットしておきました。

comment:4 follow-up: ↓ 5 Changed 11 years ago by ghana

携帯メールアドレスに仮確認メールを送信する際 本文の送信先の会員名が表示されないバグの修正

r18599 としてコミットしました

comment:5 in reply to: ↑ 4 Changed 11 years ago by shutta

r18599 を、2_4-devにマージしました。(r18600)

comment:6 Changed 11 years ago by nanasess

  • Owner changed from shutta to nanasess
  • Status changed from assigned to new
  • Milestone set to EC-CUBE2.4.4

comment:7 Changed 11 years ago by nanasess

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

version-2_4 r18706 でマージしました

Note: See TracTickets for help on using tickets.