Make WordPress Core

Opened 3 months ago

Closed 6 days ago

#50835 closed defect (bug) (fixed)

Return value DocBlock lacked attachment details for wp_prepare_attachment_for_js()

Reported by: stevenlinx Owned by: johnbillion
Milestone: 5.6 Priority: normal
Severity: normal Version:
Component: Media Keywords: has-patch commit
Focuses: docs Cc:

Attachments (4)

50835.diff (3.2 KB) - added by stevenlinx 3 months ago.
50835.2.diff (3.8 KB) - added by Mista-Flo 7 days ago.
Fix doc alignment
50835.3.diff (3.6 KB) - added by Mista-Flo 7 days ago.
Fix since prop order
50835.4.diff (4.1 KB) - added by johnbillion 7 days ago.

Download all attachments as: .zip

Change History (13)

3 months ago

#1 @stevenlinx
3 months ago

  • Keywords has-patch added; needs-patch removed

#2 @SergeyBiryukov
3 months ago

  • Milestone changed from Awaiting Review to 5.6
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

This ticket was mentioned in Slack in #core-media by antpb. View the logs.

7 days ago

#4 @Mista-Flo
7 days ago

  • Keywords needs-refresh added

It probably needs a refresh to fix the alignment, otherwise looks good.

Any thought @SergeyBiryukov?

7 days ago

Fix doc alignment

7 days ago

Fix since prop order

#5 @Mista-Flo
7 days ago

  • Keywords needs-refresh removed

Hi there, I have refreshed the patch to get a proper alignment.

#6 @johnbillion
7 days ago

  • Owner changed from SergeyBiryukov to johnbillion

7 days ago

#7 @johnbillion
7 days ago

  • Keywords commit added

This is looking good. In 50835.4.diff I've made two changes:

  1. Ordered the block by the name of the property.
  2. Remove usage of the Saxon genitive, this is something we don't typically use in core although we're definitely not consistent about it.

Unless anyone objects to the latter, this is ready for commit.

#8 @Mista-Flo
7 days ago

Nice one John thank you, looks good like this.

#9 @SergeyBiryukov
6 days ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 49281:

Docs: Document the return value of wp_prepare_attachment_for_js() using hash notation.

Props stevenlinx, Mista-Flo, johnbillion.
Fixes #50835.

Note: See TracTickets for help on using tickets.