Fix #1388

bureau/admin/dom_edit.inc.php

@@ -126,6 +126,17 @@ $dom->unlock();
 function check_type_selected() {
   if ( $('input[name=type]:radio:checked').val() ) {
     // there is a value
+    var ll = $('input[name=type]:radio:checked').val();
+    var tt = $('#t_'+ll);
+    if ( tt.length == 0 ) {
+      // this element do not exist, so OK
+      return true;
+    }
+    if ( tt.val() == '' ) {
+      alert("<?php __("Missing value for this sub-domain"); ?>");
+      return false;
+    }
+  
     return true;
   }
   alert("<?php __("Please select a type for this sub-domain"); ?>");

bureau/admin/dom_edit.php

@@ -72,16 +72,19 @@ function dnsoff() {
 }
 
 function destruction_alert() {
-	// On ne se pose pas de question si le DNS est deja sur NON
-	if (<?php echo (int)$r["dns"]; ?>!=1) {
-		return true;
-	}
-	if (document.forms["fdns"].email[1].checked) {
-		if ( confirm('<?php __("Are you sure you want to do this? This will DELETE ALL the mailboxes, messages and aliases on this domain ?"); ?>') ) {
-		  return false;
-	} else {
-		  return true;
-	}
+  // On ne se pose pas de question si le DNS est deja sur NON
+  if (<?php echo (int)$r["dns"]; ?>!=1) {
+    return true;
+  }
+  if (document.forms["fdns"].email[1].checked) {
+    if ( confirm("<?php __("Are you sure you want to do this? This will DELETE ALL the mailboxes, messages and aliases on this domain ?"); ?>") ) {
+      return true;
+    } else {
+      return false;
+    }
+  } else {
+      return true;
+  }
 }
 </script>
 <h3><?php printf(_("Editing subdomains of %s"),$domain); ?></h3>

bureau/admin/dom_editdns.php

@@ -58,6 +58,6 @@ $dom->unlock();
 // XXX: we assume the cron job is at every 5 minutes
   print strtr(_("The modifications will take effect at %time.  Server time is %now."), array('%now' => date('H:i:s', $t), '%time' => date('H:i:s', ($t-($t%300)+300)))); 
 ?><br /><br />
-<span class="ina"><a href="login.php" target="_top"><?php __("Click here to continue"); ?></a></span>
+<span class="ina"><a href="dom_edit.php?domain=<?php echo urlencode($domain) ?>" ><?php __("Click here to continue"); ?></a></span>
 </p>
 <?php include_once("foot.php"); ?>