Ticket #3152 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

WP_User->ID, not WP_User->id

Reported by: alexkingorg Assigned to: anonymous
Priority: normal Milestone: 2.1
Component: General Version: 2.1
Severity: normal Keywords: has-patch
Cc:

Description

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

Attachments

patch.diff (0.8 kB) - added by alexkingorg on 09/21/06 16:50:33.
patch attached
3152.diff (4.9 kB) - added by Nazgul on 10/02/06 11:01:27.

Change History

09/21/06 16:50:33 changed by alexkingorg

  • attachment patch.diff added.

patch attached

09/21/06 21:08:24 changed by ryan

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?

09/21/06 22:08:38 changed by foolswisdom

  • milestone set to 2.1.

10/02/06 11:01:27 changed by Nazgul

  • attachment 3152.diff added.

10/02/06 11:03:41 changed by Nazgul

  • keywords set to has-patch.
  • 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.

11/20/06 04:29:08 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

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

11/20/06 05:01:58 changed by ryan

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