Bug 695555

Summary: Transaction failed error when registering a user
Product: [Retired] Zanata Reporter: Kenichi Takemura <ktakemur>
Component: Authentication-InternalAssignee: Carlos Munoz <camunoz>
Status: CLOSED CURRENTRELEASE QA Contact: Alex Eng <aeng>
Severity: high Docs Contact:
Priority: low    
Version: 1.3-alpha-1CC: aeng, ebaak, mkim, sflaniga, zanata-bugs
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.5.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-03-07 00:06:56 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 786670    

Description Kenichi Takemura 2011-04-12 00:19:43 UTC
Description of problem:

After having registered a user, I can see 'Transaction failed error' if I add a new user that have same name and email but different Username.

Version-Release number of selected component (if applicable):
Zanata version 1.3-alpha-1 (20110330-1649). 

How reproducible:
always

Steps to Reproduce:
1. Add a new user and complete email activation
2. Add a new user whose name and email are the same but Username is different
3.
  
Actual results:
Error

    * Transaction failed
    * Unexpected error. Please try again. 

Expected results:
The system will accept the user added or 'User already registered' depending on Zanata spec.

Additional info:

Comment 1 Sean Flanigan 2011-09-07 04:32:59 UTC
Assigning to Scrum product owner for prioritisation.

Comment 2 Runa Bhattacharjee 2011-09-08 17:54:41 UTC
There should ideally be a check to ensure that an email address is mapped to only one user. The email address would be used for various purposes including account reset if required and should not be duplicated.

Comment 3 Sean Flanigan 2011-09-09 00:26:59 UTC
This error is generated because UI validation logic fails to check for duplicate email addresses, but the database *is* preventing the duplicate email address from being stored, thus causing an error.

Note that this hasn't been a high priority before now because internal authentication is only used in test environments, not in production.

Comment 5 Ding-Yi Chen 2011-09-09 02:05:30 UTC
Bug reproduced with Verion 1.4-SNAPSHOT (20110906-1403), internal authentication.

Comment 6 Carlos Munoz 2012-02-13 05:22:18 UTC
Added email validation when registering a new user with internal authentication. Zanata will now validate that the email address is well formed, as well as that the address is not already taken by another user before allowing the registration attempt to continue.

https://github.com/zanata/zanata/commit/0ac42cc809e5ea988978c55767b2bec54782787b

Comment 7 Alex Eng 2012-02-20 22:44:13 UTC
VERIFIED in Zanata version 1.5.0-alpha-2-SNAPSHOT (20120220-1431). Validation on duplicate email triggers to prevent this bug from happening.