The first parameter to t() should be a literal string. There should be no variables, concatenation, constants or other non-literal strings there. At t($item->name) in simplenews/simplenews.admin.inc on line 407. Read more at http://drupal.org/node/322732

Comments

sutharsan’s picture

sutharsan commented
Priority: Normal » Minor
Status: Needs review » Closed (won't fix)

The code is taken from local_user() function. And no, this does not make it code good, but it is as good as its currently gets.