WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#48214 closed defect (bug) (fixed)

Add missing `@param` for first parameter passed to `big_image_size_threshold`

Reported by: desrosj Owned by: SergeyBiryukov
Priority: normal Milestone: 5.3
Component: Media Version: 5.3
Severity: normal Keywords: needs-patch good-first-bug
Cc: Focuses: docs

Description

The big_image_size_threshold filter is passed 4 values, but only three are documented inline. The first one, threshold value, needs a @param tag with the appropriate parts.

Attachments (2)

48214.patch​ (1.3 KB ) - added by dkarfa 7 years ago.
48214.1.patch​ (1.3 KB ) - added by dkarfa 7 years ago.
Oops, fix typo error.

Download all attachments as: .zip

Change History (5)

#1 @desrosj
7 years ago

See #47873. Introduced in [46076].

@dkarfa
7 years ago

#2 @earnjam
7 years ago

@dkarfa Thank you for the patch.

In the future I would suggest to leave tickets marked good-first-bug for people who are new to contributing so they can learn the process of getting involved.

There is also a typo in your patch. It says patam instead of param

@dkarfa
7 years ago

Oops, fix typo error.

#3 @SergeyBiryukov
7 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 46398:

Docs: Add missing documentation for the first parameter of the big_image_size_threshold filter.

Props dkarfa, desrosj, earnjam.
Fixes #48214.

Note: See TracTickets for help on using tickets.