dev/core#2814 Fix activity:sendEmail follow up

[eileenmcnaughton]

Overview

Follow up fix on #21365

@demeritcowboy

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

@eileenmcnaughton any chance of expanding the test cover to prevent further regressions?

[eileenmcnaughton]

@seamuslee001 Yeah - but probably not for a few days - I'd vote for merging this & then I'll try to add in when I revist to remove the calls to removeCaseTokens to avoid anyone getting confused

[seamuslee001]

using the test in #21386 I was able to verify this fixed the problem Dave D outlined

[eileenmcnaughton]

Thanks @seamuslee001 appreciate that - I hope to add more tests when I convert the case tokens - although we have to work through the date issue on that first but then