Changeset 4953 in subversion

Timestamp:

Jul 22, 2011 6:49:45 AM (22 months ago)

Author:

alec

Message:

• Fix setting fieldmap and search fields, again

File:

• trunk/plugins/acl/acl.php (modified) (5 diffs)

trunk/plugins/acl/acl.php

@@ -86 +86 @@
         $this->load_config();
 
-        $search    = get_input_value('_search', RCUBE_INPUT_GPC, true);
-        $uid_field = $this->rc->config->get('acl_users_uid_field', 'uid');
-        $users     = array();
+        $search = get_input_value('_search', RCUBE_INPUT_GPC, true);
+        $users  = array();
 
         if ($this->init_ldap()) {
@@ -95 +94 @@
 
             foreach ($result->records as $record) {
-                $user = $record[$uid_field];
+                $user = $record['uid'];
 
                 if (is_array($user)) {
@@ -103 +102 @@
 
                 if ($user) {
-
                     if ($record['name'])
                         $user = $record['name'] . ' (' . $user . ')';
@@ -182 +180 @@
         $this->rc->output->set_env('acl_users_source', (bool) $this->rc->config->get('acl_users_source'));
         $this->rc->output->set_env('mailbox', $mbox_imap);
-        $this->rc->output->add_label('searching');
         $this->rc->output->add_handlers(array(
             'acltable'  => array($this, 'templ_table'),
@@ -634 +631 @@
         }
 
-        // search also in UID field
-        $ldap_config[$addressbook]['search_fields'] = array_unique(
-            array_merge((array)$uid_field, (array)$ldap_config[$addressbook]['search_fields']));
-
-        // add UID field to fieldmap, so it will be returned in a record
-        if (isset($ldap_config[$addressbook]['fieldmap'])) {
-            if (empty($ldap_config[$addressbook]['fieldmap']['uid'])) {
-                $ldap_config[$addressbook]['fieldmap']['uid'] = $uid_field;
-            }
+        $ldap_config = $ldap_config[$addressbook];
+
+        // get name attribute
+        if (!empty($ldap_config['fieldmap'])) {
+            $name_field = $ldap_config['fieldmap']['name_field'];
         }
         // ... no fieldmap, use the old method
-        else if (empty($ldap_config[$addressbook]['uid_field'])) {
-            $ldap_config[$addressbook]['uid_field'] = $uid_field;
-        }
+        if (empty($name_field)) {
+            $name_field = $ldap_config['name_field'];
+        }
+
+        // add UID field to fieldmap, so it will be returned in a record with name
+        $ldap_config['fieldmap'] = array(
+            'name' => $name_field,
+            'uid'  => $uid_field,
+        );
+
+        // search in UID and name fields
+        $ldap_config['search_fields'] = array_values($ldap_config['fieldmap']);
+
+        // disable vlv
+        $ldap_config['vlv'] = false;
 
         // Initialize LDAP connection
-        $this->ldap = new rcube_ldap(
-            $ldap_config[$addressbook],
+        $this->ldap = new rcube_ldap($ldap_config,
             $this->rc->config->get('ldap_debug'),
             $this->rc->config->mail_domain($_SESSION['imap_host']));