Problem/Motivation

When

1) uc_bitcoin module is enabled
2) Checkout completed without paying by Bitcoin,

the email confirmation message contains unnecessary text:

-- Payment to be sent to address:

Proposed resolution

Alter code in uc_bitcoin_mail_alter to check if paying by Bitcoin.

Remaining tasks

Code patch.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 UnnecessaryEmailText-2513294-2.patch1004 bytespmagunia
#1 UnnecessaryEmailText-2513294-1.patch963 bytespmagunia
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pmagunia’s picture

pmagunia
Primary language English
Location Philadelphia 🇺🇸
CreditAttribution: pmagunia as a volunteer commented
Status: Active » Needs review
FileSize
UnnecessaryEmailText-2513294-1.patch963 bytes

Attaching patch for review.

pmagunia’s picture

pmagunia
Primary language English
Location Philadelphia 🇺🇸
CreditAttribution: pmagunia as a volunteer commented
FileSize
UnnecessaryEmailText-2513294-2.patch1004 bytes

When the order_status is 'completed', the message doesn't need to be displayed either.

nerdcore’s picture

nerdcore CreditAttribution: nerdcore as a volunteer commented
Status: Needs review » Reviewed & tested by the community

#2 works as advertised.

dgtlmoon’s picture

dgtlmoon CreditAttribution: dgtlmoon commented
Status: Reviewed & tested by the community » Fixed

Thanks!

  • dgtlmoon committed ccbf3db on 7.x-1.x authored by pmagunia 
    Issue #2513294 by pmagunia: Unnecessary email text when not paying by...

Status: Fixed » Closed (fixed)

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