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%21modules%21field%21field.module/...
The page contains the following text:
- Do not use inside node (or other entities) templates, use render($content[FIELD_NAME]) instead.
- Do not use to display all fields in an entity, use field_attach_prepare_view() and field_attach_view() instead.
- The field_view_value() function can be used to output a single formatted field value, without label or wrapping field markup.
The first two sentences are comma-splices. There should be a but after the comma, or the comma should simply be replaced with a semicolon.
Comment | File | Size | Author |
---|---|---|---|
#6 | fix-comma-splice-2059785-6.patch | 878 bytes | apaderno |
#4 | fix-comma-splice-2059785-4.patch | 876 bytes | apaderno |
#2 | fix-comma-splice-2059785-2.patch | 1.13 KB | apaderno |
Comments
Comment #2
apadernoFor what I recall, in American English the items of a list should not have a period at the end.
Comment #2.0
apadernoOnly two sentences are comma-splices.
Comment #4
apadernoComment #5
longwaveWhile we are here, "node (or other entities) templates" seems like it needs changing as well; maybe "node (or any other entity) templates"?
Comment #6
apadernoThat's right; in that way, the attributive is a singular noun as it should be, and as node is.
Comment #8
apaderno#6: fix-comma-splice-2059785-6.patch queued for re-testing.
Comment #9
longwaveRTBC, no reason why this should fail!
Comment #10
jhodgdonPatch no longer applies. :(
Comment #11
longwaveStill applies for me, with no fuzz even.
#6: fix-comma-splice-2059785-6.patch queued for re-testing.
Comment #12
jhodgdonOh, you're right! Not sure what happened before.
Committed to 8.x and 7.x.
Comment #13.0
(not verified) CreditAttribution: commentedFixed grammar.