Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
On http://api.drupal.org/api/function/hook_menu/6:
The text reads:
"title": Required. The untranslated title of the menu item.
but one of the examples:
$items['blog/feed'] = array(
'title' => t('RSS feed'), // <------------- no t()??
'page callback' => 'blog_feed',
'access arguments' => array('access content'),
'type' => MENU_CALLBACK,
);
Shouldn't the t() be removed?
Comment | File | Size | Author |
---|---|---|---|
#4 | 296584-core-examples-D6.patch | 602 bytes | mr.baileys |
#4 | 296584-core-examples-D7.patch | 596 bytes | mr.baileys |
Comments
Comment #1
nielsbom CreditAttribution: nielsbom commentedChanged the component to reflect the new component categorization. See http://drupal.org/node/301443
-nielsbom
Comment #2
add1sun CreditAttribution: add1sun commentedmoving queue for API docs and yes, this should be corrected.
Comment #3
keith.smith CreditAttribution: keith.smith commentedComment #4
mr.baileysPatches for D6 and D7 attached.
Comment #5
boombatower CreditAttribution: boombatower commentedAmazing patch! now that it can be done in issue queue that is great.
Comment #6
Gábor HojtsyLet's get the D7 patch committed first.
Comment #8
mr.baileysgrmp. I fail to see how a change to an api example can cause one test in the "User language settings" suite to fail...
Comment #9
stella CreditAttribution: stella commentedAll tests pass with a clean install of HEAD on my local machine.
Comment #10
mr.baileysBack to RTC since the testbot is happy again.
Comment #11
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks!