Ticket #766 (assigned バグ指摘)

Opened 10 years ago

Last modified 7 years ago

二重に注文が入る不具合

Reported by: AMUAMU Owned by: AMUAMU
Priority: Milestone: バックログ
Component: フロント Version: 2.4.3
Keywords: Cc:
修正済み: no

Description (last modified by AMUAMU) (diff)

フォーラムにてpop2joa様の報告から、AMUAMUが調査。  http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=6189&forum=8&post_id=29437

携帯のみならずPCでも発生する恐れあり。

トランザクション処理内においてdtb_order_tempに対し明示的な行ロックをかけていない事が原因と思われる。 また、多重読み込み時の処理が無く、トランザクションの意味をあまり成していない。修正の方針はフォーラムに記載していますが手付かずです。

決済サービスモジュールによっては2重に決済がされる場合がある模様?

Change History

comment:1 Changed 10 years ago by AMUAMU

  • Description modified (diff)
  • Summary changed from 携帯で注文する時、二重注文が入ってしまうのは? to 二重に注文が入る不具合

comment:2 Changed 10 years ago by nanasess

  • Milestone set to EC-CUBE2.5.0

行ロックをかけたとしても, UPDATE が競合した状態になると, 一方のトランザクションが abort してしまい, 失注の懸念があります. DB まわりはしっかり見直したいので, milestone:EC-CUBE2.5.0 にしておきますね

comment:3 Changed 10 years ago by Seasoft

この辺りの処理は、コミュニティ版や2.5デベロッパー版では改訂済みですので、そもそも発現しなくなっているかもしれません。

まずは、再現するか否かを確認するのが先決かと思います。

comment:4 Changed 10 years ago by nanasess

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.1(仮)

comment:5 Changed 10 years ago by AMUAMU

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

comment:6 Changed 9 years ago by AMUAMU

ここ数日の大幅変更の範囲のコードを査読した限り、再発しそうに見えます。 変更点が多すぎて対策修正を掛けるのが難しい状態。

comment:7 Changed 9 years ago by kajiwara

  • 修正済み unset
  • Milestone changed from EC-CUBE2.11.1 to EC-CUBE2.11.2

引き続き、再現性の確認と必要であれば、修正作業の実施を行います。

マイルストーンを2.11.2にまわします。

comment:8 Changed 9 years ago by AMUAMU

擬似的に発生する可能性のあるようなパターンを作って確認したところ、 やはり再現する場合があるのですが、修正検討をする範囲が大きすぎて 手が回っていない状況です。GW中に出来ると良いなと・・・

comment:9 Changed 9 years ago by kajiwara

  • Milestone changed from EC-CUBE2.11.2 to EC-CUBE2.11.3(仮)

comment:10 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.3 to EC-CUBE2.11.4(仮)

comment:11 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.5 to EC-CUBE2.12.0alpha

comment:12 Changed 8 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.12.2 to EC-CUBE 2.12.3

comment:13 Changed 7 years ago by kim

  • Milestone changed from EC-CUBE2.12.3 to EC-CUBE2.12.4

AMUAMU様、こちら2.12系でも同様に発生しますでしょうか。 ひとまず、2.12.4へとマイルストーンを変更いたします。

comment:14 Changed 7 years ago by kim

  • Milestone changed from EC-CUBE2.13.0 to バックログ

2.13.0の対応内容を固めていくため、バックログに移動します。 ひとまず、2.12系での報告がないため、クローズする可能性があります。

Note: See TracTickets for help on using tickets.