php_enable() uses the following code:

 drupal_set_message(t('A !php-code text format has been created.', array('!php-code' => l('PHP code', 'admin/config/content/formats/' . $php_format->format))));

In the documentation for t() is reported the following text:

  • Incorrect HTML in t(): 

$output .= t('<p>Go to the @contact-page.</p>', array('@contact-page' => l(t('contact page'), 'contact')));
  • Correct HTML in t(): 

$output .= '<p>' . t('Go to the <a href="@contact-page">contact page</a>.', array('@contact-page' => url('contact'))) . '</p>';
CommentFileSizeAuthor
#2 908772_incorrect_use_of_t_2.patch799 bytessivaji_ganesh_jojodae

Comments

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Documentation problem with php_enable() » php_enable() uses code that is reported as wrong in the documentation for t()
Component: documentation » php.module
sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
Title: php_enable() uses code that is reported as wrong in the documentation for t() » Incorrect use of t() in php_enable()
Status: Active » Needs review
StatusFileSize
new 908772_incorrect_use_of_t_2.patch799 bytes

Agree with kiamlaluno. Attached patch will provide correct placeholder.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Reviewed & tested by the community

This is exactly right.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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