Part of meta #500866: [META] remove t() from assert message.

CommentFileSizeAuthor
#1 t_dashboard-2059685-1.patch6.43 KBstephaneq

Comments

stephaneq’s picture

stephaneq
Primary language French
Location Paris 🇫🇷
commented
Status: Active » Needs review
StatusFileSize
new t_dashboard-2059685-1.patch6.43 KB
star-szr’s picture

star-szr
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

This one looks good too, thanks @StephaneQ. This patch removes t() from all assertion messages in dashboard.test and doesn't change anything it shouldn't. This one includes a couple format_string() updates but they look correct.

star-szr’s picture

star-szr
he/him
Primary language English
commented
Issue tags: +Coding standards

Tagging.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 7.x.

Automatically closed -- issue fixed for 2 weeks with no activity.