Wed, 18 Aug 2021 15:30:49 +0200
#162 adds active flag to component
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
1 | /* |
180 | 2 | * Copyright 2021 Mike Becker. All rights reserved. |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
3 | * |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
4 | * Redistribution and use in source and binary forms, with or without |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
5 | * modification, are permitted provided that the following conditions are met: |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
6 | * |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
7 | * 1. Redistributions of source code must retain the above copyright |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
8 | * notice, this list of conditions and the following disclaimer. |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
9 | * |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
10 | * 2. Redistributions in binary form must reproduce the above copyright |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
11 | * notice, this list of conditions and the following disclaimer in the |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
12 | * documentation and/or other materials provided with the distribution. |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
13 | * |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
14 | * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
15 | * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
16 | * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
17 | * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
18 | * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
19 | * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
20 | * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
21 | * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
22 | * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
23 | * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
24 | */ |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
25 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
26 | package de.uapcore.lightpit.dao |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
27 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
28 | import de.uapcore.lightpit.entities.* |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
29 | import de.uapcore.lightpit.types.WebColor |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
30 | import de.uapcore.lightpit.util.* |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
31 | import de.uapcore.lightpit.viewmodel.ComponentSummary |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
32 | import de.uapcore.lightpit.viewmodel.IssueSummary |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
33 | import de.uapcore.lightpit.viewmodel.VersionSummary |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
34 | import org.intellij.lang.annotations.Language |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
35 | import java.sql.Connection |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
36 | import java.sql.PreparedStatement |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
37 | import java.sql.ResultSet |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
38 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
39 | class PostgresDataAccessObject(private val connection: Connection) : DataAccessObject { |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
40 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
41 | /** |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
42 | * Prepares the given [sql] statement and executes the [block] function with the prepared statement as receiver. |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
43 | * The statement is then closed properly. |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
44 | */ |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
45 | private fun <R> withStatement(@Language("SQL") sql: String, block: PreparedStatement.() -> R) = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
46 | connection.prepareStatement(sql).use(block) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
47 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
48 | /** |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
49 | * Prepares the given [sql] statement and executes the [block] function on that statement. |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
50 | * The statement is then closed properly. |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
51 | */ |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
52 | private fun <R> useStatement(@Language("SQL") sql: String, block: (PreparedStatement) -> R) = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
53 | connection.prepareStatement(sql).use(block) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
54 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
55 | /** |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
56 | * Executes the statement and iterates the whole result set extracting the rows with the given [extractor] function. |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
57 | */ |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
58 | private fun <T> PreparedStatement.queryAll(extractor: (ResultSet) -> T): List<T> = executeQuery().use { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
59 | sequence { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
60 | while (it.next()) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
61 | yield(extractor(it)) |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
62 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
63 | }.toList() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
64 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
65 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
66 | /** |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
67 | * Executes the statement and extracts a single row with the given [extractor] function. |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
68 | * If the result set is empty, null is returned. |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
69 | */ |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
70 | private fun <T> PreparedStatement.querySingle(extractor: (ResultSet) -> T): T? = executeQuery().use { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
71 | return if (it.next()) extractor(it) else null |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
72 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
73 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
74 | //<editor-fold desc="User"> |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
75 | //language=SQL |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
76 | private val userQuery = "select userid, username, lastname, givenname, mail from lpit_user" |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
77 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
78 | private fun ResultSet.extractUser() = User(getInt("userid")).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
79 | username = getString("username") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
80 | givenname = getString("givenname") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
81 | lastname = getString("lastname") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
82 | mail = getString("mail") |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
83 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
84 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
85 | private fun ResultSet.containsUserInfo(): Boolean { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
86 | getInt("userid") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
87 | return !wasNull() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
88 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
89 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
90 | private fun ResultSet.extractOptionalUser() = if (containsUserInfo()) extractUser() else null |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
91 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
92 | override fun listUsers() = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
93 | withStatement("$userQuery where userid > 0 order by username") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
94 | queryAll { it.extractUser() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
95 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
96 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
97 | override fun findUser(id: Int): User? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
98 | withStatement("$userQuery where userid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
99 | setInt(1, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
100 | querySingle { it.extractUser() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
101 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
102 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
103 | override fun findUserByName(username: String): User? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
104 | withStatement("$userQuery where lower(username) = lower(?)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
105 | setString(1, username) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
106 | querySingle { it.extractUser() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
107 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
108 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
109 | override fun insertUser(user: User) { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
110 | withStatement("insert into lpit_user (username, lastname, givenname, mail) values (?, ?, ?, ?)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
111 | with(user) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
112 | setStringSafe(1, username) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
113 | setStringOrNull(2, lastname) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
114 | setStringOrNull(3, givenname) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
115 | setStringOrNull(4, mail) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
116 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
117 | executeUpdate() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
118 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
119 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
120 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
121 | override fun updateUser(user: User) { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
122 | withStatement("update lpit_user set lastname = ?, givenname = ?, mail = ? where userid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
123 | with(user) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
124 | setStringOrNull(1, lastname) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
125 | setStringOrNull(2, givenname) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
126 | setStringOrNull(3, mail) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
127 | setInt(4, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
128 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
129 | executeUpdate() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
130 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
131 | } |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
132 | //</editor-fold> |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
133 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
134 | //<editor-fold desc="Version"> |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
135 | //language=SQL |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
136 | private val versionQuery = "select versionid, project, name, node, ordinal, status, release, eol from lpit_version" |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
137 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
138 | private fun ResultSet.extractVersion() = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
139 | Version(getInt("versionid"), getInt("project")).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
140 | name = getString("name") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
141 | node = getString("node") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
142 | ordinal = getInt("ordinal") |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
143 | release = getDate("release") |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
144 | eol = getDate("eol") |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
145 | status = getEnum("status") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
146 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
147 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
148 | override fun listVersions(project: Project): List<Version> = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
149 | withStatement("$versionQuery where project = ? order by ordinal desc, lower(name) desc") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
150 | setInt(1, project.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
151 | queryAll { it.extractVersion() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
152 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
153 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
154 | override fun listVersionSummaries(project: Project): List<VersionSummary> = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
155 | withStatement( |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
156 | """ |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
157 | with version_map(issueid, versionid, isresolved) as ( |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
158 | select issueid, versionid, true |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
159 | from lpit_issue_resolved_version |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
160 | union |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
161 | select issueid, versionid, false |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
162 | from lpit_issue_affected_version |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
163 | ), |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
164 | issues as ( |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
165 | select versionid, phase, isresolved, count(issueid) as total |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
166 | from lpit_issue |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
167 | join version_map using (issueid) |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
168 | join lpit_issue_phases using (status) |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
169 | group by versionid, phase, isresolved |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
170 | ), |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
171 | summary as ( |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
172 | select versionid, phase, isresolved, total |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
173 | from lpit_version v |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
174 | left join issues using (versionid) |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
175 | ) |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
176 | select v.versionid, project, name, node, ordinal, status, release, eol, |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
177 | ro.total as resolved_open, ra.total as resolved_active, rd.total as resolved_done, |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
178 | ao.total as affected_open, aa.total as affected_active, ad.total as affected_done |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
179 | from lpit_version v |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
180 | left join summary ro on ro.versionid = v.versionid and ro.phase = 0 and ro.isresolved |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
181 | left join summary ra on ra.versionid = v.versionid and ra.phase = 1 and ra.isresolved |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
182 | left join summary rd on rd.versionid = v.versionid and rd.phase = 2 and rd.isresolved |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
183 | left join summary ao on ao.versionid = v.versionid and ao.phase = 0 and not ao.isresolved |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
184 | left join summary aa on aa.versionid = v.versionid and aa.phase = 1 and not aa.isresolved |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
185 | left join summary ad on ad.versionid = v.versionid and ad.phase = 2 and not ad.isresolved |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
186 | where v.project = ? |
203
7d5b16379768
#152 same sort order for versions and version summaries
Mike Becker <universe@uap-core.de>
parents:
190
diff
changeset
|
187 | order by ordinal desc, lower(name) desc |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
188 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
189 | ) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
190 | setInt(1, project.id) |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
191 | queryAll { rs -> |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
192 | VersionSummary(rs.extractVersion()).apply { |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
193 | reportedTotal.open = rs.getInt("affected_open") |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
194 | reportedTotal.active = rs.getInt("affected_active") |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
195 | reportedTotal.done = rs.getInt("affected_done") |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
196 | resolvedTotal.open = rs.getInt("resolved_open") |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
197 | resolvedTotal.active = rs.getInt("resolved_active") |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
198 | resolvedTotal.done = rs.getInt("resolved_done") |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
199 | } |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
200 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
201 | } |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
202 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
203 | override fun findVersion(id: Int): Version? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
204 | withStatement("$versionQuery where versionid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
205 | setInt(1, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
206 | querySingle { it.extractVersion() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
207 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
208 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
209 | override fun findVersionByNode(project: Project, node: String): Version? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
210 | withStatement("$versionQuery where project = ? and node = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
211 | setInt(1, project.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
212 | setString(2, node) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
213 | querySingle { it.extractVersion() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
214 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
215 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
216 | override fun insertVersion(version: Version) { |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
217 | withStatement("insert into lpit_version (name, node, ordinal, status, project, release, eol) values (?, ?, ?, ?::version_status, ?, ?, ?)") { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
218 | with(version) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
219 | setStringSafe(1, name) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
220 | setStringSafe(2, node) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
221 | setInt(3, ordinal) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
222 | setEnum(4, status) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
223 | setInt(5, version.projectid) |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
224 | setDateOrNull(6, version.release) |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
225 | setDateOrNull(7, version.eol) |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
226 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
227 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
228 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
229 | |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
230 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
231 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
232 | override fun updateVersion(version: Version) { |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
233 | withStatement("update lpit_version set name = ?, node = ?, ordinal = ?, status = ?::version_status, release=?,eol=? where versionid = ?") { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
234 | with(version) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
235 | setStringSafe(1, name) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
236 | setStringSafe(2, node) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
237 | setInt(3, ordinal) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
238 | setEnum(4, status) |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
239 | setDateOrNull(5, version.release) |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
240 | setDateOrNull(6, version.eol) |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
241 | setInt(7, id) |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
242 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
243 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
244 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
245 | } |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
246 | //</editor-fold> |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
247 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
248 | //<editor-fold desc="Component"> |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
249 | //language=SQL |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
250 | private val componentQuery = |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
251 | """ |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
252 | select id, project, name, node, color, ordinal, description, active, |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
253 | userid, username, givenname, lastname, mail |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
254 | from lpit_component |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
255 | left join lpit_user on lead = userid |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
256 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
257 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
258 | private fun ResultSet.extractComponent(): Component = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
259 | Component(getInt("id"), getInt("project")).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
260 | name = getString("name") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
261 | node = getString("node") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
262 | color = try { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
263 | WebColor(getString("color")) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
264 | } catch (ex: IllegalArgumentException) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
265 | WebColor("000000") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
266 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
267 | ordinal = getInt("ordinal") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
268 | description = getString("description") |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
269 | active = getBoolean("active") |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
270 | lead = extractOptionalUser() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
271 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
272 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
273 | private fun PreparedStatement.setComponent(index: Int, component: Component): Int { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
274 | with(component) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
275 | var i = index |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
276 | setStringSafe(i++, name) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
277 | setStringSafe(i++, node) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
278 | setStringSafe(i++, color.hex) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
279 | setInt(i++, ordinal) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
280 | setStringOrNull(i++, description) |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
281 | setBoolean(i++, active) |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
282 | setIntOrNull(i++, lead?.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
283 | return i |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
284 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
285 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
286 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
287 | override fun listComponents(project: Project): List<Component> = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
288 | withStatement("$componentQuery where project = ? order by ordinal, lower(name)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
289 | setInt(1, project.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
290 | queryAll { it.extractComponent() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
291 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
292 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
293 | override fun listComponentSummaries(project: Project): List<ComponentSummary> = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
294 | withStatement( |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
295 | """ |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
296 | with issues as ( |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
297 | select component, phase, count(issueid) as total |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
298 | from lpit_issue |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
299 | join lpit_issue_phases using (status) |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
300 | group by component, phase |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
301 | ), |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
302 | summary as ( |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
303 | select c.id, phase, total |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
304 | from lpit_component c |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
305 | left join issues i on c.id = i.component |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
306 | ) |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
307 | select c.id, project, name, node, color, ordinal, description, active, |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
308 | userid, username, givenname, lastname, mail, |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
309 | open.total as open, wip.total as wip, done.total as done |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
310 | from lpit_component c |
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
311 | left join lpit_user on lead = userid |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
312 | left join summary open on c.id = open.id and open.phase = 0 |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
313 | left join summary wip on c.id = wip.id and wip.phase = 1 |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
314 | left join summary done on c.id = done.id and done.phase = 2 |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
315 | where c.project = ? |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
316 | order by ordinal, name |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
317 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
318 | ) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
319 | setInt(1, project.id) |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
320 | queryAll { rs -> |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
321 | ComponentSummary(rs.extractComponent()).apply { |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
322 | issueSummary.open = rs.getInt("open") |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
323 | issueSummary.active = rs.getInt("wip") |
190
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
324 | issueSummary.done = rs.getInt("done") |
a83f1ab56898
fixes #138 - ResultSet processing for summaries
Mike Becker <universe@uap-core.de>
parents:
189
diff
changeset
|
325 | } |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
326 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
327 | } |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
328 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
329 | override fun findComponent(id: Int): Component? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
330 | withStatement("$componentQuery where id = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
331 | setInt(1, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
332 | querySingle { it.extractComponent() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
333 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
334 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
335 | override fun findComponentByNode(project: Project, node: String): Component? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
336 | withStatement("$componentQuery where project = ? and node = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
337 | setInt(1, project.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
338 | setString(2, node) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
339 | querySingle { it.extractComponent() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
340 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
341 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
342 | override fun insertComponent(component: Component) { |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
343 | withStatement("insert into lpit_component (name, node, color, ordinal, description, active, lead, project) values (?, ?, ?, ?, ?, ?, ?, ?)") { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
344 | val col = setComponent(1, component) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
345 | setInt(col, component.projectid) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
346 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
347 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
348 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
349 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
350 | override fun updateComponent(component: Component) { |
227
f0ede8046b59
#162 adds active flag to component
Mike Becker <universe@uap-core.de>
parents:
225
diff
changeset
|
351 | withStatement("update lpit_component set name = ?, node = ?, color = ?, ordinal = ?, description = ?, active = ?, lead = ? where id = ?") { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
352 | val col = setComponent(1, component) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
353 | setInt(col, component.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
354 | executeUpdate() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
355 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
356 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
357 | |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
358 | //</editor-fold> |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
359 | |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
360 | //<editor-fold desc="Project"> |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
361 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
362 | //language=SQL |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
363 | private val projectQuery = |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
364 | """ |
175
1e6f2aace666
adds project ordering - fixes #34
Mike Becker <universe@uap-core.de>
parents:
167
diff
changeset
|
365 | select projectid, name, node, ordinal, description, repourl, |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
366 | userid, username, lastname, givenname, mail |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
367 | from lpit_project |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
368 | left join lpit_user owner on lpit_project.owner = owner.userid |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
369 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
370 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
371 | private fun ResultSet.extractProject() = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
372 | Project(getInt("projectid")).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
373 | name = getString("name") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
374 | node = getString("node") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
375 | ordinal = getInt("ordinal") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
376 | description = getString("description") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
377 | repoUrl = getString("repourl") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
378 | owner = extractOptionalUser() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
379 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
380 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
381 | private fun PreparedStatement.setProject(index: Int, project: Project): Int { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
382 | var i = index |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
383 | with(project) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
384 | setStringSafe(i++, name) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
385 | setStringSafe(i++, node) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
386 | setInt(i++, ordinal) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
387 | setStringOrNull(i++, description) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
388 | setStringOrNull(i++, repoUrl) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
389 | setIntOrNull(i++, owner?.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
390 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
391 | return i |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
392 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
393 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
394 | override fun listProjects(): List<Project> = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
395 | withStatement("$projectQuery order by ordinal, lower(name)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
396 | queryAll { it.extractProject() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
397 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
398 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
399 | override fun findProject(id: Int): Project? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
400 | withStatement("$projectQuery where projectid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
401 | setInt(1, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
402 | querySingle { it.extractProject() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
403 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
404 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
405 | override fun findProjectByNode(node: String): Project? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
406 | withStatement("$projectQuery where node = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
407 | setString(1, node) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
408 | querySingle { it.extractProject() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
409 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
410 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
411 | override fun insertProject(project: Project) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
412 | withStatement("insert into lpit_project (name, node, ordinal, description, repourl, owner) values (?, ?, ?, ?, ?, ?)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
413 | setProject(1, project) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
414 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
415 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
416 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
417 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
418 | override fun updateProject(project: Project) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
419 | withStatement("update lpit_project set name = ?, node = ?, ordinal = ?, description = ?, repourl = ?, owner = ? where projectid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
420 | val col = setProject(1, project) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
421 | setInt(col, project.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
422 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
423 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
424 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
425 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
426 | override fun collectIssueSummary(project: Project): IssueSummary = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
427 | withStatement( |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
428 | """ |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
429 | select phase, count(*) as total |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
430 | from lpit_issue |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
431 | join lpit_issue_phases using(status) |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
432 | where project = ? |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
433 | group by phase |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
434 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
435 | ) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
436 | setInt(1, project.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
437 | executeQuery().use { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
438 | val summary = IssueSummary() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
439 | while (it.next()) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
440 | val phase = it.getInt("phase") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
441 | val total = it.getInt("total") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
442 | when (phase) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
443 | 0 -> summary.open = total |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
444 | 1 -> summary.active = total |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
445 | 2 -> summary.done = total |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
446 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
447 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
448 | summary |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
449 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
450 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
451 | |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
452 | //</editor-fold> |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
453 | |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
454 | //<editor-fold desc="Issue"> |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
455 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
456 | //language=SQL |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
457 | private val issueQuery = |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
458 | """ |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
459 | select issueid, |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
460 | i.project, p.name as projectname, p.node as projectnode, |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
461 | component, c.name as componentname, c.node as componentnode, |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
462 | status, category, subject, i.description, |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
463 | userid, username, givenname, lastname, mail, |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
464 | created, updated, eta |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
465 | from lpit_issue i |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
466 | join lpit_project p on i.project = projectid |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
467 | left join lpit_component c on component = c.id |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
468 | left join lpit_user on userid = assignee |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
469 | """.trimIndent() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
470 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
471 | private fun ResultSet.extractIssue(): Issue { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
472 | val proj = Project(getInt("project")).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
473 | name = getString("projectname") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
474 | node = getString("projectnode") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
475 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
476 | val comp = getInt("component").let { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
477 | if (wasNull()) null else |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
478 | Component(it, proj.id).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
479 | name = getString("componentname") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
480 | node = getString("componentnode") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
481 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
482 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
483 | val issue = Issue(getInt("issueid"), proj).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
484 | component = comp |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
485 | status = getEnum("status") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
486 | category = getEnum("category") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
487 | subject = getString("subject") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
488 | description = getString("description") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
489 | assignee = extractOptionalUser() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
490 | created = getTimestamp("created") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
491 | updated = getTimestamp("updated") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
492 | eta = getDate("eta") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
493 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
494 | |
215
028792eda9b7
#156 fixes auto-selection overriding issue data
Mike Becker <universe@uap-core.de>
parents:
207
diff
changeset
|
495 | //language=SQL |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
496 | val queryAffected = |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
497 | """ |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
498 | $versionQuery join lpit_issue_affected_version using (versionid) |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
499 | where issueid = ? order by ordinal, name |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
500 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
501 | |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
502 | //language=SQL |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
503 | val queryResolved = |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
504 | """ |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
505 | $versionQuery join lpit_issue_resolved_version using (versionid) |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
506 | where issueid = ? order by ordinal, name |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
507 | """.trimIndent() |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
508 | |
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
509 | issue.affectedVersions = withStatement(queryAffected) { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
510 | setInt(1, issue.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
511 | queryAll { it.extractVersion() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
512 | } |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
513 | issue.resolvedVersions = withStatement(queryResolved) { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
514 | setInt(1, issue.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
515 | queryAll { it.extractVersion() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
516 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
517 | return issue |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
518 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
519 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
520 | private fun PreparedStatement.setIssue(index: Int, issue: Issue): Int { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
521 | var i = index |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
522 | with(issue) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
523 | setIntOrNull(i++, component?.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
524 | setEnum(i++, status) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
525 | setEnum(i++, category) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
526 | setStringSafe(i++, subject) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
527 | setStringOrNull(i++, description) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
528 | setIntOrNull(i++, assignee?.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
529 | setDateOrNull(i++, eta) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
530 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
531 | return i |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
532 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
533 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
534 | override fun listIssues(filter: IssueFilter): List<Issue> = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
535 | withStatement( |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
536 | """ |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
537 | with issue_version as ( |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
538 | select issueid, versionid from lpit_issue_affected_version |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
539 | union select issueid, versionid from lpit_issue_resolved_version |
183
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
540 | ), |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
541 | filtered_issues as ( |
183
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
542 | select distinct issueid from lpit_issue |
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
543 | left join issue_version using (issueid) |
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
544 | where |
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
545 | (not ? or project = ?) and |
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
546 | (not ? or versionid = ?) and (not ? or versionid is null) and |
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
547 | (not ? or component = ?) and (not ? or component is null) |
61669abf277f
fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents:
180
diff
changeset
|
548 | ) |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
549 | $issueQuery join filtered_issues using (issueid) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
550 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
551 | ) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
552 | fun <T : Entity> applyFilter(filter: Filter<T>, fflag: Int, nflag: Int, idcol: Int) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
553 | when (filter) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
554 | is AllFilter -> { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
555 | setBoolean(fflag, false) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
556 | setBoolean(nflag, false) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
557 | setInt(idcol, 0) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
558 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
559 | is NoneFilter -> { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
560 | setBoolean(fflag, false) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
561 | setBoolean(nflag, true) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
562 | setInt(idcol, 0) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
563 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
564 | is SpecificFilter -> { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
565 | setBoolean(fflag, true) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
566 | setBoolean(nflag, false) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
567 | setInt(idcol, filter.obj.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
568 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
569 | else -> { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
570 | TODO("Implement range filter.") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
571 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
572 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
573 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
574 | when (filter.project) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
575 | is AllFilter -> { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
576 | setBoolean(1, false) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
577 | setInt(2, 0) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
578 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
579 | is SpecificFilter -> { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
580 | setBoolean(1, true) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
581 | setInt(2, filter.project.obj.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
582 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
583 | else -> throw IllegalArgumentException() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
584 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
585 | applyFilter(filter.version, 3, 5, 4) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
586 | applyFilter(filter.component, 6, 8, 7) |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
587 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
588 | queryAll { it.extractIssue() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
589 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
590 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
591 | override fun findIssue(id: Int): Issue? = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
592 | withStatement("$issueQuery where issueid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
593 | setInt(1, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
594 | querySingle { it.extractIssue() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
595 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
596 | |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
597 | private fun insertVersionInfo(id: Int, issue: Issue) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
598 | withStatement("insert into lpit_issue_affected_version (issueid, versionid) values (?,?)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
599 | setInt(1, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
600 | issue.affectedVersions.forEach { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
601 | setInt(2, it.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
602 | executeUpdate() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
603 | } |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
604 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
605 | withStatement("insert into lpit_issue_resolved_version (issueid, versionid) values (?,?)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
606 | setInt(1, id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
607 | issue.resolvedVersions.forEach { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
608 | setInt(2, it.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
609 | executeUpdate() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
610 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
611 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
612 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
613 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
614 | override fun insertIssue(issue: Issue): Int { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
615 | val id = withStatement( |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
616 | """ |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
617 | insert into lpit_issue (component, status, category, subject, description, assignee, eta, project) |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
618 | values (?, ?::issue_status, ?::issue_category, ?, ?, ?, ?, ?) |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
619 | returning issueid |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
620 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
621 | ) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
622 | val col = setIssue(1, issue) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
623 | setInt(col, issue.project.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
624 | querySingle { it.getInt(1) }!! |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
625 | } |
188
2979436edd9e
fixes insertVersionInfo not using inserted issue id
Mike Becker <universe@uap-core.de>
parents:
184
diff
changeset
|
626 | insertVersionInfo(id, issue) |
184
e8eecee6aadf
completes kotlin migration
Mike Becker <universe@uap-core.de>
parents:
183
diff
changeset
|
627 | return id |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
628 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
629 | |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
630 | override fun updateIssue(issue: Issue) { |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
631 | withStatement( |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
632 | """ |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
633 | update lpit_issue set updated = now(), |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
634 | component = ?, status = ?::issue_status, category = ?::issue_category, subject = ?, |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
635 | description = ?, assignee = ?, eta = ? |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
636 | where issueid = ? |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
637 | """.trimIndent() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
638 | ) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
639 | val col = setIssue(1, issue) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
640 | setInt(col, issue.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
641 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
642 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
643 | |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
644 | // TODO: improve by only inserting / deleting changed version information |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
645 | withStatement("delete from lpit_issue_affected_version where issueid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
646 | setInt(1, issue.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
647 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
648 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
649 | withStatement("delete from lpit_issue_resolved_version where issueid = ?") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
650 | setInt(1, issue.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
651 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
652 | } |
188
2979436edd9e
fixes insertVersionInfo not using inserted issue id
Mike Becker <universe@uap-core.de>
parents:
184
diff
changeset
|
653 | insertVersionInfo(issue.id, issue) |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
654 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
655 | |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
656 | //</editor-fold> |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
657 | |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
658 | //<editor-fold desc="IssueComment"> |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
659 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
660 | private fun ResultSet.extractIssueComment() = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
661 | IssueComment(getInt("commentid"), getInt("issueid")).apply { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
662 | created = getTimestamp("created") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
663 | updated = getTimestamp("updated") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
664 | updateCount = getInt("updatecount") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
665 | comment = getString("comment") |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
666 | author = extractOptionalUser() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
667 | } |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
668 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
669 | override fun listComments(issue: Issue): List<IssueComment> = |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
670 | withStatement("select * from lpit_issue_comment left join lpit_user using (userid) where issueid = ? order by created") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
671 | setInt(1, issue.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
672 | queryAll { it.extractIssueComment() } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
673 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
674 | |
207
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
675 | override fun findComment(id: Int): IssueComment? = |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
676 | withStatement("select * from lpit_issue_comment left join lpit_user using (userid) where commentid = ?") { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
677 | setInt(1, id) |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
678 | querySingle { it.extractIssueComment() } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
679 | } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
680 | |
189
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
681 | override fun insertComment(issueComment: IssueComment) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
682 | useStatement("update lpit_issue set updated = now() where issueid = ?") { updateIssueDate -> |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
683 | withStatement("insert into lpit_issue_comment (issueid, comment, userid) values (?, ? ,?)") { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
684 | with(issueComment) { |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
685 | updateIssueDate.setInt(1, issueid) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
686 | setInt(1, issueid) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
687 | setStringSafe(2, comment) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
688 | setIntOrNull(3, author?.id) |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
689 | } |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
690 | executeUpdate() |
f7de8158b41c
fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents:
188
diff
changeset
|
691 | updateIssueDate.executeUpdate() |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
692 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
693 | } |
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
694 | } |
207
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
695 | |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
696 | override fun updateComment(issueComment: IssueComment) { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
697 | useStatement("update lpit_issue set updated = now() where issueid = ?") { updateIssueDate -> |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
698 | withStatement("update lpit_issue_comment set comment = ?, updatecount = updatecount + 1, updated = now() where commentid = ?") { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
699 | with(issueComment) { |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
700 | updateIssueDate.setInt(1, issueid) |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
701 | setStringSafe(1, comment) |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
702 | setInt(2, id) |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
703 | } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
704 | executeUpdate() |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
705 | updateIssueDate.executeUpdate() |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
706 | } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
707 | } |
479dd7993ef9
#22 adds possibility to edit own comments
Mike Becker <universe@uap-core.de>
parents:
203
diff
changeset
|
708 | } |
225
87328572e36f
#159 adds release and eol dates
Mike Becker <universe@uap-core.de>
parents:
215
diff
changeset
|
709 | //</editor-fold> |
167
3f30adba1c63
major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff
changeset
|
710 | } |