Process code review comments (PR #2069).

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);
 		}
 	});
 });