While working on a project I found a minor spelling mistake in includes/file.inc file and thought that I should let the drupal team know about this.

I know it will never break any feature of drupal neither now nor in future because it is commented, anyway it will be always be good to have correct one.

CommentFileSizeAuthor
#5 drupal_file_system-2669568-0-7.42-0.patch416 bytessoaratul
#2 drupal_file_system-2669568-7.42.patch376 bytessoaratul

Comments

soaratul created an issue. See original summary.

soaratul’s picture

soaratul commented
StatusFileSize
new drupal_file_system-2669568-7.42.patch376 bytes
soaratul’s picture

soaratul commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: drupal_file_system-2669568-7.42.patch, failed testing.

soaratul’s picture

soaratul commented
StatusFileSize
new drupal_file_system-2669568-0-7.42-0.patch416 bytes

Attaching another patch file with correct diff.

soaratul’s picture

soaratul commented
Status: Needs work » Needs review
cilefen’s picture

cilefen commented
Status: Needs review » Reviewed & tested by the community
Issue tags: -Spelling mistake +Quick fix, +Documentation

Thank you!

cilefen’s picture

cilefen commented
Version: 7.42 » 7.x-dev
cilefen’s picture

cilefen commented
Title: Incorrect method name for hook_file_download » Docblock typo in file_download_headers
fabianx’s picture

fabianx commented
Issue tags: +Pending Drupal 7 commit

  • Fabianx committed 2503818 on 7.x 
    Issue #2669568 by soaratul: Docblock typo in file_download_headers
fabianx’s picture

fabianx commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x! Thanks!

fabianx’s picture

fabianx commented
Issue tags: -Pending Drupal 7 commit

Status: Fixed » Closed (fixed)

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