Page MenuHomePhabricator

500 error when testing Endowment thank you emails in Civi
Closed, ResolvedPublic

Description

Using the Civi interface to test sending a thank you email with Financial Type = Endowment Gift gives a 500 error and no email. The normal Financial Type = Cash works fine. This happens on both Production and Staging. Elliott confirmed he sees the same, but couldn't see details in the logs.

I tried making a test donation with utm_medium=endowment, and that did get me the right email.

This is blocking me from testing T232121: Update Endowment automatic thank you email

Event Timeline

Seems to die around line 587 of thank_you.module:

$civi_mailing = $civimail_store->getMailing('thank_you', $template_info->name, $template_info->revision);

So the test code probably shouldn't try to make a CiviMail record in the first place, but this could be a sign of a bigger problem. The CiviMailStore code is directly accessing core classes, and might be possible to rewrite using API access now.

Ejegg triaged this task as Medium priority.
Ejegg moved this task from Backlog to Doing on the Fundraising Sprint Sysadmin Kane board.

Change 537653 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/crm@master] Thank you test fixes

https://gerrit.wikimedia.org/r/537653

Patch is up on staging, and we are able to test the endowment email.

Change 537653 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Thank you test fixes

https://gerrit.wikimedia.org/r/537653