Voting starts in March for the Drupal Association Board election.
Help module's help_help() incorrectly contains a mention of "context-sensitive help" which actually originates from the system module. Additionally, system module's hook_help() fails to mention this fact.
Patch in #3 adequately solves the original scope of this issue, however system module's help information needs to reflect its additional functionality. Additional patch needed. Working on this now.
|Task||Novice task?||Contributor instructions||Complete?|
|Remove mentions of "context-sensitive" from help module||Novice||Update documention. Patch in #3||by @er.pushpinderrana|
|Reroll patch in #3 to include system module edits||Novice||Add help text describing system module's System Help block. [DONE]|
|Manual test||Novice||Make sure both Help and System module help visible from admin/help is still formatted OK. Also, in the new section in System help (the section on displaying administrative help), make sure the links work, and that UI text matches the actual Drupal UI|
User interface changes
Original report by @jhodgdon
While reviewing the patch for another issue, I learned that the functionality of "providing context-sensitive help on individual pages of your site" is actually a function of the System module, not the Help module. So this text in the help module's function help_help():
$output .= '<dt>' . t('Providing context-sensitive help') . '</dt>'; $output .= '<dd>' . t('The Help module displays context-sensitive advice and explanations on various pages.') . '</dd>'; $output .= '</dl>';
and also the mention above in About, are wrong. This is not something the Help module does. It is part of the System module, and it also is only displayed if you have the System Help block shown.
So, we should remove mentions of context-sensitive help from this help text. Should be a fairly easy novice patch... it is function help_help() in core/modules/help/help.module, and inside case 'help.page.help'.