preserve accountstatus: only write to errorlog if condition is met

This commit is contained in:
Klaus Leithoff 2010-10-06 15:42:57 +00:00
parent f3bfc6bc19
commit 90723084fc

View File

@ -830,8 +830,11 @@ class addressbook_bo extends addressbook_so
// maybe we need that for id and account_id as well. // maybe we need that for id and account_id as well.
if (is_array($old) && (!isset($old['owner']) || empty($old['owner']))) if (is_array($old) && (!isset($old['owner']) || empty($old['owner'])))
{ {
error_log(__METHOD__.__LINE__." Trying to change account to owner:". $to_write['owner'].' Account affected:'.array2string($old).' Data send:'.array2string($to_write)); if (isset($to_write['owner']) && !empty($to_write['owner']))
if (isset($to_write['owner']) && !empty($to_write['owner'])) unset($to_write['owner']); {
error_log(__METHOD__.__LINE__." Trying to change account to owner:". $to_write['owner'].' Account affected:'.array2string($old).' Data send:'.array2string($to_write));
unset($to_write['owner']);
}
} }
// we dont update the content-history, if we run inside setup (admin-account-creation) // we dont update the content-history, if we run inside setup (admin-account-creation)
if(!($this->error = parent::save($to_write)) && is_object($GLOBALS['egw']->contenthistory)) if(!($this->error = parent::save($to_write)) && is_object($GLOBALS['egw']->contenthistory))