Opened 2 years ago

Closed 2 years ago

#17533 closed enhancement (fixed)

Incorrect wpdb class docs

Reported by: johnbillion Owned by: ryan
Priority: normal Milestone: 3.2
Component: Inline Docs Version: 3.2
Severity: normal Keywords: has-patch commit
Cc:

Description

The inline docs for the wpdb class says:

/**
 * WordPress Database Access Abstraction Object
 *
 * It is possible to replace this class with your own
 * by setting the $wpdb global variable in wp-content/db.php
 * file with your class. You can name it wpdb also, since
 * this file will not be included, if the other file is
 * available.

This is incorrect because wp-includes/wp-db.php is always included.

Patch coming up.

Attachments (1)

17533.patch​ (643 bytes) - added by johnbillion 2 years ago.

Download all attachments as: .zip

Change History (4)

johnbillion — 2 years ago

comment:1   johnbillion — 2 years ago

  • Keywords has-patch added

Patch which improves the inline doc for wpdb.

comment:2   scribu — 2 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.2

That is correct:

http://core.trac.wordpress.org/browser/tags/3.1.2/wp-includes/load.php#L312

Last edited 2 years ago by scribu (previous) (diff)

comment:3   ryan — 2 years ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In [18011]:

Correct phpdoc for replacing/extending thewpdb class. Props johnbillion. fixes #17533

Note: See TracTickets for help on using tickets.