comparison: src/main/java/de/uapcore/lightpit/modules/UsersModule.java
src/main/java/de/uapcore/lightpit/modules/UsersModule.java
- changeset 59
- c759c60507a2
- parent 51
- dd0a45ae25d7
- child 66
- 635ae67e73b5
equal
deleted
inserted
replaced
| 58:8d3047f78190 | 59:c759c60507a2 |
|---|---|
| 30 | 30 |
| 31 | 31 |
| 32 import de.uapcore.lightpit.*; | 32 import de.uapcore.lightpit.*; |
| 33 import de.uapcore.lightpit.dao.DataAccessObjects; | 33 import de.uapcore.lightpit.dao.DataAccessObjects; |
| 34 import de.uapcore.lightpit.entities.User; | 34 import de.uapcore.lightpit.entities.User; |
| 35 import org.slf4j.Logger; | |
| 36 import org.slf4j.LoggerFactory; | |
| 35 | 37 |
| 36 import javax.servlet.annotation.WebServlet; | 38 import javax.servlet.annotation.WebServlet; |
| 37 import javax.servlet.http.HttpServletRequest; | 39 import javax.servlet.http.HttpServletRequest; |
| 38 import java.sql.SQLException; | 40 import java.sql.SQLException; |
| 41 import java.util.NoSuchElementException; | |
| 39 import java.util.Optional; | 42 import java.util.Optional; |
| 40 | 43 |
| 41 @LightPITModule( | 44 @LightPITModule( |
| 42 bundleBaseName = "localization.users", | 45 bundleBaseName = "localization.users", |
| 43 modulePath = "teams", | 46 modulePath = "teams", |
| 46 @WebServlet( | 49 @WebServlet( |
| 47 name = "UsersModule", | 50 name = "UsersModule", |
| 48 urlPatterns = "/teams/*" | 51 urlPatterns = "/teams/*" |
| 49 ) | 52 ) |
| 50 public final class UsersModule extends AbstractLightPITServlet { | 53 public final class UsersModule extends AbstractLightPITServlet { |
| 54 | |
| 55 private static final Logger LOG = LoggerFactory.getLogger(UsersModule.class); | |
| 51 | 56 |
| 52 @RequestMapping(method = HttpMethod.GET) | 57 @RequestMapping(method = HttpMethod.GET) |
| 53 public ResponseType index(HttpServletRequest req, DataAccessObjects dao) throws SQLException { | 58 public ResponseType index(HttpServletRequest req, DataAccessObjects dao) throws SQLException { |
| 54 final var userDao = dao.getUserDao(); | 59 final var userDao = dao.getUserDao(); |
| 55 | 60 |
| 88 | 93 |
| 89 dao.getUserDao().saveOrUpdate(user); | 94 dao.getUserDao().saveOrUpdate(user); |
| 90 | 95 |
| 91 setRedirectLocation(req, "./teams/"); | 96 setRedirectLocation(req, "./teams/"); |
| 92 setDynamicFragment(req, Constants.DYN_FRAGMENT_COMMIT_SUCCESSFUL); | 97 setDynamicFragment(req, Constants.DYN_FRAGMENT_COMMIT_SUCCESSFUL); |
| 93 } catch (NullPointerException | NumberFormatException | SQLException ex) { | 98 |
| 99 LOG.debug("Successfully updated user {}", user.getUsername()); | |
| 100 } catch (NoSuchElementException | NumberFormatException | SQLException ex) { | |
| 94 // TODO: set request attribute with error text | 101 // TODO: set request attribute with error text |
| 95 req.setAttribute("user", user); | 102 req.setAttribute("user", user); |
| 96 setDynamicFragment(req, "user-form"); | 103 setDynamicFragment(req, "user-form"); |
| 104 LOG.warn("Form validation failure: {}", ex.getMessage()); | |
| 105 LOG.debug("Details:", ex); | |
| 97 } | 106 } |
| 98 | 107 |
| 99 return ResponseType.HTML; | 108 return ResponseType.HTML; |
| 100 } | 109 } |
| 101 } | 110 } |
