tonyair
/
silverstripe-userforms
mirror of https://github.com/silverstripe/silverstripe-userforms.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge pull request #258 from xini/fix-plain-text-email-format 

place colon after field name for better usability of plain text emails
This commit is contained in:
Will Rossiter 2015-04-28 17:26:19 +12:00
parent 7badef5c35 d9dde81657
commit 15f1df242b
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
code/model/UserDefinedForm.php
View File

@ -1036,7 +1036,7 @@ JS
$body = strip_tags($recipient->EmailBody) . "\n";
if(isset($emailData['Fields']) && !$recipient->HideFormData) {
foreach($emailData['Fields'] as $Field) {
$body .= $Field->Title .' - '. $Field->Value ." \n";
$body .= $Field->Title .': '. $Field->Value ." \n";
}
}

2
tests/UserDefinedFormControllerTest.php
View File

@ -41,7 +41,7 @@ class UserDefinedFormControllerTest extends FunctionalTest {
$this->assertEmailSent('nohtml@example.com', 'no-reply@example.com', 'Email Subject');
$nohtml = $this->findEmail('nohtml@example.com', 'no-reply@example.com', 'Email Subject');
$this->assertContains('Basic Text Field - Basic Value', $nohtml['content'], 'Email contains no html');
$this->assertContains('Basic Text Field: Basic Value', $nohtml['content'], 'Email contains no html');
// no data
$this->assertEmailSent('nodata@example.com', 'no-reply@example.com', 'Email Subject');