API page: https://api.drupal.org/api/drupal/core%21modules%21system%21system.insta...
This is an implementation of hook_requirements(), isn't it?
Comment | File | Size | Author |
---|---|---|---|
#7 | 2062127-system-requirements-7.patch | 496 bytes | rhm5000 |
#5 | 2062127-system-requirements-5.patch | 404 bytes | rhm5000 |
#2 | system-requirements-2062127-2.patch | 533 bytes | jlindsey15 |
Comments
Comment #1
jhodgdonI think you're right. Good catch!
Comment #2
jlindsey15 CreditAttribution: jlindsey15 commentedPatch for Drupal 8.
Comment #3
jhodgdonLooks good, thanks! I'll get this committed. Most likely I can use the same patch for D7 too.
Comment #4
jhodgdonThanks! Committed to 8.x and 7.x.
In 6.x for consistency we should probably say "Implementation of.." rather than "Implements" (the rest of 6.x uses the old standard). So if someone still cares to port this to 6.x...
Comment #5
rhm5000 CreditAttribution: rhm5000 commentedBackport to 6.x.
Comment #6
jhodgdonThanks! But this patch is not quite right. Look at the 8.x/7.x patch on #2 -- we need to *replace* the existing function body.
Comment #7
rhm5000 CreditAttribution: rhm5000 commentedThe function body, everything within {}, is unchanged in patch on #2. I provided another patch to reflect the removed comments in #2 patch.
Comment #8
rhm5000 CreditAttribution: rhm5000 commentedComment #9
jhodgdonThanks, that is the patch we needed. I will get it committed sometime soon.
Comment #10
jhodgdonThanks! Committed to 6.x.