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

Thu, 19 Aug 2021 14:51:04 +0200

author
Mike Becker <universe@uap-core.de>
date
Thu, 19 Aug 2021 14:51:04 +0200
changeset 231
dcb1d5a7ea3a
parent 227
f0ede8046b59
child 232
296e12ff8d1c
permissions
-rw-r--r--

#163 removes multi selection for versions

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

mercurial