diff options
| author | Aleksander Machniak <alec@alec.pl> | 2012-06-13 12:12:53 +0200 |
|---|---|---|
| committer | Aleksander Machniak <alec@alec.pl> | 2012-06-13 12:12:53 +0200 |
| commit | afaccf14287d06eb7bb5e33cf131b14289040865 (patch) | |
| tree | 1def4ddfc7433f59bcce1ef9155586be40476e94 /program/include/rcube_ldap.php | |
| parent | 740875e5cd55581f146fd1fbd2c4d4b904775832 (diff) | |
Fix removing contact photo using LDAP addressbook (#1488420)
Diffstat (limited to 'program/include/rcube_ldap.php')
| -rw-r--r-- | program/include/rcube_ldap.php | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/program/include/rcube_ldap.php b/program/include/rcube_ldap.php index f799df2bb..8f23a406b 100644 --- a/program/include/rcube_ldap.php +++ b/program/include/rcube_ldap.php @@ -1191,8 +1191,11 @@ class rcube_ldap extends rcube_addressbook else if ($val == '') { // Field supplied is empty, verify that it is not required. if (!in_array($fld, $this->prop['required_fields'])) { - // It is not, safe to clear. - $deletedata[$fld] = $old_data[$fld]; + // ...It is not, safe to clear. + // #1488420: Workaround "ldap_mod_del(): Modify: Inappropriate matching in..." + // jpegPhoto attribute require an array() here. It looks to me that it works for other attribs too + $deletedata[$fld] = array(); + //$deletedata[$fld] = $old_data[$fld]; } } else { |