Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
The Comment module now includes the option to configure different comment types. This is not covered in the hook_help text.
Proposed resolution
Update the hook_help text.
Check the UI text on the Comments and Comment types administration pages and update them if necessary as well.
Remaining tasks
Update the hook_help text, and possibly the UI text as well
User interface changes
This is a UI text change.
API changes
None.
Data model changes
None.
Beta phase evaluation
| Issue category | Task because it is adding a missing part to the Help text. |
|---|---|
| Issue priority | Minor |
| Unfrozen changes | Unfrozen because it only changes strings by adding another string to the hook_help(). |
| Disruption | None |
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 2572525-comment-help-3.patch | 2.39 KB | Anonymous (not verified) |
Comments
Comment #2
Anonymous (not verified) commentedI'll be working on this today.
Comment #3
ifrikApparently there is a usability issue #2496913: Don't expose entity types with string ids as a target option when creating comment types that hasn't been solved yet, even though I'm not quite sure whether this covers all of the problem.
Comment #4
ifrikLuckily, the issue mentioned above is of no concern to this issue.
Comment #5
Anonymous (not verified) commentedI updated the help-text and added a use because the functionality was changed.
Comment #6
Anonymous (not verified) commentedComment #7
ifrikThanks a lot for the patch - and for spending all the time with me to first find out how that part of the module even works :-)
This addition should save lots of sitebuilders a lot of time...
beta-evaluation added.
Comment #8
ifrikComment #9
ifrikThe UI text about adding Comment types on the administration pages looks correct.
Comment #11
jhodgdonSome random test failure. The added text looks good to me. +1.
Comment #12
xjmGreat to have this documented as this new feature might be confusing otherwise. The docs look good to me as well. Committed and pushed to 8.0.x.