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%21lib%21Drupal%21Core%21Database%...
The Schema class in \Drupal\Core\Database has no doc block. It should.
Also the @file doc block does not conform to our standards:
https://www.drupal.org/node/1354#file
Beta phase: This is just API docs.
Comment | File | Size | Author |
---|---|---|---|
#6 | interdiff-6.txt | 346 bytes | joshi.rohit100 |
#6 | database_schema_doc-10031785-6.patch | 651 bytes | joshi.rohit100 |
#5 | interdiff.txt | 343 bytes | pguillard |
#5 | database_schema_doc-10031785-5.patch | 650 bytes | pguillard |
#2 | database_schema_doc-10031785-2.patch | 648 bytes | pguillard |
Comments
Comment #1
pguillard CreditAttribution: pguillard commentedComment #2
pguillard CreditAttribution: pguillard commentedA really small first patch.
In fact many Drupal\Core\Database classes are not documented..
Comment #3
pguillard CreditAttribution: pguillard commentedComment #4
cilefen CreditAttribution: cilefen commented@pguillard Good job. Something to note:
This should be "Contains Drupal\..." and must have a period at the end.
Comment #5
pguillard CreditAttribution: pguillard commentedThanks @cilefen
Comment #6
joshi.rohit100Slightly minor change.
Comment #7
jhodgdonLooks fine to me, thanks!
Beta phase: This is just API docs.
Comment #11
zauravRTBC because the patches address the problem outlined in the issue and conform to coding standards.
1. The Schema class has been provided with a docblock style comment.
2. The @file docblock has been edited to conform with standards.
3. Follows all standards in General considerations for API module parsing and Drupal API documentation standards (general)
Comment #15
dylf CreditAttribution: dylf commentedTestbot is working!
Comment #19
jhodgdonComment #20
alexpottCommitted c65b82f and pushed to 8.0.x. Thanks!