Ticket #7576 (closed enhancement: fixed)

Opened 1 year ago

Last modified 1 year ago

[patch] Users redirected to login screen with no confirmation may fail to recognize that they are registered

Reported by: anonymous Assigned to: pacopablo
Priority: normal Component: AccountManagerPlugin
Severity: critical Keywords: user register trivial
Cc: Trac Release: 0.11

Description

When no email verification is used, the user is registered and immediately redirected to login page. Most don't understand this and try registering again and now get message "user already exists".

It should be made clear registered actually succeeded, before redirecting to login.

Attachments

acct_mgr-0.2.1dev_with-7576-fix.png (40.7 kB) - added by hasienda on 09/28/10 02:15:46.
Confirmation after successful registration of user «test» in Trac 0.12

Change History

(follow-up: ↓ 2 ) 08/29/10 23:18:31 changed by rjollos

  • summary changed from users constantly fail to recognice they are registeren to Users are redirected to the login screen with no confirmation email and fail to recognize they are registered.

It should be fairly straightforward to use add_notice to display a message to the user and show they are registered.

(in reply to: ↑ 1 ) 09/26/10 03:04:13 changed by hasienda

  • keywords set to trivial.
  • type changed from defect to enhancement.
  • severity changed from normal to critical.
  • summary changed from Users are redirected to the login screen with no confirmation email and fail to recognize they are registered to Users redirected to login screen with no confirmation may fail to recognize that they are registered.

Replying to rjollos:

It should be fairly straightforward to use add_notice to display a message to the user and show they are registered.

Right, but a patch is still needed. Only I'm just too tired to do it right now.

09/28/10 02:01:40 changed by hasienda

  • keywords changed from trivial to user register trivial.
  • summary changed from Users redirected to login screen with no confirmation may fail to recognize that they are registered to [patch] Users redirected to login screen with no confirmation may fail to recognize that they are registered.

Good news: While stumbling through the code to debug another issue I spotted a nice place, where this could be hooked into.

I've a nice working solution now, almost trivial, but need to separate from other changes I've prepared in parallel. Patch will be attached later.

09/28/10 02:15:46 changed by hasienda

  • attachment acct_mgr-0.2.1dev_with-7576-fix.png added.

Confirmation after successful registration of user «test» in Trac 0.12

09/28/10 02:19:37 changed by hasienda

See screen capture with fix applied to trunk.

This is already an heavily patched test application with i18n support and German translation in place as well. The English original message test would be 

Registration has been finished successfully. You may login as user %(user)s now.

09/28/10 21:23:41 changed by hasienda

  • status changed from new to closed.
  • resolution set to fixed.

(In [9199]) AccountManagerPlugin: Add message to confirm successful registrations, closes #7576.

Add/Change #7576 ([patch] Users redirected to login screen with no confirmation may fail to recognize that they are registered)




Change Properties
Action