Sat, 29 Aug 2020 17:28:24 +0200
fixes exception if integer parameter does not type check
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
1 | /* |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
2 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
3 | * |
24 | 4 | * Copyright 2018 Mike Becker. All rights reserved. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
5 | * |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
6 | * Redistribution and use in source and binary forms, with or without |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
7 | * modification, are permitted provided that the following conditions are met: |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
8 | * |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
9 | * 1. Redistributions of source code must retain the above copyright |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
10 | * notice, this list of conditions and the following disclaimer. |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
11 | * |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
12 | * 2. Redistributions in binary form must reproduce the above copyright |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
13 | * notice, this list of conditions and the following disclaimer in the |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
14 | * documentation and/or other materials provided with the distribution. |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
15 | * |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
16 | * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
17 | * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
18 | * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
19 | * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
20 | * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
21 | * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
22 | * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
23 | * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
24 | * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
25 | * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
26 | * POSSIBILITY OF SUCH DAMAGE. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
27 | * |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
28 | */ |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
29 | package de.uapcore.lightpit; |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
30 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
31 | import de.uapcore.lightpit.dao.DataAccessObjects; |
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
32 | import de.uapcore.lightpit.dao.postgres.PGDataAccessObjects; |
33 | 33 | import org.slf4j.Logger; |
34 | import org.slf4j.LoggerFactory; | |
35 | ||
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
36 | import javax.servlet.ServletException; |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
37 | import javax.servlet.http.HttpServlet; |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
38 | import javax.servlet.http.HttpServletRequest; |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
39 | import javax.servlet.http.HttpServletResponse; |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
40 | import javax.servlet.http.HttpSession; |
33 | 41 | import java.io.IOException; |
83
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
42 | import java.lang.reflect.*; |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
43 | import java.sql.Connection; |
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
44 | import java.sql.SQLException; |
33 | 45 | import java.util.*; |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
46 | import java.util.function.Function; |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
47 | |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
48 | /** |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
49 | * A special implementation of a HTTPServlet which is focused on implementing |
79
f64255a88d66
bloat removal 3/3 - LightPITModule annotation and ModuleManager
Mike Becker <universe@uap-core.de>
parents:
78
diff
changeset
|
50 | * the necessary functionality for LightPIT pages. |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
51 | */ |
9
20a9b2bc9063
makes LightPITServlet abstract
Mike Becker <universe@uap-core.de>
parents:
7
diff
changeset
|
52 | public abstract class AbstractLightPITServlet extends HttpServlet { |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
53 | |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
54 | private static final Logger LOG = LoggerFactory.getLogger(AbstractLightPITServlet.class); |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
55 | |
43
9abf0bf44c7b
renames some crappy constants
Mike Becker <universe@uap-core.de>
parents:
42
diff
changeset
|
56 | private static final String SITE_JSP = Functions.jspPath("site"); |
33 | 57 | |
58 | ||
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
59 | @FunctionalInterface |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
60 | protected interface SQLFindFunction<K, T> { |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
61 | T apply(K key) throws SQLException; |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
62 | |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
63 | default <V> SQLFindFunction<V, T> compose(Function<? super V, ? extends K> before) throws SQLException { |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
64 | Objects.requireNonNull(before); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
65 | return (v) -> this.apply(before.apply(v)); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
66 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
67 | |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
68 | default <V> SQLFindFunction<K, V> andThen(Function<? super T, ? extends V> after) throws SQLException { |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
69 | Objects.requireNonNull(after); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
70 | return (t) -> after.apply(this.apply(t)); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
71 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
72 | |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
73 | static <K> Function<K, K> identity() { |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
74 | return (t) -> t; |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
75 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
76 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
77 | |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
78 | /** |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
79 | * Invocation mapping gathered from the {@link RequestMapping} annotations. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
80 | * <p> |
18
a94b172c3a93
user friendly error pages for codes 404, 403 and 500
Mike Becker <universe@uap-core.de>
parents:
17
diff
changeset
|
81 | * Paths in this map must always start with a leading slash, although |
a94b172c3a93
user friendly error pages for codes 404, 403 and 500
Mike Becker <universe@uap-core.de>
parents:
17
diff
changeset
|
82 | * the specification in the annotation must not start with a leading slash. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
83 | * <p> |
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
84 | * The reason for this is the different handling of empty paths in |
18
a94b172c3a93
user friendly error pages for codes 404, 403 and 500
Mike Becker <universe@uap-core.de>
parents:
17
diff
changeset
|
85 | * {@link HttpServletRequest#getPathInfo()}. |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
86 | */ |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
87 | private final Map<HttpMethod, Map<String, Method>> mappings = new HashMap<>(); |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
88 | |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
89 | /** |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
90 | * Returns the name of the resource bundle associated with this servlet. |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
91 | * |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
92 | * @return the resource bundle base name |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
93 | */ |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
94 | protected abstract String getResourceBundleName(); |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
95 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
96 | |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
97 | /** |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
98 | * Creates a set of data access objects for the specified connection. |
33 | 99 | * |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
100 | * @param connection the SQL connection |
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
101 | * @return a set of data access objects |
17
d1036b776eee
adds getter for the database facade to the abstract servlet
Mike Becker <universe@uap-core.de>
parents:
15
diff
changeset
|
102 | */ |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
103 | private DataAccessObjects createDataAccessObjects(Connection connection) throws SQLException { |
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
104 | final var df = (DatabaseFacade) getServletContext().getAttribute(DatabaseFacade.SC_ATTR_NAME); |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
105 | if (df.getSQLDialect() == DatabaseFacade.Dialect.Postgres) { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
106 | return new PGDataAccessObjects(connection); |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
107 | } |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
108 | throw new AssertionError("Non-exhaustive if-else - this is a bug."); |
17
d1036b776eee
adds getter for the database facade to the abstract servlet
Mike Becker <universe@uap-core.de>
parents:
15
diff
changeset
|
109 | } |
33 | 110 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
111 | private ResponseType invokeMapping(Method method, HttpServletRequest req, HttpServletResponse resp, DataAccessObjects dao) throws IOException { |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
112 | try { |
14
2b270c714678
changed logger implementation to log4j2
Mike Becker <universe@uap-core.de>
parents:
13
diff
changeset
|
113 | LOG.trace("invoke {}#{}", method.getDeclaringClass().getName(), method.getName()); |
42
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
114 | final var paramTypes = method.getParameterTypes(); |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
115 | final var paramValues = new Object[paramTypes.length]; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
116 | for (int i = 0; i < paramTypes.length; i++) { |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
117 | if (paramTypes[i].isAssignableFrom(HttpServletRequest.class)) { |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
118 | paramValues[i] = req; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
119 | } else if (paramTypes[i].isAssignableFrom(HttpServletResponse.class)) { |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
120 | paramValues[i] = resp; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
121 | } |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
122 | if (paramTypes[i].isAssignableFrom(DataAccessObjects.class)) { |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
123 | paramValues[i] = dao; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
124 | } |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
125 | } |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
126 | return (ResponseType) method.invoke(this, paramValues); |
73
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
127 | } catch (InvocationTargetException ex) { |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
128 | LOG.error("invocation of method {}::{} failed: {}", |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
129 | method.getDeclaringClass().getName(), method.getName(), ex.getTargetException().getMessage()); |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
130 | LOG.debug("Details: ", ex.getTargetException()); |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
131 | resp.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, ex.getTargetException().getMessage()); |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
132 | return ResponseType.NONE; |
12 | 133 | } catch (ReflectiveOperationException | ClassCastException ex) { |
73
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
134 | LOG.error("invocation of method {}::{} failed: {}", |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
135 | method.getDeclaringClass().getName(), method.getName(), ex.getMessage()); |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
136 | LOG.debug("Details: ", ex); |
73
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
137 | resp.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, ex.getMessage()); |
12 | 138 | return ResponseType.NONE; |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
139 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
140 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
141 | |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
142 | @Override |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
143 | public void init() throws ServletException { |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
144 | scanForRequestMappings(); |
33 | 145 | |
12 | 146 | LOG.trace("{} initialized", getServletName()); |
147 | } | |
148 | ||
149 | private void scanForRequestMappings() { | |
150 | try { | |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
151 | Method[] methods = getClass().getDeclaredMethods(); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
152 | for (Method method : methods) { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
153 | Optional<RequestMapping> mapping = Optional.ofNullable(method.getAnnotation(RequestMapping.class)); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
154 | if (mapping.isPresent()) { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
155 | if (!Modifier.isPublic(method.getModifiers())) { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
156 | LOG.warn("{} is annotated with {} but is not public", |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
157 | method.getName(), RequestMapping.class.getSimpleName() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
158 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
159 | continue; |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
160 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
161 | if (Modifier.isAbstract(method.getModifiers())) { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
162 | LOG.warn("{} is annotated with {} but is abstract", |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
163 | method.getName(), RequestMapping.class.getSimpleName() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
164 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
165 | continue; |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
166 | } |
12 | 167 | if (!ResponseType.class.isAssignableFrom(method.getReturnType())) { |
168 | LOG.warn("{} is annotated with {} but has the wrong return type - 'ResponseType' required", | |
169 | method.getName(), RequestMapping.class.getSimpleName() | |
170 | ); | |
171 | continue; | |
172 | } | |
173 | ||
42
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
174 | boolean paramsInjectible = true; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
175 | for (var param : method.getParameterTypes()) { |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
176 | paramsInjectible &= HttpServletRequest.class.isAssignableFrom(param) |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
177 | || HttpServletResponse.class.isAssignableFrom(param) |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
178 | || DataAccessObjects.class.isAssignableFrom(param); |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
179 | } |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
180 | if (paramsInjectible) { |
58
8d3047f78190
fixes duplicated trailing slash if index path has a menu entry
Mike Becker <universe@uap-core.de>
parents:
57
diff
changeset
|
181 | String requestPath = "/" + mapping.get().requestPath(); |
12 | 182 | |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
183 | if (mappings |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
184 | .computeIfAbsent(mapping.get().method(), k -> new HashMap<>()) |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
185 | .putIfAbsent(requestPath, method) != null) { |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
186 | LOG.warn("{} {} has multiple mappings", |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
187 | mapping.get().method(), |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
188 | mapping.get().requestPath() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
189 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
190 | } |
12 | 191 | |
22
5a91fb7067af
minor changes to logging output
Mike Becker <universe@uap-core.de>
parents:
21
diff
changeset
|
192 | LOG.debug("{} {} maps to {}::{}", |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
193 | mapping.get().method(), |
18
a94b172c3a93
user friendly error pages for codes 404, 403 and 500
Mike Becker <universe@uap-core.de>
parents:
17
diff
changeset
|
194 | requestPath, |
22
5a91fb7067af
minor changes to logging output
Mike Becker <universe@uap-core.de>
parents:
21
diff
changeset
|
195 | getClass().getSimpleName(), |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
196 | method.getName() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
197 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
198 | } else { |
42
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
199 | LOG.warn("{} is annotated with {} but has the wrong parameters - only HttpServletRequest. HttpServletResponse, and DataAccessObjects are allowed", |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
200 | method.getName(), RequestMapping.class.getSimpleName() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
201 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
202 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
203 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
204 | } |
12 | 205 | } catch (SecurityException ex) { |
206 | LOG.error("Scan for request mappings on declared methods failed.", ex); | |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
207 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
208 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
209 | |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
210 | @Override |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
211 | public void destroy() { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
212 | mappings.clear(); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
213 | LOG.trace("{} destroyed", getServletName()); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
214 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
215 | |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
216 | /** |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
217 | * Sets the name of the content page. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
218 | * <p> |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
219 | * It is sufficient to specify the name without any extension. The extension |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
220 | * is added automatically if not specified. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
221 | * |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
222 | * @param req the servlet request object |
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
223 | * @param pageName the name of the content page |
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
224 | * @see Constants#REQ_ATTR_CONTENT_PAGE |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
225 | */ |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
226 | protected void setContentPage(HttpServletRequest req, String pageName) { |
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
227 | req.setAttribute(Constants.REQ_ATTR_CONTENT_PAGE, Functions.jspPath(pageName)); |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
228 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
229 | |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
230 | /** |
96
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
86
diff
changeset
|
231 | * Sets the navigation menu. |
71 | 232 | * |
96
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
86
diff
changeset
|
233 | * @param req the servlet request object |
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
86
diff
changeset
|
234 | * @param navigationItems the menu entries for the navigation menu |
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
86
diff
changeset
|
235 | * @see Constants#REQ_ATTR_NAVIGATION |
71 | 236 | */ |
97
602f75801644
first part of navigation redesign
Mike Becker <universe@uap-core.de>
parents:
96
diff
changeset
|
237 | protected void setNavigationMenu(HttpServletRequest req, List<MenuEntry> navigationItems) { |
96
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
86
diff
changeset
|
238 | req.setAttribute(Constants.REQ_ATTR_NAVIGATION, navigationItems); |
71 | 239 | } |
240 | ||
241 | /** | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
242 | * @param req the servlet request object |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
243 | * @param location the location where to redirect |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
244 | * @see Constants#REQ_ATTR_REDIRECT_LOCATION |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
245 | */ |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
246 | protected void setRedirectLocation(HttpServletRequest req, String location) { |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
247 | if (location.startsWith("./")) { |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
248 | location = location.replaceFirst("\\./", Functions.baseHref(req)); |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
249 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
250 | req.setAttribute(Constants.REQ_ATTR_REDIRECT_LOCATION, location); |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
251 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
252 | |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
253 | /** |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
254 | * Specifies the name of an additional stylesheet used by the module. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
255 | * <p> |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
256 | * Setting an additional stylesheet is optional, but quite common for HTML |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
257 | * output. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
258 | * <p> |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
259 | * It is sufficient to specify the name without any extension. The extension |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
260 | * is added automatically if not specified. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
261 | * |
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
262 | * @param req the servlet request object |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
263 | * @param stylesheet the name of the stylesheet |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
264 | */ |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
265 | public void setStylesheet(HttpServletRequest req, String stylesheet) { |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
266 | req.setAttribute(Constants.REQ_ATTR_STYLESHEET, Functions.enforceExt(stylesheet, ".css")); |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
267 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
268 | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
269 | /** |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
270 | * Sets the view model object. |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
271 | * The type must match the expected type in the JSP file. |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
272 | * |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
273 | * @param req the servlet request object |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
274 | * @param viewModel the view model object |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
275 | */ |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
276 | public void setViewModel(HttpServletRequest req, Object viewModel) { |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
277 | req.setAttribute(Constants.REQ_ATTR_VIEWMODEL, viewModel); |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
278 | } |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
279 | |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
280 | /** |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
281 | * Obtains a request parameter of the specified type. |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
282 | * The specified type must have a single-argument constructor accepting a string to perform conversion. |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
283 | * The constructor of the specified type may throw an exception on conversion failures. |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
284 | * |
71 | 285 | * @param req the servlet request object |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
286 | * @param clazz the class object of the expected type |
71 | 287 | * @param name the name of the parameter |
288 | * @param <T> the expected type | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
289 | * @return the parameter value or an empty optional, if no parameter with the specified name was found |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
290 | */ |
71 | 291 | protected <T> Optional<T> getParameter(HttpServletRequest req, Class<T> clazz, String name) { |
83
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
292 | if (clazz.isArray()) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
293 | final String[] paramValues = req.getParameterValues(name); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
294 | int len = paramValues == null ? 0 : paramValues.length; |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
295 | final var array = (T) Array.newInstance(clazz.getComponentType(), len); |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
296 | for (int i = 0; i < len; i++) { |
83
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
297 | try { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
298 | final Constructor<?> ctor = clazz.getComponentType().getConstructor(String.class); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
299 | Array.set(array, i, ctor.newInstance(paramValues[i])); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
300 | } catch (ReflectiveOperationException e) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
301 | throw new RuntimeException(e); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
302 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
303 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
304 | return Optional.of(array); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
305 | } else { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
306 | final String paramValue = req.getParameter(name); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
307 | if (paramValue == null) return Optional.empty(); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
308 | if (clazz.equals(Boolean.class)) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
309 | if (paramValue.toLowerCase().equals("false") || paramValue.equals("0")) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
310 | return Optional.of((T) Boolean.FALSE); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
311 | } else { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
312 | return Optional.of((T) Boolean.TRUE); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
313 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
314 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
315 | if (clazz.equals(String.class)) return Optional.of((T) paramValue); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
316 | if (java.sql.Date.class.isAssignableFrom(clazz)) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
317 | try { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
318 | return Optional.of((T) java.sql.Date.valueOf(paramValue)); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
319 | } catch (IllegalArgumentException ex) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
320 | return Optional.empty(); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
321 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
322 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
323 | try { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
324 | final Constructor<T> ctor = clazz.getConstructor(String.class); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
325 | return Optional.of(ctor.newInstance(paramValue)); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
326 | } catch (ReflectiveOperationException e) { |
106
9c6ebf67afe9
fixes exception if integer parameter does not type check
Mike Becker <universe@uap-core.de>
parents:
97
diff
changeset
|
327 | // does not type check and is not convertible - treat as if the parameter was never set |
9c6ebf67afe9
fixes exception if integer parameter does not type check
Mike Becker <universe@uap-core.de>
parents:
97
diff
changeset
|
328 | return Optional.empty(); |
80
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
329 | } |
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
330 | } |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
331 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
332 | |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
333 | /** |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
334 | * Tries to look up an entity with a key obtained from a request parameter. |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
335 | * |
71 | 336 | * @param req the servlet request object |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
337 | * @param clazz the class representing the type of the request parameter |
71 | 338 | * @param name the name of the request parameter |
339 | * @param find the find function (typically a DAO function) | |
340 | * @param <T> the type of the request parameter | |
341 | * @param <R> the type of the looked up entity | |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
342 | * @return the retrieved entity or an empty optional if there is no such entity or the request parameter was missing |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
343 | * @throws SQLException if the find function throws an exception |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
344 | */ |
71 | 345 | protected <T, R> Optional<R> findByParameter(HttpServletRequest req, Class<T> clazz, String name, SQLFindFunction<? super T, ? extends R> find) throws SQLException { |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
346 | final var param = getParameter(req, clazz, name); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
347 | if (param.isPresent()) { |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
348 | return Optional.ofNullable(find.apply(param.get())); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
349 | } else { |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
350 | return Optional.empty(); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
351 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
352 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
353 | |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
354 | private void forwardToFullView(HttpServletRequest req, HttpServletResponse resp) |
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
355 | throws IOException, ServletException { |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
356 | |
79
f64255a88d66
bloat removal 3/3 - LightPITModule annotation and ModuleManager
Mike Becker <universe@uap-core.de>
parents:
78
diff
changeset
|
357 | final String lightpitBundle = "localization.lightpit"; |
f64255a88d66
bloat removal 3/3 - LightPITModule annotation and ModuleManager
Mike Becker <universe@uap-core.de>
parents:
78
diff
changeset
|
358 | final var mainMenu = List.of( |
f64255a88d66
bloat removal 3/3 - LightPITModule annotation and ModuleManager
Mike Becker <universe@uap-core.de>
parents:
78
diff
changeset
|
359 | new MenuEntry(new ResourceKey(lightpitBundle, "menu.projects"), "projects/"), |
f64255a88d66
bloat removal 3/3 - LightPITModule annotation and ModuleManager
Mike Becker <universe@uap-core.de>
parents:
78
diff
changeset
|
360 | new MenuEntry(new ResourceKey(lightpitBundle, "menu.users"), "teams/"), |
f64255a88d66
bloat removal 3/3 - LightPITModule annotation and ModuleManager
Mike Becker <universe@uap-core.de>
parents:
78
diff
changeset
|
361 | new MenuEntry(new ResourceKey(lightpitBundle, "menu.languages"), "language/") |
f64255a88d66
bloat removal 3/3 - LightPITModule annotation and ModuleManager
Mike Becker <universe@uap-core.de>
parents:
78
diff
changeset
|
362 | ); |
71 | 363 | for (var entry : mainMenu) { |
364 | if (Functions.fullPath(req).startsWith("/" + entry.getPathName())) { | |
365 | entry.setActive(true); | |
366 | } | |
367 | } | |
368 | req.setAttribute(Constants.REQ_ATTR_MENU, mainMenu); | |
43
9abf0bf44c7b
renames some crappy constants
Mike Becker <universe@uap-core.de>
parents:
42
diff
changeset
|
369 | req.getRequestDispatcher(SITE_JSP).forward(req, resp); |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
370 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
371 | |
45
cc7f082c5ef3
simplifies menu generation, adds submenus and removes VersionsModule (versions will be part of the ProjectsModule)
Mike Becker <universe@uap-core.de>
parents:
43
diff
changeset
|
372 | private String sanitizeRequestPath(HttpServletRequest req) { |
cc7f082c5ef3
simplifies menu generation, adds submenus and removes VersionsModule (versions will be part of the ProjectsModule)
Mike Becker <universe@uap-core.de>
parents:
43
diff
changeset
|
373 | return Optional.ofNullable(req.getPathInfo()).orElse("/"); |
cc7f082c5ef3
simplifies menu generation, adds submenus and removes VersionsModule (versions will be part of the ProjectsModule)
Mike Becker <universe@uap-core.de>
parents:
43
diff
changeset
|
374 | } |
cc7f082c5ef3
simplifies menu generation, adds submenus and removes VersionsModule (versions will be part of the ProjectsModule)
Mike Becker <universe@uap-core.de>
parents:
43
diff
changeset
|
375 | |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
376 | private Optional<Method> findMapping(HttpMethod method, HttpServletRequest req) { |
45
cc7f082c5ef3
simplifies menu generation, adds submenus and removes VersionsModule (versions will be part of the ProjectsModule)
Mike Becker <universe@uap-core.de>
parents:
43
diff
changeset
|
377 | return Optional.ofNullable(mappings.get(method)).map(rm -> rm.get(sanitizeRequestPath(req))); |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
378 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
379 | |
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
380 | private void forwardAsSpecified(ResponseType type, HttpServletRequest req, HttpServletResponse resp) |
12 | 381 | throws ServletException, IOException { |
382 | switch (type) { | |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
383 | case NONE: |
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
384 | return; |
43
9abf0bf44c7b
renames some crappy constants
Mike Becker <universe@uap-core.de>
parents:
42
diff
changeset
|
385 | case HTML: |
12 | 386 | forwardToFullView(req, resp); |
387 | return; | |
388 | // TODO: implement remaining response types | |
389 | default: | |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
390 | throw new AssertionError("ResponseType switch is not exhaustive - this is a bug!"); |
12 | 391 | } |
392 | } | |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
393 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
394 | private void doProcess(HttpMethod method, HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { |
27
1f2a96efa69f
removes caching of main menu
Mike Becker <universe@uap-core.de>
parents:
24
diff
changeset
|
395 | |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
396 | // choose the requested language as session language (if available) or fall back to english, otherwise |
20
bd1a76c91d5b
module synchronization with database
Mike Becker <universe@uap-core.de>
parents:
18
diff
changeset
|
397 | HttpSession session = req.getSession(); |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
398 | if (session.getAttribute(Constants.SESSION_ATTR_LANGUAGE) == null) { |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
399 | Optional<List<String>> availableLanguages = Functions.availableLanguages(getServletContext()).map(Arrays::asList); |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
400 | Optional<Locale> reqLocale = Optional.of(req.getLocale()); |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
401 | Locale sessionLocale = reqLocale.filter((rl) -> availableLanguages.map((al) -> al.contains(rl.getLanguage())).orElse(false)).orElse(Locale.ENGLISH); |
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
402 | session.setAttribute(Constants.SESSION_ATTR_LANGUAGE, sessionLocale); |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
403 | LOG.debug("Setting language for new session {}: {}", session.getId(), sessionLocale.getDisplayLanguage()); |
14
2b270c714678
changed logger implementation to log4j2
Mike Becker <universe@uap-core.de>
parents:
13
diff
changeset
|
404 | } else { |
15
bb594abac796
language selector completed
Mike Becker <universe@uap-core.de>
parents:
14
diff
changeset
|
405 | Locale sessionLocale = (Locale) session.getAttribute(Constants.SESSION_ATTR_LANGUAGE); |
bb594abac796
language selector completed
Mike Becker <universe@uap-core.de>
parents:
14
diff
changeset
|
406 | resp.setLocale(sessionLocale); |
bb594abac796
language selector completed
Mike Becker <universe@uap-core.de>
parents:
14
diff
changeset
|
407 | LOG.trace("Continuing session {} with language {}", session.getId(), sessionLocale); |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
408 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
409 | |
21
b213fef2539e
adds first part of a module manager UI
Mike Becker <universe@uap-core.de>
parents:
20
diff
changeset
|
410 | // set some internal request attributes |
53
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
411 | final String fullPath = Functions.fullPath(req); |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
412 | req.setAttribute(Constants.REQ_ATTR_BASE_HREF, Functions.baseHref(req)); |
53
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
413 | req.setAttribute(Constants.REQ_ATTR_PATH, fullPath); |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
414 | req.setAttribute(Constants.REQ_ATTR_RESOURCE_BUNDLE, getResourceBundleName()); |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
415 | |
53
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
416 | // if this is an error path, bypass the normal flow |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
417 | if (fullPath.startsWith("/error/")) { |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
418 | final var mapping = findMapping(method, req); |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
419 | if (mapping.isPresent()) { |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
420 | forwardAsSpecified(invokeMapping(mapping.get(), req, resp, null), req, resp); |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
421 | } |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
422 | return; |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
423 | } |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
424 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
425 | // obtain a connection and create the data access objects |
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
426 | final var db = (DatabaseFacade) req.getServletContext().getAttribute(DatabaseFacade.SC_ATTR_NAME); |
53
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
427 | final var ds = db.getDataSource(); |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
428 | if (ds == null) { |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
429 | resp.sendError(HttpServletResponse.SC_SERVICE_UNAVAILABLE, "JNDI DataSource lookup failed. See log for details."); |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
430 | return; |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
431 | } |
6a8498291606
fixes bug where displaying an error page for missing data source would also require that data source (error pages don't try to get database connections now)
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
432 | try (final var connection = ds.getConnection()) { |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
433 | final var dao = createDataAccessObjects(connection); |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
434 | try { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
435 | connection.setAutoCommit(false); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
436 | // call the handler, if available, or send an HTTP 404 error |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
437 | final var mapping = findMapping(method, req); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
438 | if (mapping.isPresent()) { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
439 | forwardAsSpecified(invokeMapping(mapping.get(), req, resp, dao), req, resp); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
440 | } else { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
441 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
442 | } |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
443 | connection.commit(); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
444 | } catch (SQLException ex) { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
445 | LOG.warn("Database transaction failed (Code {}): {}", ex.getErrorCode(), ex.getMessage()); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
446 | LOG.debug("Details: ", ex); |
54
77e01cda5a40
adds missing spaces in error messages
Mike Becker <universe@uap-core.de>
parents:
53
diff
changeset
|
447 | resp.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, "Unhandled Transaction Error - Code: " + ex.getErrorCode()); |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
448 | connection.rollback(); |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
449 | } |
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
450 | } catch (SQLException ex) { |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
451 | LOG.error("Severe Database Exception (Code {}): {}", ex.getErrorCode(), ex.getMessage()); |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
452 | LOG.debug("Details: ", ex); |
54
77e01cda5a40
adds missing spaces in error messages
Mike Becker <universe@uap-core.de>
parents:
53
diff
changeset
|
453 | resp.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, "Database Error - Code: " + ex.getErrorCode()); |
12 | 454 | } |
455 | } | |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
456 | |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
457 | @Override |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
458 | protected final void doGet(HttpServletRequest req, HttpServletResponse resp) |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
459 | throws ServletException, IOException { |
12 | 460 | doProcess(HttpMethod.GET, req, resp); |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
461 | } |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
462 | |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
463 | @Override |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
464 | protected final void doPost(HttpServletRequest req, HttpServletResponse resp) |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
465 | throws ServletException, IOException { |
12 | 466 | doProcess(HttpMethod.POST, req, resp); |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
467 | } |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
468 | } |