Fixed: FS#3372 - Web browser password autofill function overwrites settings

Till Brehm

2014-03-08 71c74b1b572f3bdf00ba99009cc23f8f48072a1c

 interface/lib/classes/remoting.inc.php

@@ -126,8 +126,11 @@
      $session_id = $app->db->quote($session_id);

      $sql = "DELETE FROM remote_session WHERE remote_session = '$session_id'";
      $app->db->query($sql);
      return $app->db->affectedRows() == 1;
      if($app->db->query($sql) != false) {
         return true;
      } else {
         return false;
      }
   }


@@ -136,20 +139,54 @@
    @param int session id
    @param int server id
    @param string  section of the config field in the server table. Could be 'web', 'dns', 'mail', 'dns', 'cron', etc
    @author Julio Montoya <gugli100@gmail.com> BeezNest 2010
    @author Julio Montoya <gugli100@gmail.com> BeezNest 2010, extended by M. Cramer <m.cramer@pixcept.de> 2014
    */


   public function server_get($session_id, $server_id, $section ='') {
   public function server_get($session_id, $server_id = null, $section ='') {
      global $app;
      if(!$this->checkPerm($session_id, 'server_get')) {
         $this->server->fault('permission_denied', 'You do not have the permissions to access this function.');
         return false;
      }
      if (!empty($session_id) && !empty($server_id)) {
      if (!empty($session_id)) {
         $app->uses('remoting_lib , getconf');
         $section_config =  $app->getconf->get_server_config($server_id, $section);
         return $section_config;
         if(!empty($server_id)) {
            $section_config =  $app->getconf->get_server_config($server_id, $section);
            return $section_config;
         } else {
            $servers = array();
            $sql = "SELECT server_id FROM server WHERE 1";
            $all = $app->db->queryAllRecords($sql);
            foreach($all as $s) {
               $servers[$s['server_id']] = $app->getconf->get_server_config($s['server_id'], $section);
            }
            unset($all);
            unset($s);
            return $servers;
         }
      } else {
         return false;
      }
   }
	
   /**
       Gets a list of all servers
       @param int session_id
       @param int server_name
       @author Marius Cramer <m.cramer@pixcept.de> 2014
    */
   public function server_get_all($session_id)
    {
        global $app;
      if(!$this->checkPerm($session_id, 'server_get')) {
           $this->server->fault('permission_denied', 'You do not have the permissions to access this function.');
            return false;
      }
      if (!empty($session_id)) {
         $sql = "SELECT server_id, server_name FROM server WHERE 1";
         $servers = $app->db->queryAllRecords($sql);
         return $servers;
      } else {
         return false;
      }
@@ -294,6 +331,20 @@
      return $all;
   }

   //* Get server ips
   public function server_ip_get($session_id, $primary_id)
   {
      global $app;

      if(!$this->checkPerm($session_id, 'server_ip_get')) {
         $this->server->fault('permission_denied', 'You do not have the permissions to access this function.');
         return false;
      }
      $app->uses('remoting_lib');
      $app->remoting_lib->loadFormDef('../admin/form/server_ip.tform.php');
      return $app->remoting_lib->getDataRecord($primary_id);
   }
	
   //* Add a IP address record
   public function server_ip_add($session_id, $client_id, $params)
   {
@@ -1712,7 +1763,18 @@
         $this->dataRecord = $params;
         $app->sites_database_plugin->processDatabaseInsert($this);

         return $this->insertQueryExecute($sql, $params);
         $retval = $this->insertQueryExecute($sql, $params);
			
         // set correct values for backup_interval and backup_copies
         if(isset($params['backup_interval']) || isset($params['backup_copies'])){
            $sql_set = array();
            if(isset($params['backup_interval'])) $sql_set[] = "backup_interval = '".$app->db->quote($params['backup_interval'])."'";
            if(isset($params['backup_copies'])) $sql_set[] = "backup_copies = ".$app->functions->intval($params['backup_copies']);
            //$app->db->query("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$retval);
            $this->updateQueryExecute("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$retval, $retval, $params);
         }
			
         return $retval;
      }

      return false;
@@ -1735,7 +1797,18 @@
         $this->id = $primary_id;
         $this->dataRecord = $params;
         $app->sites_database_plugin->processDatabaseUpdate($this);
         return $this->updateQueryExecute($sql, $primary_id, $params);
         $retval = $this->updateQueryExecute($sql, $primary_id, $params);
			
         // set correct values for backup_interval and backup_copies
         if(isset($params['backup_interval']) || isset($params['backup_copies'])){
            $sql_set = array();
            if(isset($params['backup_interval'])) $sql_set[] = "backup_interval = '".$app->db->quote($params['backup_interval'])."'";
            if(isset($params['backup_copies'])) $sql_set[] = "backup_copies = ".$app->functions->intval($params['backup_copies']);
            //$app->db->query("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$primary_id);
            $this->updateQueryExecute("UPDATE web_database SET ".implode(', ', $sql_set)." WHERE database_id = ".$primary_id, $primary_id, $params);
         }
			
         return $retval;
      }

      return false;