Mon, 02 Aug 2021 17:04:17 +0200
#22 adds possibility to edit own comments
184 | 1 | /* |
2 | * Copyright 2021 Mike Becker. All rights reserved. | |
3 | * | |
4 | * Redistribution and use in source and binary forms, with or without | |
5 | * modification, are permitted provided that the following conditions are met: | |
6 | * | |
7 | * 1. Redistributions of source code must retain the above copyright | |
8 | * notice, this list of conditions and the following disclaimer. | |
9 | * | |
10 | * 2. Redistributions in binary form must reproduce the above copyright | |
11 | * notice, this list of conditions and the following disclaimer in the | |
12 | * documentation and/or other materials provided with the distribution. | |
13 | * | |
14 | * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" | |
15 | * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE | |
16 | * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | |
17 | * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE | |
18 | * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL | |
19 | * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR | |
20 | * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER | |
21 | * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, | |
22 | * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | |
23 | * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | |
24 | */ | |
25 | ||
26 | package de.uapcore.lightpit.servlet | |
27 | ||
28 | import de.uapcore.lightpit.AbstractServlet | |
29 | import de.uapcore.lightpit.HttpRequest | |
30 | import de.uapcore.lightpit.dao.DataAccessObject | |
31 | import de.uapcore.lightpit.entities.* | |
32 | import de.uapcore.lightpit.types.IssueCategory | |
33 | import de.uapcore.lightpit.types.IssueStatus | |
34 | import de.uapcore.lightpit.types.VersionStatus | |
35 | import de.uapcore.lightpit.types.WebColor | |
36 | import de.uapcore.lightpit.util.AllFilter | |
37 | import de.uapcore.lightpit.util.IssueFilter | |
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
38 | import de.uapcore.lightpit.util.IssueSorter.Companion.DEFAULT_ISSUE_SORTER |
184 | 39 | import de.uapcore.lightpit.util.SpecificFilter |
40 | import de.uapcore.lightpit.viewmodel.* | |
41 | import java.sql.Date | |
42 | import javax.servlet.annotation.WebServlet | |
43 | ||
44 | @WebServlet(urlPatterns = ["/projects/*"]) | |
45 | class ProjectServlet : AbstractServlet() { | |
46 | ||
47 | init { | |
48 | get("/", this::projects) | |
49 | get("/%project", this::project) | |
50 | get("/%project/issues/%version/%component/", this::project) | |
51 | get("/%project/edit", this::projectForm) | |
52 | get("/-/create", this::projectForm) | |
53 | post("/-/commit", this::projectCommit) | |
54 | ||
55 | get("/%project/versions/", this::versions) | |
56 | get("/%project/versions/%version/edit", this::versionForm) | |
57 | get("/%project/versions/-/create", this::versionForm) | |
58 | post("/%project/versions/-/commit", this::versionCommit) | |
59 | ||
60 | get("/%project/components/", this::components) | |
61 | get("/%project/components/%component/edit", this::componentForm) | |
62 | get("/%project/components/-/create", this::componentForm) | |
63 | post("/%project/components/-/commit", this::componentCommit) | |
64 | ||
65 | get("/%project/issues/%version/%component/%issue", this::issue) | |
66 | get("/%project/issues/%version/%component/%issue/edit", this::issueForm) | |
186
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
67 | post("/%project/issues/%version/%component/%issue/comment", this::issueComment) |
184 | 68 | get("/%project/issues/%version/%component/-/create", this::issueForm) |
186
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
69 | post("/%project/issues/%version/%component/-/commit", this::issueCommit) |
184 | 70 | } |
71 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
72 | private fun projects(http: HttpRequest, dao: DataAccessObject) { |
184 | 73 | val projects = dao.listProjects() |
74 | val projectInfos = projects.map { | |
75 | ProjectInfo( | |
76 | project = it, | |
77 | versions = dao.listVersions(it), | |
78 | components = emptyList(), // not required in this view | |
79 | issueSummary = dao.collectIssueSummary(it) | |
80 | ) | |
81 | } | |
82 | ||
83 | with(http) { | |
84 | view = ProjectsView(projectInfos) | |
85 | navigationMenu = projectNavMenu(projects) | |
86 | styleSheets = listOf("projects") | |
87 | render("projects") | |
88 | } | |
89 | } | |
90 | ||
91 | private fun activeProjectNavMenu( | |
92 | projects: List<Project>, | |
93 | projectInfo: ProjectInfo, | |
94 | selectedVersion: Version? = null, | |
95 | selectedComponent: Component? = null | |
96 | ) = | |
97 | projectNavMenu( | |
98 | projects, | |
99 | projectInfo.versions, | |
100 | projectInfo.components, | |
101 | projectInfo.project, | |
102 | selectedVersion, | |
103 | selectedComponent | |
104 | ) | |
105 | ||
106 | sealed class LookupResult<T> { | |
107 | class NotFound<T> : LookupResult<T>() | |
108 | data class Found<T>(val elem: T?) : LookupResult<T>() | |
109 | } | |
110 | ||
111 | private fun <T : HasNode> HttpRequest.lookupPathParam(paramName: String, list: List<T>): LookupResult<T> { | |
112 | val node = pathParams[paramName] | |
113 | return if (node == null || node == "-") { | |
114 | LookupResult.Found(null) | |
115 | } else { | |
116 | val result = list.find { it.node == node } | |
117 | if (result == null) { | |
118 | LookupResult.NotFound() | |
119 | } else { | |
120 | LookupResult.Found(result) | |
121 | } | |
122 | } | |
123 | } | |
124 | ||
125 | private fun obtainProjectInfo(http: HttpRequest, dao: DataAccessObject): ProjectInfo? { | |
126 | val project = dao.findProjectByNode(http.pathParams["project"] ?: "") ?: return null | |
127 | ||
128 | val versions: List<Version> = dao.listVersions(project) | |
129 | val components: List<Component> = dao.listComponents(project) | |
130 | ||
131 | return ProjectInfo( | |
132 | project, | |
133 | versions, | |
134 | components, | |
135 | dao.collectIssueSummary(project) | |
136 | ) | |
137 | } | |
138 | ||
139 | private fun sanitizeNode(name: String): String { | |
140 | val san = name.replace(Regex("[/\\\\]"), "-") | |
141 | return if (san.startsWith(".")) { | |
142 | "v$san" | |
143 | } else { | |
144 | san | |
145 | } | |
146 | } | |
147 | ||
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
148 | private fun feedPath(project: Project) = "feed/${project.node}/issues.rss" |
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
149 | |
184 | 150 | data class PathInfos( |
151 | val projectInfo: ProjectInfo, | |
152 | val version: Version?, | |
153 | val component: Component? | |
154 | ) { | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
155 | val project = projectInfo.project |
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
156 | val issuesHref by lazyOf("projects/${project.node}/issues/${version?.node ?: "-"}/${component?.node ?: "-"}/") |
184 | 157 | } |
158 | ||
159 | private fun withPathInfo(http: HttpRequest, dao: DataAccessObject): PathInfos? { | |
160 | val projectInfo = obtainProjectInfo(http, dao) | |
161 | if (projectInfo == null) { | |
162 | http.response.sendError(404) | |
163 | return null | |
164 | } | |
165 | ||
166 | val version = when (val result = http.lookupPathParam("version", projectInfo.versions)) { | |
167 | is LookupResult.NotFound -> { | |
168 | http.response.sendError(404) | |
169 | return null | |
170 | } | |
171 | is LookupResult.Found -> { | |
172 | result.elem | |
173 | } | |
174 | } | |
175 | val component = when (val result = http.lookupPathParam("component", projectInfo.components)) { | |
176 | is LookupResult.NotFound -> { | |
177 | http.response.sendError(404) | |
178 | return null | |
179 | } | |
180 | is LookupResult.Found -> { | |
181 | result.elem | |
182 | } | |
183 | } | |
184 | ||
185 | return PathInfos(projectInfo, version, component) | |
186 | } | |
187 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
188 | private fun project(http: HttpRequest, dao: DataAccessObject) { |
184 | 189 | withPathInfo(http, dao)?.run { |
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
190 | |
184 | 191 | val issues = dao.listIssues(IssueFilter( |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
192 | project = SpecificFilter(project), |
184 | 193 | version = version?.let { SpecificFilter(it) } ?: AllFilter(), |
194 | component = component?.let { SpecificFilter(it) } ?: AllFilter() | |
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
195 | )).sortedWith(DEFAULT_ISSUE_SORTER) |
184 | 196 | |
197 | with(http) { | |
205
7725a79416f3
#115 adds custom page titles
Mike Becker <universe@uap-core.de>
parents:
200
diff
changeset
|
198 | pageTitle = project.name |
184 | 199 | view = ProjectDetails(projectInfo, issues, version, component) |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
200 | feedPath = feedPath(project) |
184 | 201 | navigationMenu = activeProjectNavMenu( |
202 | dao.listProjects(), | |
203 | projectInfo, | |
204 | version, | |
205 | component | |
206 | ) | |
207 | styleSheets = listOf("projects") | |
208 | render("project-details") | |
209 | } | |
210 | } | |
211 | } | |
212 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
213 | private fun projectForm(http: HttpRequest, dao: DataAccessObject) { |
200
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
214 | if (!http.pathParams.containsKey("project")) { |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
215 | http.view = ProjectEditView(Project(-1), dao.listUsers()) |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
216 | http.navigationMenu = projectNavMenu(dao.listProjects()) |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
217 | } else { |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
218 | val projectInfo = obtainProjectInfo(http, dao) |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
219 | if (projectInfo == null) { |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
220 | http.response.sendError(404) |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
221 | return |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
222 | } |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
223 | http.view = ProjectEditView(projectInfo.project, dao.listUsers()) |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
224 | http.navigationMenu = activeProjectNavMenu( |
184 | 225 | dao.listProjects(), |
226 | projectInfo | |
227 | ) | |
228 | } | |
200
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
229 | http.styleSheets = listOf("projects") |
a5ddfaf6b469
fixes project creation not working
Mike Becker <universe@uap-core.de>
parents:
198
diff
changeset
|
230 | http.render("project-form") |
184 | 231 | } |
232 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
233 | private fun projectCommit(http: HttpRequest, dao: DataAccessObject) { |
184 | 234 | // TODO: replace defaults with throwing validator exceptions |
235 | val project = Project(http.param("id")?.toIntOrNull() ?: -1).apply { | |
236 | name = http.param("name") ?: "" | |
237 | node = http.param("node") ?: "" | |
238 | description = http.param("description") ?: "" | |
239 | ordinal = http.param("ordinal")?.toIntOrNull() ?: 0 | |
240 | repoUrl = http.param("repoUrl") ?: "" | |
241 | owner = (http.param("owner")?.toIntOrNull() ?: -1).let { | |
242 | if (it < 0) null else dao.findUser(it) | |
243 | } | |
244 | // intentional defaults | |
245 | if (node.isBlank()) node = name | |
246 | // sanitizing | |
247 | node = sanitizeNode(node) | |
248 | } | |
249 | ||
250 | if (project.id < 0) { | |
251 | dao.insertProject(project) | |
252 | } else { | |
253 | dao.updateProject(project) | |
254 | } | |
255 | ||
256 | http.renderCommit("projects/${project.node}") | |
257 | } | |
258 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
259 | private fun versions(http: HttpRequest, dao: DataAccessObject) { |
184 | 260 | val projectInfo = obtainProjectInfo(http, dao) |
261 | if (projectInfo == null) { | |
262 | http.response.sendError(404) | |
263 | return | |
264 | } | |
265 | ||
266 | with(http) { | |
205
7725a79416f3
#115 adds custom page titles
Mike Becker <universe@uap-core.de>
parents:
200
diff
changeset
|
267 | pageTitle = "${projectInfo.project.name} - ${i18n("navmenu.versions")}" |
184 | 268 | view = VersionsView( |
269 | projectInfo, | |
270 | dao.listVersionSummaries(projectInfo.project) | |
271 | ) | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
272 | feedPath = feedPath(projectInfo.project) |
184 | 273 | navigationMenu = activeProjectNavMenu( |
274 | dao.listProjects(), | |
275 | projectInfo | |
276 | ) | |
277 | styleSheets = listOf("projects") | |
278 | render("versions") | |
279 | } | |
280 | } | |
281 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
282 | private fun versionForm(http: HttpRequest, dao: DataAccessObject) { |
184 | 283 | val projectInfo = obtainProjectInfo(http, dao) |
284 | if (projectInfo == null) { | |
285 | http.response.sendError(404) | |
286 | return | |
287 | } | |
288 | ||
289 | val version: Version | |
290 | when (val result = http.lookupPathParam("version", projectInfo.versions)) { | |
291 | is LookupResult.NotFound -> { | |
292 | http.response.sendError(404) | |
293 | return | |
294 | } | |
295 | is LookupResult.Found -> { | |
296 | version = result.elem ?: Version(-1, projectInfo.project.id) | |
297 | } | |
298 | } | |
299 | ||
300 | with(http) { | |
301 | view = VersionEditView(projectInfo, version) | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
302 | feedPath = feedPath(projectInfo.project) |
184 | 303 | navigationMenu = activeProjectNavMenu( |
304 | dao.listProjects(), | |
305 | projectInfo, | |
306 | selectedVersion = version | |
307 | ) | |
308 | styleSheets = listOf("projects") | |
309 | render("version-form") | |
310 | } | |
311 | } | |
312 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
313 | private fun versionCommit(http: HttpRequest, dao: DataAccessObject) { |
184 | 314 | val id = http.param("id")?.toIntOrNull() |
315 | val projectid = http.param("projectid")?.toIntOrNull() ?: -1 | |
316 | val project = dao.findProject(projectid) | |
317 | if (id == null || project == null) { | |
318 | http.response.sendError(400) | |
319 | return | |
320 | } | |
321 | ||
322 | // TODO: replace defaults with throwing validator exceptions | |
323 | val version = Version(id, projectid).apply { | |
324 | name = http.param("name") ?: "" | |
325 | node = http.param("node") ?: "" | |
326 | ordinal = http.param("ordinal")?.toIntOrNull() ?: 0 | |
327 | status = http.param("status")?.let(VersionStatus::valueOf) ?: VersionStatus.Future | |
328 | // intentional defaults | |
329 | if (node.isBlank()) node = name | |
330 | // sanitizing | |
331 | node = sanitizeNode(node) | |
332 | } | |
333 | ||
334 | if (id < 0) { | |
335 | dao.insertVersion(version) | |
336 | } else { | |
337 | dao.updateVersion(version) | |
338 | } | |
339 | ||
340 | http.renderCommit("projects/${project.node}/versions/") | |
341 | } | |
342 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
343 | private fun components(http: HttpRequest, dao: DataAccessObject) { |
184 | 344 | val projectInfo = obtainProjectInfo(http, dao) |
345 | if (projectInfo == null) { | |
346 | http.response.sendError(404) | |
347 | return | |
348 | } | |
349 | ||
350 | with(http) { | |
205
7725a79416f3
#115 adds custom page titles
Mike Becker <universe@uap-core.de>
parents:
200
diff
changeset
|
351 | pageTitle = "${projectInfo.project.name} - ${i18n("navmenu.components")}" |
184 | 352 | view = ComponentsView( |
353 | projectInfo, | |
354 | dao.listComponentSummaries(projectInfo.project) | |
355 | ) | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
356 | feedPath = feedPath(projectInfo.project) |
184 | 357 | navigationMenu = activeProjectNavMenu( |
358 | dao.listProjects(), | |
359 | projectInfo | |
360 | ) | |
361 | styleSheets = listOf("projects") | |
362 | render("components") | |
363 | } | |
364 | } | |
365 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
366 | private fun componentForm(http: HttpRequest, dao: DataAccessObject) { |
184 | 367 | val projectInfo = obtainProjectInfo(http, dao) |
368 | if (projectInfo == null) { | |
369 | http.response.sendError(404) | |
370 | return | |
371 | } | |
372 | ||
373 | val component: Component | |
374 | when (val result = http.lookupPathParam("component", projectInfo.components)) { | |
375 | is LookupResult.NotFound -> { | |
376 | http.response.sendError(404) | |
377 | return | |
378 | } | |
379 | is LookupResult.Found -> { | |
380 | component = result.elem ?: Component(-1, projectInfo.project.id) | |
381 | } | |
382 | } | |
383 | ||
384 | with(http) { | |
385 | view = ComponentEditView(projectInfo, component, dao.listUsers()) | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
386 | feedPath = feedPath(projectInfo.project) |
184 | 387 | navigationMenu = activeProjectNavMenu( |
388 | dao.listProjects(), | |
389 | projectInfo, | |
390 | selectedComponent = component | |
391 | ) | |
392 | styleSheets = listOf("projects") | |
393 | render("component-form") | |
394 | } | |
395 | } | |
396 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
397 | private fun componentCommit(http: HttpRequest, dao: DataAccessObject) { |
184 | 398 | val id = http.param("id")?.toIntOrNull() |
399 | val projectid = http.param("projectid")?.toIntOrNull() ?: -1 | |
400 | val project = dao.findProject(projectid) | |
401 | if (id == null || project == null) { | |
402 | http.response.sendError(400) | |
403 | return | |
404 | } | |
405 | ||
406 | // TODO: replace defaults with throwing validator exceptions | |
407 | val component = Component(id, projectid).apply { | |
408 | name = http.param("name") ?: "" | |
409 | node = http.param("node") ?: "" | |
410 | ordinal = http.param("ordinal")?.toIntOrNull() ?: 0 | |
411 | color = WebColor(http.param("color") ?: "#000000") | |
412 | description = http.param("description") | |
413 | lead = (http.param("lead")?.toIntOrNull() ?: -1).let { | |
414 | if (it < 0) null else dao.findUser(it) | |
415 | } | |
416 | // intentional defaults | |
417 | if (node.isBlank()) node = name | |
418 | // sanitizing | |
419 | node = sanitizeNode(node) | |
420 | } | |
421 | ||
422 | if (id < 0) { | |
423 | dao.insertComponent(component) | |
424 | } else { | |
425 | dao.updateComponent(component) | |
426 | } | |
427 | ||
428 | http.renderCommit("projects/${project.node}/components/") | |
429 | } | |
430 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
431 | private fun issue(http: HttpRequest, dao: DataAccessObject) { |
184 | 432 | withPathInfo(http, dao)?.run { |
433 | val issue = dao.findIssue(http.pathParams["issue"]?.toIntOrNull() ?: -1) | |
434 | if (issue == null) { | |
435 | http.response.sendError(404) | |
436 | return | |
437 | } | |
438 | ||
439 | val comments = dao.listComments(issue) | |
440 | ||
441 | with(http) { | |
205
7725a79416f3
#115 adds custom page titles
Mike Becker <universe@uap-core.de>
parents:
200
diff
changeset
|
442 | pageTitle = "${projectInfo.project.name}: #${issue.id} ${issue.subject}" |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
443 | view = IssueDetailView(issue, comments, project, version, component) |
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
444 | feedPath = feedPath(projectInfo.project) |
184 | 445 | navigationMenu = activeProjectNavMenu( |
446 | dao.listProjects(), | |
447 | projectInfo, | |
448 | version, | |
449 | component | |
450 | ) | |
451 | styleSheets = listOf("projects") | |
207
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
452 | javascript = "issue-editor" |
184 | 453 | render("issue-view") |
454 | } | |
455 | } | |
456 | } | |
457 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
458 | private fun issueForm(http: HttpRequest, dao: DataAccessObject) { |
184 | 459 | withPathInfo(http, dao)?.run { |
460 | val issue = dao.findIssue(http.pathParams["issue"]?.toIntOrNull() ?: -1) ?: Issue( | |
461 | -1, | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
462 | project, |
184 | 463 | ) |
464 | ||
465 | // pre-select component, if available in the path info | |
466 | issue.component = component | |
467 | ||
191
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
468 | // pre-select version, if available in the path info |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
469 | if (version != null) { |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
470 | if (version.status.isReleased) { |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
471 | issue.affectedVersions = listOf(version) |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
472 | } else { |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
473 | issue.resolvedVersions = listOf(version) |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
474 | } |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
475 | } |
193ee4828767
fixes #134 - automatic version selection
Mike Becker <universe@uap-core.de>
parents:
186
diff
changeset
|
476 | |
184 | 477 | with(http) { |
478 | view = IssueEditView( | |
479 | issue, | |
480 | projectInfo.versions, | |
481 | projectInfo.components, | |
482 | dao.listUsers(), | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
483 | project, |
184 | 484 | version, |
485 | component | |
486 | ) | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
487 | feedPath = feedPath(projectInfo.project) |
184 | 488 | navigationMenu = activeProjectNavMenu( |
489 | dao.listProjects(), | |
490 | projectInfo, | |
491 | version, | |
492 | component | |
493 | ) | |
494 | styleSheets = listOf("projects") | |
207
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
495 | javascript = "issue-editor" |
184 | 496 | render("issue-form") |
497 | } | |
498 | } | |
499 | } | |
500 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
501 | private fun issueComment(http: HttpRequest, dao: DataAccessObject) { |
184 | 502 | withPathInfo(http, dao)?.run { |
503 | val issue = dao.findIssue(http.pathParams["issue"]?.toIntOrNull() ?: -1) | |
504 | if (issue == null) { | |
505 | http.response.sendError(404) | |
506 | return | |
507 | } | |
508 | ||
509 | // TODO: throw validator exception instead of using a default | |
207
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
510 | |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
511 | val commentId = http.param("commentid")?.toIntOrNull() ?: -1 |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
512 | if (commentId > 0) { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
513 | val comment = dao.findComment(commentId) |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
514 | val originalAuthor = comment?.author?.username |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
515 | if (originalAuthor != null && originalAuthor == http.remoteUser) { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
516 | comment.comment = http.param("comment") ?: "" |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
517 | dao.updateComment(comment) |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
518 | } else { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
519 | http.response.sendError(403) |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
520 | return |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
521 | } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
522 | } else { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
523 | val comment = IssueComment(-1, issue.id).apply { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
524 | author = http.remoteUser?.let { dao.findUserByName(it) } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
525 | comment = http.param("comment") ?: "" |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
526 | } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
205
diff
changeset
|
527 | dao.insertComment(comment) |
184 | 528 | } |
529 | ||
530 | http.renderCommit("${issuesHref}${issue.id}") | |
531 | } | |
532 | } | |
533 | ||
193
1e4044d29b1c
fixes missing issue sorting
Mike Becker <universe@uap-core.de>
parents:
191
diff
changeset
|
534 | private fun issueCommit(http: HttpRequest, dao: DataAccessObject) { |
184 | 535 | withPathInfo(http, dao)?.run { |
536 | // TODO: throw validator exception instead of using defaults | |
537 | val issue = Issue( | |
538 | http.param("id")?.toIntOrNull() ?: -1, | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
539 | project |
184 | 540 | ).apply { |
541 | component = dao.findComponent(http.param("component")?.toIntOrNull() ?: -1) | |
542 | category = IssueCategory.valueOf(http.param("category") ?: "") | |
543 | status = IssueStatus.valueOf(http.param("status") ?: "") | |
544 | subject = http.param("subject") ?: "" | |
545 | description = http.param("description") ?: "" | |
546 | assignee = http.param("assignee")?.toIntOrNull()?.let { | |
547 | when (it) { | |
548 | -1 -> null | |
549 | -2 -> component?.lead | |
550 | else -> dao.findUser(it) | |
551 | } | |
552 | } | |
186
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
553 | eta = http.param("eta")?.let { if (it.isBlank()) null else Date.valueOf(it) } |
184 | 554 | |
555 | affectedVersions = http.paramArray("affected") | |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
556 | .mapNotNull { param -> param.toIntOrNull()?.let { Version(it, project.id) } } |
184 | 557 | resolvedVersions = http.paramArray("resolved") |
198
94f174d591ab
fixes wrong handling of feeds - only one channel per feed is allowed
Mike Becker <universe@uap-core.de>
parents:
193
diff
changeset
|
558 | .mapNotNull { param -> param.toIntOrNull()?.let { Version(it, project.id) } } |
184 | 559 | } |
560 | ||
186
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
561 | val openId = if (issue.id < 0) { |
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
562 | dao.insertIssue(issue) |
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
563 | } else { |
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
564 | dao.updateIssue(issue) |
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
565 | issue.id |
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
566 | } |
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
567 | |
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
568 | if (http.param("more") != null) { |
185
5ec9fcfbdf9c
re-enables the "create another" feature
Mike Becker <universe@uap-core.de>
parents:
184
diff
changeset
|
569 | http.renderCommit("${issuesHref}-/create") |
5ec9fcfbdf9c
re-enables the "create another" feature
Mike Becker <universe@uap-core.de>
parents:
184
diff
changeset
|
570 | } else { |
186
05eec764facd
fixes some minor migration regressions
Mike Becker <universe@uap-core.de>
parents:
185
diff
changeset
|
571 | http.renderCommit("${issuesHref}${openId}") |
185
5ec9fcfbdf9c
re-enables the "create another" feature
Mike Becker <universe@uap-core.de>
parents:
184
diff
changeset
|
572 | } |
184 | 573 | } |
574 | } | |
575 | } |