Tue, 05 Jan 2021 19:19:31 +0100
migrates the utility classes for the AbstractServlet
universe@41 | 1 | /* |
universe@41 | 2 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. |
universe@41 | 3 | * |
universe@41 | 4 | * Copyright 2018 Mike Becker. All rights reserved. |
universe@41 | 5 | * |
universe@41 | 6 | * Redistribution and use in source and binary forms, with or without |
universe@41 | 7 | * modification, are permitted provided that the following conditions are met: |
universe@41 | 8 | * |
universe@41 | 9 | * 1. Redistributions of source code must retain the above copyright |
universe@41 | 10 | * notice, this list of conditions and the following disclaimer. |
universe@41 | 11 | * |
universe@41 | 12 | * 2. Redistributions in binary form must reproduce the above copyright |
universe@41 | 13 | * notice, this list of conditions and the following disclaimer in the |
universe@41 | 14 | * documentation and/or other materials provided with the distribution. |
universe@41 | 15 | * |
universe@41 | 16 | * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" |
universe@41 | 17 | * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE |
universe@41 | 18 | * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE |
universe@41 | 19 | * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE |
universe@41 | 20 | * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR |
universe@41 | 21 | * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF |
universe@41 | 22 | * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS |
universe@41 | 23 | * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN |
universe@41 | 24 | * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) |
universe@41 | 25 | * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE |
universe@41 | 26 | * POSSIBILITY OF SUCH DAMAGE. |
universe@41 | 27 | * |
universe@41 | 28 | */ |
universe@41 | 29 | package de.uapcore.lightpit.modules; |
universe@41 | 30 | |
universe@41 | 31 | |
universe@41 | 32 | import de.uapcore.lightpit.*; |
universe@167 | 33 | import de.uapcore.lightpit.dao.DataAccessObject; |
universe@64 | 34 | import de.uapcore.lightpit.entities.*; |
universe@167 | 35 | import de.uapcore.lightpit.filter.AllFilter; |
universe@167 | 36 | import de.uapcore.lightpit.filter.IssueFilter; |
universe@167 | 37 | import de.uapcore.lightpit.filter.NoneFilter; |
universe@167 | 38 | import de.uapcore.lightpit.filter.SpecificFilter; |
universe@167 | 39 | import de.uapcore.lightpit.types.IssueCategory; |
universe@167 | 40 | import de.uapcore.lightpit.types.IssueStatus; |
universe@167 | 41 | import de.uapcore.lightpit.types.VersionStatus; |
universe@134 | 42 | import de.uapcore.lightpit.types.WebColor; |
universe@86 | 43 | import de.uapcore.lightpit.viewmodel.*; |
universe@121 | 44 | import de.uapcore.lightpit.viewmodel.util.IssueSorter; |
universe@59 | 45 | import org.slf4j.Logger; |
universe@59 | 46 | import org.slf4j.LoggerFactory; |
universe@41 | 47 | |
universe@157 | 48 | import javax.servlet.ServletException; |
universe@41 | 49 | import javax.servlet.annotation.WebServlet; |
universe@41 | 50 | import javax.servlet.http.HttpServletRequest; |
universe@59 | 51 | import javax.servlet.http.HttpServletResponse; |
universe@59 | 52 | import java.io.IOException; |
universe@75 | 53 | import java.sql.Date; |
universe@47 | 54 | import java.sql.SQLException; |
universe@86 | 55 | import java.util.NoSuchElementException; |
universe@136 | 56 | import java.util.Optional; |
universe@83 | 57 | import java.util.stream.Collectors; |
universe@83 | 58 | import java.util.stream.Stream; |
universe@41 | 59 | |
universe@41 | 60 | @WebServlet( |
universe@41 | 61 | name = "ProjectsModule", |
universe@41 | 62 | urlPatterns = "/projects/*" |
universe@41 | 63 | ) |
universe@179 | 64 | public final class ProjectsModule extends AbstractServlet { |
universe@41 | 65 | |
universe@59 | 66 | private static final Logger LOG = LoggerFactory.getLogger(ProjectsModule.class); |
universe@59 | 67 | |
universe@78 | 68 | @Override |
universe@78 | 69 | protected String getResourceBundleName() { |
universe@78 | 70 | return "localization.projects"; |
universe@78 | 71 | } |
universe@71 | 72 | |
universe@158 | 73 | private static int parseIntOrZero(String str) { |
universe@158 | 74 | try { |
universe@158 | 75 | return Integer.parseInt(str); |
universe@158 | 76 | } catch (NumberFormatException ex) { |
universe@158 | 77 | return 0; |
universe@158 | 78 | } |
universe@158 | 79 | } |
universe@158 | 80 | |
universe@167 | 81 | private void populate(ProjectView viewModel, PathParameters pathParameters, DataAccessObject dao) { |
universe@167 | 82 | dao.listProjects().stream().map(ProjectInfo::new).forEach(viewModel.getProjectList()::add); |
universe@99 | 83 | |
universe@131 | 84 | if (pathParameters == null) |
universe@131 | 85 | return; |
universe@131 | 86 | |
universe@99 | 87 | // Select Project |
universe@167 | 88 | final var project = dao.findProjectByNode(pathParameters.get("project")); |
universe@138 | 89 | if (project == null) |
universe@138 | 90 | return; |
universe@138 | 91 | |
universe@138 | 92 | final var info = new ProjectInfo(project); |
universe@167 | 93 | info.setVersions(dao.listVersions(project)); |
universe@167 | 94 | info.setComponents(dao.listComponents(project)); |
universe@167 | 95 | info.setIssueSummary(dao.collectIssueSummary(project)); |
universe@138 | 96 | viewModel.setProjectInfo(info); |
universe@99 | 97 | |
universe@99 | 98 | // Select Version |
universe@138 | 99 | final var versionNode = pathParameters.get("version"); |
universe@161 | 100 | if (versionNode != null) { |
universe@161 | 101 | if ("no-version".equals(versionNode)) { |
universe@161 | 102 | viewModel.setVersionFilter(ProjectView.NO_VERSION); |
universe@161 | 103 | } else if ("all-versions".equals(versionNode)) { |
universe@161 | 104 | viewModel.setVersionFilter(ProjectView.ALL_VERSIONS); |
universe@161 | 105 | } else { |
universe@167 | 106 | viewModel.setVersionFilter(dao.findVersionByNode(project, versionNode)); |
universe@161 | 107 | } |
universe@131 | 108 | } |
universe@131 | 109 | |
universe@131 | 110 | // Select Component |
universe@138 | 111 | final var componentNode = pathParameters.get("component"); |
universe@161 | 112 | if (componentNode != null) { |
universe@161 | 113 | if ("no-component".equals(componentNode)) { |
universe@161 | 114 | viewModel.setComponentFilter(ProjectView.NO_COMPONENT); |
universe@161 | 115 | } else if ("all-components".equals(componentNode)) { |
universe@161 | 116 | viewModel.setComponentFilter(ProjectView.ALL_COMPONENTS); |
universe@161 | 117 | } else { |
universe@167 | 118 | viewModel.setComponentFilter(dao.findComponentByNode(project, componentNode)); |
universe@161 | 119 | } |
universe@138 | 120 | } |
universe@138 | 121 | } |
universe@138 | 122 | |
universe@138 | 123 | private static String sanitizeNode(String node, String defaultValue) { |
universe@138 | 124 | String result = node == null || node.isBlank() ? defaultValue : node; |
universe@138 | 125 | result = result.replace('/', '-'); |
universe@138 | 126 | if (result.equals(".") || result.equals("..")) { |
universe@138 | 127 | return "_"+result; |
universe@138 | 128 | } else { |
universe@138 | 129 | return result; |
universe@129 | 130 | } |
universe@99 | 131 | } |
universe@99 | 132 | |
universe@157 | 133 | private void forwardView(HttpServletRequest req, HttpServletResponse resp, ProjectView viewModel, String name) throws ServletException, IOException { |
universe@99 | 134 | setViewModel(req, viewModel); |
universe@99 | 135 | setContentPage(req, name); |
universe@99 | 136 | setStylesheet(req, "projects"); |
universe@109 | 137 | setNavigationMenu(req, "project-navmenu"); |
universe@157 | 138 | renderSite(req, resp); |
universe@64 | 139 | } |
universe@64 | 140 | |
universe@61 | 141 | @RequestMapping(method = HttpMethod.GET) |
universe@167 | 142 | public void index(HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws ServletException, IOException { |
universe@99 | 143 | final var viewModel = new ProjectView(); |
universe@131 | 144 | populate(viewModel, null, dao); |
universe@86 | 145 | |
universe@99 | 146 | for (var info : viewModel.getProjectList()) { |
universe@167 | 147 | info.setVersions(dao.listVersions(info.getProject())); |
universe@167 | 148 | info.setIssueSummary(dao.collectIssueSummary(info.getProject())); |
universe@86 | 149 | } |
universe@86 | 150 | |
universe@157 | 151 | forwardView(req, resp, viewModel, "projects"); |
universe@45 | 152 | } |
universe@45 | 153 | |
universe@167 | 154 | private void configureProjectEditor(ProjectEditView viewModel, Project project, DataAccessObject dao) { |
universe@99 | 155 | viewModel.setProject(project); |
universe@167 | 156 | viewModel.setUsers(dao.listUsers()); |
universe@71 | 157 | } |
universe@71 | 158 | |
universe@131 | 159 | @RequestMapping(requestPath = "$project/edit", method = HttpMethod.GET) |
universe@167 | 160 | public void edit(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParams, DataAccessObject dao) throws IOException, SQLException, ServletException { |
universe@99 | 161 | final var viewModel = new ProjectEditView(); |
universe@131 | 162 | populate(viewModel, pathParams, dao); |
universe@47 | 163 | |
universe@134 | 164 | if (!viewModel.isProjectInfoPresent()) { |
universe@131 | 165 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 166 | return; |
universe@131 | 167 | } |
universe@47 | 168 | |
universe@131 | 169 | configureProjectEditor(viewModel, viewModel.getProjectInfo().getProject(), dao); |
universe@157 | 170 | forwardView(req, resp, viewModel, "project-form"); |
universe@131 | 171 | } |
universe@131 | 172 | |
universe@131 | 173 | @RequestMapping(requestPath = "create", method = HttpMethod.GET) |
universe@167 | 174 | public void create(HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws SQLException, ServletException, IOException { |
universe@131 | 175 | final var viewModel = new ProjectEditView(); |
universe@131 | 176 | populate(viewModel, null, dao); |
universe@131 | 177 | configureProjectEditor(viewModel, new Project(-1), dao); |
universe@157 | 178 | forwardView(req, resp, viewModel, "project-form"); |
universe@47 | 179 | } |
universe@47 | 180 | |
universe@47 | 181 | @RequestMapping(requestPath = "commit", method = HttpMethod.POST) |
universe@167 | 182 | public void commit(HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws IOException, ServletException { |
universe@47 | 183 | |
universe@47 | 184 | try { |
universe@131 | 185 | final var project = new Project(getParameter(req, Integer.class, "pid").orElseThrow()); |
universe@47 | 186 | project.setName(getParameter(req, String.class, "name").orElseThrow()); |
universe@138 | 187 | |
universe@138 | 188 | final var node = getParameter(req, String.class, "node").orElse(null); |
universe@138 | 189 | project.setNode(sanitizeNode(node, project.getName())); |
universe@175 | 190 | getParameter(req, Integer.class, "ordinal").ifPresent(project::setOrdinal); |
universe@138 | 191 | |
universe@47 | 192 | getParameter(req, String.class, "description").ifPresent(project::setDescription); |
universe@47 | 193 | getParameter(req, String.class, "repoUrl").ifPresent(project::setRepoUrl); |
universe@47 | 194 | getParameter(req, Integer.class, "owner").map( |
universe@47 | 195 | ownerId -> ownerId >= 0 ? new User(ownerId) : null |
universe@47 | 196 | ).ifPresent(project::setOwner); |
universe@47 | 197 | |
mike@159 | 198 | if (project.getId() > 0) { |
universe@167 | 199 | dao.updateProject(project); |
mike@159 | 200 | } else { |
universe@167 | 201 | dao.insertProject(project); |
mike@159 | 202 | } |
universe@47 | 203 | |
universe@118 | 204 | setRedirectLocation(req, "./projects/"); |
universe@74 | 205 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
universe@59 | 206 | LOG.debug("Successfully updated project {}", project.getName()); |
universe@99 | 207 | |
universe@157 | 208 | renderSite(req, resp); |
mike@159 | 209 | } catch (NoSuchElementException | IllegalArgumentException ex) { |
universe@131 | 210 | resp.sendError(HttpServletResponse.SC_NOT_IMPLEMENTED); |
universe@131 | 211 | // TODO: implement - fix issue #21 |
universe@47 | 212 | } |
universe@47 | 213 | } |
universe@47 | 214 | |
universe@134 | 215 | @RequestMapping(requestPath = "$project/$component/$version/issues/", method = HttpMethod.GET) |
universe@167 | 216 | public void issues(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParams, DataAccessObject dao) throws SQLException, IOException, ServletException { |
universe@99 | 217 | final var viewModel = new ProjectDetailsView(); |
universe@131 | 218 | populate(viewModel, pathParams, dao); |
universe@86 | 219 | |
universe@134 | 220 | if (!viewModel.isEveryFilterValid()) { |
universe@131 | 221 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 222 | return; |
universe@80 | 223 | } |
universe@47 | 224 | |
universe@134 | 225 | final var project = viewModel.getProjectInfo().getProject(); |
universe@134 | 226 | final var version = viewModel.getVersionFilter(); |
universe@134 | 227 | final var component = viewModel.getComponentFilter(); |
universe@134 | 228 | |
universe@167 | 229 | // TODO: use new IssueFilter class for the ViewModel |
universe@70 | 230 | |
universe@167 | 231 | final var projectFilter = new SpecificFilter<>(project); |
universe@167 | 232 | final IssueFilter filter; |
universe@134 | 233 | if (version.equals(ProjectView.NO_VERSION)) { |
universe@134 | 234 | if (component.equals(ProjectView.ALL_COMPONENTS)) { |
universe@167 | 235 | filter = new IssueFilter(projectFilter, |
universe@167 | 236 | new NoneFilter<>(), |
universe@167 | 237 | new AllFilter<>() |
universe@167 | 238 | ); |
universe@134 | 239 | } else if (component.equals(ProjectView.NO_COMPONENT)) { |
universe@167 | 240 | filter = new IssueFilter(projectFilter, |
universe@167 | 241 | new NoneFilter<>(), |
universe@167 | 242 | new NoneFilter<>() |
universe@167 | 243 | ); |
universe@134 | 244 | } else { |
universe@167 | 245 | filter = new IssueFilter(projectFilter, |
universe@167 | 246 | new NoneFilter<>(), |
universe@167 | 247 | new SpecificFilter<>(component) |
universe@167 | 248 | ); |
universe@134 | 249 | } |
universe@134 | 250 | } else if (version.equals(ProjectView.ALL_VERSIONS)) { |
universe@134 | 251 | if (component.equals(ProjectView.ALL_COMPONENTS)) { |
universe@167 | 252 | filter = new IssueFilter(projectFilter, |
universe@167 | 253 | new AllFilter<>(), |
universe@167 | 254 | new AllFilter<>() |
universe@167 | 255 | ); |
universe@134 | 256 | } else if (component.equals(ProjectView.NO_COMPONENT)) { |
universe@167 | 257 | filter = new IssueFilter(projectFilter, |
universe@167 | 258 | new AllFilter<>(), |
universe@167 | 259 | new NoneFilter<>() |
universe@167 | 260 | ); |
universe@134 | 261 | } else { |
universe@167 | 262 | filter = new IssueFilter(projectFilter, |
universe@167 | 263 | new AllFilter<>(), |
universe@167 | 264 | new SpecificFilter<>(component) |
universe@167 | 265 | ); |
universe@134 | 266 | } |
universe@134 | 267 | } else { |
universe@134 | 268 | if (component.equals(ProjectView.ALL_COMPONENTS)) { |
universe@167 | 269 | filter = new IssueFilter(projectFilter, |
universe@167 | 270 | new SpecificFilter<>(version), |
universe@167 | 271 | new AllFilter<>() |
universe@167 | 272 | ); |
universe@134 | 273 | } else if (component.equals(ProjectView.NO_COMPONENT)) { |
universe@167 | 274 | filter = new IssueFilter(projectFilter, |
universe@167 | 275 | new SpecificFilter<>(version), |
universe@167 | 276 | new NoneFilter<>() |
universe@167 | 277 | ); |
universe@134 | 278 | } else { |
universe@167 | 279 | filter = new IssueFilter(projectFilter, |
universe@167 | 280 | new SpecificFilter<>(version), |
universe@167 | 281 | new SpecificFilter<>(component) |
universe@167 | 282 | ); |
universe@134 | 283 | } |
universe@134 | 284 | } |
universe@134 | 285 | |
universe@167 | 286 | final var issues = dao.listIssues(filter); |
universe@121 | 287 | issues.sort(new IssueSorter( |
universe@141 | 288 | new IssueSorter.Criteria(IssueSorter.Field.DONE, true), |
universe@121 | 289 | new IssueSorter.Criteria(IssueSorter.Field.ETA, true), |
universe@121 | 290 | new IssueSorter.Criteria(IssueSorter.Field.UPDATED, false) |
universe@121 | 291 | )); |
universe@134 | 292 | |
universe@134 | 293 | |
universe@134 | 294 | viewModel.getProjectDetails().updateDetails(issues); |
universe@134 | 295 | if (version.getId() > 0) |
universe@134 | 296 | viewModel.getProjectDetails().updateVersionInfo(version); |
universe@80 | 297 | |
universe@157 | 298 | forwardView(req, resp, viewModel, "project-details"); |
universe@71 | 299 | } |
universe@71 | 300 | |
universe@131 | 301 | @RequestMapping(requestPath = "$project/versions/", method = HttpMethod.GET) |
universe@167 | 302 | public void versions(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, SQLException, ServletException { |
universe@109 | 303 | final var viewModel = new VersionsView(); |
universe@131 | 304 | populate(viewModel, pathParameters, dao); |
universe@109 | 305 | |
universe@109 | 306 | final var projectInfo = viewModel.getProjectInfo(); |
universe@109 | 307 | if (projectInfo == null) { |
universe@131 | 308 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 309 | return; |
universe@109 | 310 | } |
universe@109 | 311 | |
universe@167 | 312 | final var issues = dao.listIssues( |
universe@167 | 313 | new IssueFilter( |
universe@167 | 314 | new SpecificFilter<>(projectInfo.getProject()), |
universe@167 | 315 | new AllFilter<>(), |
universe@167 | 316 | new AllFilter<>() |
universe@167 | 317 | ) |
universe@167 | 318 | ); |
universe@109 | 319 | viewModel.update(projectInfo.getVersions(), issues); |
universe@109 | 320 | |
universe@157 | 321 | forwardView(req, resp, viewModel, "versions"); |
universe@109 | 322 | } |
universe@109 | 323 | |
universe@131 | 324 | @RequestMapping(requestPath = "$project/versions/$version/edit", method = HttpMethod.GET) |
universe@167 | 325 | public void editVersion(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, ServletException { |
universe@99 | 326 | final var viewModel = new VersionEditView(); |
universe@131 | 327 | populate(viewModel, pathParameters, dao); |
universe@99 | 328 | |
universe@131 | 329 | if (viewModel.getProjectInfo() == null || viewModel.getVersionFilter() == null) { |
universe@131 | 330 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 331 | return; |
universe@110 | 332 | } |
universe@110 | 333 | |
universe@131 | 334 | viewModel.setVersion(viewModel.getVersionFilter()); |
universe@59 | 335 | |
universe@157 | 336 | forwardView(req, resp, viewModel, "version-form"); |
universe@59 | 337 | } |
universe@59 | 338 | |
universe@131 | 339 | @RequestMapping(requestPath = "$project/create-version", method = HttpMethod.GET) |
universe@167 | 340 | public void createVersion(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, ServletException { |
universe@131 | 341 | final var viewModel = new VersionEditView(); |
universe@131 | 342 | populate(viewModel, pathParameters, dao); |
universe@59 | 343 | |
universe@131 | 344 | if (viewModel.getProjectInfo() == null) { |
universe@131 | 345 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 346 | return; |
universe@131 | 347 | } |
universe@131 | 348 | |
universe@167 | 349 | viewModel.setVersion(new Version(-1, viewModel.getProjectInfo().getProject().getId())); |
universe@131 | 350 | |
universe@157 | 351 | forwardView(req, resp, viewModel, "version-form"); |
universe@131 | 352 | } |
universe@131 | 353 | |
universe@131 | 354 | @RequestMapping(requestPath = "commit-version", method = HttpMethod.POST) |
universe@167 | 355 | public void commitVersion(HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws IOException, ServletException { |
universe@131 | 356 | |
universe@59 | 357 | try { |
universe@167 | 358 | final var project = dao.findProject(getParameter(req, Integer.class, "pid").orElseThrow()); |
universe@138 | 359 | if (project == null) { |
universe@138 | 360 | // TODO: improve error handling, because not found is not correct for this POST request |
universe@138 | 361 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 362 | return; |
universe@138 | 363 | } |
universe@167 | 364 | final var version = new Version(getParameter(req, Integer.class, "id").orElseThrow(), project.getId()); |
universe@59 | 365 | version.setName(getParameter(req, String.class, "name").orElseThrow()); |
universe@138 | 366 | |
universe@138 | 367 | final var node = getParameter(req, String.class, "node").orElse(null); |
universe@138 | 368 | version.setNode(sanitizeNode(node, version.getName())); |
universe@138 | 369 | |
universe@59 | 370 | getParameter(req, Integer.class, "ordinal").ifPresent(version::setOrdinal); |
universe@59 | 371 | version.setStatus(VersionStatus.valueOf(getParameter(req, String.class, "status").orElseThrow())); |
mike@159 | 372 | |
mike@159 | 373 | if (version.getId() > 0) { |
universe@167 | 374 | dao.updateVersion(version); |
mike@159 | 375 | } else { |
universe@167 | 376 | dao.insertVersion(version); |
mike@159 | 377 | } |
universe@59 | 378 | |
universe@138 | 379 | setRedirectLocation(req, "./projects/" + project.getNode() + "/versions/"); |
universe@74 | 380 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
universe@157 | 381 | |
universe@157 | 382 | renderSite(req, resp); |
mike@159 | 383 | } catch (NoSuchElementException | IllegalArgumentException ex) { |
universe@131 | 384 | resp.sendError(HttpServletResponse.SC_NOT_IMPLEMENTED); |
universe@131 | 385 | // TODO: implement - fix issue #21 |
universe@59 | 386 | } |
universe@41 | 387 | } |
universe@64 | 388 | |
universe@134 | 389 | @RequestMapping(requestPath = "$project/components/", method = HttpMethod.GET) |
universe@167 | 390 | public void components(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, SQLException, ServletException { |
universe@134 | 391 | final var viewModel = new ComponentsView(); |
universe@134 | 392 | populate(viewModel, pathParameters, dao); |
universe@134 | 393 | |
universe@134 | 394 | final var projectInfo = viewModel.getProjectInfo(); |
universe@134 | 395 | if (projectInfo == null) { |
universe@134 | 396 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 397 | return; |
universe@134 | 398 | } |
universe@134 | 399 | |
universe@167 | 400 | final var issues = dao.listIssues( |
universe@167 | 401 | new IssueFilter( |
universe@167 | 402 | new SpecificFilter<>(projectInfo.getProject()), |
universe@167 | 403 | new AllFilter<>(), |
universe@167 | 404 | new AllFilter<>() |
universe@167 | 405 | ) |
universe@167 | 406 | ); |
universe@134 | 407 | viewModel.update(projectInfo.getComponents(), issues); |
universe@134 | 408 | |
universe@157 | 409 | forwardView(req, resp, viewModel, "components"); |
universe@134 | 410 | } |
universe@134 | 411 | |
universe@134 | 412 | @RequestMapping(requestPath = "$project/components/$component/edit", method = HttpMethod.GET) |
universe@167 | 413 | public void editComponent(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, ServletException { |
universe@134 | 414 | final var viewModel = new ComponentEditView(); |
universe@134 | 415 | populate(viewModel, pathParameters, dao); |
universe@134 | 416 | |
universe@134 | 417 | if (viewModel.getProjectInfo() == null || viewModel.getComponentFilter() == null) { |
universe@134 | 418 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 419 | return; |
universe@134 | 420 | } |
universe@134 | 421 | |
universe@134 | 422 | viewModel.setComponent(viewModel.getComponentFilter()); |
universe@167 | 423 | viewModel.setUsers(dao.listUsers()); |
universe@134 | 424 | |
universe@157 | 425 | forwardView(req, resp, viewModel, "component-form"); |
universe@134 | 426 | } |
universe@134 | 427 | |
universe@134 | 428 | @RequestMapping(requestPath = "$project/create-component", method = HttpMethod.GET) |
universe@167 | 429 | public void createComponent(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, ServletException { |
universe@134 | 430 | final var viewModel = new ComponentEditView(); |
universe@134 | 431 | populate(viewModel, pathParameters, dao); |
universe@134 | 432 | |
universe@134 | 433 | if (viewModel.getProjectInfo() == null) { |
universe@134 | 434 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 435 | return; |
universe@134 | 436 | } |
universe@134 | 437 | |
universe@167 | 438 | viewModel.setComponent(new Component(-1, viewModel.getProjectInfo().getProject().getId())); |
universe@167 | 439 | viewModel.setUsers(dao.listUsers()); |
universe@134 | 440 | |
universe@157 | 441 | forwardView(req, resp, viewModel, "component-form"); |
universe@134 | 442 | } |
universe@134 | 443 | |
universe@134 | 444 | @RequestMapping(requestPath = "commit-component", method = HttpMethod.POST) |
universe@167 | 445 | public void commitComponent(HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws IOException, ServletException { |
universe@134 | 446 | |
universe@134 | 447 | try { |
universe@167 | 448 | final var project = dao.findProject(getParameter(req, Integer.class, "pid").orElseThrow()); |
universe@138 | 449 | if (project == null) { |
universe@138 | 450 | // TODO: improve error handling, because not found is not correct for this POST request |
universe@138 | 451 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 452 | return; |
universe@138 | 453 | } |
universe@167 | 454 | final var component = new Component(getParameter(req, Integer.class, "id").orElseThrow(), project.getId()); |
universe@134 | 455 | component.setName(getParameter(req, String.class, "name").orElseThrow()); |
universe@138 | 456 | |
universe@138 | 457 | final var node = getParameter(req, String.class, "node").orElse(null); |
universe@138 | 458 | component.setNode(sanitizeNode(node, component.getName())); |
universe@138 | 459 | |
universe@134 | 460 | component.setColor(getParameter(req, WebColor.class, "color").orElseThrow()); |
universe@134 | 461 | getParameter(req, Integer.class, "ordinal").ifPresent(component::setOrdinal); |
universe@134 | 462 | getParameter(req, Integer.class, "lead").map( |
universe@134 | 463 | userid -> userid >= 0 ? new User(userid) : null |
universe@134 | 464 | ).ifPresent(component::setLead); |
universe@134 | 465 | getParameter(req, String.class, "description").ifPresent(component::setDescription); |
universe@134 | 466 | |
mike@159 | 467 | if (component.getId() > 0) { |
universe@167 | 468 | dao.updateComponent(component); |
mike@159 | 469 | } else { |
universe@167 | 470 | dao.insertComponent(component); |
mike@159 | 471 | } |
universe@134 | 472 | |
universe@138 | 473 | setRedirectLocation(req, "./projects/" + project.getNode() + "/components/"); |
universe@134 | 474 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
universe@157 | 475 | |
universe@157 | 476 | renderSite(req, resp); |
mike@159 | 477 | } catch (NoSuchElementException | IllegalArgumentException ex) { |
universe@134 | 478 | resp.sendError(HttpServletResponse.SC_NOT_IMPLEMENTED); |
universe@134 | 479 | // TODO: implement - fix issue #21 |
universe@134 | 480 | } |
universe@134 | 481 | } |
universe@134 | 482 | |
universe@167 | 483 | private void configureIssueEditor(IssueEditView viewModel, Issue issue, DataAccessObject dao) { |
universe@134 | 484 | final var project = viewModel.getProjectInfo().getProject(); |
universe@146 | 485 | issue.setProject(project); // automatically set current project for new issues |
universe@99 | 486 | viewModel.setIssue(issue); |
universe@99 | 487 | viewModel.configureVersionSelectors(viewModel.getProjectInfo().getVersions()); |
universe@167 | 488 | viewModel.setUsers(dao.listUsers()); |
universe@167 | 489 | viewModel.setComponents(dao.listComponents(project)); |
universe@71 | 490 | } |
universe@71 | 491 | |
universe@146 | 492 | @RequestMapping(requestPath = "$project/issues/$issue/view", method = HttpMethod.GET) |
universe@167 | 493 | public void viewIssue(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, ServletException { |
universe@146 | 494 | final var viewModel = new IssueDetailView(); |
universe@146 | 495 | populate(viewModel, pathParameters, dao); |
universe@146 | 496 | |
universe@146 | 497 | final var projectInfo = viewModel.getProjectInfo(); |
universe@146 | 498 | if (projectInfo == null) { |
universe@146 | 499 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 500 | return; |
universe@146 | 501 | } |
universe@146 | 502 | |
universe@167 | 503 | final var issue = dao.findIssue(parseIntOrZero(pathParameters.get("issue"))); |
universe@146 | 504 | if (issue == null) { |
universe@146 | 505 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 506 | return; |
universe@146 | 507 | } |
universe@146 | 508 | |
universe@146 | 509 | viewModel.setIssue(issue); |
universe@167 | 510 | viewModel.setComments(dao.listComments(issue)); |
universe@146 | 511 | |
universe@162 | 512 | viewModel.processMarkdown(); |
universe@162 | 513 | |
universe@157 | 514 | forwardView(req, resp, viewModel, "issue-view"); |
universe@146 | 515 | } |
universe@146 | 516 | |
universe@146 | 517 | // TODO: why should the issue editor be child of $project? |
universe@131 | 518 | @RequestMapping(requestPath = "$project/issues/$issue/edit", method = HttpMethod.GET) |
universe@167 | 519 | public void editIssue(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, SQLException, ServletException { |
universe@99 | 520 | final var viewModel = new IssueEditView(); |
universe@131 | 521 | populate(viewModel, pathParameters, dao); |
universe@99 | 522 | |
universe@131 | 523 | final var projectInfo = viewModel.getProjectInfo(); |
universe@131 | 524 | if (projectInfo == null) { |
universe@131 | 525 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 526 | return; |
universe@86 | 527 | } |
universe@64 | 528 | |
universe@167 | 529 | final var issue = dao.findIssue(parseIntOrZero(pathParameters.get("issue"))); |
universe@131 | 530 | if (issue == null) { |
universe@131 | 531 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 532 | return; |
universe@131 | 533 | } |
universe@131 | 534 | |
universe@133 | 535 | configureIssueEditor(viewModel, issue, dao); |
universe@131 | 536 | |
universe@157 | 537 | forwardView(req, resp, viewModel, "issue-form"); |
universe@64 | 538 | } |
universe@64 | 539 | |
universe@131 | 540 | @RequestMapping(requestPath = "$project/create-issue", method = HttpMethod.GET) |
universe@168 | 541 | public void createIssue(HttpServletRequest req, HttpServletResponse resp, PathParameters pathParameters, DataAccessObject dao) throws IOException, ServletException { |
universe@131 | 542 | final var viewModel = new IssueEditView(); |
universe@131 | 543 | populate(viewModel, pathParameters, dao); |
universe@131 | 544 | |
universe@131 | 545 | final var projectInfo = viewModel.getProjectInfo(); |
universe@131 | 546 | if (projectInfo == null) { |
universe@131 | 547 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 548 | return; |
universe@131 | 549 | } |
universe@131 | 550 | |
universe@168 | 551 | setAttributeFromParameter(req, "more"); |
universe@178 | 552 | setAttributeFromParameter(req, "cid"); |
universe@178 | 553 | setAttributeFromParameter(req, "vid"); |
universe@168 | 554 | |
universe@167 | 555 | final var issue = new Issue(-1, projectInfo.getProject(), null); |
universe@131 | 556 | issue.setProject(projectInfo.getProject()); |
universe@133 | 557 | configureIssueEditor(viewModel, issue, dao); |
universe@131 | 558 | |
universe@157 | 559 | forwardView(req, resp, viewModel, "issue-form"); |
universe@131 | 560 | } |
universe@131 | 561 | |
universe@131 | 562 | @RequestMapping(requestPath = "commit-issue", method = HttpMethod.POST) |
universe@167 | 563 | public void commitIssue(HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws IOException, ServletException { |
universe@64 | 564 | try { |
universe@167 | 565 | final var project = dao.findProject(getParameter(req, Integer.class, "pid").orElseThrow()); |
universe@138 | 566 | if (project == null) { |
universe@138 | 567 | // TODO: improve error handling, because not found is not correct for this POST request |
universe@138 | 568 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 569 | return; |
universe@138 | 570 | } |
universe@167 | 571 | final var componentId = getParameter(req, Integer.class, "component"); |
universe@167 | 572 | final Component component; |
universe@167 | 573 | if (componentId.isPresent()) { |
universe@167 | 574 | component = dao.findComponent(componentId.get()); |
universe@167 | 575 | } else { |
universe@167 | 576 | component = null; |
universe@167 | 577 | } |
universe@167 | 578 | final var issue = new Issue(getParameter(req, Integer.class, "id").orElseThrow(), project, component); |
universe@75 | 579 | getParameter(req, String.class, "category").map(IssueCategory::valueOf).ifPresent(issue::setCategory); |
universe@75 | 580 | getParameter(req, String.class, "status").map(IssueStatus::valueOf).ifPresent(issue::setStatus); |
universe@75 | 581 | issue.setSubject(getParameter(req, String.class, "subject").orElseThrow()); |
universe@136 | 582 | getParameter(req, Integer.class, "assignee").map(userid -> { |
universe@136 | 583 | if (userid >= 0) { |
universe@136 | 584 | return new User(userid); |
universe@136 | 585 | } else if (userid == -2) { |
universe@136 | 586 | return Optional.ofNullable(component).map(Component::getLead).orElse(null); |
universe@136 | 587 | } else { |
universe@136 | 588 | return null; |
universe@136 | 589 | } |
universe@136 | 590 | } |
universe@75 | 591 | ).ifPresent(issue::setAssignee); |
universe@75 | 592 | getParameter(req, String.class, "description").ifPresent(issue::setDescription); |
universe@75 | 593 | getParameter(req, Date.class, "eta").ifPresent(issue::setEta); |
universe@83 | 594 | |
universe@83 | 595 | getParameter(req, Integer[].class, "affected") |
universe@83 | 596 | .map(Stream::of) |
universe@83 | 597 | .map(stream -> |
universe@167 | 598 | stream.map(id -> new Version(id, project.getId())) |
universe@167 | 599 | .collect(Collectors.toList()) |
universe@83 | 600 | ).ifPresent(issue::setAffectedVersions); |
universe@83 | 601 | getParameter(req, Integer[].class, "resolved") |
universe@83 | 602 | .map(Stream::of) |
universe@83 | 603 | .map(stream -> |
universe@167 | 604 | stream.map(id -> new Version(id, project.getId())) |
universe@167 | 605 | .collect(Collectors.toList()) |
universe@83 | 606 | ).ifPresent(issue::setResolvedVersions); |
universe@83 | 607 | |
mike@159 | 608 | if (issue.getId() > 0) { |
universe@167 | 609 | dao.updateIssue(issue); |
mike@159 | 610 | } else { |
universe@167 | 611 | dao.insertIssue(issue); |
mike@159 | 612 | } |
universe@64 | 613 | |
universe@168 | 614 | if (getParameter(req, Boolean.class, "create-another").orElse(false)) { |
universe@168 | 615 | // TODO: fix #38 - automatically select component (and version) |
universe@168 | 616 | setRedirectLocation(req, "./projects/" + issue.getProject().getNode() + "/create-issue?more=true"); |
universe@168 | 617 | } else{ |
universe@168 | 618 | setRedirectLocation(req, "./projects/" + issue.getProject().getNode() + "/issues/" + issue.getId() + "/view"); |
universe@168 | 619 | } |
universe@74 | 620 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
universe@124 | 621 | |
universe@157 | 622 | renderSite(req, resp); |
mike@159 | 623 | } catch (NoSuchElementException | IllegalArgumentException ex) { |
universe@131 | 624 | resp.sendError(HttpServletResponse.SC_NOT_IMPLEMENTED); |
universe@131 | 625 | // TODO: implement - fix issue #21 |
universe@64 | 626 | } |
universe@124 | 627 | } |
universe@64 | 628 | |
universe@131 | 629 | @RequestMapping(requestPath = "commit-issue-comment", method = HttpMethod.POST) |
universe@167 | 630 | public void commentIssue(HttpServletRequest req, HttpServletResponse resp, DataAccessObject dao) throws IOException, ServletException { |
universe@124 | 631 | final var issueIdParam = getParameter(req, Integer.class, "issueid"); |
universe@124 | 632 | if (issueIdParam.isEmpty()) { |
universe@124 | 633 | resp.sendError(HttpServletResponse.SC_FORBIDDEN, "Detected manipulated form."); |
universe@157 | 634 | return; |
universe@124 | 635 | } |
universe@167 | 636 | final var issue = dao.findIssue(issueIdParam.get()); |
universe@137 | 637 | if (issue == null) { |
universe@137 | 638 | resp.sendError(HttpServletResponse.SC_NOT_FOUND); |
universe@157 | 639 | return; |
universe@137 | 640 | } |
universe@124 | 641 | try { |
universe@167 | 642 | final var issueComment = new IssueComment(getParameter(req, Integer.class, "commentid").orElse(-1), issue.getId()); |
universe@124 | 643 | issueComment.setComment(getParameter(req, String.class, "comment").orElse("")); |
universe@124 | 644 | |
universe@124 | 645 | if (issueComment.getComment().isBlank()) { |
universe@124 | 646 | throw new IllegalArgumentException("comment.null"); |
universe@124 | 647 | } |
universe@124 | 648 | |
universe@124 | 649 | LOG.debug("User {} is commenting on issue #{}", req.getRemoteUser(), issue.getId()); |
universe@124 | 650 | if (req.getRemoteUser() != null) { |
universe@167 | 651 | Optional.ofNullable(dao.findUserByName(req.getRemoteUser())).ifPresent(issueComment::setAuthor); |
universe@124 | 652 | } |
universe@124 | 653 | |
universe@167 | 654 | dao.insertComment(issueComment); |
universe@124 | 655 | |
universe@146 | 656 | setRedirectLocation(req, "./projects/" + issue.getProject().getNode()+"/issues/"+issue.getId()+"/view"); |
universe@124 | 657 | setContentPage(req, Constants.JSP_COMMIT_SUCCESSFUL); |
universe@124 | 658 | |
universe@157 | 659 | renderSite(req, resp); |
mike@159 | 660 | } catch (NoSuchElementException | IllegalArgumentException ex) { |
universe@131 | 661 | resp.sendError(HttpServletResponse.SC_NOT_IMPLEMENTED); |
universe@131 | 662 | // TODO: implement - fix issue #21 |
universe@124 | 663 | } |
universe@64 | 664 | } |
universe@41 | 665 | } |