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.
API page: https://api.drupal.org/api/drupal/core%21core.api.php/function/hook_mail...
The 'see also' isn't forming a link.
Also, the first mention of MailManagerInterface->mail() should probably be changed to API makes it into a link.
Comment | File | Size | Author |
---|---|---|---|
#11 | interdiff-2777451-5-11.txt | 1.21 KB | markdorison |
#11 | see_references_not-2777451-11.patch | 3.9 KB | markdorison |
#5 | references_to-2777451-5.patch | 3.86 KB | markdorison |
#2 | D8-MailManagerInterface-2777451-2.patch | 483 bytes | er.pushpinderrana |
Comments
Comment #2
er.pushpinderrana CreditAttribution: er.pushpinderrana as a volunteer and at Publicis Sapient for Publicis Sapient commentedCorrected the
See also
link but the one that is coming to description looks fine to me. It seems to be consistent with hook_mail_alter().Comment #3
markdorisonComment #4
alexpottLet's fix all the instances of @see.*->
There are seven to fix in total.
Comment #5
markdorisonComment #6
markdorisonComment #8
quietone CreditAttribution: quietone as a volunteer commentedLooks good to me. Applied patch and used grep to confirm that all 7 instances were fixed.
Comment #9
alexpottShould be
@see \Drupal\Core\Extension\ModuleHandlerInterface::invokeAll()
Should be @see \Drupal\Core\Database\Driver\pgsql\Schema::queryTableInformation()
Comment #10
alexpottNeed to file a followup to fix or perhaps even remove this documentation. It's not useful tbh.
Comment #11
markdorisonFixed issues described in #9.
Comment #12
quietone CreditAttribution: quietone as a volunteer commentedMade a follow up issue as per #9, #2791375: Fix $tableInformation documentation in pgsql and mysql schema.php
Comment #13
quietone CreditAttribution: quietone as a volunteer commentedApplied patch locally and the links are generated. Thx.
Comment #17
catchCommitted/pushed to all three 8.x branches, thanks!
Comment #19
daffie CreditAttribution: daffie commentedThe patch was committed, so it will not apply again.