Mercurial > dive4elements > framework
comparison artifact-database/src/main/java/de/intevation/artifactdatabase/Backend.java @ 154:1a72f08ce8d7
Propagate delete user errors correctly.
artifacts/trunk@1379 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author | Sascha L. Teichmann <sascha.teichmann@intevation.de> |
---|---|
date | Thu, 03 Mar 2011 09:36:03 +0000 |
parents | bf596b83d984 |
children | a76de72ad6d1 |
comparison
equal
deleted
inserted
replaced
153:bf596b83d984 | 154:1a72f08ce8d7 |
---|---|
893 } | 893 } |
894 } | 894 } |
895 return null; | 895 return null; |
896 } | 896 } |
897 | 897 |
898 public void deleteUser(String identifier) { | 898 public boolean deleteUser(String identifier) { |
899 | 899 |
900 Connection conn = null; | 900 Connection conn = null; |
901 ResultSet result = null; | 901 ResultSet result = null; |
902 PreparedStatement stmnt = null; | 902 PreparedStatement stmnt = null; |
903 | 903 |
904 if (!StringUtils.checkUUID(identifier)) { | 904 if (!StringUtils.checkUUID(identifier)) { |
905 return; | 905 return false; |
906 } | 906 } |
907 | 907 |
908 DataSource dataSource = DBConnection.getDataSource(); | 908 DataSource dataSource = DBConnection.getDataSource(); |
909 try { | 909 try { |
910 conn = dataSource.getConnection(); | 910 conn = dataSource.getConnection(); |
916 | 916 |
917 stmnt.setString(1, identifier); | 917 stmnt.setString(1, identifier); |
918 result = stmnt.executeQuery(); | 918 result = stmnt.executeQuery(); |
919 | 919 |
920 if (!result.next()) { // No such user | 920 if (!result.next()) { // No such user |
921 return; | 921 return false; |
922 } | 922 } |
923 | 923 |
924 int id = result.getInt(1); | 924 int id = result.getInt(1); |
925 | 925 |
926 result.close(); result = null; | 926 result.close(); result = null; |
954 stmnt = conn.prepareStatement(SQL_USERS_DELETE_ID); | 954 stmnt = conn.prepareStatement(SQL_USERS_DELETE_ID); |
955 stmnt.setInt(1, id); | 955 stmnt.setInt(1, id); |
956 stmnt.execute(); | 956 stmnt.execute(); |
957 | 957 |
958 conn.commit(); | 958 conn.commit(); |
959 | |
960 return true; | |
959 } | 961 } |
960 catch (SQLException sqle) { | 962 catch (SQLException sqle) { |
961 conn.rollback(); | 963 conn.rollback(); |
962 throw sqle; | 964 throw sqle; |
963 } | 965 } |
977 if (conn != null) { | 979 if (conn != null) { |
978 try { conn.close(); } | 980 try { conn.close(); } |
979 catch (SQLException sqle) {} | 981 catch (SQLException sqle) {} |
980 } | 982 } |
981 } | 983 } |
984 | |
985 return false; | |
982 } | 986 } |
983 | 987 |
984 public User getUser(String identifier) { | 988 public User getUser(String identifier) { |
985 | 989 |
986 Connection conn = null; | 990 Connection conn = null; |