From ffc0bcc3a00ff007e5ede2804ff33e48ee5eb24b Mon Sep 17 00:00:00 2001
From: zzz <zzz@mail.i2p>
Date: Mon, 12 Mar 2018 11:42:52 +0000
Subject: [PATCH] Susimail: Fix up compose.js (ticket #2176) Enable for
 configuration page Version the js

---
 apps/susimail/src/js/compose.js                |  7 ++-----
 .../src/src/i2p/susi/webmail/WebMail.java      | 18 ++++++------------
 2 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/apps/susimail/src/js/compose.js b/apps/susimail/src/js/compose.js
index 1f897526d..77b101bb2 100644
--- a/apps/susimail/src/js/compose.js
+++ b/apps/susimail/src/js/compose.js
@@ -1,5 +1,2 @@
-// onbeforeunload() is in the servlet because it has a translated string
-
-function cancelPopup() {
-	window.onbeforeunload = null;
-}
+let beforePopup = true;
+window.addEventListener('beforeunload', (e)=>{if (beforePopup) e.returnValue=true;} );
diff --git a/apps/susimail/src/src/i2p/susi/webmail/WebMail.java b/apps/susimail/src/src/i2p/susi/webmail/WebMail.java
index 499074d8a..fbd3a7a49 100644
--- a/apps/susimail/src/src/i2p/susi/webmail/WebMail.java
+++ b/apps/susimail/src/src/i2p/susi/webmail/WebMail.java
@@ -524,8 +524,9 @@ public class WebMail extends HttpServlet
 		StringBuilder buf = new StringBuilder(128);
 		buf.append("<input type=\"submit\" class=\"").append(name).append("\" name=\"")
 		   .append(name).append("\" value=\"").append(label).append('"');
-		if (name.equals(SEND) || name.equals(CANCEL) || name.equals(DELETE_ATTACHMENT) || name.equals(NEW_UPLOAD))
-			buf.append(" onclick=\"cancelPopup()\"");
+		if (name.equals(SEND) || name.equals(CANCEL) || name.equals(DELETE_ATTACHMENT) || name.equals(NEW_UPLOAD) ||
+		    name.equals(SETPAGESIZE) || name.equals(SAVE))  // config page
+			buf.append(" onclick=\"beforePopup=false;\"");
 		// These are icons only now, via the CSS, so add a tooltip
 		if (name.equals(FIRSTPAGE) || name.equals(PREVPAGE) || name.equals(NEXTPAGE) || name.equals(LASTPAGE) ||
 		    name.equals(PREV) || name.equals(LIST) || name.equals(NEXT))
@@ -2166,18 +2167,11 @@ public class WebMail extends HttpServlet
 				}
 				if(state != State.AUTH)
 					out.println("<link rel=\"stylesheet\" href=\"/susimail/css/print.css?" + CoreVersion.VERSION + "\" type=\"text/css\" media=\"print\" />");
-				if (state == State.NEW) {
+				if (state == State.NEW || state == State.CONFIG) {
 					// TODO cancel if to and body are empty
-					out.println(
-						"<script type=\"text/javascript\">\n" +
-							"window.onbeforeunload = function () {" +
-								"return \"" + _t("Message has not been sent. Do you want to discard it?") + "\";" +
-							"};\n" +
-						"</script>"
-					);
-					out.println("<script src=\"/susimail/js/compose.js\" type=\"text/javascript\"></script>");
+					out.println("<script src=\"/susimail/js/compose.js?" + CoreVersion.VERSION + "\" type=\"text/javascript\"></script>");
 				} else if (state == State.LIST) {
-					out.println("<script src=\"/susimail/js/folder.js\" type=\"text/javascript\"></script>");
+					out.println("<script src=\"/susimail/js/folder.js?" + CoreVersion.VERSION + "\" type=\"text/javascript\"></script>");
 				} else if (state == State.LOADING) {
 					// TODO JS?
 			                out.println("<meta http-equiv=\"refresh\" content=\"5;url=" + myself + "\">");