I am getting the error message "Administrative menu items should have a description" from Coder Tough Love on a few lines that are MENU_CALLBACK and MENU_DEFAULT_LOCAL_TASK. I'm not necessarily against putting a description on them for code documentation purposes, but I don't think that description is ever displayed anywhere, so I think maybe a code comment would be a better way to document those menu items. If a #description is put there, it would need to be in t() and then translators would have extra work to do to translate text that is never displayed in the UI. Or am I wrong and those descriptions would be displayed somewhere? I don't think so...
So my opinion is that for MENU_CALLBACK and MENU_LOCAL_TASK and probably several other types of menu items, that warning should not be generated. Willing to be told I'm wrong, too. :)
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | menu_callback.patch | 1.82 KB | aspilicious |
Comments
Comment #1
tr commentedI agree - the warning should not be generated for MENU_CALLBACK and MENU_LOCAL_TASK items.
Comment #2
aspilicious commentedThis is a D7 patch probably works for D6 to.
Comment #3
xjmTested and confirmed that #2 resolves the issue. Thanks @aspilicious!
Comment #4
xjmCommitted to both branches:
http://drupal.org/commitlog/commit/7948/407e9d12f6454d544c2695c9b76da23e...
http://drupal.org/commitlog/commit/7948/4726d24d11f8dbc5b7ec4c41fb683278...