Stack trace in reset password if user doesn't have a name entered

Description

The name fields are not required for a user, and if these are left blank, and you attempt to reset the password, you get a stack trace. See testing tab for more info about how to reproduce on nightly.

Caused by: java.sql.BatchUpdateException: ORA-01400: cannot insert NULL into ("SAK"."VALIDATIONACCOUNT_ITEM"."FIRST_NAME")

at oracle.jdbc.driver.DatabaseError.throwBatchUpdateException(DatabaseError.java:343)
at oracle.jdbc.driver.OraclePreparedStatement.executeBatch(OraclePreparedStatement.java:10720)
at org.apache.commons.dbcp.DelegatingStatement.executeBatch(DelegatingStatement.java:297)
at org.apache.commons.dbcp.DelegatingStatement.executeBatch(DelegatingStatement.java:297)
at org.hibernate.jdbc.BatchingBatcher.doExecuteBatch(BatchingBatcher.java:70)
at org.hibernate.jdbc.AbstractBatcher.executeBatch(AbstractBatcher.java:268)
... 99 more
2014-01-09 15:17:57,921 WARN http-bio-8082-exec-8 PonderUtilCore - Exception rendering view:
java.lang.ClassCastException: java.lang.String cannot be cast to [Ljava.lang.String;

Attachments

2

Activity

Show:

Hudson CI Server May 30, 2014 at 12:04 PM

Integrated in sakai-10-java-1.7 #82 (See http://builds.sakaiproject.org:8080/job/sakai-10-java-1.7/82/)
database conversion scripts for 10 (Revision 309979)

Result = UNSTABLE

Earle Nietzel May 30, 2014 at 10:04 AM

svn commit -m "INFRSTR-257 database conversion scripts for 10" reference/
Adding reference/docs/conversion/sakai_10_0_mysql_conversion.sql
Adding reference/docs/conversion/sakai_10_0_oracle_conversion.sql
Transmitting file data ..
Committed revision 309979.

Matthew Jones May 6, 2014 at 9:49 AM

I think there was that question that came up on the list. If you either don't have an email address, or two people have the same email address (which Sakai also allows) then what happens with reset password?

That's not related to this ticket though.

Neal Caidin May 6, 2014 at 9:26 AM

Okay, I think this one can be verified based on testing and looking at the log files. The link in the email in the log file worked like a charm for the password reset and no stack trace.

However, I found one thing and I don't know if it is a bug or not.

The Email address field is not required. Of course, if I don't have an email address, I cannot reset my password!

Should the Email address field be required?

sakai/trunk on Oracle - Built: 05/06/14 08:00 - Sakai Revision: 309188 (Kernel 11-SNAPSHOT) - Server sakai-nightly.uits.iupui.edu

Hudson CI Server May 1, 2014 at 3:06 PM

Integrated in sakai-trunk-java-1.7 #218 (See http://builds.sakaiproject.org:8080/job/sakai-trunk-java-1.7/218/)
/ - Stack trace in reset password if user does not have a name entered (Revision 309054)

Result = UNSTABLE

Fixed

Details

Priority

Affects versions

Fix versions

Components

Assignee

Reporter

Conversion Script Required

Yes

Created January 9, 2014 at 2:34 PM
Updated July 1, 2014 at 7:44 AM
Resolved April 30, 2014 at 3:34 PM