do NOT try to read empty contact_id

it wont find anything anyway, but gives an exception under AD, because of wrong formated GUID
This commit is contained in:
Ralf Becker 2019-01-28 10:01:12 +01:00
parent 32095efd4b
commit fdfcc15c6b
2 changed files with 5 additions and 1 deletions

View File

@ -525,7 +525,7 @@ var et2_avatar = (function(){ "use strict"; return et2_image.extend(
{
this._super.apply(this,arguments);
var self = this;
if (this.options.editable)
if (this.options.contact_id && this.options.editable)
{
egw(window).json(
'addressbook.addressbook_ui.ajax_noPhotoExists',

View File

@ -603,6 +603,10 @@ class Storage
*/
function read($contact_id)
{
if (empty($contact_id))
{
return false; // no need to pass to backend, will fail anyway
}
if (!is_array($contact_id) && substr($contact_id,0,8) == 'account:')
{
$contact_id = array('account_id' => (int) substr($contact_id,8));