The description of the item that comes back from authorize.net in emails currently just has the SKU. This isn't always good for customers. The attached patch adds a the product title - with the SKU in parentheses - to the description if we have one to use. Otherwise, it keeps the current format. Note that the else condition may not be necssary because of the call to commerce_product_line_item_types(), but I left it in just in case.

There may be a more robust way to handle this (a form settings using tokens, etc.), but this is a quick fix to at least make the information passed to authorize.net usable for customers.

Patch is built against dev, but should work on 1.0-rc1 also.

CommentFileSizeAuthor
commerce_authnet_simdpm_item_description.patch1 KBseanbfuller
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jerry’s picture

jerry CreditAttribution: jerry commented
Assigned: Unassigned » jerry
Status: Needs review » Postponed

Thanks for the submission. I'll consider adding this to a future release. The downside of doing so is that the description field has a maximum length of 255, so increasing the length of each item in the list also increases the likelihood that items will be truncated. A better solution might be to use one of the available Commerce modules to generate and E-mail an itemized invoice, rather than relying on Authorize.Net to do so.

jerry’s picture

jerry CreditAttribution: jerry commented
Status: Postponed » Closed (won't fix)

Upon reflection, I'm going to mark this "won't fix" based my my reasoning above. Your patch will remain here for those who prefer your solution.

seanbfuller’s picture

seanbfuller CreditAttribution: seanbfuller commented

Hey jerry,

Thanks for the response. I wasn't aware of the 255 character limit. Your alternate suggestion to use a commerce contrib module to generate the invoice email from Drupal instead makes more sense. If we get a chance to explore that, I'll report back here. Thanks again!