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

Marius Cramer

2015-08-06 37b29231e47a0c4458dc1c15d98588f16f07e1e2

 interface/lib/classes/remoting.inc.php

@@ -93,9 +93,6 @@
      $sql = "DELETE FROM remote_session WHERE tstamp < UNIX_TIMSTAMP()";
      $app->db->query($sql);

      $username = $app->db->quote($username);
      $password = $app->db->quote($password);

      if($client_login == true) {
         $sql = "SELECT * FROM sys_user WHERE USERNAME = ?";
         $user = $app->db->queryOneRecord($sql, $username);
@@ -175,8 +172,6 @@
         return false;
      }

      $session_id = $app->db->quote($session_id);

      $sql = "DELETE FROM remote_session WHERE remote_session = ?";
      if($app->db->query($sql, $session_id) != false) {
         return true;
@@ -201,7 +196,7 @@
      $sql = $app->remoting_lib->getSQL($params, 'INSERT', 0);

      //* Check if no system user with that username exists
      $username = $app->db->quote($params["username"]);
      $username = $params["username"];
      $tmp = $app->db->queryOneRecord("SELECT count(userid) as number FROM sys_user WHERE username = ?", $username);
      if($tmp['number'] > 0) $app->remoting_lib->errorMessage .= "Duplicate username<br />";

@@ -341,10 +336,19 @@
      
      //* get old record and merge with params, so only new values have to be set in $params
      $old_rec = $app->remoting_lib->getDataRecord($primary_id);
		
      foreach ($app->remoting_lib->formDef['fields'] as $fieldName => $fieldConf)
        {
            if ($fieldConf['formtype'] === 'PASSWORD' && empty($params[$fieldName])) {
                unset($old_rec[$fieldName]);
            }
        }
		
      $params = $app->functions->array_merge($old_rec,$params);

      //* Get the SQL query
      $sql = $app->remoting_lib->getSQL($params, 'UPDATE', $primary_id);
		
      // throw new SoapFault('debug', $sql);
      if($app->remoting_lib->errorMessage != '') {
         throw new SoapFault('data_processing_error', $app->remoting_lib->errorMessage);
@@ -470,8 +474,6 @@
         throw new SoapFault('session_id_empty', 'The SessionID is empty.');
         return false;
      }

      $session_id = $app->db->quote($session_id);

      $sql = "SELECT * FROM remote_session WHERE remote_session = ? AND tstamp >= UNIX_TIMSTAMP()";
      $session = $app->db->queryOneRecord($sql, $session_id);