Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
10 Aug 2013 at 12:09 UTC
Updated:
4 Jan 2014 at 03:57 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jhodgdonI think you're right. Good catch!
Comment #2
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
michaelhiiva 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
michaelhiiva 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
michaelhiiva commentedComment #9
jhodgdonThanks, that is the patch we needed. I will get it committed sometime soon.
Comment #10
jhodgdonThanks! Committed to 6.x.