opendev/gerrit
Code Issues Proposed changes

Remove unnecessary throws OrmException in AccountResolver

Browse Source 
This method doesn't do database lookups directly, so it doesn't
throw the exception from them.

Change-Id: I742435a6e6dd92b07d1dff9464d73f940e4b0509
Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce
2010-08-04 13:53:55 -07:00
parent 92793d4bdc
commit 5e8dbd4f7e
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/account/AccountResolver.java
View File

@@ -115,7 +115,7 @@ public class AccountResolver {
return r.size() == 1 ? r.get(0) : null; return r.size() == 1 ? r.get(0) : null;
} }
private Account findByUserName(final String userName) throws OrmException { private Account findByUserName(final String userName) {
AccountState as = byId.getByUsername(userName); AccountState as = byId.getByUsername(userName);
return as != null ? as.getAccount() : null; return as != null ? as.getAccount() : null;
} }