Hello,

I've tried this module and I think it is not working... when user is redirected back to the site after payment, although the payment was done, the module is not catching Sermepa response successfully.
The system throws two errors after payment completion and order can't be never completed.

path payment/notify/credit_card:

Error: Call to a member function get() on null en Drupal\commerce_sermepa\Plugin\Commerce\PaymentGateway\Sermepa->processRequest() (line 340 of ...modules/contrib/commerce_sermepa/src/Plugin/Commerce/PaymentGateway/Sermepa.php)

path checkout/%/payment/return

Bad feedback response, missing feedback parameter.

Could somebody help me with this error?

Thanks,
Dani

CommentFileSizeAuthor
#11 commerce_sermepa-error-response-2948804-11-8.x.patch1.09 KBfacine
#7 commerce_sermepa-error-response-2948804-7.patch774 bytesfastangel
#6 commerce_sermepa-error-response-2948804-6.patch773 bytesfastangel

Comments

dromansab created an issue. See original summary.

dromansab’s picture

dromansab commented
Issue summary: View changes
dromansab’s picture

dromansab commented
Issue summary: View changes
dromansab’s picture

dromansab commented

Nobody using this module for Drupal 8?

carles.zapater’s picture

carles.zapater commented

We have solved the problem with this modification:

Sermepa.php
Line 340

if ($credit_card = $order->get('payment_method')->first()->get('entity')) {
if($credit_card = $order->get('payment_method')->entity) {

fastangel’s picture

fastangel commented
Status: Active » Needs review
StatusFileSize
new commerce_sermepa-error-response-2948804-6.patch773 bytes

Here a patch.

fastangel’s picture

fastangel commented
StatusFileSize
new commerce_sermepa-error-response-2948804-7.patch774 bytes

Sorry error in the patch.

jncruces’s picture

jncruces
Primary language Spanish
Location Sevilla
commented

The attached patch in #7 works great!

Thanks.

interdruper’s picture

interdruper commented
Status: Needs review » Reviewed & tested by the community

#7 RTBC IMHO, thanks!

trebormc’s picture

trebormc
Primary language Catalan
commented

The attached patch in #7 works great!

Thanks.

facine’s picture

facine commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new commerce_sermepa-error-response-2948804-11-8.x.patch1.09 KB

Thanks to all for the work!

I have implemented the patch making a small modification.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Reviewed & tested by the community

LGTM

facine’s picture

facine commented
Status: Reviewed & tested by the community » Fixed

Fixed!

  • facine committed 3cfdf29 on 8.x-2.x authored by fastangel 
    Issue #2948804 by fastangel, facine, carles.zapater, pcambra: Error with...
  • facine committed a4e947d on 8.x-2.x
    Issue #2948804 by fastangel, facine, carles.zapater, pcambra: Error with...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.