Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
API page: https://api.drupal.org/api/drupal/core%21includes%21bootstrap.inc/functi...
In the documentation, this text "In order for strings to be localized, make them available in one of the ways supported by the Localization API. " contains a link to the localization API page here https://www.drupal.org/node/322729, but that page is for D6/7. This should probably link to https://www.drupal.org/developing/api/8/localization instead.
Comment | File | Size | Author |
---|---|---|---|
#12 | 2651766-12-update_link.patch | 1.04 KB | imalabya |
#10 | 2651766-10-update_link.patch | 716 bytes | imalabya |
| |||
#6 | 2651766-6-update_link.patch | 1.04 KB | eojthebrave |
#4 | 2651766-4-update_link.patch | 973 bytes | eojthebrave |
#2 | 2651766-1-update_link.patch | 1.02 KB | eojthebrave |
Comments
Comment #2
eojthebraveAnd, here's a patch with the change.
Comment #3
jhodgdonThanks! I think we would actually be better off linking to
https://api.drupal.org/api/drupal/core!lib!Drupal!Core!Language!language...
which would look something like
Comment #4
eojthebraveHere's a new patch using "@link i18n Localization API. @endlink" instead, and re-warping the comment to 80 chars.
Comment #5
jhodgdonIt looks like the @link ... @endlink would fit on the previous line?
I like the idea of warping comments. ;)
Comment #6
eojthebraveYou, see, that comment was warped. But now it's properly wrapped. :)
Comment #7
jhodgdonLooks good, thanks!
Comment #8
alexpottI think there are too many fullstops. I think the one after
API
is not necessary.Comment #9
jhodgdonYeah, I think we should take out the one after @endlink -- sorry I didn't notice the duplication.
Comment #10
imalabyaAdded an updated patch with the above comment.
Comment #11
jhodgdonThis latest patch is missing a lot from the earlier patch. Looks like it's more like an interdiff? We need a full patch. Thanks!
Comment #12
imalabyaHi @jhodgdon, sorry for the confusion. I was too quick. Have added the complete patch with the updates. Hope this is good.
Comment #13
imalabyaComment #14
jhodgdonThanks!
Comment #16
alexpottCommitted 5b912b4 and pushed to 8.0.x and 8.1.x. Thanks!