A lot of translatable strings have two spaces between the period closing a sentence and the first character of a new sentence. This should just be one space. Example from /includes/handlers/gateways/pay_method.inc:

t('This is how the payment method will be listed in admin interfaces.  Use something that will set this apart from other payment methods.')
CommentFileSizeAuthor
#4 pay.double_spaces.928906.1.patch19.54 KBgoogletorp

Comments

allie micka’s picture

allie micka
she/her
commented
Status: Active » Needs work

This is the type of issue where a patch would be most welcome!

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

The infamous "File a patch!" :D

I've been trying to capture these occurences with the following regular expression, but for some reason I can only find matches if I use only one space character instead of two:

t\((.+?)[:space:][:space:](.+?)\)
mstrelan’s picture

mstrelan commented

Just search for ". " as a string instead of a regular expression.

googletorp’s picture

googletorp commented
Status: Needs work » Needs review
StatusFileSize
new pay.double_spaces.928906.1.patch19.54 KB

I don't know if I got them all, but I got a lot.

jerdavis’s picture

jerdavis
Location Minneapolis, MN
commented
Status: Needs review » Fixed

Committed, thanks googletorp!

Status: Fixed » Closed (fixed)

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