Make WordPress Core

Opened 9 years ago

Last modified 5 years ago

#34690 new enhancement

when user login is required to comment, there should be a link to registration

Reported by: mark-k's profile mark-k Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Comments Keywords: has-patch
Focuses: template Cc:

Description

Right now when user login is required the login form only shows a message about the need to login, but if registration is open, there is no reason not to send the visitor directly to the registretion page.

At the very least the default text of the link to the login page should be changed to some thing like "Login or Register" when registration is open.

Attachments (1)

34690.patch (1.9 KB) - added by subharanjan 8 years ago.
Display register link when enabled

Download all attachments as: .zip

Change History (6)

#1 @rachelbaker
9 years ago

  • Focuses template added
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release

Makes sense. @mark-k want to work on a patch?

#2 @jignesh.nakrani
9 years ago

I don't think we require registration link on comment template form. As user will get registration link on wp-login.php page if registration is open.

And if we need this link to be add I think text should be You must be logged in to post a comment or register here.

Last edited 9 years ago by jignesh.nakrani (previous) (diff)

#3 @mark-k
8 years ago

@jignesh.nakrani yes, the flow exists for the user, but when a non registered user sees this message for the first time he do not have any idea that the login form will also enable him to register, and in any case it is a two steps thing that could have been made into one.

@rachelbaker sorry for being lazy but it is early morning over here.... Is there any patch creation flow using the git repository, or does everything still needs to go via SVN?

#4 @Mista-Flo
8 years ago

@mark-k You can take the official miror repo on Github (sync every 15 min) no push, issue or pull request on this repo : https://github.com/WordPress/WordPress

Then, create a new branch for your ticket, do your commit, and then create a patch (you can use SmartGit for example or other GUI GIT tools).

Last edited 8 years ago by Mista-Flo (previous) (diff)

@subharanjan
8 years ago

Display register link when enabled

#5 @subharanjan
8 years ago

  • Keywords has-patch added; needs-patch removed
Note: See TracTickets for help on using tickets.