From b62836ad7ea74b6ad0e2c9946e2d9305c66dba79 Mon Sep 17 00:00:00 2001
From: Dimitri Huisman <diman@huisman.xyz>
Date: Wed, 1 Dec 2021 20:46:10 +0000
Subject: [PATCH] Process code review comments (PR #2069).

---
 setup/static/render.js | 36 +++++++++---------------------------
 1 file changed, 9 insertions(+), 27 deletions(-)

diff --git a/setup/static/render.js b/setup/static/render.js
index 7e36de0f..84ac5cc2 100644
--- a/setup/static/render.js
+++ b/setup/static/render.js
@@ -36,7 +36,7 @@ $(document).ready(function() {
 $(document).ready(function() {
 	if ($("#database").val() == 'sqlite') {
 		$("#external_db").hide();
-	} else if ($("#database").val() != 'sqlite') {
+	} else {
 		$("#external_db").show();
 	}
 
@@ -45,47 +45,29 @@ $(document).ready(function() {
 			$("#db_flavor_rc_sel").show();
 		} else {
 			$("#db_flavor_rc_sel").hide();
-			$("#roundcube_db_user").prop('required',false);
-			$("#roundcube_db_pw").prop('required',false);
-			$("#roundcube_db_url").prop('required',false);
-			$("#roundcube_db_name").prop('required',false);
+			$("#roundcube_db_user,#roundcube_db_pw,#roundcube_db_url,#roundcube_db_name").prop('required',false);
 		}
 	});
 
 	$("#database").click(function() {
 		if (this.value == 'sqlite') {
 			$("#external_db").hide();
-			$("#db_user").prop('required',false);
-			$("#db_pw").prop('required',false);
-			$("#db_url").prop('required',false);
-			$("#db_name").prop('required',false);
-			$("#roundcube_db_user").prop('required',false);
-			$("#roundcube_db_pw").prop('required',false);
-			$("#roundcube_db_url").prop('required',false);
-			$("#roundcube_db_name").prop('required',false);
-		} else if (this.value != 'sqlite') {
+			$("#db_user,#db_pw,#db_url,#db_name").prop('required',false);
+			$("#roundcube_db_user,#roundcube_db_pw,#roundcube_db_url,#roundcube_db_name").prop('required',false);
+		} else {
 			$("#external_db").show();
-			$("#db_user").prop('required',true);
-			$("#db_pw").prop('required',true);
-			$("#db_url").prop('required',true);
-			$("#db_name").prop('required',true);
+			$("#db_user,#db_pw,#db_url,#db_name").prop('required',true);
 		}
 	});
 
 	$("#database_rc").click(function() {
 		if (this.value == 'sqlite'){
 			$("#roundcube_external_db").hide();
-			$("#roundcube_db_user").prop('required',false);
-			$("#roundcube_db_pw").prop('required',false);
-			$("#roundcube_db_url").prop('required',false);
-			$("#roundcube_db_name").prop('required',false);
+			$("#roundcube_db_user,#roundcube_db_pw,#roundcube_db_url,#roundcube_db_name").prop('required',false);
 		}
-		if (this.value != 'sqlite' && $("#webmail").val() == 'roundcube') {
+		else if ($("#webmail").val() == 'roundcube') {
 			$("#roundcube_external_db").show();
-			$("#roundcube_db_user").prop('required',true);
-			$("#roundcube_db_pw").prop('required',true);
-			$("#roundcube_db_url").prop('required',true);
-			$("#roundcube_db_name").prop('required',true);
+			$("#roundcube_db_user,#roundcube_db_pw,#roundcube_db_url,#roundcube_db_name").prop('required',true);
 		}
 	});
 });