View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007376 | module PayPal Checkout | module PayPal checkout - sub | public | 2022-11-02 18:19 | 2022-12-14 14:30 |
| Reporter | thorsten.schneider | Assigned To | |||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 2.2.2 / 1.2.2 | ||||
| Fixed in Version | 2.2.3 / 1.2.3 | ||||
| Summary | 0007376: _executePayment(...) breaks with other payment module | ||||
| Description | Using Paypal checkout together with tronet sofortüberweisung breaks the checkout as there is an inconsistency in the function call: [message Declaration of Tronet\Trosofortueberweisung\Application\Model\TrosofortueberweisungOrder::_executePayment(OxidEsales\Eshop\Application\Model\Basket $oBasket, $oUserPayment) must be compatible with OxidSolutionCatalysts\PayPal\Model\Order::_executePayment($basket, $userpayment)] IMHO is the function declaration in the PayPal Module not correct as the base function out of the shop framework and the declaration in the tronet module are equal. Adding "OxidEsales\Eshop\Application\Model\Basket" to the function parameter $basket fixes the problem. | ||||
| Steps To Reproduce | - Install Shop - Install and activate Paypal checkout and Tronet sofortüberweisung - Make an order -> shop in maintenance mode + error in log file | ||||
| Additional Information | In our case it was sufficient to add the declaration as stated in the description. As this is the same as in the base Order Model Class it seems to me that this is just missing in the Paypal Module. | ||||
| Tags | Payment | ||||
