The documentation at http://api.drupal.org/api/function/hook_file_insert/7 is incorrect -- requiring &$file throws a warning. Patch forthcoming.

CommentFileSizeAuthor
#1 hook_file_insert.no-referenced-param.791986.1.patch875 bytesaaron
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aaron’s picture

aaron CreditAttribution: aaron commented
Status: Active » Needs review
FileSize
hook_file_insert.no-referenced-param.791986.1.patch875 bytes
aaron’s picture

aaron CreditAttribution: aaron commented
Priority: Normal » Minor

definitely minor.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Needs review » Reviewed & tested by the community

Makes sense.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks.

Status: Fixed » Closed (fixed)

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