fix(model-server): @RequiresTransaction annotation to help avoiding MissingTransactionException

slisson · slisson · commit cfebf704f5ce · 2024-12-12T16:32:49.000+01:00
diff --git a/model-server/src/main/kotlin/org/modelix/model/server/handlers/ui/DiffView.kt b/model-server/src/main/kotlin/org/modelix/model/server/handlers/ui/DiffView.kt
@@ -72,7 +72,7 @@ class DiffView(private val repositoryManager: RepositoriesManager) {
             requiresLogin {
                 get("/diff") {
                     @OptIn(RequiresTransaction::class)
-                    call.respondHtmlTemplate(PageWithMenuBar("diff", "..")) {
+                    call.respondHtmlTemplateInTransaction(repositoryManager.getTransactionManager(), PageWithMenuBar("diff", "..")) {
                         val visibleRepositories = repositoryManager.getRepositories().filter {
                             call.hasPermission(ModelServerPermissionSchema.repository(it).list)
                         }

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ class DiffView(private val repositoryManager: RepositoriesManager) {`
`72`	`72`	`requiresLogin {`
`73`	`73`	`get("/diff") {`
`74`	`74`	`@OptIn(RequiresTransaction::class)`
`75`		`- call.respondHtmlTemplate(PageWithMenuBar("diff", "..")) {`
	`75`	`+ call.respondHtmlTemplateInTransaction(repositoryManager.getTransactionManager(), PageWithMenuBar("diff", "..")) {`
`76`	`76`	`val visibleRepositories = repositoryManager.getRepositories().filter {`
`77`	`77`	`call.hasPermission(ModelServerPermissionSchema.repository(it).list)`
`78`	`78`	`}`