Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#46891 closed defect (bug) (duplicate)

Docs: Add missing description for global wpdb param

Reported by: immeet94 Owned by:
Priority: normal Milestone:
Component: General Version:
Severity: normal Keywords: has-patch needs-refresh
Cc: Focuses: docs, coding-standards

Description

i found that there is missing $wpdb declaration in files.

Attachments (2)

46891.diff (2.1 KB ) - added by immeet94 7 years ago.
Please see this file for patch
46891.2.diff (2.2 KB ) - added by immeet94 7 years ago.
Update patch and follow coding standards

Download all attachments as: .zip

Change History (7)

@immeet94
7 years ago

Please see this file for patch

#1 @swissspidy
7 years ago

  • Focuses coding-standards removed
  • Milestone Awaiting ReviewFuture Release
  • Priority normallow
  • Severity normalminor
  • Summary Needs documentation improvement.Docs: Add missing description for global wpdb param
  • Type defect (bug)enhancement
  • Version trunk

#2 @afercia
7 years ago

  • Keywords needs-refresh added
  • Priority lownormal
  • Severity minornormal
  • Type enhancementdefect (bug)

Thanks @immeet94 ! Searching through the codebase for @global wpdb, it seems there are more occurrences where wpdb misses a description. Would be great to fix also the other occurrences.

Also, when there are multiple globals (same for params), the type, varname, and descriptions should be aligned with spaces. An example would be:

 * @global wpdb   $wpdb         The WordPress database class.
 * @global string $table_prefix The database table prefix.

Reference to the documentation coding standards: https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#docblock-formatting

@immeet94
7 years ago

Update patch and follow coding standards

#3 @immeet94
7 years ago

Hello @afercia

i just update my patch and follow coding standards so please review it.

Thanks Meet

#4 @immeet94
7 years ago

  • Focuses coding-standards added

#5 @SergeyBiryukov
7 years ago

  • Milestone Future Release
  • Resolutionduplicate
  • Status newclosed

Hi @immeet94, thanks for the patch! We're already tracking this issue in #45604.

Note: See TracTickets for help on using tickets.