A simple typo.

"If a image toolkit"
should be
"If an image toolkit"

The typo is present in both d8 and d7:
https://api.drupal.org/api/drupal/core!modules!file!file.module/function...
https://api.drupal.org/api/drupal/includes%21file.inc/function/file_vali...

Beta Evaluation

Documentation changes are unfrozen.

CommentFileSizeAuthor
#8 typo_d7-2579625.patch536 byteslaurencemercer
#2 typo-2579625.patch614 byteslaurencemercer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

laurencemercer created an issue. See original summary.

laurencemercer’s picture

laurencemercer
Primary language English
CreditAttribution: laurencemercer commented
Status: Active » Needs review
FileSize
typo-2579625.patch614 bytes
cilefen’s picture

cilefen CreditAttribution: cilefen commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix, +Documentation, +Needs backport to D7
xjm’s picture

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

Nice catch and nice clear fix. Committed and pushed to 8.0.x. Thanks!

  • xjm committed ef4ded0 on 8.0.x 
    Issue #2579625 by laurencemercer: Typo in file_validate_image_resolution...
cilefen’s picture

cilefen CreditAttribution: cilefen commented
Version: 8.0.x-dev » 7.x-dev
Status: Fixed » Patch (to be ported)

Moving to D7 for the backport.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented

Oops thanks. :)

laurencemercer’s picture

laurencemercer
Primary language English
CreditAttribution: laurencemercer commented
Status: Patch (to be ported) » Needs review
FileSize
typo_d7-2579625.patch536 bytes

Ported D7 patch attached.

nicrodgers’s picture

nicrodgers
Primary language English
Location Monmouthshire, UK
CreditAttribution: nicrodgers commented
Status: Needs review » Reviewed & tested by the community

Nice and simple patch.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x - thanks!

  • David_Rothstein committed 36e2726 on 7.x 
    Issue #2579625 by laurencemercer: Typo in file_validate_image_resolution...

Status: Fixed » Closed (fixed)

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