FIX Fixes #814 Renamed 'SubmittedFormEmailHtml' to 'SubmittedFormEmail'

following peer feedback.  Updated respective code and unit test
This commit is contained in:
Luke Amos 2019-01-17 10:30:05 +13:00
parent 2cb649abbb
commit 1efeeca6db
3 changed files with 3 additions and 3 deletions

View File

@ -290,7 +290,7 @@ JS
if ($recipients = $this->FilteredEmailRecipients($data, $form)) { if ($recipients = $this->FilteredEmailRecipients($data, $form)) {
foreach ($recipients as $recipient) { foreach ($recipients as $recipient) {
$email = Email::create() $email = Email::create()
->setHTMLTemplate('email/SubmittedFormEmailHtml') ->setHTMLTemplate('email/SubmittedFormEmail')
->setPlainTemplate('email/SubmittedFormEmailPlain'); ->setPlainTemplate('email/SubmittedFormEmailPlain');
// Merge fields are used for CMS authors to reference specific form fields in email content // Merge fields are used for CMS authors to reference specific form fields in email content

View File

@ -175,8 +175,8 @@ class UserDefinedFormTest extends FunctionalTest
$result = $recipient->getEmailTemplateDropdownValues(); $result = $recipient->getEmailTemplateDropdownValues();
// Installation path can be as a project when testing in Travis, so check partial match // Installation path can be as a project when testing in Travis, so check partial match
$this->assertContains('email' . DIRECTORY_SEPARATOR . 'SubmittedFormEmailHtml', key($result)); $this->assertContains('email' . DIRECTORY_SEPARATOR . 'SubmittedFormEmail', key($result));
$this->assertSame('SubmittedFormEmailHtml', current($result)); $this->assertSame('SubmittedFormEmail', current($result));
} }
public function testEmailTemplateExists() public function testEmailTemplateExists()