Fixed: FS#2421 - Domain module does not check for valid selection Impleme...

mcramer

2012-09-11 4e18bd4210f5c79d341436b1ed026300d59ba208

 interface/web/sites/web_aliasdomain_edit.php

@@ -85,7 +85,7 @@
         /*
          * The admin can select ALL domains, the user only the domains assigned to him
          */
         $sql = "SELECT domain FROM domain ";
         $sql = "SELECT domain_id, domain FROM domain ";
         if ($_SESSION["s"]["user"]["typ"] != 'admin') {
            $sql .= "WHERE sys_groupid =" . $client_group_id;
         }
@@ -95,7 +95,7 @@
         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'] ;
               $domain_select .= "<option value=" . $domain['domain_id'] ;
               if ($domain['domain'] == $this->dataRecord["domain"]) {
                  $domain_select .= " selected";
               }
@@ -120,6 +120,25 @@
   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') {
         $client_group_id = intval($_SESSION["s"]["user"]["default_group"]);
			
            $sql = "SELECT domain_id, domain FROM domain WHERE domain_id = " . intval($this->dataRecord['domain']);
         if ($_SESSION["s"]["user"]["typ"] != 'admin') {
            $sql .= "AND sys_groupid =" . $client_group_id;
         }
         $domain_check = $app->db->queryOneRecord($sql);
            if(!$domain_check) {
                // invalid domain selected
                $app->tform->errorMessage .= $app->tform->lng("domain_error_empty")."<br />";
            } else {
                $this->dataRecord['domain'] = $domain_check['domain'];
            }
        }
        
      // Get the record of the parent domain
      $parent_domain = $app->db->queryOneRecord("select * FROM web_domain WHERE domain_id = ".intval(@$this->dataRecord["parent_domain_id"]));
      
@@ -155,15 +174,7 @@
         //* Update the old website, so that the vhost alias gets removed
         //* We force the update by inserting a transaction record without changes manually.
         $old_website = $app->db->queryOneRecord('SELECT * FROM web_domain WHERE domain_id = '.$this->oldDataRecord['domain_id']);
         $diffrec_full = array();
         $diffrec_full['old'] = $old_website;
         $diffrec_full['new'] = $old_website;
         $diffstr = $app->db->quote(serialize($diffrec_full));
         $username = $app->db->quote($_SESSION['s']['user']['username']);
         $dbidx = 'domsin_id:'.$this->id;
         $server_id = $this->oldDataRecord['server_id'];
         $sql = "INSERT INTO sys_datalog (dbtable,dbidx,server_id,action,tstamp,user,data) VALUES ('web_domain','$dbidx','$server_id','u','".time()."','$username','$diffstr')";
         $app->db->query($sql);
            $app->db->datalogSave('web_domain', 'UPDATE', 'domain_id', $this->oldDataRecord['parent_domain_id'], $old_website, $old_website, true);
      }
      
   }