Sat, 26 Dec 2020 20:19:09 +0100
adds "Create another" checkbox to issue form - fixes #110
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 | |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
31 | import de.uapcore.lightpit.dao.DataAccessObject; |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
32 | import de.uapcore.lightpit.dao.PostgresDataAccessObject; |
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; |
163
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
47 | import java.util.stream.Collectors; |
7
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 | /** |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
50 | * 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
|
51 | * 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
|
52 | */ |
9
20a9b2bc9063
makes LightPITServlet abstract
Mike Becker <universe@uap-core.de>
parents:
7
diff
changeset
|
53 | 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
|
54 | |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
55 | 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
|
56 | |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
57 | private static final String SITE_JSP = jspPath("site"); |
33 | 58 | |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
59 | /** |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
60 | * 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
|
61 | * <p> |
18
a94b172c3a93
user friendly error pages for codes 404, 403 and 500
Mike Becker <universe@uap-core.de>
parents:
17
diff
changeset
|
62 | * 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
|
63 | * 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
|
64 | * <p> |
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
65 | * 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
|
66 | * {@link HttpServletRequest#getPathInfo()}. |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
67 | */ |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
68 | private final Map<HttpMethod, Map<PathPattern, Method>> mappings = new HashMap<>(); |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
69 | |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
70 | /** |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
71 | * 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
|
72 | * |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
73 | * @return the resource bundle base name |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
74 | */ |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
75 | protected abstract String getResourceBundleName(); |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
76 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
77 | |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
78 | /** |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
79 | * Creates a set of data access objects for the specified connection. |
33 | 80 | * |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
81 | * @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
|
82 | * @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
|
83 | */ |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
84 | private DataAccessObject createDataAccessObjects(Connection connection) { |
151
b3f14cd4f3ab
migrate DataSourceProvider
Mike Becker <universe@uap-core.de>
parents:
145
diff
changeset
|
85 | final var df = (DataSourceProvider) getServletContext().getAttribute(DataSourceProvider.Companion.getSC_ATTR_NAME()); |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
86 | if (df.getDialect() == DataSourceProvider.Dialect.Postgres) { |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
87 | return new PostgresDataAccessObject(connection); |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
88 | } |
151
b3f14cd4f3ab
migrate DataSourceProvider
Mike Becker <universe@uap-core.de>
parents:
145
diff
changeset
|
89 | throw new UnsupportedOperationException("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
|
90 | } |
33 | 91 | |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
92 | private void invokeMapping(Map.Entry<PathPattern, Method> mapping, HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws IOException { |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
93 | final var pathPattern = mapping.getKey(); |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
94 | final var method = mapping.getValue(); |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
95 | try { |
14
2b270c714678
changed logger implementation to log4j2
Mike Becker <universe@uap-core.de>
parents:
13
diff
changeset
|
96 | 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
|
97 | final var paramTypes = method.getParameterTypes(); |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
98 | 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
|
99 | 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
|
100 | 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
|
101 | paramValues[i] = req; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
102 | } 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
|
103 | paramValues[i] = resp; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
104 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
105 | if (paramTypes[i].isAssignableFrom(DataAccessObject.class)) { |
42
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
106 | paramValues[i] = dao; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
107 | } |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
108 | if (paramTypes[i].isAssignableFrom(PathParameters.class)) { |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
109 | paramValues[i] = pathPattern.obtainPathParameters(sanitizeRequestPath(req)); |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
110 | } |
42
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
111 | } |
157
1e6f16fad3a5
removes ResponseType enum
Mike Becker <universe@uap-core.de>
parents:
151
diff
changeset
|
112 | method.invoke(this, paramValues); |
73
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
113 | } catch (InvocationTargetException ex) { |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
114 | LOG.error("invocation of method {}::{} failed: {}", |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
115 | method.getDeclaringClass().getName(), method.getName(), ex.getTargetException().getMessage()); |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
116 | LOG.debug("Details: ", ex.getTargetException()); |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
117 | resp.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, ex.getTargetException().getMessage()); |
12 | 118 | } catch (ReflectiveOperationException | ClassCastException ex) { |
73
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
119 | LOG.error("invocation of method {}::{} failed: {}", |
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
120 | 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
|
121 | LOG.debug("Details: ", ex); |
73
672b5003cafe
improves error message for InvocationTargetExceptions
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
122 | resp.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, ex.getMessage()); |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
123 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
124 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
125 | |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
126 | @Override |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
127 | public void init() throws ServletException { |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
128 | scanForRequestMappings(); |
33 | 129 | |
12 | 130 | LOG.trace("{} initialized", getServletName()); |
131 | } | |
132 | ||
133 | private void scanForRequestMappings() { | |
134 | try { | |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
135 | Method[] methods = getClass().getDeclaredMethods(); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
136 | for (Method method : methods) { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
137 | Optional<RequestMapping> mapping = Optional.ofNullable(method.getAnnotation(RequestMapping.class)); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
138 | if (mapping.isPresent()) { |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
139 | if (mapping.get().requestPath().isBlank()) { |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
140 | LOG.warn("{} is annotated with {} but request path is empty", |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
141 | method.getName(), RequestMapping.class.getSimpleName() |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
142 | ); |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
143 | continue; |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
144 | } |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
145 | |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
146 | if (!Modifier.isPublic(method.getModifiers())) { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
147 | LOG.warn("{} is annotated with {} but is not public", |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
148 | method.getName(), RequestMapping.class.getSimpleName() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
149 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
150 | continue; |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
151 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
152 | if (Modifier.isAbstract(method.getModifiers())) { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
153 | LOG.warn("{} is annotated with {} but is abstract", |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
154 | method.getName(), RequestMapping.class.getSimpleName() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
155 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
156 | continue; |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
157 | } |
12 | 158 | |
42
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
159 | boolean paramsInjectible = true; |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
160 | for (var param : method.getParameterTypes()) { |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
161 | paramsInjectible &= HttpServletRequest.class.isAssignableFrom(param) |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
162 | || HttpServletResponse.class.isAssignableFrom(param) |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
163 | || PathParameters.class.isAssignableFrom(param) |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
164 | || DataAccessObject.class.isAssignableFrom(param); |
42
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
165 | } |
f962ff9dd44e
makes every parameter in a handler method optional
Mike Becker <universe@uap-core.de>
parents:
40
diff
changeset
|
166 | if (paramsInjectible) { |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
167 | try { |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
168 | PathPattern pathPattern = new PathPattern(mapping.get().requestPath()); |
12 | 169 | |
131
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
170 | final var methodMappings = mappings.computeIfAbsent(mapping.get().method(), k -> new HashMap<>()); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
171 | final var currentMapping = methodMappings.putIfAbsent(pathPattern, method); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
172 | if (currentMapping != null) { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
173 | LOG.warn("Cannot map {} {} to {} in class {} - this would override the mapping to {}", |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
174 | mapping.get().method(), |
131
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
175 | mapping.get().requestPath(), |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
176 | method.getName(), |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
177 | getClass().getSimpleName(), |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
178 | currentMapping.getName() |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
179 | ); |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
180 | } |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
181 | |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
182 | LOG.debug("{} {} maps to {}::{}", |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
183 | mapping.get().method(), |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
184 | mapping.get().requestPath(), |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
185 | getClass().getSimpleName(), |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
186 | method.getName() |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
187 | ); |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
188 | } catch (IllegalArgumentException ex) { |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
189 | LOG.warn("Request mapping for {} failed: path pattern '{}' is syntactically invalid", |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
190 | method.getName(), mapping.get().requestPath() |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
191 | ); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
192 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
193 | } else { |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
194 | LOG.warn("{} is annotated with {} but has the wrong parameters - only HttpServletRequest, HttpServletResponse, PathParameters, and DataAccessObjects are allowed", |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
195 | method.getName(), RequestMapping.class.getSimpleName() |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
196 | ); |
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 | } |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
199 | } |
12 | 200 | } catch (SecurityException ex) { |
201 | 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
|
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 | |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
205 | @Override |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
206 | public void destroy() { |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
207 | mappings.clear(); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
208 | LOG.trace("{} destroyed", getServletName()); |
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
209 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
210 | |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
211 | /** |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
212 | * 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
|
213 | * <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
|
214 | * 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
|
215 | * 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
|
216 | * |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
217 | * @param req the servlet request object |
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
218 | * @param pageName the name of the content page |
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
219 | * @see Constants#REQ_ATTR_CONTENT_PAGE |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
220 | */ |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
73
diff
changeset
|
221 | protected void setContentPage(HttpServletRequest req, String pageName) { |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
222 | req.setAttribute(Constants.REQ_ATTR_CONTENT_PAGE, 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
|
223 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
224 | |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
225 | /** |
96
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
86
diff
changeset
|
226 | * Sets the navigation menu. |
71 | 227 | * |
109 | 228 | * @param req the servlet request object |
229 | * @param jspName the name of the menu's jsp file | |
96
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
86
diff
changeset
|
230 | * @see Constants#REQ_ATTR_NAVIGATION |
71 | 231 | */ |
109 | 232 | protected void setNavigationMenu(HttpServletRequest req, String jspName) { |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
233 | req.setAttribute(Constants.REQ_ATTR_NAVIGATION, jspPath(jspName)); |
71 | 234 | } |
235 | ||
236 | /** | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
237 | * @param req the servlet request object |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
238 | * @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
|
239 | * @see Constants#REQ_ATTR_REDIRECT_LOCATION |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
240 | */ |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
241 | 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
|
242 | if (location.startsWith("./")) { |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
243 | location = location.replaceFirst("\\./", baseHref(req)); |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
244 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
245 | 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
|
246 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
247 | |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
248 | /** |
163
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
249 | * Specifies the names of additional stylesheets used by this Servlet. |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
250 | * <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
|
251 | * 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
|
252 | * 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
|
253 | * |
163
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
254 | * @param req the servlet request object |
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
255 | * @param stylesheets the names of the stylesheets |
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 | */ |
163
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
257 | public void setStylesheet(HttpServletRequest req, String ... stylesheets) { |
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
258 | req.setAttribute(Constants.REQ_ATTR_STYLESHEET, Arrays |
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
259 | .stream(stylesheets) |
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
260 | .map(s -> enforceExt(s, ".css")) |
a5b9632729b6
adds possibility to specify multiple additional CSS files
Mike Becker <universe@uap-core.de>
parents:
160
diff
changeset
|
261 | .collect(Collectors.toUnmodifiableList())); |
10
89e3e6e28b69
implements automatic menu generation from module information
Mike Becker <universe@uap-core.de>
parents:
9
diff
changeset
|
262 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
263 | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
264 | /** |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
265 | * Sets the view model object. |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
266 | * 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
|
267 | * |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
268 | * @param req the servlet request object |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
269 | * @param viewModel the view model object |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
270 | */ |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
271 | 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
|
272 | 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
|
273 | } |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
274 | |
131
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
275 | private <T> Optional<T> parseParameter(String paramValue, Class<T> clazz) { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
276 | if (paramValue == null) return Optional.empty(); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
277 | if (clazz.equals(Boolean.class)) { |
168
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
278 | if (paramValue.equalsIgnoreCase("false") || paramValue.equals("0")) { |
131
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
279 | return Optional.of((T) Boolean.FALSE); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
280 | } else { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
281 | return Optional.of((T) Boolean.TRUE); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
282 | } |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
283 | } |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
284 | if (clazz.equals(String.class)) return Optional.of((T) paramValue); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
285 | if (java.sql.Date.class.isAssignableFrom(clazz)) { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
286 | try { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
287 | return Optional.of((T) java.sql.Date.valueOf(paramValue)); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
288 | } catch (IllegalArgumentException ex) { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
289 | return Optional.empty(); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
290 | } |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
291 | } |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
292 | try { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
293 | final Constructor<T> ctor = clazz.getConstructor(String.class); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
294 | return Optional.of(ctor.newInstance(paramValue)); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
295 | } catch (ReflectiveOperationException e) { |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
296 | // does not type check and is not convertible - treat as if the parameter was never set |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
297 | return Optional.empty(); |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
298 | } |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
299 | } |
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
300 | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
301 | /** |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
302 | * 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
|
303 | * 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
|
304 | * 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
|
305 | * |
71 | 306 | * @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
|
307 | * @param clazz the class object of the expected type |
71 | 308 | * @param name the name of the parameter |
309 | * @param <T> the expected type | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
310 | * @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
|
311 | */ |
71 | 312 | 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
|
313 | if (clazz.isArray()) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
314 | final String[] paramValues = req.getParameterValues(name); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
315 | int len = paramValues == null ? 0 : paramValues.length; |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
316 | 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
|
317 | 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
|
318 | try { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
319 | 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
|
320 | 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
|
321 | } catch (ReflectiveOperationException e) { |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
322 | throw new RuntimeException(e); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
323 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
324 | } |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
325 | return Optional.of(array); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
80
diff
changeset
|
326 | } else { |
131
67df332e3146
changes request mapping to contain project and version ID as path parameters (this removes the session storage)
Mike Becker <universe@uap-core.de>
parents:
130
diff
changeset
|
327 | return parseParameter(req.getParameter(name), clazz); |
80
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
328 | } |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
329 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
330 | |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
331 | /** |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
332 | * 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
|
333 | * |
71 | 334 | * @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
|
335 | * @param clazz the class representing the type of the request parameter |
71 | 336 | * @param name the name of the request parameter |
337 | * @param find the find function (typically a DAO function) | |
338 | * @param <T> the type of the request parameter | |
339 | * @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
|
340 | * @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
|
341 | * @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
|
342 | */ |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
163
diff
changeset
|
343 | protected <T, R> Optional<R> findByParameter(HttpServletRequest req, Class<T> clazz, String name, Function<? super T, ? extends R> find) { |
63
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
344 | 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
|
345 | 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
|
346 | 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
|
347 | } else { |
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.empty(); |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
349 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
350 | } |
51aa5e267c7f
adds utility function to find an entity by ID (reduces code duplication)
Mike Becker <universe@uap-core.de>
parents:
58
diff
changeset
|
351 | |
168
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
352 | protected void setAttributeFromParameter(HttpServletRequest req, String name) { |
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
353 | final var parm = req.getParameter(name); |
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
354 | if (parm != null) { |
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
355 | req.setAttribute(name, parm); |
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
356 | } |
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
357 | } |
1c3694ae224c
adds "Create another" checkbox to issue form - fixes #110
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
358 | |
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
|
359 | 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
|
360 | 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
|
361 | } |
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
|
362 | |
130
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
363 | private Optional<Map.Entry<PathPattern, Method>> findMapping(HttpMethod method, HttpServletRequest req) { |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
364 | return Optional.ofNullable(mappings.get(method)).flatMap(rm -> |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
365 | rm.entrySet().stream().filter( |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
366 | kv -> kv.getKey().matches(sanitizeRequestPath(req)) |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
367 | ).findAny() |
7ef369744fd1
adds the possibility to specify path parameters to RequestMapping
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
368 | ); |
11
737ab27e37b3
implements simple request mapper
Mike Becker <universe@uap-core.de>
parents:
10
diff
changeset
|
369 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
370 | |
157
1e6f16fad3a5
removes ResponseType enum
Mike Becker <universe@uap-core.de>
parents:
151
diff
changeset
|
371 | protected void renderSite(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { |
1e6f16fad3a5
removes ResponseType enum
Mike Becker <universe@uap-core.de>
parents:
151
diff
changeset
|
372 | req.getRequestDispatcher(SITE_JSP).forward(req, resp); |
12 | 373 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
374 | |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
375 | protected Optional<String[]> availableLanguages() { |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
376 | return Optional.ofNullable(getServletContext().getInitParameter(Constants.CTX_ATTR_LANGUAGES)).map((x) -> x.split("\\s*,\\s*")); |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
377 | } |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
378 | |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
379 | private static String baseHref(HttpServletRequest req) { |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
380 | return String.format("%s://%s:%d%s/", |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
381 | req.getScheme(), |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
382 | req.getServerName(), |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
383 | req.getServerPort(), |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
384 | req.getContextPath()); |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
385 | } |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
386 | |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
387 | private static String enforceExt(String filename, String ext) { |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
388 | return filename.endsWith(ext) ? filename : filename + ext; |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
389 | } |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
390 | |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
391 | private static String jspPath(String filename) { |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
392 | return enforceExt(Constants.JSP_PATH_PREFIX + filename, ".jsp"); |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
393 | } |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
394 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
395 | private void doProcess(HttpMethod method, HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { |
160
e2d09cf3fb96
forces the use of utf-8 as request character encoding - fixes #108
Mike Becker <universe@uap-core.de>
parents:
159
diff
changeset
|
396 | // the very first thing to do is to force UTF-8 |
e2d09cf3fb96
forces the use of utf-8 as request character encoding - fixes #108
Mike Becker <universe@uap-core.de>
parents:
159
diff
changeset
|
397 | req.setCharacterEncoding("UTF-8"); |
27
1f2a96efa69f
removes caching of main menu
Mike Becker <universe@uap-core.de>
parents:
24
diff
changeset
|
398 | |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
399 | // 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
|
400 | 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
|
401 | if (session.getAttribute(Constants.SESSION_ATTR_LANGUAGE) == null) { |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
402 | Optional<List<String>> availableLanguages = availableLanguages().map(Arrays::asList); |
13
f4608ad6c947
adds dynamic fragments to LightPIT request handling framework + basic language recognition code
Mike Becker <universe@uap-core.de>
parents:
12
diff
changeset
|
403 | 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
|
404 | 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
|
405 | 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
|
406 | 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
|
407 | } else { |
15
bb594abac796
language selector completed
Mike Becker <universe@uap-core.de>
parents:
14
diff
changeset
|
408 | Locale sessionLocale = (Locale) session.getAttribute(Constants.SESSION_ATTR_LANGUAGE); |
bb594abac796
language selector completed
Mike Becker <universe@uap-core.de>
parents:
14
diff
changeset
|
409 | resp.setLocale(sessionLocale); |
bb594abac796
language selector completed
Mike Becker <universe@uap-core.de>
parents:
14
diff
changeset
|
410 | 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
|
411 | } |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
412 | |
21
b213fef2539e
adds first part of a module manager UI
Mike Becker <universe@uap-core.de>
parents:
20
diff
changeset
|
413 | // set some internal request attributes |
158
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
414 | final String fullPath = req.getServletPath() + Optional.ofNullable(req.getPathInfo()).orElse(""); |
4f912cd42876
migrates constants and removes global functions
Mike Becker <universe@uap-core.de>
parents:
157
diff
changeset
|
415 | req.setAttribute(Constants.REQ_ATTR_BASE_HREF, 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
|
416 | req.setAttribute(Constants.REQ_ATTR_PATH, fullPath); |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
75
diff
changeset
|
417 | 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
|
418 | |
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
|
419 | // 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
|
420 | 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
|
421 | 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
|
422 | if (mapping.isPresent()) { |
157
1e6f16fad3a5
removes ResponseType enum
Mike Becker <universe@uap-core.de>
parents:
151
diff
changeset
|
423 | invokeMapping(mapping.get(), req, resp, null); |
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
|
424 | } |
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
|
425 | 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
|
426 | } |
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 | |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
428 | // obtain a connection and create the data access objects |
151
b3f14cd4f3ab
migrate DataSourceProvider
Mike Becker <universe@uap-core.de>
parents:
145
diff
changeset
|
429 | final var db = (DataSourceProvider) req.getServletContext().getAttribute(DataSourceProvider.Companion.getSC_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
|
430 | 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
|
431 | 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
|
432 | 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
|
433 | 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
|
434 | } |
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
|
435 | 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
|
436 | 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
|
437 | try { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
438 | connection.setAutoCommit(false); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
439 | // 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
|
440 | 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
|
441 | if (mapping.isPresent()) { |
157
1e6f16fad3a5
removes ResponseType enum
Mike Becker <universe@uap-core.de>
parents:
151
diff
changeset
|
442 | invokeMapping(mapping.get(), req, resp, dao); |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
443 | } else { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
444 | 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
|
445 | } |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
446 | connection.commit(); |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
447 | } catch (SQLException ex) { |
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
448 | 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
|
449 | LOG.debug("Details: ", ex); |
54
77e01cda5a40
adds missing spaces in error messages
Mike Becker <universe@uap-core.de>
parents:
53
diff
changeset
|
450 | 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
|
451 | connection.rollback(); |
38
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
452 | } |
cf85ef18f231
adds DAO for Project entity and save/update methods
Mike Becker <universe@uap-core.de>
parents:
36
diff
changeset
|
453 | } catch (SQLException ex) { |
39
e722861558bb
fixes minor issues that were reported by default inspection
Mike Becker <universe@uap-core.de>
parents:
38
diff
changeset
|
454 | 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
|
455 | LOG.debug("Details: ", ex); |
54
77e01cda5a40
adds missing spaces in error messages
Mike Becker <universe@uap-core.de>
parents:
53
diff
changeset
|
456 | resp.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, "Database Error - Code: " + ex.getErrorCode()); |
12 | 457 | } |
458 | } | |
34
824d4042c857
cleanup and simplification of database access layer
Mike Becker <universe@uap-core.de>
parents:
33
diff
changeset
|
459 | |
7
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
460 | @Override |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
461 | 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
|
462 | throws ServletException, IOException { |
12 | 463 | 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
|
464 | } |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
465 | |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
466 | @Override |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
467 | 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
|
468 | throws ServletException, IOException { |
12 | 469 | 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
|
470 | } |
598670d5b0b8
core functionality should also use the modules system, changed the code structure accordingly
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
471 | } |