<em> tag appears in revision log message when a revision entry is reverted

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?

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?

. 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.

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

> I don't think that's a good idea, that could be tens or hundreds of thousands of revisions.

Also, I might have some contrib/custom module modifying that message, so we shouldn't touch existing messages at all.

Reviewed the code, checked the test coverage, this looks safe to me.

We still need to cover #6 tho.

IMO #6 should be done in a separate issue, this is tightly scoped to just removing the placeholder markup from automatically generated messages from reverting.