Problem/Motivation

If the API calls to the Digital Commerce API fail, the module catches the error properly, but there is no information about which order caused the error. This would be especially useful when using the action in batches. Currently the user has to guess which order failed and is unable to use the batch functionality.

Currently #3216580: Issues if house number is in address line 2 is causing an error like this.

Steps to reproduce

Proposed resolution

Display the order number/id in the error message.

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Lukas von Blarer created an issue. See original summary.

Berdir’s picture

Title: Display information about which order casuses an error » Display information about which order causes an error
Status: Active » Needs review
FileSize
6.45 KB

This adds the shipment and order ID to the exception messages, also added a basic test to the exception handling.

Berdir’s picture

Status: Needs review » Fixed

Committing this so I can build on the test changes.

  • Berdir committed 02986b0 on 1.0.x
    Issue #3227082 by Berdir: Display information about which order causes...

Status: Fixed » Closed (fixed)

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