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.

Comments

soaratul created an issue. See original summary.

soaratul’s picture

StatusFileSize
new376 bytes
soaratul’s picture

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

StatusFileSize
new416 bytes

Attaching another patch file with correct diff.

soaratul’s picture

Status: Needs work » Needs review
cilefen’s picture

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

Thank you!

cilefen’s picture

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

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

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

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x! Thanks!

fabianx’s picture

Issue tags: -Pending Drupal 7 commit

Status: Fixed » Closed (fixed)

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