Make WordPress Core

Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#45604 closed enhancement (fixed)

Docs: Added description for global $wpdb

Reported by: mukesh27's profile mukesh27 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: normal Version: 5.0
Component: Database Keywords: has-patch dev-feedback good-first-bug
Focuses: docs Cc:

Description

In many core files document of global $wpdb id not added.

Attachments (2)

45604.patch (16.4 KB) - added by mukesh27 6 years ago.
Patch.
45604.2.patch (34.0 KB) - added by mukesh27 6 years ago.
Updated patch.

Download all attachments as: .zip

Change History (16)

@mukesh27
6 years ago

Patch.

#1 @SergeyBiryukov
6 years ago

  • Milestone changed from Awaiting Review to Future Release

Hi @mukesh27, thanks for the patch!

When adding the description for $wpdb in DocBlocks that have other global variables ($wp_query, $wp_locale, etc.), I think it would be a good idea to document those variables as well for consistency.

@mukesh27
6 years ago

Updated patch.

#2 @mukesh27
6 years ago

@SergeyBiryukov, updated patch for global WP_Screen, wp_locale, wp_query and wpdb.

#3 @SergeyBiryukov
6 years ago

#45772 was marked as a duplicate.

#4 @SergeyBiryukov
6 years ago

#46891 was marked as a duplicate.

This ticket was mentioned in Slack in #core by hareesh-pillai. View the logs.


6 years ago

#6 @SergeyBiryukov
6 years ago

  • Component changed from General to Database
  • Milestone changed from Future Release to 5.3
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#7 @SergeyBiryukov
5 years ago

In 45734:

Docs: Add missing description for $wpdb, $wp_db_version, and $wp_current_db_version globals.

Props mukesh27, utsav72640, immeet94, SergeyBiryukov.
See #45604.

#8 @SergeyBiryukov
5 years ago

In 45735:

Docs: Add missing description for $wp_rewrite global.

See #45604, #47110.

#9 @SergeyBiryukov
5 years ago

In 45736:

Docs: Add missing description for $wp global.

See #45604, #47110.

#10 @SergeyBiryukov
5 years ago

In 45737:

Docs: Add missing description for $wp_locale global.

Props mukesh27.
See #45604, #47110.

#11 @SergeyBiryukov
5 years ago

In 45738:

Docs: Add missing description for $wp_locale_switcher global.

See #45604, #47110.

#12 @SergeyBiryukov
5 years ago

In 45739:

Docs: Add missing description for $wp_query and $wp_the_query globals.

Props mukesh27.
See #45604, #47110.

#13 @SergeyBiryukov
5 years ago

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

In 45740:

Docs: Add missing description for $current_screen global.

Props mukesh27.
Fixes #45604. See #47110.

#14 @valentinbora
5 years ago

#41478 was marked as a duplicate.

Note: See TracTickets for help on using tickets.