Custom fields in contact mails are rendered with HTML#40557
Custom fields in contact mails are rendered with HTML#40557obuisard merged 8 commits intojoomla:4.3-devfrom
Conversation
|
Yes it fixes, thanks for the hint, updated. |
|
We should probably remove the lines, where do we delete these files in the script..? |
@laoneo I leave it to you if you do that with this PR or if I do it with my PR for script.php, but if I do it and the merge of my PR takes longer, the functionality might be broken when people update with nightly builds, so maybe you should fix that here. |
|
Thanks for the hint @Kostelano ! |
|
@laoneo It needs to remove also the deleted folders in script.php here: joomla-cms/administrator/components/com_admin/script.php Lines 7923 to 7926 in 3f468ad |
|
@richard67 thanks |
|
@laoneo I've just tested, also with the testing instructions from PR #37716 . What I've noticed is that when using a subform field in the mail context, i.e. the contact form, the email still contains some HTML markup, just less as before. I don't know how that's been in 4.2.8 or older before that PR had been merged, I haven't tested that yet with 4.2.8. Besides this, everything works as expected, i.e. the contact fields and the article fields look the same, and when using single fields and not subforms for the contact form, there is no HTML markup in the email. |
|
It is probably the same on 4.2.8. All this pr does is to remove the markup around the field. But the subform probably adds some more markup to align the fields. So this should be fixed in another pr. |
|
@laoneo I've just tested with 4.2.8 and there I get an error when using a subform in the contact form, so your PR doesn't break anything. |
|
I have tested this item ✅ successfully on a056f90 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40557. |
|
Testet, the Fix works... |
@Uwe-MrT Could you mark your test result in the issue tracker? Just go to https://issues.joomla.org/tracker/joomla-cms/40557 and use the blue "Test this" button at the top left corner, then select your test result (success) and submit. Thanks in advance. |
|
I have tested this item ✅ successfully on a056f90 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40557. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40557. |
|
Thanks Allon @laoneo for the PR and Richard @richard67 for looking into it! |
Pull Request for Issue #40338.
Summary of Changes
This pr restores the behavior that custom fields in mails are rendered without HTML code. It was like that forever but the pr #37716 broke it.
Testing Instructions
You can also do the testing instructions from #37716 to ensure that the contact data is rendered like an article.
Actual result BEFORE applying this Pull Request
The custom field is rendered like on articles. The mail which got sent to the contact contains some HTML output.
Expected result AFTER applying this Pull Request
The custom field is rendered like on articles. The mail which got sent to the contact contains no HTML output.
Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed