WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#36968 closed defect (bug) (fixed)

image_memory_limit filter docblock for wp-includes/class-wp-image-editor-imagick.php references self

Reported by: bobbingwide Owned by: DrewAPicture
Milestone: 4.6 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords:
Focuses: docs Cc:

Description

The documentation for the image_memory_limit filter in wp-includes/class-wp-image-editor-imagick.php currently says

This filter is documented in wp-includes/class-wp-image-editor-imagick.php 

I believe it should be

This filter is documented in wp-includes/class-wp-image-editor-gd.php 

Attachments (1)

36968.patch (840 bytes) - added by bobbingwide 4 years ago.
Fix docblock for image_memory_limit filter

Download all attachments as: .zip

Change History (5)

@bobbingwide
4 years ago

Fix docblock for image_memory_limit filter

#1 @swissspidy
4 years ago

  • Focuses docs added

#2 @joemcgill
4 years ago

  • Milestone changed from Awaiting Review to 4.6

Good catch @bobbingwide. Patch looks right to me.

#3 @DrewAPicture
4 years ago

In 37591:

Docs: Correct a duplicate filter comment for the image_memory_limit hook to point to the correct file.

Props bobbingwide.
See #36968.

#4 @DrewAPicture
4 years ago

  • Owner set to DrewAPicture
  • Resolution set to fixed
  • Status changed from new to closed

In 37592:

Docs: Ensure hook and duplicate hook docs directly precede all instances of the image_memory_limit filter.

Hook docs must directly precede hooks in order for the Code Reference parser to correctly identify them as such.

Fixes #36968.

Note: See TracTickets for help on using tickets.