I'm attaching a patch from a translatable strings review. Most of the strings are fine, but very many are not translatable as they are missing t() function.

I may also found a security issue in:
'#title' => t('@label display options', array('@label' => $options[$selected])),

I have not verified the security issue itself, but trusting other modules/code to provide clean option strings could lead to code injection.

Comments

hass’s picture

Heck, one bracket was wrong. New patch.

codycraven’s picture

Status: Needs review » Fixed

Committed and pushed to 7.x-1.x

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