Opened 2 years ago

Closed 2 years ago

#15868 closed defect (bug) (fixed)

Undefined property: stdClass::$id in wp-admin\network\edit.php

Reported by: ocean90 Owned by:
Priority: normal Milestone: 3.1
Component: Users Version: 3.1
Severity: normal Keywords: has-patch commit
Cc:

Description

Go to network/users and delete a user:

Notice: Undefined property: stdClass::$id in H:\xampplite\htdocs\wp\wp-admin\network\edit.php on line 61

Notice: Undefined property: stdClass::$id in H:\xampplite\htdocs\wp\wp-admin\network\edit.php on line 62

Attachments (1)

15868.patch​ (683 bytes) - added by ocean90 2 years ago.
relativ

Download all attachments as: .zip

Change History (7)

comment:1 follow-up: ↓ 2   ocean90 — 2 years ago

Maybe we can use wp_dropdown_user here?

comment:2 in reply to: ↑ 1   scribu — 2 years ago

  • Keywords commit added

Patch works fine.

Replying to ocean90:

Maybe we can use wp_dropdown_user here?

We could, but not in 3.1

ocean90 — 2 years ago

relativ

comment:3   ryan — 2 years ago

Looks good.

comment:4 follow-up: ↓ 5   nacin — 2 years ago

$user should be a WP_User object, no? According to the WP_User docs, $id should still be set (though deprecated, with ID being canonical). What am I missing?

comment:5 in reply to: ↑ 4   scribu — 2 years ago

No, it's a stdClass now, for performance reasons.

comment:6   nacin — 2 years ago

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

(In [17042]) s/id/ID/. props ocean90, fixes #15868.

Note: See TracTickets for help on using tickets.