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 docblock looks like this:
{#
/**
* @file
…
* @ingroup themeable
*/
#}
This actually makes the page on api.d.o look broken: https://api.drupal.org/api/drupal/core%21modules%21language%21templates%...
Why is this RC eligible?
Because it only changes documentation.
Proposed resolution
It should look like this:
{#
/**
* @file
…
* @ingroup themeable
*/
#}
A space needs to be added before each single asterisk after the /**
, see https://www.drupal.org/node/1354#general.
Remaining tasks
Patch
User interface changes
n/a
API changes
n/a
Data model changes
n/a
Comment | File | Size | Author |
---|---|---|---|
#4 | interdiff-2-4.txt | 859 bytes | Nitesh Pawar |
#4 | 2607824-4.patch | 2.07 KB | Nitesh Pawar |
#2 | 2607824-2.patch | 2.06 KB | r_sharma08 |
Comments
Comment #2
r_sharma08 CreditAttribution: r_sharma08 at Publicis Sapient for Publicis Sapient commentedDocumented now, please review.
Comment #3
star-szrThanks @r_sharma08! That patch only adds the spaces, doesn't change anything else (as it should), but this does put the 'description' line over 80 characters, so that should be wrapped now: https://www.drupal.org/node/1354#drupal
Also I don't think what we're doing in this issue is documenting but we're fixing things :)
Normally I'd make this kind of thing minor but it does actually break things on api.d.o, adding that to the issue summary.
Comment #4
Nitesh Pawar CreditAttribution: Nitesh Pawar at Trigyn Technologies Ltd commentedComment #5
Nitesh Pawar CreditAttribution: Nitesh Pawar at Trigyn Technologies Ltd commentedComment #6
star-szrThank you @Nitesh Pawar, I think this is ready to go now!
Comment #7
star-szrForgot to tag it.
Comment #9
alexpottCommitted 7bef4e6 and pushed to 8.0.x. Thanks!