API page: http://api.drupal.org/api/drupal/modules%21field%21field.info.inc/functi...

Enter a descriptive title (above) relating to field_info_instance, then describe the problem you have found:

In particular, this should state what is returned when no instance is found (looks like NULL to me).

CommentFileSizeAuthor
#5 @return_doc_field_info_instance-1734482-5.patch648 bytesplopesc
#3 @return_doc_field_info_instance-1734482-3.patch643 bytesplopesc
#1 @return_doc_field_info_instance-1734482-1.patch640 bytesplopesc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

plopesc’s picture

plopesc
he/him
Primary language Spanish
CreditAttribution: plopesc commented
FileSize
@return_doc_field_info_instance-1734482-1.patch640 bytes

Attaching a patch that includes the @return info.

plopesc’s picture

plopesc
he/him
Primary language Spanish
CreditAttribution: plopesc commented
Status: Active » Needs review
plopesc’s picture

plopesc
he/him
Primary language Spanish
CreditAttribution: plopesc commented
FileSize
@return_doc_field_info_instance-1734482-3.patch643 bytes

Attaching new patch according to 80 chars per line limitation.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Needs work

Looks good, thanks! The only thing is that we need a blank line between the @param section and the @return. See
http://drupal.org/node/1354#functions

plopesc’s picture

plopesc
he/him
Primary language Spanish
CreditAttribution: plopesc commented
Status: Needs work » Needs review
FileSize
@return_doc_field_info_instance-1734482-5.patch648 bytes

Hello.
Blank line added.
Thanks.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Reviewed & tested by the community

Looks good! I'll get it committed shortly.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 8.x and 7.x.

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