Opened 19 months ago

Closed 4 months ago

#19134 closed enhancement (fixed)

Update the user role hook to include old roles.

Reported by: awellis13 Owned by: SergeyBiryukov
Priority: normal Milestone: 3.6
Component: Role/Capability Version: 3.2.1
Severity: normal Keywords: has-patch commit 3.4-early
Cc: alexkingorg, awellis13, devin@…, eddie.moya+wptrac@…

Description

While developing a plugin I needed to see the transition in roles. This is because I am delaying the sending of the new user welcome email until a certain role has been set. So I want to see if a certain role existed before when setting the new role. If yes, send the email.

Of course, I could hook on earlier in the actions and cache the current user's role, but it would be much easier if the old roles were passed into "set_user_role".

Attached is a diff of the change.

Attachments (1)

roleaction.diff (708 bytes) - added by awellis13 19 months ago.

Download all attachments as: .zip

Change History (6)

  • Cc devin@… added
  • Keywords commit 3.4-early added
  • Milestone changed from Awaiting Review to Future Release
  • Version changed from 3.3 to 3.2.1

Reminded me of the previous change here: #14708.

Looks good.

  • Cc eddie.moya+wptrac@… added
  • Milestone changed from Future Release to 3.6
  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 23358:

Pass old roles to set_user_role action. props awellis13. fixes #19134.

Note: See TracTickets for help on using tickets.