From 021ef646e889702ea62e3dd5030a47c703bfed1d Mon Sep 17 00:00:00 2001
From: Sergio Cambra <sergio@programatica.es>
Date: Tue, 08 Jul 2014 04:02:20 -0400
Subject: [PATCH] use domain option on dns forms, fixes FS#3028

---
 interface/web/dns/templates/dns_soa_edit.htm   |    8 +
 interface/web/dns/templates/dns_import.htm     |    9 ++
 interface/web/dns/templates/dns_wizard.htm     |    8 +
 interface/web/dns/lib/lang/es_dns_import.lng   |    2 
 interface/web/dns/templates/dns_slave_edit.htm |    8 +
 interface/web/dns/dns_wizard.php               |   48 +++++++++++
 interface/web/dns/dns_slave_edit.php           |   46 +++++++++++
 interface/web/dns/dns_soa_edit.php             |   46 +++++++++++
 interface/web/dns/dns_import.php               |   38 +++++++++
 9 files changed, 206 insertions(+), 7 deletions(-)

diff --git a/interface/web/dns/dns_import.php b/interface/web/dns/dns_import.php
index c78e41a..b213110 100644
--- a/interface/web/dns/dns_import.php
+++ b/interface/web/dns/dns_import.php
@@ -151,6 +151,44 @@
 
 }
 
+/*
+ * Now we have to check, if we should use the domain-module to select the domain
+ * or not
+ */
+$app->uses('ini_parser,getconf');
+$settings = $app->getconf->get_global_config('domains');
+if ($settings['use_domain_module'] == 'y') {
+	/*
+	 * The domain-module is in use.
+	*/
+	$domains = $app->tools_sites->getDomainModuleDomains();
+	/*
+	 * We can leave domain empty if domain is filename
+	*/
+	$domain_select = "<option value=''></option>\r\n";
+	if(is_array($domains) && sizeof($domains) > 0) {
+		/* We have domains in the list, so create the drop-down-list */
+		foreach( $domains as $domain) {
+			$domain_select .= "<option value=" . $domain['domain_id'] ;
+			if ($domain['domain'] == $_POST['domain']) {
+				$domain_select .= " selected";
+			}
+			$domain_select .= ">" . $app->functions->idn_decode($domain['domain']) . ".</option>\r\n";
+		}
+	}
+	$app->tpl->setVar("domain_option", $domain_select);
+	/* check if the selected domain can be used! */
+	if ($domain) {
+		$domain_check = $app->tools_sites->checkDomainModuleDomain($domain);
+		if(!$domain_check) {
+			// invalid domain selected
+			$domain = NULL;
+		} else {
+			$domain = $domain_check;
+		}
+	}
+}
+
 $lng_file = 'lib/lang/'.$_SESSION['s']['language'].'_dns_import.lng';
 include $lng_file;
 $app->tpl->setVar($wb);
diff --git a/interface/web/dns/dns_slave_edit.php b/interface/web/dns/dns_slave_edit.php
index dfca92e..ff06821 100644
--- a/interface/web/dns/dns_slave_edit.php
+++ b/interface/web/dns/dns_slave_edit.php
@@ -106,6 +106,39 @@
 
 			}
 
+		/*
+		 * Now we have to check, if we should use the domain-module to select the domain
+		 * or not
+		 */
+		$app->uses('ini_parser,getconf');
+		$settings = $app->getconf->get_global_config('domains');
+		if ($settings['use_domain_module'] == 'y') {
+			/*
+			 * The domain-module is in use.
+			*/
+			$domains = $app->tools_sites->getDomainModuleDomains();
+			$domain_select = '';
+			if(is_array($domains) && sizeof($domains) > 0) {
+				/* We have domains in the list, so create the drop-down-list */
+				foreach( $domains as $domain) {
+					$domain_select .= "<option value=" . $domain['domain_id'] ;
+					if ($domain['domain'].'.' == $this->dataRecord["origin"]) {
+						$domain_select .= " selected";
+					}
+					$domain_select .= ">" . $app->functions->idn_decode($domain['domain']) . ".</option>\r\n";
+				}
+			}
+			else {
+				/*
+				 * We have no domains in the domain-list. This means, we can not add ANY new domain.
+				 * To avoid, that the variable "domain_option" is empty and so the user can
+				 * free enter a domain, we have to create a empty option!
+				*/
+				$domain_select .= "<option value=''></option>\r\n";
+			}
+			$app->tpl->setVar("domain_option", $domain_select);
+		}
+
 		if($this->id > 0) {
 			//* we are editing a existing record
 			$app->tpl->setVar("edit_disabled", 1);
@@ -120,6 +153,19 @@
 	function onSubmit() {
 		global $app, $conf;
 
+		/* check if the domain module is used - and check if the selected domain can be used! */
+		$app->uses('ini_parser,getconf');
+		$settings = $app->getconf->get_global_config('domains');
+		if ($settings['use_domain_module'] == 'y') {
+			$domain_check = $app->tools_sites->checkDomainModuleDomain($this->dataRecord['origin']);
+			if(!$domain_check) {
+				// invalid domain selected
+				$app->tform->errorMessage .= $app->tform->lng("origin_error_empty")."<br />";
+			} else {
+				$this->dataRecord['origin'] = $domain_check.'.';
+			}
+		}
+
 		if($_SESSION["s"]["user"]["typ"] != 'admin') {
 			// Get the limits of the client
 			$client_group_id = intval($_SESSION["s"]["user"]["default_group"]);
diff --git a/interface/web/dns/dns_soa_edit.php b/interface/web/dns/dns_soa_edit.php
index 17c704e..3352885 100644
--- a/interface/web/dns/dns_soa_edit.php
+++ b/interface/web/dns/dns_soa_edit.php
@@ -147,6 +147,39 @@
 
 	}
 
+	/*
+	 * Now we have to check, if we should use the domain-module to select the domain
+	 * or not
+	 */
+	$app->uses('ini_parser,getconf');
+	$settings = $app->getconf->get_global_config('domains');
+	if ($settings['use_domain_module'] == 'y') {
+		/*
+		 * The domain-module is in use.
+		*/
+		$domains = $app->tools_sites->getDomainModuleDomains();
+		$domain_select = '';
+		if(is_array($domains) && sizeof($domains) > 0) {
+			/* We have domains in the list, so create the drop-down-list */
+			foreach( $domains as $domain) {
+				$domain_select .= "<option value=" . $domain['domain_id'] ;
+				if ($domain['domain'].'.' == $this->dataRecord["origin"]) {
+					$domain_select .= " selected";
+				}
+				$domain_select .= ">" . $app->functions->idn_decode($domain['domain']) . ".</option>\r\n";
+			}
+		}
+		else {
+			/*
+			 * We have no domains in the domain-list. This means, we can not add ANY new domain.
+			 * To avoid, that the variable "domain_option" is empty and so the user can
+			 * free enter a domain, we have to create a empty option!
+			*/
+			$domain_select .= "<option value=''></option>\r\n";
+		}
+		$app->tpl->setVar("domain_option", $domain_select);
+	}
+
 	if($this->id > 0) {
 		//* we are editing a existing record
 		$app->tpl->setVar("edit_disabled", 1);
@@ -161,6 +194,19 @@
 function onSubmit() {
 	global $app, $conf;
 
+	/* check if the domain module is used - and check if the selected domain can be used! */
+	$app->uses('ini_parser,getconf');
+	$settings = $app->getconf->get_global_config('domains');
+	if ($settings['use_domain_module'] == 'y') {
+		$domain_check = $app->tools_sites->checkDomainModuleDomain($this->dataRecord['origin']);
+		if(!$domain_check) {
+			// invalid domain selected
+			$app->tform->errorMessage .= $app->tform->lng("origin_error_empty")."<br />";
+		} else {
+			$this->dataRecord['origin'] = $domain_check.'.';
+		}
+	}
+
 	if($_SESSION["s"]["user"]["typ"] != 'admin') {
 		// Get the limits of the client
 		$client_group_id = $_SESSION["s"]["user"]["default_group"];
diff --git a/interface/web/dns/dns_wizard.php b/interface/web/dns/dns_wizard.php
index 370d3df..a462d0f 100644
--- a/interface/web/dns/dns_wizard.php
+++ b/interface/web/dns/dns_wizard.php
@@ -157,6 +157,39 @@
 	}
 }
 
+/*
+ * Now we have to check, if we should use the domain-module to select the domain
+ * or not
+ */
+$app->uses('ini_parser,getconf');
+$settings = $app->getconf->get_global_config('domains');
+if ($settings['use_domain_module'] == 'y') {
+	/*
+	 * The domain-module is in use.
+	*/
+	$domains = $app->tools_sites->getDomainModuleDomains();
+	$domain_select = '';
+	if(is_array($domains) && sizeof($domains) > 0) {
+		/* We have domains in the list, so create the drop-down-list */
+		foreach( $domains as $domain) {
+			$domain_select .= "<option value=" . $domain['domain_id'] ;
+			if ($domain['domain'] == $_POST['domain']) {
+				$domain_select .= " selected";
+			}
+			$domain_select .= ">" . $app->functions->idn_decode($domain['domain']) . ".</option>\r\n";
+		}
+	}
+	else {
+		/*
+		 * We have no domains in the domain-list. This means, we can not add ANY new domain.
+		 * To avoid, that the variable "domain_option" is empty and so the user can
+		 * free enter a domain, we have to create a empty option!
+		*/
+		$domain_select .= "<option value=''></option>\r\n";
+	}
+	$app->tpl->setVar("domain_option", $domain_select);
+}
+
 if($_POST['create'] == 1) {
 
 	$error = '';
@@ -180,8 +213,19 @@
 
 	// apply filters
 	if(isset($_POST['domain']) && $_POST['domain'] != ''){
-		$_POST['domain'] = $app->functions->idn_encode($_POST['domain']);
-		$_POST['domain'] = strtolower($_POST['domain']);
+		/* check if the domain module is used - and check if the selected domain can be used! */
+		if ($settings['use_domain_module'] == 'y') {
+			$domain_check = $app->tools_sites->checkDomainModuleDomain($_POST['domain']);
+			if(!$domain_check) {
+				// invalid domain selected
+				$_POST['domain'] = '';
+			} else {
+				$_POST['domain'] = $domain_check;
+			}
+		} else {
+			$_POST['domain'] = $app->functions->idn_encode($_POST['domain']);
+			$_POST['domain'] = strtolower($_POST['domain']);
+		}
 	}
 	if(isset($_POST['ns1']) && $_POST['ns1'] != ''){
 		$_POST['ns1'] = $app->functions->idn_encode($_POST['ns1']);
diff --git a/interface/web/dns/lib/lang/es_dns_import.lng b/interface/web/dns/lib/lang/es_dns_import.lng
index 3f91027..7223f5c 100644
--- a/interface/web/dns/lib/lang/es_dns_import.lng
+++ b/interface/web/dns/lib/lang/es_dns_import.lng
@@ -9,6 +9,6 @@
 $wb['zonefile_to_import_txt'] = 'Archivo de zona';
 $wb['domain_field_desc_txt'] = 'Se puede dejar vacío si el nombre de dominio es el nombre del archivo o el contenido de la zona de archivo.';
 $wb['title'] = 'Archivos de zona de importación';
-$wb['no_file_uploaded_error'] = 'No hay fichero en zona subida';
+$wb['no_file_uploaded_error'] = 'No se ha subido un fichero de zona';
 $wb['zone_file_import_txt'] = 'Importar Archivo de Zona';
 ?>
diff --git a/interface/web/dns/templates/dns_import.htm b/interface/web/dns/templates/dns_import.htm
index 98049a4..d4806e1 100644
--- a/interface/web/dns/templates/dns_import.htm
+++ b/interface/web/dns/templates/dns_import.htm
@@ -32,7 +32,14 @@
             </tmpl_if>
             <div class="ctrlHolder">
                 <label for="domain">{tmpl_var name='domain_txt'}</label>
-                <input name="domain" id="domain" value="{tmpl_var name='domain'}" size="30" maxlength="255" type="text" class="textInput" onkeydown="keydown(event.which);" /><p class="value">{tmpl_var name='domain_field_desc_txt'}</p>
+                <tmpl_if name="domain_option">
+                    <select name="domain" id="domain" class="selectInput">
+                        {tmpl_var name='domain_option'}
+                    </select>
+                    <tmpl_else>
+                        <input name="domain" id="domain" value="{tmpl_var name='domain'}" size="30" maxlength="255" type="text" class="textInput" onkeydown="keydown(event.which);" />
+                </tmpl_if>
+                <p class="value"><tmpl_var name='domain_field_desc_txt'}</p>
             </div>    
             <div class="ctrlHolder">
                 <label for="file">{tmpl_var name='zonefile_to_import_txt'}</label>
diff --git a/interface/web/dns/templates/dns_slave_edit.htm b/interface/web/dns/templates/dns_slave_edit.htm
index 8aa9b03..7df6cc7 100644
--- a/interface/web/dns/templates/dns_slave_edit.htm
+++ b/interface/web/dns/templates/dns_slave_edit.htm
@@ -37,7 +37,13 @@
             </tmpl_if>
             <div class="ctrlHolder">
                 <label for="origin">{tmpl_var name='origin_txt'}</label>
-                <input name="origin" id="origin" value="{tmpl_var name='origin'}" size="30" maxlength="255" type="text" class="textInput" />
+                <tmpl_if name="domain_option">
+                    <select name="origin" id="origin" class="selectInput">
+                        {tmpl_var name='domain_option'}
+                    </select>
+                    <tmpl_else>
+                        <input name="origin" id="origin" value="{tmpl_var name='origin'}" size="30" maxlength="255" type="text" class="textInput" />
+                </tmpl_if>
                 <p class="formHint">{tmpl_var name='eg_domain_tld'}</p>
             </div>
             <div class="ctrlHolder">
diff --git a/interface/web/dns/templates/dns_soa_edit.htm b/interface/web/dns/templates/dns_soa_edit.htm
index 27bb761..ee42c34 100644
--- a/interface/web/dns/templates/dns_soa_edit.htm
+++ b/interface/web/dns/templates/dns_soa_edit.htm
@@ -56,7 +56,13 @@
             </tmpl_if>
             <div class="ctrlHolder">
                 <label for="origin">{tmpl_var name='origin_txt'}</label>
-                <input name="origin" id="origin" value="{tmpl_var name='origin'}" size="30" maxlength="255" type="text" class="textInput" />
+                <tmpl_if name="domain_option">
+                    <select name="origin" id="origin" class="selectInput">
+                        {tmpl_var name='domain_option'}
+                    </select>
+                    <tmpl_else>
+                        <input name="origin" id="origin" value="{tmpl_var name='origin'}" size="30" maxlength="255" type="text" class="textInput" />
+                </tmpl_if>
                 <p class="formHint">{tmpl_var name='eg_domain_tld'}</p>
             </div>
             <div class="ctrlHolder">
diff --git a/interface/web/dns/templates/dns_wizard.htm b/interface/web/dns/templates/dns_wizard.htm
index c6cddb0..94f6a46 100644
--- a/interface/web/dns/templates/dns_wizard.htm
+++ b/interface/web/dns/templates/dns_wizard.htm
@@ -51,7 +51,13 @@
             <tmpl_if name="DOMAIN_VISIBLE">
                 <div class="ctrlHolder">
                     <label for="domain">{tmpl_var name='domain_txt'}</label>
-                    <input name="domain" id="domain" value="{tmpl_var name='domain'}" size="30" maxlength="255" type="text" class="textInput" />
+                    <tmpl_if name="domain_option">
+                        <select name="domain" id="domain" class="selectInput">
+                            {tmpl_var name='domain_option'}
+                        </select>
+                        <tmpl_else>
+                            <input name="domain" id="domain" value="{tmpl_var name='domain'}" size="30" maxlength="255" type="text" class="textInput" />
+                    </tmpl_if>
                 </div>
             </tmpl_if>
             <tmpl_if name="IP_VISIBLE">

--
Gitblit v1.9.1