-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7175/
-----------------------------------------------------------

(Updated Sept. 21, 2012, 7:38 p.m.)


Review request for rave.


Changes
-------

Code cleanup and added test cases


Description
-------

OpenID users user their URL to login into Rave. Get a username from the user 
and register the user when he logs in for the first time.


This addresses bug Rave-775.
    https://issues.apache.org/jira/browse/Rave-775


Diffs (updated)
-----

  https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/pom.xml 
1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/repository/UserRepository.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/UserService.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultNewAccountService.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultUserService.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/test/java/org/apache/rave/portal/service/impl/DefaultUserServiceTest.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaUser.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/repository/impl/JpaUserRepository.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-jpa/src/test/java/org/apache/rave/portal/repository/impl/JpaUserRepositoryTest.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-jpa/src/test/resources/test_data.sql
 1388609 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/controller/NewAccountController.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-web/src/test/java/org/apache/rave/portal/web/controller/NewAccountControllerTest.java
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/resources/messages.properties
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/applicationContext-security.xml
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/db/initial_data.sql
 1388609 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/jsp/views/includes/new_user_form.jsp
 1388565 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/tags/login_navbar.tag
 1388565 

Diff: https://reviews.apache.org/r/7175/diff/


Testing
-------


Thanks,

Viknes B

Reply via email to