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.
Problem/Motivation
The functionality of the Link module has changed since the hook_help text got committed.
There now is an option to restrict links to internal or external links as "Allowed link type"
Proposed resolution
Add another use to describe the Allowed link type option.
Remaining tasks
User interface changes
API changes
Comment | File | Size | Author |
---|---|---|---|
#7 | drupal_core-update_hook_help_link_2470960-7.patch | 1.42 KB | meramo |
#6 | drupal_core-update_hook_help_link_2470960-6.patch | 1.42 KB | meramo |
#4 | link_help.png | 23.48 KB | Reno Greenleaf |
#3 | drupal_core-update_hook_help_link_2470960-3.patch | 1.42 KB | meramo |
#1 | drupal_core-update_hook_help_link_2470960-1.patch | 2.38 KB | meramo |
Comments
Comment #1
meramo CreditAttribution: meramo commentedSee the patch
Comment #2
jhodgdonThanks for the patch! It's a good start... a couple of comments:
We use serial commas in Drupal docs. So before "or" you need a ,
Also ... we don't normally want to tell people that something is "easy". Better to be descriptive of the interface. Is it an auto-complete interface? or ... something like that.
I think these two lines are extra, given that you added something similar above?
Comment #3
meramo CreditAttribution: meramo commentedWhoops, my bad!
Resubmitting the amended patch.
Comment #4
Reno Greenleaf CreditAttribution: Reno Greenleaf commentedChecked last patch.
Looks fine.
Comment #5
ifrikThanks Igor Kandyba,
just one tiny thing. "internal linking" sounds a bit akward.
"enable an autocomplete widget for internal links." should be sufficient. Don't worry about repeating the existing wording from the interface text.
Comment #6
meramo CreditAttribution: meramo commentedGood spot!
Comment #7
meramo CreditAttribution: meramo commentedSorry, haven't uploaded the file
Comment #8
meramo CreditAttribution: meramo commentedSetting for review.
Comment #9
ifrikThanks!
Comment #10
webchickYay docs! :)
Committed and pushed to 8.0.x. Thanks!