From cbdc7acabd3c61c3e8da7a3b04421f0d061fc5fa Mon Sep 17 00:00:00 2001 From: Marius Cramer <m.cramer@pixcept.de> Date: Tue, 08 Jul 2014 10:05:41 -0400 Subject: [PATCH] Merge branch 'hide_used_domains' into 'master' --- interface/web/dns/templates/dns_soa_edit.htm | 8 + interface/web/sites/web_vhost_domain_edit.php | 2 interface/web/dns/templates/dns_import.htm | 9 ++ interface/web/dns/templates/dns_wizard.htm | 8 + interface/lib/classes/tools_sites.inc.php | 14 +++ interface/web/dns/templates/dns_slave_edit.htm | 8 + interface/web/mail/mail_domain_edit.php | 2 interface/web/dns/dns_slave_edit.php | 46 +++++++++++ interface/web/sites/web_childdomain_edit.php | 2 interface/web/dns/dns_import.php | 38 +++++++++ interface/web/dns/lib/lang/es_dns_import.lng | 2 interface/web/dns/dns_wizard.php | 48 +++++++++++ interface/web/dns/dns_soa_edit.php | 46 +++++++++++ 13 files changed, 221 insertions(+), 12 deletions(-) diff --git a/interface/lib/classes/tools_sites.inc.php b/interface/lib/classes/tools_sites.inc.php index c636bbc..690a4f0 100644 --- a/interface/lib/classes/tools_sites.inc.php +++ b/interface/lib/classes/tools_sites.inc.php @@ -144,10 +144,20 @@ return $res; } - function getDomainModuleDomains() { + function getDomainModuleDomains($not_used_in_table = null, $selected_domain = null) { global $app; $sql = "SELECT domain_id, domain FROM domain WHERE"; + if ($not_used_in_table) { + if (strpos($not_used_in_table, 'dns') !== false) { + $field = "origin"; + $select = "SUBSTRING($field, 1, CHAR_LENGTH($field) - 1)"; + } else { + $field = "domain"; + $select = $field; + } + $sql .= " domain NOT IN (SELECT $select FROM ?? WHERE $field != ?) AND"; + } if ($_SESSION["s"]["user"]["typ"] == 'admin') { $sql .= " 1"; } else { @@ -155,7 +165,7 @@ $sql .= " sys_groupid IN (".$groups.")"; } $sql .= " ORDER BY domain"; - return $app->db->queryAllRecords($sql); + return $app->db->queryAllRecords($sql, $not_used_in_table, $selected_domain); } function checkDomainModuleDomain($domain_id) { diff --git a/interface/web/dns/dns_import.php b/interface/web/dns/dns_import.php index c78e41a..a391c2e 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("dns_soa"); + /* + * 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..eb25f22 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("dns_slave", $this->dataRecord["origin"]); + $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..2fcb9f9 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("dns_soa", $this->dataRecord["origin"]); + $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 93996ed..6b8de3e 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("dns_soa"); + $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"> diff --git a/interface/web/mail/mail_domain_edit.php b/interface/web/mail/mail_domain_edit.php index 8220c2c..6f48c7f 100644 --- a/interface/web/mail/mail_domain_edit.php +++ b/interface/web/mail/mail_domain_edit.php @@ -150,7 +150,7 @@ /* * The domain-module is in use. */ - $domains = $app->tools_sites->getDomainModuleDomains(); + $domains = $app->tools_sites->getDomainModuleDomains("mail_domain", $this->dataRecord["domain"]); $domain_select = ''; if(is_array($domains) && sizeof($domains) > 0) { /* We have domains in the list, so create the drop-down-list */ diff --git a/interface/web/sites/web_childdomain_edit.php b/interface/web/sites/web_childdomain_edit.php index 435b901..8cdda48 100644 --- a/interface/web/sites/web_childdomain_edit.php +++ b/interface/web/sites/web_childdomain_edit.php @@ -105,7 +105,7 @@ /* * The domain-module is in use. */ - $domains = $app->tools_sites->getDomainModuleDomains(); + $domains = $app->tools_sites->getDomainModuleDomains($this->_vhostdomain_type == 'subdomain' ? null : "web_domain", $this->dataRecord["domain"]); $domain_select = ''; $selected_domain = ''; if(is_array($domains) && sizeof($domains) > 0) { diff --git a/interface/web/sites/web_vhost_domain_edit.php b/interface/web/sites/web_vhost_domain_edit.php index de731bd..00b4985 100644 --- a/interface/web/sites/web_vhost_domain_edit.php +++ b/interface/web/sites/web_vhost_domain_edit.php @@ -631,7 +631,7 @@ /* * The domain-module is in use. */ - $domains = $app->tools_sites->getDomainModuleDomains(); + $domains = $app->tools_sites->getDomainModuleDomains($this->_vhostdomain_type == 'subdomain' ? null : "web_domain", $this->dataRecord["domain"]); $domain_select = ''; $selected_domain = ''; if(is_array($domains) && sizeof($domains) > 0) { -- Gitblit v1.9.1