src/main/kotlin/de/uapcore/lightpit/dao/PostgresDataAccessObject.kt

Tue, 11 May 2021 16:00:28 +0200

author
Mike Becker <universe@uap-core.de>
date
Tue, 11 May 2021 16:00:28 +0200
changeset 189
f7de8158b41c
parent 188
2979436edd9e
child 190
a83f1ab56898
permissions
-rw-r--r--

fixes #137 - leaking prepared statements

167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
1 /*
180
009700915269 merge resource bundles
Mike Becker <universe@uap-core.de>
parents: 176
diff changeset
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 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
132 //</editor-fold>
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
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
136 private val versionQuery = "select versionid, project, name, node, ordinal, status from lpit_version"
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")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
143 status = getEnum("status")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
144 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
145
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
146 override fun listVersions(project: Project): List<Version> =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
147 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
148 setInt(1, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
149 queryAll { it.extractVersion() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
150 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
151
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
152 override fun listVersionSummaries(project: Project): List<VersionSummary> =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
153 withStatement(
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
154 """
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
155 with version_map(issueid, versionid, isresolved) as (
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
156 select issueid, versionid, 1
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
157 from lpit_issue_resolved_version
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
158 union
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
159 select issueid, versionid, 0
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
160 from lpit_issue_affected_version
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
161 ),
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
162 issues as (
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
163 select versionid, phase, isresolved, count(issueid) as total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
164 from lpit_issue
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
165 join version_map using (issueid)
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
166 join lpit_issue_phases using (status)
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
167 group by versionid, phase, isresolved
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
168 ),
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
169 summary as (
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
170 select versionid, phase, isresolved, total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
171 from lpit_version v
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
172 left join issues using (versionid)
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
173 where v.project = ?
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
174 )
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
175 select versionid, project, name, node, ordinal, status, phase, isresolved, total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
176 from lpit_version
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
177 join summary using (versionid)
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
178 order by ordinal, name
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
179 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
180 ) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
181 setInt(1, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
182 executeQuery().use { rs ->
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
183 sequence {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
184 // TODO: fix bug: this extractor is not grouping the results
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
185 val versionSummary = VersionSummary(rs.extractVersion())
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
186 val phase = rs.getInt("phase")
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
187 val total = rs.getInt("total")
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
188 val issueSummary =
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
189 if (rs.getBoolean("isresolved")) versionSummary.resolvedTotal else versionSummary.reportedTotal
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
190 when (phase) {
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
191 0 -> issueSummary.open = total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
192 1 -> issueSummary.active = total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
193 2 -> issueSummary.done = total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
194 }
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
195 yield(versionSummary)
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
196 }.toList()
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
197 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
198 }
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
199
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
200 override fun findVersion(id: Int): Version? =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
201 withStatement("$versionQuery where versionid = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
202 setInt(1, id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
203 querySingle { it.extractVersion() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
204 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
205
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
206 override fun findVersionByNode(project: Project, node: String): Version? =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
207 withStatement("$versionQuery where project = ? and node = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
208 setInt(1, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
209 setString(2, node)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
210 querySingle { it.extractVersion() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
211 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
212
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
213 override fun insertVersion(version: Version) {
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
214 withStatement("insert into lpit_version (name, node, ordinal, status, project) values (?, ?, ?, ?::version_status, ?)") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
215 with(version) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
216 setStringSafe(1, name)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
217 setStringSafe(2, node)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
218 setInt(3, ordinal)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
219 setEnum(4, status)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
220 setInt(5, version.projectid)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
221 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
222 executeUpdate()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
223 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
224
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
225 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
226
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
227 override fun updateVersion(version: Version) {
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
228 withStatement("update lpit_version set name = ?, node = ?, ordinal = ?, status = ?::version_status where versionid = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
229 with(version) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
230 setStringSafe(1, name)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
231 setStringSafe(2, node)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
232 setInt(3, ordinal)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
233 setEnum(4, status)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
234 setInt(5, id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
235 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
236 executeUpdate()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
237 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
238 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
239 //</editor-fold>
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
240
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
241 //<editor-fold desc="Component">
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
242 //language=SQL
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
243 private val componentQuery =
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
244 """
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
245 select id, project, name, node, color, ordinal, description,
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
246 userid, username, givenname, lastname, mail
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
247 from lpit_component
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
248 left join lpit_user on lead = userid
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
249 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
250
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
251 private fun ResultSet.extractComponent(): Component =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
252 Component(getInt("id"), getInt("project")).apply {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
253 name = getString("name")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
254 node = getString("node")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
255 color = try {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
256 WebColor(getString("color"))
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
257 } catch (ex: IllegalArgumentException) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
258 WebColor("000000")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
259 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
260 ordinal = getInt("ordinal")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
261 description = getString("description")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
262 lead = extractOptionalUser()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
263 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
264
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
265 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
266 with(component) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
267 var i = index
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
268 setStringSafe(i++, name)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
269 setStringSafe(i++, node)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
270 setStringSafe(i++, color.hex)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
271 setInt(i++, ordinal)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
272 setStringOrNull(i++, description)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
273 setIntOrNull(i++, lead?.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
274 return i
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
275 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
276 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
277
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
278 override fun listComponents(project: Project): List<Component> =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
279 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
280 setInt(1, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
281 queryAll { it.extractComponent() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
282 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
283
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
284 override fun listComponentSummaries(project: Project): List<ComponentSummary> =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
285 withStatement(
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
286 """
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
287 with issues as (
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
288 select component, phase, count(issueid) as total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
289 from lpit_issue
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
290 join lpit_issue_phases using (status)
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
291 group by component, phase
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
292 ),
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
293 summary as (
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
294 select c.id, phase, total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
295 from lpit_component c
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
296 left join issues i on c.id = i.component
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
297 where c.project = ?
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
298 )
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
299 select c.id, project, name, node, color, ordinal, description,
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
300 userid, username, givenname, lastname, mail, phase, total
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
301 from lpit_component c
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
302 left join lpit_user on lead = userid
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
303 join summary s on c.id = s.id
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
304 order by ordinal, name
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
305 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
306 ) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
307 setInt(1, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
308 executeQuery().use { rs ->
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
309 // TODO: fix bug: this extractor is not grouping the results
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
310 sequence {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
311 val componentSummary = ComponentSummary(rs.extractComponent()).also {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
312 val phase = rs.getInt("phase")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
313 val total = rs.getInt("total")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
314 when (phase) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
315 0 -> it.issueSummary.open = total
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
316 1 -> it.issueSummary.active = total
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
317 2 -> it.issueSummary.done = total
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
318 }
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
319 }
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
320 yield(componentSummary)
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
321 }.toList()
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
322 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
323 }
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
324
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
325 override fun findComponent(id: Int): Component? =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
326 withStatement("$componentQuery where id = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
327 setInt(1, id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
328 querySingle { it.extractComponent() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
329 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
330
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
331 override fun findComponentByNode(project: Project, node: String): Component? =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
332 withStatement("$componentQuery where project = ? and node = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
333 setInt(1, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
334 setString(2, node)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
335 querySingle { it.extractComponent() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
336 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
337
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
338 override fun insertComponent(component: Component) {
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
339 withStatement("insert into lpit_component (name, node, color, ordinal, description, lead, project) values (?, ?, ?, ?, ?, ?, ?)") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
340 val col = setComponent(1, component)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
341 setInt(col, component.projectid)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
342 executeUpdate()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
343 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
344 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
345
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
346 override fun updateComponent(component: Component) {
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
347 withStatement("update lpit_component set name = ?, node = ?, color = ?, ordinal = ?, description = ?, lead = ? where id = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
348 val col = setComponent(1, component)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
349 setInt(col, component.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
350 executeUpdate()
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
351 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
352 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
353
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
354 //</editor-fold>
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
355
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
356 //<editor-fold desc="Project">
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
357
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
358 //language=SQL
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
359 private val projectQuery =
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
360 """
175
1e6f2aace666 adds project ordering - fixes #34
Mike Becker <universe@uap-core.de>
parents: 167
diff changeset
361 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
362 userid, username, lastname, givenname, mail
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
363 from lpit_project
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
364 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
365 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
366
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
367 private fun ResultSet.extractProject() =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
368 Project(getInt("projectid")).apply {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
369 name = getString("name")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
370 node = getString("node")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
371 ordinal = getInt("ordinal")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
372 description = getString("description")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
373 repoUrl = getString("repourl")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
374 owner = extractOptionalUser()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
375 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
376
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
377 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
378 var i = index
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
379 with(project) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
380 setStringSafe(i++, name)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
381 setStringSafe(i++, node)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
382 setInt(i++, ordinal)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
383 setStringOrNull(i++, description)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
384 setStringOrNull(i++, repoUrl)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
385 setIntOrNull(i++, owner?.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
386 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
387 return i
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
388 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
389
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
390 override fun listProjects(): List<Project> =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
391 withStatement("$projectQuery order by ordinal, lower(name)") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
392 queryAll { it.extractProject() }
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
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
395 override fun findProject(id: Int): Project? =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
396 withStatement("$projectQuery where projectid = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
397 setInt(1, id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
398 querySingle { it.extractProject() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
399 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
400
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
401 override fun findProjectByNode(node: String): Project? =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
402 withStatement("$projectQuery where node = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
403 setString(1, node)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
404 querySingle { it.extractProject() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
405 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
406
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
407 override fun insertProject(project: Project) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
408 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
409 setProject(1, project)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
410 executeUpdate()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
411 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
412 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
413
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
414 override fun updateProject(project: Project) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
415 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
416 val col = setProject(1, project)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
417 setInt(col, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
418 executeUpdate()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
419 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
420 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
421
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
422 override fun collectIssueSummary(project: Project): IssueSummary =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
423 withStatement(
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
424 """
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
425 select phase, count(*) as total
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
426 from lpit_issue
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
427 join lpit_issue_phases using(status)
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
428 where project = ?
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
429 group by phase
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
430 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
431 ) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
432 setInt(1, project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
433 executeQuery().use {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
434 val summary = IssueSummary()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
435 while (it.next()) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
436 val phase = it.getInt("phase")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
437 val total = it.getInt("total")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
438 when (phase) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
439 0 -> summary.open = total
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
440 1 -> summary.active = total
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
441 2 -> summary.done = total
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
442 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
443 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
444 summary
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
445 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
446 }
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 //</editor-fold>
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
449
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
450 //<editor-fold desc="Issue">
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
451
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
452 //language=SQL
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
453 private val issueQuery =
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
454 """
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
455 select issueid,
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
456 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
457 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
458 status, category, subject, i.description,
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
459 userid, username, givenname, lastname, mail,
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
460 created, updated, eta
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
461 from lpit_issue i
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
462 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
463 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
464 left join lpit_user on userid = assignee
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
465 """.trimIndent()
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
466
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
467 private fun ResultSet.extractIssue(): Issue {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
468 val proj = Project(getInt("project")).apply {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
469 name = getString("projectname")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
470 node = getString("projectnode")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
471 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
472 val comp = getInt("component").let {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
473 if (wasNull()) null else
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
474 Component(it, proj.id).apply {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
475 name = getString("componentname")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
476 node = getString("componentnode")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
477 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
478 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
479 val issue = Issue(getInt("issueid"), proj).apply {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
480 component = comp
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
481 status = getEnum("status")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
482 category = getEnum("category")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
483 subject = getString("subject")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
484 description = getString("description")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
485 assignee = extractOptionalUser()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
486 created = getTimestamp("created")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
487 updated = getTimestamp("updated")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
488 eta = getDate("eta")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
489 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
490
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
491 fun versionQuery(table: String) =
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
492 """
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
493 select versionid, project, name, status, ordinal, node
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
494 from lpit_version join $table using (versionid)
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
495 where issueid = ?
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
496 order by ordinal, name
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
497 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
498
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
499 issue.affectedVersions = withStatement(versionQuery("lpit_issue_affected_version")) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
500 setInt(1, issue.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
501 queryAll { it.extractVersion() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
502 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
503 issue.resolvedVersions = withStatement(versionQuery("lpit_issue_resolved_version")) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
504 setInt(1, issue.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
505 queryAll { it.extractVersion() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
506 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
507 return issue
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
508 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
509
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
510 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
511 var i = index
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
512 with(issue) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
513 setIntOrNull(i++, component?.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
514 setEnum(i++, status)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
515 setEnum(i++, category)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
516 setStringSafe(i++, subject)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
517 setStringOrNull(i++, description)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
518 setIntOrNull(i++, assignee?.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
519 setDateOrNull(i++, eta)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
520 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
521 return i
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
522 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
523
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
524 override fun listIssues(filter: IssueFilter): List<Issue> =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
525 withStatement(
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
526 """
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
527 with issue_version as (
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
528 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
529 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
530 ),
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
531 filtered_issues as (
183
61669abf277f fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents: 180
diff changeset
532 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
533 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
534 where
61669abf277f fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents: 180
diff changeset
535 (not ? or project = ?) and
61669abf277f fixes issue query returning issues more than once
Mike Becker <universe@uap-core.de>
parents: 180
diff changeset
536 (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
537 (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
538 )
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
539 $issueQuery join filtered_issues using (issueid)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
540 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
541 ) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
542 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
543 when (filter) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
544 is AllFilter -> {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
545 setBoolean(fflag, false)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
546 setBoolean(nflag, false)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
547 setInt(idcol, 0)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
548 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
549 is NoneFilter -> {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
550 setBoolean(fflag, false)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
551 setBoolean(nflag, true)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
552 setInt(idcol, 0)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
553 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
554 is SpecificFilter -> {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
555 setBoolean(fflag, true)
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, filter.obj.id)
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 else -> {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
560 TODO("Implement range filter.")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
561 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
562 }
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 when (filter.project) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
565 is AllFilter -> {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
566 setBoolean(1, false)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
567 setInt(2, 0)
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 is SpecificFilter -> {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
570 setBoolean(1, true)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
571 setInt(2, filter.project.obj.id)
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 else -> throw IllegalArgumentException()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
574 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
575 applyFilter(filter.version, 3, 5, 4)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
576 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
577
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
578 queryAll { it.extractIssue() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
579 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
580
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
581 override fun findIssue(id: Int): Issue? =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
582 withStatement("$issueQuery where issueid = ?") {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
583 setInt(1, id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
584 querySingle { it.extractIssue() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
585 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
586
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
587 private fun insertVersionInfo(id: Int, issue: Issue) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
588 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
589 setInt(1, id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
590 issue.affectedVersions.forEach {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
591 setInt(2, it.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
592 executeUpdate()
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
593 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
594 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
595 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
596 setInt(1, id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
597 issue.resolvedVersions.forEach {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
598 setInt(2, it.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
599 executeUpdate()
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
600 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
601 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
602 }
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 override fun insertIssue(issue: Issue): Int {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
605 val id = withStatement(
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
606 """
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
607 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
608 values (?, ?::issue_status, ?::issue_category, ?, ?, ?, ?, ?)
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
609 returning issueid
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
610 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
611 ) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
612 val col = setIssue(1, issue)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
613 setInt(col, issue.project.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
614 querySingle { it.getInt(1) }!!
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
615 }
188
2979436edd9e fixes insertVersionInfo not using inserted issue id
Mike Becker <universe@uap-core.de>
parents: 184
diff changeset
616 insertVersionInfo(id, issue)
184
e8eecee6aadf completes kotlin migration
Mike Becker <universe@uap-core.de>
parents: 183
diff changeset
617 return id
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
618 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
619
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
620 override fun updateIssue(issue: Issue) {
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
621 withStatement(
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
622 """
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
623 update lpit_issue set updated = now(),
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
624 component = ?, status = ?::issue_status, category = ?::issue_category, subject = ?,
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
625 description = ?, assignee = ?, eta = ?
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
626 where issueid = ?
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
627 """.trimIndent()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
628 ) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
629 val col = setIssue(1, issue)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
630 setInt(col, issue.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
631 executeUpdate()
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
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
634 // 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
635 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
636 setInt(1, issue.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
637 executeUpdate()
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 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
640 setInt(1, 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 }
188
2979436edd9e fixes insertVersionInfo not using inserted issue id
Mike Becker <universe@uap-core.de>
parents: 184
diff changeset
643 insertVersionInfo(issue.id, issue)
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
644 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
645
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
646 //</editor-fold>
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
647
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
648 //<editor-fold desc="IssueComment">
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
649
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
650 private fun ResultSet.extractIssueComment() =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
651 IssueComment(getInt("commentid"), getInt("issueid")).apply {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
652 created = getTimestamp("created")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
653 updated = getTimestamp("updated")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
654 updateCount = getInt("updatecount")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
655 comment = getString("comment")
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
656 author = extractOptionalUser()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
657 }
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
658
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
659 override fun listComments(issue: Issue): List<IssueComment> =
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
660 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
661 setInt(1, issue.id)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
662 queryAll { it.extractIssueComment() }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
663 }
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
664
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
665 override fun insertComment(issueComment: IssueComment) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
666 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
667 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
668 with(issueComment) {
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
669 updateIssueDate.setInt(1, issueid)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
670 setInt(1, issueid)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
671 setStringSafe(2, comment)
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
672 setIntOrNull(3, author?.id)
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 executeUpdate()
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
675 updateIssueDate.executeUpdate()
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
676 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
677 }
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
678 }
189
f7de8158b41c fixes #137 - leaking prepared statements
Mike Becker <universe@uap-core.de>
parents: 188
diff changeset
679 //</editor-fold>
167
3f30adba1c63 major refactoring of DAO architecture - also fixes #114
Mike Becker <universe@uap-core.de>
parents:
diff changeset
680 }

mercurial