Oops. :)
The change does indeed work, but it required the "*_field" values to be
all lower case. The values in "search_fields" seem to be
case-insensitive (which I believe they should be).
Kyle
Terminal Addict wrote:
> Okay...
>
> program/steps/addressbook/show.inc uses $CONTACTS->get_result()
>
> program/steps/addressbook/list.inc uses $CONTACTS->list_records()
>
> I tried changing both to get_result() and it didn't work. Changed both
> to list_records() and both work fine (with case-sensitive fields
> specified for search_fields and name_field arrays.
>
> So, line 33 in show.inc changed to the following yields expected results.
>
> if (!(($result = $CONTACTS->list_records()) && ($record =
> $result->first()))) {
>
> Kyle
>
> Terminal Addict wrote:
>> LDAP being used is AD. name_field is 'displayName'.
>>
>> If I specify name_field as 'displayname', then it properly displays
>> names in the list (for search results), but it won't show the display
>> name in the contact details.
>>
>> If I specify name_field as 'displayName', then it displays empty (but
>> clickable) fields in the list, but the display name shows up properly in
>> the contact details.
>>
>> By dumping variables, I can see that the same is happening for
>> firstname_field (givenName in LDAP), but that isn't a big deal since
>> that isn't used in the listing, only in the details.
>>
>> Any thoughts?
>>
>> Kyle
>> _______________________________________________
>> List info: http://lists.roundcube.net/dev/
> _______________________________________________
> List info: http://lists.roundcube.net/dev/
_______________________________________________
List info: http://lists.roundcube.net/dev/
