This results in lots and lots of notices appearing in _bootstrap_colorize_text().

CommentFileSizeAuthor
#7 t_no_longer_returns_a-2573541-7.patch1.57 KBhaza
#2 2573541_1.patch2.86 KBslashrsm

Comments

slashrsm created an issue. See original summary.

slashrsm’s picture

slashrsm commented
StatusFileSize
new 2573541_1.patch2.86 KB

Solution in attached patch is not the nicest one, but it removes notices.

slashrsm’s picture

slashrsm commented
Status: Active » Needs review

  • neardark committed 252af68 on 8.x-3.x authored by slashrsm 
    Issue #2573541 by slashrsm: t() no longer returns a string, it returns \...
neardark’s picture

neardark commented
Status: Needs review » Fixed

Merged.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Fixed » Needs review

This cannot be the only place this needs to be changed? The t() function is used through the base theme.

haza’s picture

haza
Primary language French
commented
StatusFileSize
new t_no_longer_returns_a-2573541-7.patch1.57 KB

Here is a patch that adds the same logic to the other t() implementations.

It only contains the new addition since the other patch has already be commited.

neardark’s picture

neardark commented
Status: Needs review » Fixed

Merged

  • Haza authored 75f24cb on 8.x-3.x 
    Issue #2573541 by slashrsm, Haza: t() no longer returns a string, it...
neardark’s picture

neardark commented
Status: Fixed » Closed (fixed)