Hi,
In the commerce_paypal_wps.module file there are 2 lines of code that need to be rearranged.
if (!in_array(trim(strtolower($ipn['receiver_email'])), $receiver_emails)) {
commerce_payment_redirect_pane_previous_page($order);
watchdog('commerce_paypal_wps', 'IPN rejected: invalid receiver e-mail specified (@receiver_email); must match the primary e-mail address on the PayPal account.', array('@receiver_email' => $ipn['receiver_email']), WATCHDOG_NOTICE);
return FALSE;
}
Should be:
if (!in_array(trim(strtolower($ipn['receiver_email'])), $receiver_emails)) {
watchdog('commerce_paypal_wps', 'IPN rejected: invalid receiver e-mail specified (@receiver_email); must match the primary e-mail address on the PayPal account.', array('@receiver_email' => $ipn['receiver_email']), WATCHDOG_NOTICE);
commerce_payment_redirect_pane_previous_page($order);
return FALSE;
}
Otherwise this wont be logged and the log does not show where the problem relays.
I cannot create patches, so it would be nice if someone else does this.
Comments
Comment #1
harings_rob commentedComment #2
discipolo commentedsince you requested a patch so nicely, here it is
Comment #3
mglamanComment #4
anybodyMakes sense and patch works. RTBC!
Nothing will ever be logged after the redirect...
Comment #5
anybodyAny chance to commit this finally? Any active maintainer?
I guess it's clear that this makes sense as the log line can never be called!
Comment #7
anybodyKudos to @discipolo! Created a MR from his patch in #2
Comment #8
anybodyAny maintainer to review the MR and plan to merge this? I guess it's clear from the code logic that this makes a lot of sense and is safe to be committed?
Comment #9
grevil commentedMakes sense! RTBC+1
Comment #10
tomtech commentedAutomatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.