Make WordPress Core

Opened 4 years ago

Last modified 20 months ago

#17950 closed enhancement

Introduce wp_registration_url() — at Version 7

Reported by: scribu Owned by:
Milestone: 3.6 Priority: low
Severity: normal Version:
Component: General Keywords: has-patch commit
Focuses: Cc:

Description (last modified by scribu)

It would be handy to have a wp_registration_url() template tag, to complete the wp_login_url() and wp_logout_url() triad.

Change History (11)

@scribu4 years ago

@scribu4 years ago

$redirect param isn't useful

comment:1 @scribu4 years ago

I noticed that wp-login.php itself never uses wp_login_url() so I didn't use wp_register_url() there either.

comment:2 @scribu3 years ago

Related: #19898

comment:3 @meloniq3 years ago

  • Cc meloniq@… added

comment:4 @kovshenin3 years ago

In 17950.2.diff since wp_register_url is filterable, we should probably esc_url when printing the register link. Also, you might want to remove the $redirect variable since it's undefined and we're not going to be using it anyway :)

@scribu3 years ago

comment:5 @scribu3 years ago

Fixed in 17950.3.diff and also use it in wp-login.php.

comment:6 @MikeSchinkel3 years ago

Something to consider, wp_register_url() could easily be confused to provide functionality to register a URL like register_taxonomy() registers a taxonomy.

Suggestion: Change from verb to noun and call it wp_registration_url().

Last edited 3 years ago by MikeSchinkel (previous) (diff)

@scribu3 years ago


comment:7 @scribu3 years ago

  • Description modified (diff)
  • Summary changed from Introduce wp_register_url() to Introduce wp_registration_url()

Good idea!

Note: See TracTickets for help on using tickets.