-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
完了画面での JavaScript によるトラッキングが多重に行われる #107
Comments
@seasoftjapan nocacheがあることで逆にリロードなどの挙動の際に複数形状される可能性があるのではないでしょうか。 |
私もそう考えていましたが、実際に検証したところ、なりました。金額・数量が2倍・3倍と増加する事象を確認済みです。
最終的にはコンバージョンの JavaScript として何を送出するか次第ですが、EC-CUBE の注文番号のセッション変数は1回目でクリアされています。その場合に、eコマース計上されないように、カスタマイズするのは自然では無いでしょうか。 |
nocacheの弊害が無いか確認したい |
当該画面は、GET で表示し、且つ、何度もロードできるので、支障ないと考える。あえて、重隅するなら、アクセスログには計上されやすくなるため、(本件で想定している JavaScript ではなく) WEB サーバーのログベースで追跡する場合にはノイズが増えるかもしれない。 また、類似した画面である LC_Page_Entry_Complete は、現況で |
異論が他の人から出なければnocacheで行きましょうか |
[fix] refs #107 完了画面での JavaScript によるトラッキングが多重に行われる
下記でも
$this->httpCacheControl('nocache');
を行うべき。$tpl_order_id
で分岐することで、重複計上を回避できるようになる。$tpl_customer_id
で分岐することで、重複計上を回避できるようになる。会員登録は、会員IDの連携が不完全だった。少なくともデフォルトでは完了画面に引き継がれない。この不具合も同時に改修した(f73a268)。値渡しを、URLの問い合わせ文字列から、セッション変数に変更する。これにより、この値の信頼性は高まると期待するが、本当に頼って大丈夫かは未検証である。
The text was updated successfully, but these errors were encountered: