Make WordPress Core

Opened 11 months ago

Closed 8 months 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 11 months ago.
50835.2.diff (3.8 KB) - added by Mista-Flo 8 months ago.
Fix doc alignment
50835.3.diff (3.6 KB) - added by Mista-Flo 8 months ago.
Fix since prop order
50835.4.diff (4.1 KB) - added by johnbillion 8 months ago.

Download all attachments as: .zip

Change History (13)

11 months ago

#1 @stevenlinx
11 months ago

  • Keywords has-patch added; needs-patch removed

#2 @SergeyBiryukov
11 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.

8 months ago

#4 @Mista-Flo
8 months ago

  • Keywords needs-refresh added

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

Any thought @SergeyBiryukov?

8 months ago

Fix doc alignment

8 months ago

Fix since prop order

#5 @Mista-Flo
8 months ago

  • Keywords needs-refresh removed

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

#6 @johnbillion
8 months ago

  • Owner changed from SergeyBiryukov to johnbillion

8 months ago

#7 @johnbillion
8 months 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
8 months ago

Nice one John thank you, looks good like this.

#9 @SergeyBiryukov
8 months 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.