If a user figures out a way to get to the payment processing system without adding an address, the error message returned is the generic "you didn't pass me a parameter I require" message, but somehow less helpful:

Object reference not set to an instance of an object.

Since we're adding an address in a function that returns false anyway, why not kick out of the payment processing at that point, and display a more helpful error message?

Attaching a patch for that. The same patch applies to 7.3 and 7.2.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mariacha1 created an issue. See original summary.

mariacha1’s picture

Status: Active » Needs review

  • ruscoe committed 381be78 on 7.x-2.x authored by mariacha1
    Issue #2750339 by mariacha1: Provide better error handling if no address...
ruscoe’s picture

Status: Needs review » Fixed

Thank you!

Status: Fixed » Closed (fixed)

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