Don't save 'internal' or empty serialized values

Thomas Bruederli

2013-02-13 49cb6984ed35ff4342ae31b73269ff2d6a2186e8

Don't save 'internal' or empty serialized values

 program/lib/Roundcube/rcube_ldap.php

@@ -38,7 +38,7 @@
    /** private properties */
    protected $ldap;
    protected $prop = array();
    protected $fieldmap = array('_objclass' => 'objectclass');
    protected $fieldmap = array();
    protected $sub_filter;
    protected $filter = '';
    protected $result = null;
@@ -63,6 +63,8 @@
    {
        $this->prop = $p;

        $fetch_attributes = array('objectClass');

        if (isset($p['searchonly']))
            $this->searchonly = $p['searchonly'];

@@ -80,8 +82,8 @@
            if (empty($this->prop['groups']['scope']))
                $this->prop['groups']['scope'] = 'sub';

            // add group name attrib to fieldmap in order to have it fetched
            $this->fieldmap['_groupname'] = $this->prop['groups']['name_attr'];
            // add group name attrib to the list of attributes to be fetched
            $fetch_attributes[] = $this->prop['groups']['name_attr'];
        }
        else if (is_array($p['group_filters']) && count($p['group_filters'])) {
            $this->groups = true;
@@ -192,7 +194,7 @@
        $this->cache = $rcube->get_cache('LDAP.' . asciiwords($this->prop['name']), 'db', 600);

        // initialize ldap wrapper object
        $this->prop['attributes'] = array_values($this->fieldmap);
        $this->prop['attributes'] = array_unique(array_merge(array_values($this->fieldmap), $fetch_attributes));
        $this->ldap = new rcube_ldap_generic($this->prop, true);
        $this->ldap->set_cache($this->cache);

@@ -565,8 +567,7 @@
            return $group_members;

        // read these attributes for all members
        $attrib = $count ? array('dn') : array_values($this->fieldmap);
        $attrib[] = 'objectClass';
        $attrib = $count ? array('dn') : $this->prop['attributes'];
        $attrib[] = 'member';
        $attrib[] = 'uniqueMember';
        $attrib[] = 'memberURL';
@@ -1288,7 +1289,7 @@
        if (array_intersect(array('groupofuniquenames','kolabgroupofuniquenames'), array_map('strtolower', (array)$rec['objectclass']))) {
            $out['_type'] = 'group';
            $out['readonly'] = true;
            $fieldmap['name'] = $fieldmap['_groupname'];
            $fieldmap['name'] = $this->prop['groups']['name_attr'];
        }

        foreach ($fieldmap as $rf => $lf)
@@ -1353,8 +1354,10 @@
            if (is_array($colprop['serialized'])) {
               foreach ($colprop['serialized'] as $subtype => $delim) {
                  $key = $col.':'.$subtype;
                  foreach ((array)$save_cols[$key] as $i => $val)
                     $save_cols[$key][$i] = join($delim, array($val['street'], $val['locality'], $val['zipcode'], $val['country']));
                  foreach ((array)$save_cols[$key] as $i => $val) {
                     $values = array($val['street'], $val['locality'], $val['zipcode'], $val['country']);
                     $save_cols[$key][$i] = count(array_filter($values)) ? join($delim, $values) : null;
                 }
               }
            }
        }