Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#3152 closed defect (bug) (fixed)

WP_User->ID, not WP_User->id

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


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 9 years ago.
patch attached
3152.diff (4.9 KB) - added by Nazgul 9 years ago.

Download all attachments as: .zip

Change History (7)

9 years ago

patch attached

#1 @ryan
9 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
9 years ago

  • Milestone set to 2.1

9 years ago

#3 @Nazgul
9 years ago

  • Keywords has-patch added
  • Summary changed from WP_User->id, not WP_User->ID to WP_User->ID, not WP_User->id

Created a patch, based on Ryan's suggestion.

#4 @ryan
9 years ago

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

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

#5 @ryan
9 years ago

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

Note: See TracTickets for help on using tickets.