search

kentyeh / gwtsecurity

Automatically exported from code.google.com/p/gwtsecurity
0 stars 0 forks source link

Add Additional Exceptions for Translation #8

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
I have created a patch that adds the following exceptions:

AccountExpiredException
AccountStatusException
CredentialsExpiredException
DisabledException
LockedException
UsernameNotFoundException

The patch moves the gwt exceptions into a separate package and refactors the 
GwtResponseUtil.processGwtException method.

The refactoring of the processGwtException method allows for easy addition of 
new exceptions that need to be added in the future by calling the 
createGwtException method.

I tried to provide the Chinese translation for my comments (using Google 
Translate) but they may not be correct.

Original issue reported on code.google.com by steven.j...@gmail.com on 8 Jun 2011 at 6:14

GoogleCodeExporter commented 8 years ago 
I have decided not to create an additional package for exceptions at this time. 
 This way we don't break others code.

Ivanenok,  I do think this would be a good idea for the branch you are working 
on.  We are going to have quite a few exceptions to deal with and I think it 
may warrant a separate package.

Anyway, whatever you think.

Original comment by steven.j...@gmail.com on 9 Jun 2011 at 3:41

GoogleCodeExporter commented 8 years ago 
Ok, I think we can talk about this change in project structure in my branch. I 
had delay in completion of my branch by some reasons independent from me :( We 
can discuss about it in skype.

Original comment by ivane...@gmail.com on 9 Jun 2011 at 4:33

GoogleCodeExporter commented 8 years ago 
my skype - ivanenok

Original comment by ivane...@gmail.com on 9 Jun 2011 at 4:49

GoogleCodeExporter commented 8 years ago 
Sounds good.  I will give you a call a little later.  Thanks!

Original comment by steven.j...@gmail.com on 9 Jun 2011 at 7:00