The current field.info.inc file (in the modules sub-folder) has some incoherent language on lines 76-77. This should be remedied and patched for clarity. It will also need to be updated at https://www.drupal.org/node/1156554.

CommentFileSizeAuthor
#6 2386545-6-entity-field-info-documentation.patch735 bytesTR
#5 2386545-5-entity-field-info-documentation.patch739 bytesTR
#2 entity-fix-field-info-inc-grammer-2386545-2.patch666 byteslaughnan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

laughnan’s picture

laughnan
he/him/his
CreditAttribution: laughnan commented
Title: Fix function comments for clarity » Fix field.info.inc function comments for clarity
laughnan’s picture

laughnan
he/him/his
CreditAttribution: laughnan commented
Status: Active » Needs review
FileSize
entity-fix-field-info-inc-grammer-2386545-2.patch666 bytes

Adding patch

laughnan’s picture

laughnan
he/him/his
CreditAttribution: laughnan commented
Issue summary: View changes
Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer commented

The 4 year old patch in #2 to field.info.inc applied cleanly to the latest entity 7.x-1.x-dev and (if still relevant) needs review.

TR’s picture

TR CreditAttribution: TR commented
FileSize
2386545-5-entity-field-info-documentation.patch739 bytes

New patch with coding standards fixed.

TR’s picture

TR CreditAttribution: TR commented
FileSize
2386545-6-entity-field-info-documentation.patch735 bytes

  • TR committed 523aec8 on 7.x-1.x 
    Issue #2386545 by laughnan, TR: Fix field.info.inc function comments for...
TR’s picture

TR CreditAttribution: TR commented
Status: Needs review » Fixed

Committed #6.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.