Thu, 15 Oct 2020 14:01:49 +0200
add components to navmenu
41
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
1 | /* |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
2 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
3 | * |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
4 | * Copyright 2018 Mike Becker. All rights reserved. |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
5 | * |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
6 | * Redistribution and use in source and binary forms, with or without |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
7 | * modification, are permitted provided that the following conditions are met: |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
8 | * |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
9 | * 1. Redistributions of source code must retain the above copyright |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
10 | * notice, this list of conditions and the following disclaimer. |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
11 | * |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
12 | * 2. Redistributions in binary form must reproduce the above copyright |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
13 | * notice, this list of conditions and the following disclaimer in the |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
14 | * documentation and/or other materials provided with the distribution. |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
15 | * |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
16 | * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
17 | * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
18 | * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
19 | * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
20 | * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
21 | * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
22 | * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
23 | * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
24 | * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) |
4f1c026a8aab
adds files for ProjectsModule
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 |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
26 | * POSSIBILITY OF SUCH DAMAGE. |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
27 | * |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
28 | */ |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
29 | package de.uapcore.lightpit.modules; |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
30 | |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
31 | |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
32 | import de.uapcore.lightpit.*; |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
33 | import de.uapcore.lightpit.dao.DataAccessObjects; |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
34 | import de.uapcore.lightpit.entities.*; |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
35 | import de.uapcore.lightpit.viewmodel.*; |
121
428dca747d6b
adds application level issue sorting (fixes #19)
Mike Becker <universe@uap-core.de>
parents:
118
diff
changeset
|
36 | import de.uapcore.lightpit.viewmodel.util.IssueSorter; |
59 | 37 | import org.slf4j.Logger; |
38 | import org.slf4j.LoggerFactory; | |
41
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
39 | |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
40 | import javax.servlet.annotation.WebServlet; |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
41 | import javax.servlet.http.HttpServletRequest; |
59 | 42 | import javax.servlet.http.HttpServletResponse; |
43 | import java.io.IOException; | |
75
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
44 | import java.sql.Date; |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
45 | import java.sql.SQLException; |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
46 | import java.util.NoSuchElementException; |
99 | 47 | import java.util.Optional; |
83
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
48 | import java.util.stream.Collectors; |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
49 | import java.util.stream.Stream; |
41
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
50 | |
52 | 51 | import static de.uapcore.lightpit.Functions.fqn; |
52 | ||
41
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
53 | @WebServlet( |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
54 | name = "ProjectsModule", |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
55 | urlPatterns = "/projects/*" |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
56 | ) |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
57 | public final class ProjectsModule extends AbstractLightPITServlet { |
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
58 | |
59 | 59 | private static final Logger LOG = LoggerFactory.getLogger(ProjectsModule.class); |
60 | ||
99 | 61 | private static final String SESSION_ATTR_SELECTED_PROJECT = fqn(ProjectsModule.class, "selected_project"); |
62 | private static final String SESSION_ATTR_SELECTED_VERSION = fqn(ProjectsModule.class, "selected_version"); | |
129
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
63 | private static final String SESSION_ATTR_SELECTED_COMPONENT = fqn(ProjectsModule.class, "selected_component"); |
99 | 64 | private static final String PARAMETER_SELECTED_PROJECT = "pid"; |
65 | private static final String PARAMETER_SELECTED_VERSION = "vid"; | |
129
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
66 | private static final String PARAMETER_SELECTED_COMPONENT = "cid"; |
71 | 67 | |
78
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
76
diff
changeset
|
68 | @Override |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
76
diff
changeset
|
69 | protected String getResourceBundleName() { |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
76
diff
changeset
|
70 | return "localization.projects"; |
bb4c52bf3439
bloat removal 2/3 - moduleInfo
Mike Becker <universe@uap-core.de>
parents:
76
diff
changeset
|
71 | } |
71 | 72 | |
99 | 73 | private int syncParamWithSession(HttpServletRequest req, String param, String attr) { |
74 | final var session = req.getSession(); | |
75 | final var idParam = getParameter(req, Integer.class, param); | |
76 | final int id; | |
77 | if (idParam.isPresent()) { | |
78 | id = idParam.get(); | |
79 | session.setAttribute(attr, id); | |
80 | } else { | |
81 | id = Optional.ofNullable(session.getAttribute(attr)).map(x->(Integer)x).orElse(-1); | |
82 | } | |
83 | return id; | |
84 | } | |
85 | ||
86 | private void populate(ProjectView viewModel, HttpServletRequest req, DataAccessObjects dao) throws SQLException { | |
87 | final var projectDao = dao.getProjectDao(); | |
88 | final var versionDao = dao.getVersionDao(); | |
129
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
89 | final var componentDao = dao.getComponentDao(); |
99 | 90 | |
91 | projectDao.list().stream().map(ProjectInfo::new).forEach(viewModel.getProjectList()::add); | |
92 | ||
93 | // Select Project | |
94 | final int pid = syncParamWithSession(req, PARAMETER_SELECTED_PROJECT, SESSION_ATTR_SELECTED_PROJECT); | |
95 | if (pid >= 0) { | |
96 | final var project = projectDao.find(pid); | |
107
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
97 | if (project == null) { |
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
98 | req.setAttribute(SESSION_ATTR_SELECTED_PROJECT, -1); |
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
99 | } else { |
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
100 | final var info = new ProjectInfo(project); |
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
101 | info.setVersions(versionDao.list(project)); |
129
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
102 | info.setComponents(componentDao.list(project)); |
107
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
103 | info.setIssueSummary(projectDao.getIssueSummary(project)); |
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
104 | viewModel.setProjectInfo(info); |
b5f740a87af4
fixes misbehavior when a non-existing project ID is selected
Mike Becker <universe@uap-core.de>
parents:
105
diff
changeset
|
105 | } |
99 | 106 | } |
107 | ||
108 | // Select Version | |
109 | final int vid = syncParamWithSession(req, PARAMETER_SELECTED_VERSION, SESSION_ATTR_SELECTED_VERSION); | |
129
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
110 | if (vid > 0) { |
99 | 111 | viewModel.setVersionFilter(versionDao.find(vid)); |
129
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
112 | } else { |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
113 | // NULL for version means: show all unassigned |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
114 | viewModel.setVersionFilter(null); |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
115 | } |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
116 | |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
117 | // Select Component |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
118 | final int cid = syncParamWithSession(req, PARAMETER_SELECTED_COMPONENT, SESSION_ATTR_SELECTED_COMPONENT); |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
119 | if (cid > 0) { |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
120 | viewModel.setComponentFilter(componentDao.find(cid)); |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
121 | } else if (cid <= 0) { |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
122 | // -1 means: filter for unassigned, null means: show all |
a09d5c59351a
add components to navmenu
Mike Becker <universe@uap-core.de>
parents:
128
diff
changeset
|
123 | viewModel.setComponentFilter(new Component(-1)); |
99 | 124 | } |
125 | } | |
126 | ||
127 | private ResponseType forwardView(HttpServletRequest req, ProjectView viewModel, String name) { | |
128 | setViewModel(req, viewModel); | |
129 | setContentPage(req, name); | |
130 | setStylesheet(req, "projects"); | |
109 | 131 | setNavigationMenu(req, "project-navmenu"); |
99 | 132 | return ResponseType.HTML; |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
133 | } |
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
134 | |
61
3e287f361c7a
moves project index to separate sub path
Mike Becker <universe@uap-core.de>
parents:
59
diff
changeset
|
135 | @RequestMapping(method = HttpMethod.GET) |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
136 | public ResponseType index(HttpServletRequest req, DataAccessObjects dao) throws SQLException { |
99 | 137 | final var viewModel = new ProjectView(); |
138 | populate(viewModel, req, dao); | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
139 | |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
140 | final var projectDao = dao.getProjectDao(); |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
141 | final var versionDao = dao.getVersionDao(); |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
142 | |
99 | 143 | for (var info : viewModel.getProjectList()) { |
144 | info.setVersions(versionDao.list(info.getProject())); | |
145 | info.setIssueSummary(projectDao.getIssueSummary(info.getProject())); | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
146 | } |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
147 | |
99 | 148 | return forwardView(req, viewModel, "projects"); |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
149 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
150 | |
99 | 151 | private void configure(ProjectEditView viewModel, Project project, DataAccessObjects dao) throws SQLException { |
152 | viewModel.setProject(project); | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
153 | viewModel.setUsers(dao.getUserDao().list()); |
71 | 154 | } |
155 | ||
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
156 | @RequestMapping(requestPath = "edit", method = HttpMethod.GET) |
51
dd0a45ae25d7
adds the possibility to add users / developers
Mike Becker <universe@uap-core.de>
parents:
47
diff
changeset
|
157 | public ResponseType edit(HttpServletRequest req, DataAccessObjects dao) throws SQLException { |
99 | 158 | final var viewModel = new ProjectEditView(); |
159 | populate(viewModel, req, dao); | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
160 | |
99 | 161 | final var project = Optional.ofNullable(viewModel.getProjectInfo()) |
162 | .map(ProjectInfo::getProject) | |
163 | .orElse(new Project(-1)); | |
164 | configure(viewModel, project, dao); | |
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
|
165 | |
99 | 166 | return forwardView(req, viewModel, "project-form"); |
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
|
167 | } |
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
|
168 | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
169 | @RequestMapping(requestPath = "commit", method = HttpMethod.POST) |
68
ded1b5639bd3
adds request attribute users in commit() error path
Mike Becker <universe@uap-core.de>
parents:
67
diff
changeset
|
170 | public ResponseType commit(HttpServletRequest req, DataAccessObjects dao) throws SQLException { |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
171 | |
75
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
172 | Project project = new Project(-1); |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
173 | try { |
99 | 174 | project = new Project(getParameter(req, Integer.class, "pid").orElseThrow()); |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
175 | project.setName(getParameter(req, String.class, "name").orElseThrow()); |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
176 | getParameter(req, String.class, "description").ifPresent(project::setDescription); |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
177 | getParameter(req, String.class, "repoUrl").ifPresent(project::setRepoUrl); |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
178 | getParameter(req, Integer.class, "owner").map( |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
179 | ownerId -> ownerId >= 0 ? new User(ownerId) : null |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
180 | ).ifPresent(project::setOwner); |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
181 | |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
182 | dao.getProjectDao().saveOrUpdate(project); |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
183 | |
118
d3583959c875
adding a new project redirects to the projects overview (because we do not retrieve the ID of the inserted project yet)
Mike Becker <universe@uap-core.de>
parents:
110
diff
changeset
|
184 | setRedirectLocation(req, "./projects/"); |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
185 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
59 | 186 | LOG.debug("Successfully updated project {}", project.getName()); |
99 | 187 | |
188 | return ResponseType.HTML; | |
75
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
189 | } catch (NoSuchElementException | IllegalArgumentException | SQLException ex) { |
59 | 190 | LOG.warn("Form validation failure: {}", ex.getMessage()); |
191 | LOG.debug("Details:", ex); | |
99 | 192 | final var viewModel = new ProjectEditView(); |
193 | populate(viewModel, req, dao); | |
194 | configure(viewModel, project, dao); | |
195 | // TODO: error text | |
196 | return forwardView(req, viewModel, "project-form"); | |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
197 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
198 | } |
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
199 | |
70
821c4950b619
removes the sub menu and removes the home module
Mike Becker <universe@uap-core.de>
parents:
68
diff
changeset
|
200 | @RequestMapping(requestPath = "view", method = HttpMethod.GET) |
80
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
201 | public ResponseType view(HttpServletRequest req, HttpServletResponse resp, DataAccessObjects dao) throws SQLException, IOException { |
99 | 202 | final var viewModel = new ProjectDetailsView(); |
203 | populate(viewModel, req, dao); | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
204 | |
99 | 205 | if (viewModel.getProjectInfo() == null) { |
80
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
206 | resp.sendError(HttpServletResponse.SC_NOT_FOUND, "No project selected."); |
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
207 | return ResponseType.NONE; |
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
208 | } |
47
57cfb94ab99f
projects can now be added and updated
Mike Becker <universe@uap-core.de>
parents:
45
diff
changeset
|
209 | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
210 | final var issueDao = dao.getIssueDao(); |
70
821c4950b619
removes the sub menu and removes the home module
Mike Becker <universe@uap-core.de>
parents:
68
diff
changeset
|
211 | |
105
250c5cbb8276
simplifies issues per version view and re-adds edit version button
Mike Becker <universe@uap-core.de>
parents:
104
diff
changeset
|
212 | final var version = viewModel.getVersionFilter(); |
71 | 213 | |
99 | 214 | final var detailView = viewModel.getProjectDetails(); |
105
250c5cbb8276
simplifies issues per version view and re-adds edit version button
Mike Becker <universe@uap-core.de>
parents:
104
diff
changeset
|
215 | final var issues = issueDao.list(version); |
100
7e3c61c340d3
fixes issues and version information not set in correct order
Mike Becker <universe@uap-core.de>
parents:
99
diff
changeset
|
216 | for (var issue : issues) issueDao.joinVersionInformation(issue); |
121
428dca747d6b
adds application level issue sorting (fixes #19)
Mike Becker <universe@uap-core.de>
parents:
118
diff
changeset
|
217 | issues.sort(new IssueSorter( |
428dca747d6b
adds application level issue sorting (fixes #19)
Mike Becker <universe@uap-core.de>
parents:
118
diff
changeset
|
218 | new IssueSorter.Criteria(IssueSorter.Field.PHASE, true), |
428dca747d6b
adds application level issue sorting (fixes #19)
Mike Becker <universe@uap-core.de>
parents:
118
diff
changeset
|
219 | new IssueSorter.Criteria(IssueSorter.Field.ETA, true), |
428dca747d6b
adds application level issue sorting (fixes #19)
Mike Becker <universe@uap-core.de>
parents:
118
diff
changeset
|
220 | new IssueSorter.Criteria(IssueSorter.Field.UPDATED, false) |
428dca747d6b
adds application level issue sorting (fixes #19)
Mike Becker <universe@uap-core.de>
parents:
118
diff
changeset
|
221 | )); |
105
250c5cbb8276
simplifies issues per version view and re-adds edit version button
Mike Becker <universe@uap-core.de>
parents:
104
diff
changeset
|
222 | detailView.updateDetails(issues, version); |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
223 | |
99 | 224 | return forwardView(req, viewModel, "project-details"); |
71 | 225 | } |
226 | ||
109 | 227 | @RequestMapping(requestPath = "versions", method = HttpMethod.GET) |
228 | public ResponseType versions(HttpServletRequest req, HttpServletResponse resp, DataAccessObjects dao) throws IOException, SQLException { | |
229 | final var viewModel = new VersionsView(); | |
230 | populate(viewModel, req, dao); | |
231 | viewModel.setVersionFilter(null); | |
232 | ||
233 | final var projectInfo = viewModel.getProjectInfo(); | |
234 | if (projectInfo == null) { | |
235 | resp.sendError(HttpServletResponse.SC_NOT_FOUND, "No project selected."); | |
236 | return ResponseType.NONE; | |
237 | } | |
238 | ||
239 | final var issueDao = dao.getIssueDao(); | |
240 | final var issues = issueDao.list(projectInfo.getProject()); | |
241 | for (var issue : issues) issueDao.joinVersionInformation(issue); | |
242 | viewModel.update(projectInfo.getVersions(), issues); | |
243 | ||
244 | return forwardView(req, viewModel, "versions"); | |
245 | } | |
246 | ||
59 | 247 | @RequestMapping(requestPath = "versions/edit", method = HttpMethod.GET) |
110
9d0be0b1580f
adds indicators for version status
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
248 | public ResponseType editVersion(HttpServletRequest req, HttpServletResponse resp, DataAccessObjects dao) throws IOException, SQLException { |
99 | 249 | final var viewModel = new VersionEditView(); |
250 | populate(viewModel, req, dao); | |
251 | ||
110
9d0be0b1580f
adds indicators for version status
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
252 | if (viewModel.getProjectInfo() == null) { |
9d0be0b1580f
adds indicators for version status
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
253 | resp.sendError(HttpServletResponse.SC_NOT_FOUND, "No project selected."); |
9d0be0b1580f
adds indicators for version status
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
254 | return ResponseType.NONE; |
9d0be0b1580f
adds indicators for version status
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
255 | } |
9d0be0b1580f
adds indicators for version status
Mike Becker <universe@uap-core.de>
parents:
109
diff
changeset
|
256 | |
128
947d0f6a6a83
changes the way how to deal with child entities + adds component lead
Mike Becker <universe@uap-core.de>
parents:
124
diff
changeset
|
257 | viewModel.setVersion(Optional.ofNullable(viewModel.getVersionFilter()).orElse(new Version(-1))); |
59 | 258 | |
99 | 259 | return forwardView(req, viewModel, "version-form"); |
59 | 260 | } |
261 | ||
262 | @RequestMapping(requestPath = "versions/commit", method = HttpMethod.POST) | |
80
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
263 | public ResponseType commitVersion(HttpServletRequest req, HttpServletResponse resp, DataAccessObjects dao) throws SQLException { |
59 | 264 | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
265 | var version = new Version(-1); |
59 | 266 | try { |
128
947d0f6a6a83
changes the way how to deal with child entities + adds component lead
Mike Becker <universe@uap-core.de>
parents:
124
diff
changeset
|
267 | final var project = new Project(getParameter(req, Integer.class, "pid").orElseThrow()); |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
268 | version = new Version(getParameter(req, Integer.class, "id").orElseThrow()); |
59 | 269 | version.setName(getParameter(req, String.class, "name").orElseThrow()); |
270 | getParameter(req, Integer.class, "ordinal").ifPresent(version::setOrdinal); | |
271 | version.setStatus(VersionStatus.valueOf(getParameter(req, String.class, "status").orElseThrow())); | |
128
947d0f6a6a83
changes the way how to deal with child entities + adds component lead
Mike Becker <universe@uap-core.de>
parents:
124
diff
changeset
|
272 | dao.getVersionDao().saveOrUpdate(version, project); |
59 | 273 | |
128
947d0f6a6a83
changes the way how to deal with child entities + adds component lead
Mike Becker <universe@uap-core.de>
parents:
124
diff
changeset
|
274 | setRedirectLocation(req, "./projects/versions?pid=" + project.getId()); |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
275 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
75
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
276 | } catch (NoSuchElementException | IllegalArgumentException | SQLException ex) { |
59 | 277 | LOG.warn("Form validation failure: {}", ex.getMessage()); |
278 | LOG.debug("Details:", ex); | |
99 | 279 | final var viewModel = new VersionEditView(); |
280 | populate(viewModel, req, dao); | |
281 | viewModel.setVersion(version); | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
282 | // TODO: set Error Text |
99 | 283 | return forwardView(req, viewModel, "version-form"); |
59 | 284 | } |
41
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
285 | |
43
9abf0bf44c7b
renames some crappy constants
Mike Becker <universe@uap-core.de>
parents:
42
diff
changeset
|
286 | return ResponseType.HTML; |
41
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
287 | } |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
288 | |
99 | 289 | private void configure(IssueEditView viewModel, Issue issue, DataAccessObjects dao) throws SQLException { |
290 | issue.setProject(viewModel.getProjectInfo().getProject()); | |
291 | viewModel.setIssue(issue); | |
292 | viewModel.configureVersionSelectors(viewModel.getProjectInfo().getVersions()); | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
293 | viewModel.setUsers(dao.getUserDao().list()); |
124 | 294 | if (issue.getId() >= 0) { |
295 | viewModel.setComments(dao.getIssueDao().listComments(issue)); | |
296 | } | |
71 | 297 | } |
298 | ||
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
299 | @RequestMapping(requestPath = "issues/edit", method = HttpMethod.GET) |
80
27a25f32048e
adds project overview page
Mike Becker <universe@uap-core.de>
parents:
79
diff
changeset
|
300 | public ResponseType editIssue(HttpServletRequest req, HttpServletResponse resp, DataAccessObjects dao) throws SQLException { |
99 | 301 | final var viewModel = new IssueEditView(); |
124 | 302 | populate(viewModel, req, dao); |
99 | 303 | |
304 | final var issueParam = getParameter(req, Integer.class, "issue"); | |
305 | if (issueParam.isPresent()) { | |
104
8be80ea4f52b
fixes issue not containing version information in issue form
Mike Becker <universe@uap-core.de>
parents:
102
diff
changeset
|
306 | final var issueDao = dao.getIssueDao(); |
8be80ea4f52b
fixes issue not containing version information in issue form
Mike Becker <universe@uap-core.de>
parents:
102
diff
changeset
|
307 | final var issue = issueDao.find(issueParam.get()); |
8be80ea4f52b
fixes issue not containing version information in issue form
Mike Becker <universe@uap-core.de>
parents:
102
diff
changeset
|
308 | issueDao.joinVersionInformation(issue); |
99 | 309 | req.getSession().setAttribute(SESSION_ATTR_SELECTED_PROJECT, issue.getProject().getId()); |
310 | configure(viewModel, issue, dao); | |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
311 | } else { |
99 | 312 | configure(viewModel, new Issue(-1), dao); |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
313 | } |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
314 | |
99 | 315 | return forwardView(req, viewModel, "issue-form"); |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
316 | } |
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
317 | |
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
318 | @RequestMapping(requestPath = "issues/commit", method = HttpMethod.POST) |
124 | 319 | public ResponseType commitIssue(HttpServletRequest req, DataAccessObjects dao) throws SQLException { |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
320 | Issue issue = new Issue(-1); |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
321 | try { |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
322 | issue = new Issue(getParameter(req, Integer.class, "id").orElseThrow()); |
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
323 | issue.setProject(new Project(getParameter(req, Integer.class, "pid").orElseThrow())); |
75
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
324 | getParameter(req, String.class, "category").map(IssueCategory::valueOf).ifPresent(issue::setCategory); |
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
325 | getParameter(req, String.class, "status").map(IssueStatus::valueOf).ifPresent(issue::setStatus); |
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
326 | issue.setSubject(getParameter(req, String.class, "subject").orElseThrow()); |
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
327 | getParameter(req, Integer.class, "assignee").map( |
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
328 | userid -> userid >= 0 ? new User(userid) : null |
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
329 | ).ifPresent(issue::setAssignee); |
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
330 | getParameter(req, String.class, "description").ifPresent(issue::setDescription); |
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
331 | getParameter(req, Date.class, "eta").ifPresent(issue::setEta); |
83
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
332 | |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
333 | getParameter(req, Integer[].class, "affected") |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
334 | .map(Stream::of) |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
335 | .map(stream -> |
96
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
88
diff
changeset
|
336 | stream.map(Version::new).collect(Collectors.toList()) |
83
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
337 | ).ifPresent(issue::setAffectedVersions); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
338 | getParameter(req, Integer[].class, "resolved") |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
339 | .map(Stream::of) |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
340 | .map(stream -> |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
341 | stream.map(Version::new).collect(Collectors.toList()) |
83
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
342 | ).ifPresent(issue::setResolvedVersions); |
24a3596b8f98
adds version selection in issue editor
Mike Becker <universe@uap-core.de>
parents:
81
diff
changeset
|
343 | |
128
947d0f6a6a83
changes the way how to deal with child entities + adds component lead
Mike Becker <universe@uap-core.de>
parents:
124
diff
changeset
|
344 | dao.getIssueDao().saveOrUpdate(issue, issue.getProject()); |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
345 | |
96
b7b685f31e39
breadcrumb menu is now a sidebar navigation menu
Mike Becker <universe@uap-core.de>
parents:
88
diff
changeset
|
346 | // specifying the issue parameter keeps the edited issue as menu item |
102
7eee7cad093d
fixes wrong redirect url after committing an issue
Mike Becker <universe@uap-core.de>
parents:
100
diff
changeset
|
347 | setRedirectLocation(req, "./projects/view?pid=" + issue.getProject().getId()); |
74
91d1fc2a3a14
removes that dynamic_fragment bullshit
Mike Becker <universe@uap-core.de>
parents:
71
diff
changeset
|
348 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
124 | 349 | |
350 | return ResponseType.HTML; | |
75
33b6843fdf8a
adds the ability to create and edit issues
Mike Becker <universe@uap-core.de>
parents:
74
diff
changeset
|
351 | } catch (NoSuchElementException | IllegalArgumentException | SQLException ex) { |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
352 | // TODO: set request attribute with error text |
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
353 | LOG.warn("Form validation failure: {}", ex.getMessage()); |
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
354 | LOG.debug("Details:", ex); |
99 | 355 | final var viewModel = new IssueEditView(); |
124 | 356 | populate(viewModel, req, dao); |
99 | 357 | configure(viewModel, issue, dao); |
86
0a658e53177c
improves issue overview and adds progress information
Mike Becker <universe@uap-core.de>
parents:
83
diff
changeset
|
358 | // TODO: set Error Text |
99 | 359 | return forwardView(req, viewModel, "issue-form"); |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
360 | } |
124 | 361 | } |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
362 | |
124 | 363 | @RequestMapping(requestPath = "issues/comment", method = HttpMethod.POST) |
364 | public ResponseType commentIssue(HttpServletRequest req, HttpServletResponse resp, DataAccessObjects dao) throws SQLException, IOException { | |
365 | final var issueIdParam = getParameter(req, Integer.class, "issueid"); | |
366 | if (issueIdParam.isEmpty()) { | |
367 | resp.sendError(HttpServletResponse.SC_FORBIDDEN, "Detected manipulated form."); | |
368 | return ResponseType.NONE; | |
369 | } | |
370 | final var issue = new Issue(issueIdParam.get()); | |
371 | try { | |
372 | final var issueComment = new IssueComment(getParameter(req, Integer.class, "commentid").orElse(-1), issue); | |
373 | issueComment.setComment(getParameter(req, String.class, "comment").orElse("")); | |
374 | ||
375 | if (issueComment.getComment().isBlank()) { | |
376 | throw new IllegalArgumentException("comment.null"); | |
377 | } | |
378 | ||
379 | LOG.debug("User {} is commenting on issue #{}", req.getRemoteUser(), issue.getId()); | |
380 | if (req.getRemoteUser() != null) { | |
381 | dao.getUserDao().findByUsername(req.getRemoteUser()).ifPresent(issueComment::setAuthor); | |
382 | } | |
383 | ||
384 | dao.getIssueDao().saveComment(issueComment); | |
385 | ||
386 | // specifying the issue parameter keeps the edited issue as menu item | |
387 | setRedirectLocation(req, "./projects/issues/edit?issue=" + issue.getId()); | |
388 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); | |
389 | ||
390 | return ResponseType.HTML; | |
391 | } catch (NoSuchElementException | IllegalArgumentException | SQLException ex) { | |
392 | // TODO: set request attribute with error text | |
393 | LOG.warn("Form validation failure: {}", ex.getMessage()); | |
394 | LOG.debug("Details:", ex); | |
395 | final var viewModel = new IssueEditView(); | |
396 | populate(viewModel, req, dao); | |
397 | configure(viewModel, issue, dao); | |
398 | // TODO: set Error Text | |
399 | return forwardView(req, viewModel, "issue-form"); | |
400 | } | |
64
0f1746c6abfb
adds backend methods for issues (TODO: implement commitIssue())
Mike Becker <universe@uap-core.de>
parents:
61
diff
changeset
|
401 | } |
41
4f1c026a8aab
adds files for ProjectsModule
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
402 | } |