|
In-Portal Issue Tracker - In-Commerce
|
|
Viewing Issue Advanced Details |
|
|
ID:
|
Category:
|
Type:
|
Reproducibility:
|
Date Submitted:
|
Last Update:
|
|
660 |
[In-Portal CMS] Front End |
bug report |
always |
2010-03-27 17:22 |
2011-03-30 09:58 |
|
|
Reporter:
|
alex |
Platform:
|
|
|
|
Assigned To:
|
alex |
OS:
|
|
|
|
Priority:
|
normal |
OS Version:
|
|
|
|
Status:
|
closed |
Product Version:
|
5.0.3-B1 |
|
|
Product Build:
|
|
Resolution:
|
fixed |
|
| |
|
ETA:
|
none |
Fixed in Version:
|
5.1.1-B1 |
|
|
Reference:
|
http://groups.google.com/group/in-portal-bugs/browse_thread/thread/9d25bc78e14b98b0 |
|
Change Log Message:
|
|
|
Estimate Points:
|
0 |
|
|
Summary:
|
0000660: Order's user could reset to Guest during payment gateway processing |
|
Description:
|
Order's user could reset to Guest during payment gateway processing. This can happen because of check made in OrdersEventHandler::CheckUser method. That method checks, that user, who is currently being logged in is also mentioned as order's owner. In case if they doesn't match, then currency logged in user is placed in order. This raises a problem with "stupid" payment gateways, who doesn't understand dynamic callback url.
For example we have same installation (one copy of code, one database, multiple DNS records only) running on multiple domains, but payment processing always happens on central domain. In this case CheckUser method will reset owner of order in case order wasn't initially created on domain used for payment processing.
I propose to check for GW_NOTIFY constant (proposed to be added in another discussion) and not to perform any such checks when payment gateway notification is being processed. |
|
Steps To Reproduce:
|
|
|
Additional Information:
|
|
| Relationships | |
|
Attached Files:
|
 user_lost_during_payment_gw_processing.patch (549) 2010-09-08 09:03
http://tracker.in-portal.org/file_download.php?file_id=749&type=bug |
|
|
Issue History |
|
Date Modified |
Username |
Field |
Change |
|
2011-03-30 09:58 |
Dmitry |
Note Added: 0003390 |
|
|
2011-03-30 09:58 |
Dmitry |
Status |
resolved => closed |
|
2010-09-10 03:03 |
alex |
Note Added: 0002804 |
|
|
2010-09-10 03:03 |
alex |
Status |
reviewed and tested => resolved |
|
2010-09-10 03:03 |
alex |
Fixed in Version |
=> 5.1.1-B1 |
|
2010-09-10 03:03 |
alex |
Resolution |
open => fixed |
|
2010-09-10 03:03 |
alex |
Changeset attached |
5.1.x r13930 |
|
2010-09-08 23:26 |
Dmitry |
Note Added: 0002793 |
|
|
2010-09-08 23:26 |
Dmitry |
Assigned To |
Dmitry => alex |
|
2010-09-08 23:26 |
Dmitry |
Status |
needs testing => reviewed and tested |
|
2010-09-08 09:03 |
alex |
Time Estimate Removed |
1 => |
|
2010-09-08 09:03 |
alex |
Assigned To |
alex => Dmitry |
|
2010-09-08 09:03 |
alex |
Developer |
=> alex |
|
2010-09-08 09:03 |
alex |
Status |
needs work => needs testing |
|
2010-09-08 09:03 |
alex |
File Added: user_lost_during_payment_gw_processing.patch |
|
|
2010-08-27 13:16 |
alex |
Time Estimate Added |
1 |
|
2010-07-31 13:34 |
Dmitry |
Fixed in Version |
5.1.1 => |
|
2010-07-31 13:34 |
Dmitry |
Target Version |
Icebox => 5.1.1 |
|
2010-07-31 13:33 |
Dmitry |
Assigned To |
=> alex |
|
2010-07-31 13:33 |
Dmitry |
Status |
active => needs work |
|
2010-07-31 13:33 |
Dmitry |
Fixed in Version |
=> 5.1.1 |
|
2010-05-10 14:51 |
alex |
Target Version |
5.0.4 => Icebox |
|
2010-03-27 17:22 |
alex |
New Issue |
|
|
2010-03-27 17:22 |
alex |
Reference |
=> http://groups.google.com/group/in-portal-bugs/browse_thread/thread/9d25bc78e14b98b0 |