WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#35962 closed defect (bug) (fixed)

Documentation fix for `get_meta_sql` filter

Reported by: sudar Owned by: DrewAPicture
Priority: normal Milestone: 4.5
Component: Database Version: 4.1
Severity: normal Keywords: has-patch commit
Cc: Focuses: docs

Description

The get_meta_sql filter has 6 separate parameters, but the phpdoc documentation wrongly documents them as an array.

Because of this, the generated documentation for the filter at ​https://developer.wordpress.org/reference/hooks/get_meta_sql/ is also wrong.

I am attaching a patch which fixes the phpdocs documentation error. It doesn't have any code changes.

Attachments (1)

35962.patch​ (1.3 KB ) - added by sudar 11 years ago.

Download all attachments as: .zip

Change History (5)

@sudar
11 years ago

#1 @DrewAPicture
11 years ago

  • Keywords has-patch commit added
  • Milestone Awaiting Review → 4.5

@sudar Thanks for the patch, good catch!

#2 @DrewAPicture
11 years ago

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

In 36735:

Docs: Correctly document parameters in the hook doc for the get_meta_sql filter as individual parameters rather than an array.

When an array of parameters is passed to do_action|apply_filters_ref_array, parameters should be documented as individuals instead of as part of the whole.

Props sudar.
Fixes #35962.

#3 @sudar
11 years ago

@DrewAPicture You're welcome :) I just happened to look at the documentation for that hook.

Meanwhile I also found a similar issue with phpdoc and developer docs. I have created #35974 (with patch) for it.

This one affects the documentation about a dozen hooks.

#4 @johnbillion
11 years ago

  • Version trunk → 4.1
Note: See TracTickets for help on using tickets.