This is an automated notification sent by LCG Savannah.
It relates to:
bugs #59949, project CDS Invenio
==============================================================================
OVERVIEW of bugs #59949:
==============================================================================
URL:
<http://savannah.cern.ch/bugs/?59949>
Summary: Registration failure displays wrong error message
Project: CDS Invenio
Submitted by: jluker
Submitted on: 2009-12-01 23:02
Category: None
Severity: 3 - Normal
Priority: 5 - Normal
Item Group: None
Status: None
Privacy: Public
Assigned to: None
Originator Email:
Open/Closed: Open
Discussion Lock: Any
_______________________________________________________
When there are SMTP problems (such as missing or incorrect configuration) the
error message displayed on a failed registration is incorrect.
The message I see:
"Supplied email address [email protected] is invalid. Please try
again."
The message I should see:
"The site is having troubles in sending you an email for confirming your
email..."
This is because the registerUser function in webuser.py is returning the
wrong ruid value. It should be returning "6" but is returning "1". It will
only return "6" if it catches a smtplib.SMTPException or socket.error
exception, but those are being caught in the send_email function and not
re-raised.
I considered trying to tackle this myself but I'm not familiar enough with
the testing frameworks being used to confidently write a test, fix the code,
etc.
_______________________________________________________
Carbon-Copy List:
CC Address | Comment
------------------------------------+-----------------------------
5714 | -SUB-
==============================================================================
This item URL is:
<http://savannah.cern.ch/bugs/?59949>
_______________________________________________
Message sent via/by LCG Savannah
http://savannah.cern.ch/
