WordPress.org

Make WordPress Core

Opened 3 weeks ago

Last modified 3 weeks ago

#53992 reopened enhancement

Lets add WP_User object to "registration_redirect" filter

Reported by: Collizo4sky Owned by:
Milestone: 5.9 Priority: normal
Severity: normal Version:
Component: Login and Registration Keywords: has-patch needs-testing
Focuses: Cc:

Description

I am the maintainer of Peters login redirect plugin. One of the most request feature is to redirect users after registration based on certain user characteristics.

Please, I want

$redirect_to = apply_filters( 'registration_redirect', $registration_redirect );

To change to

$redirect_to = apply_filters( 'registration_redirect', $registration_redirect, $errors );

Attachments (2)

53992.patch (696 bytes) - added by aadilali 3 weeks ago.
I have crated the patch for this.
53992-alt.patch (811 bytes) - added by aadilali 3 weeks ago.
Patch with @since note

Download all attachments as: .zip

Change History (10)

#1 @mukesh27
3 weeks ago

  • Component changed from General to Login and Registration

#2 @mukesh27
3 weeks ago

  • Keywords needs-patch added

Hi there!

If we add new argument then we also need to add document for it. Also i think we have to change that $errors variable with proper name so other don't get confuse.

#3 @Collizo4sky
3 weeks ago

@mukesh27 totally agreed. Currently away from my computer.

If you can submit a patch, that would be great.

#4 @SergeyBiryukov
3 weeks ago

  • Milestone changed from Awaiting Review to 5.9

@aadilali
3 weeks ago

I have crated the patch for this.

#5 @aadilali
3 weeks ago

  • Keywords has-patch needs-testing added; needs-patch removed
  • Resolution set to invalid
  • Status changed from new to closed

Hi,

Patch has been added but it need testing.

#6 @aadilali
3 weeks ago

  • Resolution invalid deleted
  • Status changed from closed to reopened

#7 follow-up: @mukesh27
3 weeks ago

@aadilali thanks for the patch.

I think we have to add a @since note for the new filter parameter

Last edited 3 weeks ago by mukesh27 (previous) (diff)

#8 in reply to: ↑ 7 @aadilali
3 weeks ago

Replying to mukesh27:

@aadilali thanks for the patch.

I think we have to add a @since note new filter parameter

Should I add @since note and update the patch?

@aadilali
3 weeks ago

Patch with @since note

Note: See TracTickets for help on using tickets.