WordPress.org
Get WordPress

Make WordPress Core

Opened 20 years ago

Closed 20 years ago

Last modified 3 years ago

#3152 closed defect (bug) (fixed)

WP_User->ID, not WP_User->id

Reported by: alexkingorg Owned by:
Priority: normal Milestone: 2.1
Component: General Version: 2.1
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Fixed several references to WP_User->ID, the id is a lowercase var in this class.

Attachments (2)

patch.diff​ (853 bytes ) - added by alexkingorg 20 years ago.
patch attached
3152.diff​ (4.9 KB ) - added by Nazgul 20 years ago.

Download all attachments as: .zip

Change History (7)

@alexkingorg
20 years ago

patch attached

#1 @ryan
20 years ago

ID is set when we pull the users table data into the WP_User class. I'd like to stick with uppercase ID since it is more widely used due to it being in the table. How about adding var $ID = 0 to the WP_User class and putting a comment next to $id noting that it is deprecated?

#2 @foolswisdom
20 years ago

  • Milestone → 2.1

@Nazgul
20 years ago

#3 @Nazgul
20 years ago

  • Keywords has-patch added
  • Summary WP_User->id, not WP_User->ID → WP_User->ID, not WP_User->id

Created a patch, based on Ryan's suggestion.

#4 @ryan
20 years ago

  • Resolution → fixed
  • Status new → closed

(In [4504]) ID instead of id. Props Nazgul. fixes #3152

#5 @ryan
20 years ago

(In [4505]) ID instead of id. Props Nazgul. fixes #3152

Note: See TracTickets for help on using tickets.