updateOrderNumberOnPlace incorrectly attempts to run against non Klarna orders, although the try/catch statement includes ApiException | NonKlarnaOrderException every non Klarna order ends up with a Failed to update merchant references error in the log

CommentFileSizeAuthor
#2 updateOrderNumberOnPlace_attempts_to_run_against_non_Klarna_orders-3470754-2.patch657 bytesintrafusion

Comments

intrafusion created an issue. See original summary.

intrafusion’s picture

intrafusion
Primary language English
Location Edinburgh, UK
commented
Status: Active » Needs review
StatusFileSize
new updateOrderNumberOnPlace_attempts_to_run_against_non_Klarna_orders-3470754-2.patch657 bytes

Moved NonKlarnaOrderException into a separate try/catch block

  • tuutti authored 19f1c5f6 on 3.x 
    Issue #3470754 by intrafusion: updateOrderNumberOnPlace attempts to run...
tuutti’s picture

tuutti commented
Status: Needs review » Fixed

Thanks for the patch! This should be fixed in the latest release.

Status: Fixed » Closed (fixed)

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