Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 8 years ago

Closed 7 years ago

Last modified 6 years ago

#45604 closed enhancement (fixed)

Docs: Added description for global $wpdb

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

Description

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

Attachments (2)

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

Download all attachments as: .zip

Change History (16)

@mukesh27
8 years ago

Attachment 45604.patch added

Patch.

#1 @SergeyBiryukov
8 years ago

Milestone Awaiting Review → 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
8 years ago

Attachment 45604.2.patch added

Updated patch.

#2 @mukesh27
8 years ago

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

#3 @SergeyBiryukov
8 years ago

#45772 was marked as a duplicate.

#4 @SergeyBiryukov
7 years ago

#46891 was marked as a duplicate.

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

7 years ago

#6 @SergeyBiryukov
7 years ago

Component General → Database
Milestone Future Release → 5.3
Owner set to SergeyBiryukov
Status new → reviewing

#7 @SergeyBiryukov
7 years ago

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

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

#8 @SergeyBiryukov
7 years ago

Docs: Add missing description for $wp_rewrite global.

See #45604, #47110.

#9 @SergeyBiryukov
7 years ago

Docs: Add missing description for $wp global.

See #45604, #47110.

#10 @SergeyBiryukov
7 years ago

Docs: Add missing description for $wp_locale global.

Props mukesh27.
See #45604, #47110.

#11 @SergeyBiryukov
7 years ago

Docs: Add missing description for $wp_locale_switcher global.

See #45604, #47110.

#12 @SergeyBiryukov
7 years ago

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

Props mukesh27.
See #45604, #47110.

#13 @SergeyBiryukov
7 years ago

Resolution → fixed
Status reviewing → closed

Docs: Add missing description for $current_screen global.

Props mukesh27.
Fixes #45604. See #47110.

#14 @valentinbora
6 years ago

#41478 was marked as a duplicate.

Note: See TracTickets for help on using tickets.

Download in other formats: