Problem/Motivation

My taxes are a bit more dynamic than out of the box and thus need some help. Realized there is no order context in the transaction alter hook which makes it tricky to alter the transaction without re-loading the order from the ID.

Proposed resolution

Pass along the $order to the transaction alter.

Remaining tasks

None

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#1 pass_order_through_to-2529382-1.patch667 bytesjoelpittet

Comments

joelpittet’s picture

Status: Active » Needs review
StatusFileSize
new667 bytes

This should do the trick.

parisek’s picture

Thank you for the patch, I am currently on vacation. I will merge it later.

  • parisek committed 9a95b18 on 7.x-1.x authored by joelpittet
    Issue #2529382 by joelpittet: Pass $order through to the transaction...
parisek’s picture

Issue summary: View changes
Status: Needs review » Fixed

joelpittet: patch looks good, committed to 7.x-1.x, thank you, will make new release soon

joelpittet’s picture

Cool thanks @parisek

Status: Fixed » Closed (fixed)

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