View Issue Details

IDProjectCategoryView StatusLast Update
0002439OXID eShop (all versions)1.03. Basket, checkout processpublic2012-12-07 15:17
ReporterMoehlis 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.4.5 revision 31315 
Target VersionFixed in Version4.5.4 revision 39463 
Summary0002439: wrong oxuserpayment entry in checkout
DescriptionIf you change your payment data (dynvalues) and order again (so there will be two entries in oxuserpayments), the next checkout will use the outdated payment values for default dynvalues.

Bug lies in oxuserpayment.php:217 getPaymentByPaymentType EE 4.4.5 31315

Wrong:
$sSelect = 'select oxid from oxuserpayments where oxpaymentsid=' . oxDb::getDb()->quote( $sPaymentType ) . ' and oxuserid="' . $oUser->getId() . '" ';

Fix:
                $sSelect = 'select oxpaymentid
                            from oxorder
                            where oxuserid = "' . $oUser->getId() . '"
                                and oxpaymenttype = ' . oxDb::getDb()->quote( $sPaymentType ) . '
                            order by oxorderdate desc';



Bug also exists in the 4.5.0_beta1
TagsPayment
Theme
BrowserAll
PHP Versionany
Database Versionany

Relationships

related to 0003308 resolvedarvydas_vapsva Payment info is lost if user makes the second order with the same payment type 

Activities

birute_meilutyte

2011-05-05 17:05

reporter   ~0004487

@developers: check from source code side if such issue still exist

arvydas_vapsva

2011-09-09 16:41

reporter   ~0005197

the real issue here is that payment info is not updated, but written again and again for each order using same payment method.