Actually differentiate between GET and POST parameters

[Sone.git] / src / main / kotlin / net / pterodactylus / sone / web / pages / DeleteReplyPage.kt

diff --git a/src/main/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPage.kt b/src/main/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPage.kt
index f9c51a3..0b922b0 100644 (file)
--- a/src/main/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPage.kt
+++ b/src/main/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPage.kt
@@ -14,15 +14,13 @@ class DeleteReplyPage(template: Template, webInterface: WebInterface):
                SoneTemplatePage("deleteReply.html", template, "Page.DeleteReply.Title", webInterface, true) {
 
        override fun handleRequest(request: FreenetRequest, templateContext: TemplateContext) {
-               val replyId = request.httpRequest.getPartAsStringFailsafe("reply", 36)
-               templateContext["reply"] = replyId
-               val returnPage = request.httpRequest.getPartAsStringFailsafe("returnPage", 256)
-               templateContext["returnPage"] = returnPage
                if (request.isPOST) {
+                       val replyId = request.httpRequest.getPartAsStringFailsafe("reply", 36)
                        val reply = webInterface.core.getPostReply(replyId).orNull() ?: throw RedirectException("noPermission.html")
                        if (!reply.sone.isLocal) {
                                throw RedirectException("noPermission.html")
                        }
+                       val returnPage = request.httpRequest.getPartAsStringFailsafe("returnPage", 256)
                        if (request.httpRequest.isPartSet("confirmDelete")) {
                                webInterface.core.deleteReply(reply)
                                throw RedirectException(returnPage)
@@ -30,7 +28,12 @@ class DeleteReplyPage(template: Template, webInterface: WebInterface):
                        if (request.httpRequest.isPartSet("abortDelete")) {
                                throw RedirectException(returnPage)
                        }
+                       templateContext["reply"] = replyId
+                       templateContext["returnPage"] = returnPage
+                       return
                }
+               templateContext["reply"] = request.httpRequest.getParam("reply")
+               templateContext["returnPage"] = request.httpRequest.getParam("returnPage")
        }
 
 }