Skip to content

Update ContactTable.class.php#10

Open
filip1 wants to merge 1 commit intowebvariants:masterfrom
filip1:master
Open

Update ContactTable.class.php#10
filip1 wants to merge 1 commit intowebvariants:masterfrom
filip1:master

Conversation

@filip1
Copy link
Copy Markdown

@filip1 filip1 commented Sep 17, 2018

Remove the two newlines added as part of the #PERSONAL-SALUTATION# keyword substitution as they lead to unexpected additional line breaks.

For example, I might have the following in the email template:

#PERSONAL-SALUTATION#

lorem ipsum dolor sit amet ...

Which I expect to get replaced by

Sehr geehrter Herr Max Mustermann,

lorem ipsum dolor sit amet ...

Instead, it currently gets replaced by

Sehr geehrter Herr Max Mustermann,



lorem ipsum dolor sit amet ...

Which means that in order to get a correct result, I currently have to write the following email template, which looks confusing to the user:

#PERSONAL-SALUTATION#lorem ipsum dolor sit amet ...

Remove the two newlines added as part of the #PERSONAL-SALUTATION# keyword substitution as they lead to unexpected additional line breaks
@mcnilz
Copy link
Copy Markdown
Contributor

mcnilz commented Dec 19, 2018

hmmm yes. But it fix could break existing actions. We need a database migration to add the removed newlines in the database.

mcnilz pushed a commit that referenced this pull request Mar 24, 2020
- remove sentry
- remove password-compat
- update dompdf
- update swiftmailer (update your factories.yml !)
- update htmlpurifier
- update php-markdown
- update paypal sdk
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants