src/main/java/de/uapcore/lightpit/modules/UsersModule.java

changeset 59
c759c60507a2
parent 51
dd0a45ae25d7
child 66
635ae67e73b5
     1.1 --- a/src/main/java/de/uapcore/lightpit/modules/UsersModule.java	Sun May 17 16:00:13 2020 +0200
     1.2 +++ b/src/main/java/de/uapcore/lightpit/modules/UsersModule.java	Sun May 17 16:23:39 2020 +0200
     1.3 @@ -32,10 +32,13 @@
     1.4  import de.uapcore.lightpit.*;
     1.5  import de.uapcore.lightpit.dao.DataAccessObjects;
     1.6  import de.uapcore.lightpit.entities.User;
     1.7 +import org.slf4j.Logger;
     1.8 +import org.slf4j.LoggerFactory;
     1.9  
    1.10  import javax.servlet.annotation.WebServlet;
    1.11  import javax.servlet.http.HttpServletRequest;
    1.12  import java.sql.SQLException;
    1.13 +import java.util.NoSuchElementException;
    1.14  import java.util.Optional;
    1.15  
    1.16  @LightPITModule(
    1.17 @@ -49,6 +52,8 @@
    1.18  )
    1.19  public final class UsersModule extends AbstractLightPITServlet {
    1.20  
    1.21 +    private static final Logger LOG = LoggerFactory.getLogger(UsersModule.class);
    1.22 +
    1.23      @RequestMapping(method = HttpMethod.GET)
    1.24      public ResponseType index(HttpServletRequest req, DataAccessObjects dao) throws SQLException {
    1.25          final var userDao = dao.getUserDao();
    1.26 @@ -90,10 +95,14 @@
    1.27  
    1.28              setRedirectLocation(req, "./teams/");
    1.29              setDynamicFragment(req, Constants.DYN_FRAGMENT_COMMIT_SUCCESSFUL);
    1.30 -        } catch (NullPointerException | NumberFormatException | SQLException ex) {
    1.31 +
    1.32 +            LOG.debug("Successfully updated user {}", user.getUsername());
    1.33 +        } catch (NoSuchElementException | NumberFormatException | SQLException ex) {
    1.34              // TODO: set request attribute with error text
    1.35              req.setAttribute("user", user);
    1.36              setDynamicFragment(req, "user-form");
    1.37 +            LOG.warn("Form validation failure: {}", ex.getMessage());
    1.38 +            LOG.debug("Details:", ex);
    1.39          }
    1.40  
    1.41          return ResponseType.HTML;

mercurial