- don't set password via remoting if field is empty

Marius Cramer

2015-08-06 37b29231e47a0c4458dc1c15d98588f16f07e1e2

 interface/web/dns/dns_soa_edit.php

@@ -71,6 +71,9 @@
         if(!$app->tform->checkResellerLimit('limit_dns_zone')) {
            $app->error('Reseller: '.$app->tform->wordbook["limit_dns_zone_txt"]);
         }
      } else {
         $settings = $app->getconf->get_global_config('dns');
         $app->tform->formDef['tabs']['dns_soa']['fields']['server_id']['default'] = intval($settings['default_dnsserver']);
      }

      parent::onShowNew();
@@ -79,31 +82,39 @@
   function onShowEnd() {
      global $app, $conf;

      // If user is admin, we will allow him to select to whom this record belongs
      if($_SESSION["s"]["user"]["typ"] == 'admin') {
         // Getting Domains of the user
         $sql = "SELECT sys_group.groupid, sys_group.name, CONCAT(IF(client.company_name != '', CONCAT(client.company_name, ' :: '), ''), client.contact_name, ' (', client.username, IF(client.customer_no != '', CONCAT(', ', client.customer_no), ''), ')') as contactname FROM sys_group, client WHERE sys_group.client_id = client.client_id AND sys_group.client_id > 0 ORDER BY sys_group.name";
         $clients = $app->db->queryAllRecords($sql);
         $client_select = '';
         if($_SESSION["s"]["user"]["typ"] == 'admin') $client_select .= "<option value='0'></option>";
         //$tmp_data_record = $app->tform->getDataRecord($this->id);
         if(is_array($clients)) {
            foreach( $clients as $client) {
               $selected = @(is_array($this->dataRecord) && ($client["groupid"] == $this->dataRecord['client_group_id'] || $client["groupid"] == $this->dataRecord['sys_groupid']))?'SELECTED':'';
               $client_select .= "<option value='$client[groupid]' $selected>$client[contactname]</option>\r\n";
      $app->uses('ini_parser,getconf');
      $settings = $app->getconf->get_global_config('domains');

      /*
       * Now we have to check, if we should use the domain-module to select the domain
       * or not
       */
      if ($settings['use_domain_module'] != 'y') {
         // If user is admin, we will allow him to select to whom this record belongs
         if($_SESSION["s"]["user"]["typ"] == 'admin') {
            // Getting Domains of the user
            $sql = "SELECT sys_group.groupid, sys_group.name, CONCAT(IF(client.company_name != '', CONCAT(client.company_name, ' :: '), ''), client.contact_name, ' (', client.username, IF(client.customer_no != '', CONCAT(', ', client.customer_no), ''), ')') as contactname FROM sys_group, client WHERE sys_group.client_id = client.client_id AND sys_group.client_id > 0 ORDER BY client.company_name, client.contact_name, sys_group.name";
            $clients = $app->db->queryAllRecords($sql);
            $client_select = '';
            if($_SESSION["s"]["user"]["typ"] == 'admin') $client_select .= "<option value='0'></option>";
            //$tmp_data_record = $app->tform->getDataRecord($this->id);
            if(is_array($clients)) {
               foreach( $clients as $client) {
                  $selected = @(is_array($this->dataRecord) && ($client["groupid"] == $this->dataRecord['client_group_id'] || $client["groupid"] == $this->dataRecord['sys_groupid']))?'SELECTED':'';
                  $client_select .= "<option value='$client[groupid]' $selected>$client[contactname]</option>\r\n";
               }
            }
         }
         $app->tpl->setVar("client_group_id", $client_select);
      } else if($app->auth->has_clients($_SESSION['s']['user']['userid'])) {
            $app->tpl->setVar("client_group_id", $client_select);
         } else if($app->auth->has_clients($_SESSION['s']['user']['userid'])) {

            // Get the limits of the client
            $client_group_id = intval($_SESSION["s"]["user"]["default_group"]);
            $client = $app->db->queryOneRecord("SELECT client.client_id, client.contact_name, CONCAT(IF(client.company_name != '', CONCAT(client.company_name, ' :: '), ''), client.contact_name, ' (', client.username, IF(client.customer_no != '', CONCAT(', ', client.customer_no), ''), ')') as contactname, sys_group.name FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id");
            $client = $app->db->queryOneRecord("SELECT client.client_id, client.contact_name, CONCAT(IF(client.company_name != '', CONCAT(client.company_name, ' :: '), ''), client.contact_name, ' (', client.username, IF(client.customer_no != '', CONCAT(', ', client.customer_no), ''), ')') as contactname, sys_group.name FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ?", $client_group_id);

            // Fill the client select field
            $sql = "SELECT sys_group.groupid, sys_group.name, CONCAT(IF(client.company_name != '', CONCAT(client.company_name, ' :: '), ''), client.contact_name, ' (', client.username, IF(client.customer_no != '', CONCAT(', ', client.customer_no), ''), ')') as contactname FROM sys_group, client WHERE sys_group.client_id = client.client_id AND client.parent_client_id = ".$client['client_id']." ORDER BY sys_group.name";
            $clients = $app->db->queryAllRecords($sql);
            $tmp = $app->db->queryOneRecord("SELECT groupid FROM sys_group WHERE client_id = ".$client['client_id']);
            $sql = "SELECT sys_group.groupid, sys_group.name, CONCAT(IF(client.company_name != '', CONCAT(client.company_name, ' :: '), ''), client.contact_name, ' (', client.username, IF(client.customer_no != '', CONCAT(', ', client.customer_no), ''), ')') as contactname FROM sys_group, client WHERE sys_group.client_id = client.client_id AND client.parent_client_id = ? ORDER BY client.company_name, client.contact_name, sys_group.name";
            $clients = $app->db->queryAllRecords($sql, $client['client_id']);
            $tmp = $app->db->queryOneRecord("SELECT groupid FROM sys_group WHERE client_id = ?", $client['client_id']);
            $client_select = '<option value="'.$tmp['groupid'].'">'.$client['contactname'].'</option>';
            //$tmp_data_record = $app->tform->getDataRecord($this->id);
            if(is_array($clients)) {
@@ -115,14 +126,14 @@
            $app->tpl->setVar("client_group_id", $client_select);

         }
      $app->tpl->setVar("client_group_id", $client_select);
      }

//   }

   if($_SESSION["s"]["user"]["typ"] != 'admin')
   {
      $client_group_id = $_SESSION["s"]["user"]["default_group"];
      $client_dns = $app->db->queryOneRecord("SELECT dns_servers FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id");
      $client_dns = $app->db->queryOneRecord("SELECT dns_servers FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ?", $client_group_id);

      $client_dns['dns_servers_ids'] = explode(',', $client_dns['dns_servers']);

@@ -133,18 +144,45 @@
         $app->tpl->setVar('server_id_value', $client_dns['dns_servers_ids'][0]);
      }

      $sql = "SELECT server_id, server_name FROM server WHERE server_id IN (" . $client_dns['dns_servers'] . ");";
      $dns_servers = $app->db->queryAllRecords($sql);
      $sql = "SELECT server_id, server_name FROM server WHERE server_id IN ?";
      $dns_servers = $app->db->queryAllRecords($sql, $client_dns['dns_servers_ids']);

      $options_dns_servers = "";

      foreach ($dns_servers as $dns_server) {
         $options_dns_servers .= "<option value='$dns_server[server_id]'>$dns_server[server_name]</option>";
         $options_dns_servers .= '<option value="'.$dns_server['server_id'].'"'.($this->id > 0 && $this->dataRecord["server_id"] == $dns_server['server_id'] ? ' selected="selected"' : '').'>'.$dns_server['server_name'].'</option>';
      }

      $app->tpl->setVar("client_server_id", $options_dns_servers);
      unset($options_dns_servers);

   }

   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) {
@@ -161,82 +199,73 @@
function onSubmit() {
   global $app, $conf;

   if($_SESSION["s"]["user"]["typ"] != 'admin') {
      // Get the limits of the client
      $client_group_id = $_SESSION["s"]["user"]["default_group"];
      $client = $app->db->queryOneRecord("SELECT limit_dns_zone, dns_servers FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id");

      $client['dns_servers_ids'] = explode(',', $client['dns_servers']);

      // Check if chosen server is in authorized servers for this client
      if (!(is_array($client['dns_servers_ids']) && in_array($this->dataRecord["server_id"], $client['dns_servers_ids'])) && $_SESSION["s"]["user"]["typ"] != 'admin') {
         $app->error($app->tform->wordbook['error_not_allowed_server_id']);
   if ($app->tform->getCurrentTab() == 'dns_soa') {
      /* 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') {
         if ($_SESSION["s"]["user"]["typ"] == 'admin' || $app->auth->has_clients($_SESSION['s']['user']['userid'])) {
            $this->dataRecord['client_group_id'] = $app->tools_sites->getClientIdForDomain($this->dataRecord['origin']);
         }
         $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.'.';
         }
      }

      // When the record is updated
      if($this->id > 0) {
         // restore the server ID if the user is not admin and record is edited
         $tmp = $app->db->queryOneRecord("SELECT server_id FROM dns_soa WHERE id = ".$app->functions->intval($this->id));
         $this->dataRecord["server_id"] = $tmp["server_id"];
         unset($tmp);
         // When the record is inserted
      } else {
         // Check if the user may add another maildomain.
         if($client["limit_dns_zone"] >= 0) {
            $tmp = $app->db->queryOneRecord("SELECT count(id) as number FROM dns_soa WHERE sys_groupid = $client_group_id");
            if($tmp["number"] >= $client["limit_dns_zone"]) {
               $app->error($app->tform->wordbook["limit_dns_zone_txt"]);
      if($_SESSION["s"]["user"]["typ"] != 'admin') {
         // Get the limits of the client
         $client_group_id = $_SESSION["s"]["user"]["default_group"];
         $client = $app->db->queryOneRecord("SELECT limit_dns_zone, dns_servers FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ?", $client_group_id);

         $client['dns_servers_ids'] = explode(',', $client['dns_servers']);

         // Check if chosen server is in authorized servers for this client
         if (!(is_array($client['dns_servers_ids']) && in_array($this->dataRecord["server_id"], $client['dns_servers_ids'])) && $_SESSION["s"]["user"]["typ"] != 'admin') {
            $app->error($app->tform->wordbook['error_not_allowed_server_id']);
         }

         // When the record is updated
         if($this->id > 0) {
            // restore the server ID if the user is not admin and record is edited
            $tmp = $app->db->queryOneRecord("SELECT server_id FROM dns_soa WHERE id = ?", $this->id);
            $this->dataRecord["server_id"] = $tmp["server_id"];
            unset($tmp);
            // When the record is inserted
         } else {
            // Check if the user may add another maildomain.
            if($client["limit_dns_zone"] >= 0) {
               $tmp = $app->db->queryOneRecord("SELECT count(id) as number FROM dns_soa WHERE sys_groupid = ?", $client_group_id);
               if($tmp["number"] >= $client["limit_dns_zone"]) {
                  $app->error($app->tform->wordbook["limit_dns_zone_txt"]);
               }
            }
         }
      }

      //* Check if soa, ns and mbox have a dot at the end
      if(strlen($this->dataRecord["origin"]) > 0 && substr($this->dataRecord["origin"], -1, 1) != '.') $this->dataRecord["origin"] .= '.';
      if(strlen($this->dataRecord["ns"]) > 0 && substr($this->dataRecord["ns"], -1, 1) != '.') $this->dataRecord["ns"] .= '.';
      if(strlen($this->dataRecord["mbox"]) > 0 && substr($this->dataRecord["mbox"], -1, 1) != '.') $this->dataRecord["mbox"] .= '.';

      //* Replace @ in mbox
      if(stristr($this->dataRecord["mbox"], '@')) {
         $this->dataRecord["mbox"] = str_replace('@', '.', $this->dataRecord["mbox"]);
      }

      $this->dataRecord["xfer"] = preg_replace('/\s+/', '', $this->dataRecord["xfer"]);
      $this->dataRecord["also_notify"] = preg_replace('/\s+/', '', $this->dataRecord["also_notify"]);

      //* Check if a secondary zone with the same name already exists
      $tmp = $app->db->queryOneRecord("SELECT count(id) as number FROM dns_slave WHERE origin = ? AND server_id = ?", $this->dataRecord["origin"], $this->dataRecord["server_id"]);
      if($tmp["number"] > 0) {
         $app->error($app->tform->wordbook["origin_error_unique"]);
      }
   }

   /*
      // Update the serial number of the SOA record
      $soa = $app->db->queryOneRecord("SELECT serial FROM dns_soa WHERE id = ".$this->id);
      $this->dataRecord["serial"] = $app->validate_dns->increase_serial($soa["serial"]);
      */


   //* Check if soa, ns and mbox have a dot at the end
   if(strlen($this->dataRecord["origin"]) > 0 && substr($this->dataRecord["origin"], -1, 1) != '.') $this->dataRecord["origin"] .= '.';
   if(strlen($this->dataRecord["ns"]) > 0 && substr($this->dataRecord["ns"], -1, 1) != '.') $this->dataRecord["ns"] .= '.';
   if(strlen($this->dataRecord["mbox"]) > 0 && substr($this->dataRecord["mbox"], -1, 1) != '.') $this->dataRecord["mbox"] .= '.';

   //* Replace @ in mbox
   if(stristr($this->dataRecord["mbox"], '@')) {
      $this->dataRecord["mbox"] = str_replace('@', '.', $this->dataRecord["mbox"]);
   }

   $this->dataRecord["xfer"] = preg_replace('/\s+/', '', $this->dataRecord["xfer"]);
   $this->dataRecord["also_notify"] = preg_replace('/\s+/', '', $this->dataRecord["also_notify"]);

   //* Check if a secondary zone with the same name already exists
   $tmp = $app->db->queryOneRecord("SELECT count(id) as number FROM dns_slave WHERE origin = ? AND server_id = ?", $this->dataRecord["origin"], $this->dataRecord["server_id"]);
   if($tmp["number"] > 0) {
      $app->error($app->tform->wordbook["origin_error_unique"]);
   }

   parent::onSubmit();
}

function onAfterInsert() {
   global $app, $conf;

   // make sure that the record belongs to the client group and not the admin group when a dmin inserts it
   if($_SESSION["s"]["user"]["typ"] == 'admin' && isset($this->dataRecord["client_group_id"])) {
      $client_group_id = $app->functions->intval($this->dataRecord["client_group_id"]);
      $app->db->query("UPDATE dns_soa SET sys_groupid = $client_group_id, sys_perm_group = 'ru' WHERE id = ".$this->id);
      // And we want to update all rr records too, that belong to this record
      $app->db->query("UPDATE dns_rr SET sys_groupid = $client_group_id WHERE zone = ".$this->id);
   }
   if($app->auth->has_clients($_SESSION['s']['user']['userid']) && isset($this->dataRecord["client_group_id"])) {
      $client_group_id = $app->functions->intval($this->dataRecord["client_group_id"]);
      $app->db->query("UPDATE dns_soa SET sys_groupid = $client_group_id, sys_perm_group = 'riud' WHERE id = ".$this->id);
      // And we want to update all rr records too, that belong to this record
      $app->db->query("UPDATE dns_rr SET sys_groupid = $client_group_id WHERE zone = ".$this->id);
   }

}

function onBeforeUpdate () {
@@ -244,52 +273,18 @@

   //* Check if the server has been changed
   // We do this only for the admin or reseller users, as normal clients can not change the server ID anyway
   if($_SESSION["s"]["user"]["typ"] != 'admin' && !$app->auth->has_clients($_SESSION['s']['user']['userid'])) {
   if($_SESSION["s"]["user"]["typ"] != 'admin' && !$app->auth->has_clients($_SESSION['s']['user']['userid']) && isset($this->dataRecord['origin'])) {
      //* We do not allow users to change a domain which has been created by the admin
      $rec = $app->db->queryOneRecord("SELECT origin from dns_soa WHERE id = ".$this->id);
      if(isset($this->dataRecord["origin"]) && $rec['origin'] != $this->dataRecord["origin"] && $app->tform->checkPerm($this->id, 'u')) {
      $rec = $app->db->queryOneRecord("SELECT origin from dns_soa WHERE id = ?", $this->id);
      $drOrigin = $app->functions->idn_encode($this->dataRecord['origin']);

      if($rec['origin'] !== $drOrigin && $app->tform->checkPerm($this->id, 'u')) {
         //* Add a error message and switch back to old server
         $app->tform->errorMessage .= $app->lng('The Zone (soa) can not be changed. Please ask your Administrator if you want to change the Zone name.');
         $app->tform->errorMessage .= $app->tform->wordbook["soa_cannot_be_changed_txt"];
         $this->dataRecord["origin"] = $rec['origin'];
      }
      unset($rec);
   }
}

function onAfterUpdate() {
   global $app, $conf;

   $tmp = $app->db->diffrec($this->oldDataRecord, $app->tform->getDataRecord($this->id));
   if($tmp['diff_num'] > 0) {
      // Update the serial number of the SOA record
      $soa = $app->db->queryOneRecord("SELECT serial FROM dns_soa WHERE id = ".$this->id);
      $app->db->query("UPDATE dns_soa SET serial = '".$app->validate_dns->increase_serial($soa["serial"])."' WHERE id = ".$this->id);
   }

   // make sure that the record belongs to the client group and not the admin group when a dmin inserts it
   if($_SESSION["s"]["user"]["typ"] == 'admin' && isset($this->dataRecord["client_group_id"])) {
      $client_group_id = $app->functions->intval($this->dataRecord["client_group_id"]);
      $app->db->query("UPDATE dns_soa SET sys_groupid = $client_group_id, sys_perm_group = 'ru' WHERE id = ".$this->id);
      // And we want to update all rr records too, that belong to this record
      $app->db->query("UPDATE dns_rr SET sys_groupid = $client_group_id WHERE zone = ".$this->id);
   }
   if($app->auth->has_clients($_SESSION['s']['user']['userid']) && isset($this->dataRecord["client_group_id"])) {
      $client_group_id = $app->functions->intval($this->dataRecord["client_group_id"]);
      $app->db->query("UPDATE dns_soa SET sys_groupid = $client_group_id, sys_perm_group = 'riud' WHERE id = ".$this->id);
      // And we want to update all rr records too, that belong to this record
      $app->db->query("UPDATE dns_rr SET sys_groupid = $client_group_id WHERE zone = ".$this->id);
   }

   //** When the client group has changed, change also the owner of the record if the owner is not the admin user
   if($this->oldDataRecord["client_group_id"] != $this->dataRecord["client_group_id"] && $this->dataRecord["sys_userid"] != 1) {
      $client_group_id = $app->functions->intval($this->dataRecord["client_group_id"]);
      $tmp = $app->db->queryOneREcord("SELECT userid FROM sys_user WHERE default_group = ".$client_group_id);
      if($tmp["userid"] > 0) {
         $app->db->query("UPDATE dns_soa SET sys_userid = ".$tmp["userid"]." WHERE id = ".$this->id);
         $app->db->query("UPDATE dns_rr SET sys_userid = ".$tmp["userid"]." WHERE zone = ".$this->id);
      }
   }

}

}