When I view the transaction list, each transaction has a transaction received drop-down list that allows one to print invoice or cancel transaction etc. Whenever I select cancel transaction, I get the following error message:

Mail template for ec_mail_cancel_txn could not be found!

I am still apparently able to fill out the mail form and send the mail, but then when I am returned to the transaction list, it says my transaction is still pending.

So, how do I

a) create that imssing mail template?
b) actually cancel and remove a transaction?

CommentFileSizeAuthor
#6 store.module_4.patch494 bytesghing

Comments

criznach’s picture

criznach commented

Did you upgrade from an ecommerce beta?

I had the same problem on an upgraded ecommerce install on 4.7. The variables that map ec_mail message ids to message types were not set correctly. So I either the update script didn't run properly, or it didn't do what it was supposed to. I'm going to do a clean install of drupal 4.7.6 with ecommerce 3 and look at the ec_mail and variables tables to see how they should look.

criznach’s picture

criznach commented

Also on 4.7, after patching up the variable table to fix the above error, the transaction is still not cancelled. It says it was, and the message gets sent, but nothing else seems to happen.

criznach’s picture

criznach commented

You can delete the transaction by editing it then clicking the delete button at the bottom of the page. Stepping through the code, this takes a completely different path than cancelling it. So maybe they're meant to be two different features?

cpelham’s picture

cpelham commented

Ah, yes I am able to edit the transaction and then click the delete button at the bottom and get rid of it.
I am using Drupal 5.1 and not 4.7.6 as I believe you said you are, but I did earlier install and get rid of eCommerce beta before installing the current 5..x-3.0 version, and i had a really hard time with the install, so that may be part of the problem with that missing mail template. The installer wouldn't seem to run and I had to manually install through phpMyAdmin. maybe i should wipe the whole thing out and try again?

criznach’s picture

criznach commented

If we're talking about the same issue, you might be able to fix the template not found errors by copying the ec_mail table from a clean install and patching the message id variables in the variable table to match these records. I'm not a druapl 5 expert or an ecommerce module developer, but it's working for me so far.

ghing’s picture

ghing commented
Component: ec_mail » store
StatusFileSize
new store.module_4.patch494 bytes

This actually appears to be a bug in the store.module implementation. I've attached a patch to the 4.7-3.0 version of the ecommerce module, but looking at the patch, you should see how to fix the Drupal 5 version. If anyone has questions about the cause of the bug and how the patch works, you can email me at geoff [at] terrorware [dot] com.

jp.stacey’s picture

jp.stacey commented

ghing, I've emailed you about a possible Drupal 5 patch separately as it still seems to exhibit a partial bug (might be my bad). If anyone can test the v4 patch here - both with and without "Notify customer" checkbox checked - then that'd be great.

I'm also finding that "shipping notify" is doing nothing: I can't shift any of my transactions from "transaction received", except to cancel them using my buggy 5.1 patch. I'd never get that workaround past the accountants :) Anyone else having that problem? Could it have the same cause as this bug?

brmassa’s picture

brmassa commented
Status: Active » Reviewed & tested by the community

i confirm the bug and patch works.

for drupal 5.0, the same solution. (but on a different line)

regards,

massa

tknospdr’s picture

tknospdr commented

So, can someone post a patch that works for 5? Or maybe just update the official v3 with all the recently found and fixed bugs. It hasn't been updated since May 10th now and I know lots of behind the scenes work has gone into it.

Thanks,
David
http://www.FloridaPets.org

aymerick’s picture

aymerick commented

Yes, this patch fixes the cancelation issue.

This should be commited.

brmassa’s picture

brmassa commented
Status: Reviewed & tested by the community » Patch (to be ported)

Guys,

bug fixed on eC4. eC still needs the patch.

regards,

massa

gordon’s picture

gordon commented
Status: Patch (to be ported) » Fixed

Ported to 5.x-3.x-dev and 4.7.x-3.x-dev

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)