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

Opened 12 years ago

Last modified 12 years ago

重複申込チェックのついたキャンペーンに申し込みできなくなる不具合

Reported by: kajiwara Owned by: somebody
Priority: Milestone: EC-CUBE2.4.3
Component: フロント Version: 2.4.1
Keywords: Cc:
修正済み:

Description

開発コミュニティ kaorinstar様からのご報告

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?forum=10&post_id=19399&topic_id=1049#forumpost19399

・バグ:
一度、キャンペーンに申し込みすると、以後、重複申込チェックのついたキャンペーンに申し込みできなくなる。

・原因:
重複申込チェックの際に、キャンペーンIDのチェックしていないため、すべてのキャンペーンがチェック対象となっている。

Attachments

LC_Page_CampaignApplication.php Download (9.5 KB) - added by kajiwara 12 years ago.
Ver2.4.2からの修正済ファイルを添付します。

Change History

comment:1 Changed 12 years ago by kajiwara

以下のファイル2点を修正すれば正常に動作すると思います。

/data/class/pages/campaign/LC_Page_CampaignApplication.php

前)
122 if($this->lfOverlappingCheck($objCustomer->getValue('customer_id'), $objQuery)) {
後)
122  if($this->lfOverlappingCheck($objCustomer->getValue('customer_id'), $objCampaignSess->getCampaignId(), $objQuery)) {

前)
255 function lfOverlappingCheck($customer_id, &$objQuery) {
256 $count = $objQuery->count("dtb_campaign_order", "customer_id = ?", array($customer_id));
後)
255 function lfOverlappingCheck($customer_id, $campaign_id, &$objQuery) {
256 $count = $objQuery->count("dtb_campaign_order", "customer_id = ? AND campaign_id = ?", array($customer_id, $campaign_id));

こちらに反映済み。
 http://svn.ec-cube.net/sandbox_demo/version-2_4/html/

ご確認お願いします。

Changed 12 years ago by kajiwara

Ver2.4.2からの修正済ファイルを添付します。

comment:2 Changed 12 years ago by kajiwara

デベロッパー版 にて r18537 で対応いたしました。 

comment:3 Changed 12 years ago by kajiwara

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

プレビュー版にて r18541 でマージを行いました。

Note: See TracTickets for help on using tickets.