letter.tt did not pass the parameter 'lang' of 'is_html' to letter.pl, resulting in the failure to save the 'is_html' value of the corresponding language.
Created attachment 93820 [details] [review] Bug 23762 - Editing is_html status of email template fails under multi-languages To test: 1.Enable multi-languages 2.Set the preference 'TranslateNotices' on 'Allow' 3.Go to: tools==>Notices & slips==>Edit (one notice) 4.Click on the "Email" in the "Default" tab, change the checkbox status of 'HTML Messages'. 5.Save 6.Reopen the notice, nothing changed. 7.Apply the patch 8.Repeat the steps 4 and 5 9.Success.
Hi Bin, I tested this according to your test plan, but the HTML checkbox saves correctly for me on master. Can you please check your test plan again? What I did: - Edit MEMBERSHIP_EXPIRY - Pick default tab - Check checkbox for HTML - Save - Reopen - Checkbox was saved (patch wasn't applied)
Actually, it should be: Don't use default, but any other. I can see the issue when picking German.
(In reply to Katrin Fischer from comment #3) > Actually, it should be: Don't use default, but any other. I can see the > issue when picking German. Actually no, this happens with and without the patch if no subject and text is entered. I am sorry, I can't replicate the problem :(
Ok... it cannot be unchecked it seems.
Created attachment 93906 [details] [review] Bug 23762: Editing is_html status of email template fails under multi-languages To test: 1.Enable multi-languages 2.Set the preference 'TranslateNotices' on 'Allow' 3.Go to: tools==>Notices & slips==>Edit (one notice) 4.Click on the "Email" in the Default, uncheck the checkbox for 'HTML Messages'. 5.Save 6.Reopen the notice, nothing changed. 7.Apply the patch 8.Repeat the steps 4 and 5 9.Success. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
(In reply to Katrin Fischer from comment #5) > Ok... it cannot be unchecked it seems. Hi Katrin, thanks for reminding me to add it to the test plan. Only when you enter the subject and content and save it will the corresponding language email template be generated in the table 'letter'. To test: 1.Enable multi-languages 2.Set the preference 'TranslateNotices' on 'Allow' 3.Go to: tools==>Notices & slips==>Edit (one notice) 4.Make sure that multilingual email templates are not empty. 5.Click on the "Email" in the "Default" tab, change the checkbox status of 'HTML Messages'. 6.Save 7.Reopen the notice, nothing changed. 8.Apply the patch 9.Repeat the steps 4 and 5 10.Success.
Created attachment 94948 [details] [review] Bug 23762: Editing is_html status of email template fails under multi-languages To test: 1.Enable multi-languages 2.Set the preference 'TranslateNotices' on 'Allow' 3.Go to: tools==>Notices & slips==>Edit (one notice) 4.Click on the "Email" in the Default, uncheck the checkbox for 'HTML Messages'. 5.Save 6.Reopen the notice, nothing changed. 7.Apply the patch 8.Repeat the steps 4 and 5 9.Success. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Code makes sense, bug replicated and proved to be resolved by this patch. Passes QA scripts and although I'd love to have seen an accompanying selenium test that's not a blocker. Passing QA. Minor note.. please remember to alter the 'Importance' of bugs.. This was labelled as an enhancement when in my mind it's a bug. Nice work finding and fixing though Bin Wen
Nice work! Pushed to master for 19.11.00