- 🇦🇺Australia acbramley
This is still an issue and is because things like NodeRevisionRevertForm use
%placeholders. We get around this with a custom formatter than formats strings as markup. Since the revision_log is a string field, I think the easiest approach here is to just change the placeholder used to remove the markup in the first place? - Merge request !11235Issue #2736401: Change revert revision_log placeholders to remove markup → (Open) created by acbramley
- 🇺🇸United States smustgrave
Tested out and the issue is fixed for newly reverted revisions but for existing revisions they still have em tag. So do we need an upgrade hook to retroactively fix that? Are we okay with a mix of of the 2 on the same screen?
- 🇦🇺Australia acbramley
. So do we need an upgrade hook to retroactively fix that
I don't think that's a good idea, that could be tens or hundreds of thousands of revisions.
- 🇺🇸United States smustgrave
I’ll leave in review for others but seems odd to have mix and match. Can already see a ticket opened asking why some are italicized and some aren’t
