WordPress.org
Get WordPress

Make WordPress Core

Opened 23 months ago

Closed 23 months ago

Last modified 23 months ago

#56839 closed defect (bug) (fixed)

Document the $wp_query global in WP_Media_List_Table::display_rows()

Reported by: upadalavipul's profile upadalavipul Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.2 Priority: normal
Severity: normal Version: 6.1
Component: Media Keywords: has-patch
Focuses: docs, administration Cc:

Description

I am checking for all function commenting details for good documentation and well comment and in which I have found one file that needs improvement. please check the below details.

file path: wp-admin/includes/class-wp-media-list-table.php

function name: display_rows()

Attachments (1)

56839.patch​ (445 bytes) - added by upadalavipul 23 months ago.

Download all attachments as: .zip

Change History (6)

@upadalavipul
23 months ago

This ticket was mentioned in ​PR #3495 on ​WordPress/wordpress-develop by ​@saumil1611.
23 months ago #1

  • Keywords has-patch added

Add param in doc
Ticket Ref: https://core.trac.wordpress.org/ticket/56839

#2 @SergeyBiryukov
23 months ago

  • Component changed from General to Media
  • Focuses administration added
  • Owner set to SergeyBiryukov
  • Status changed from new to accepted
  • Summary changed from Class function details need to improvement in class-wp-media-list-table.php file. to Document the $wp_query global in WP_Media_List_Table::display_rows()

#3 @SergeyBiryukov
23 months ago

  • Milestone changed from Awaiting Review to 6.2

#4 @SergeyBiryukov
23 months ago

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

In 54661:

Docs: Document the usage of $wp_query global in WP_Media_List_Table::display_rows().

Follow-up to [34127].

Props upadalavipul, saumil1611.
Fixes #56839.

​@SergeyBiryukov commented on ​PR #3495:
23 months ago #5

Thanks for the PR! Merged in r54661.

Note: See TracTickets for help on using tickets.