Bug 2044

Summary: email completion according to more fields p.ex last name
Product: Claws Mail (GTK 2) Reporter: whats_up
Component: UI/Compose WindowAssignee: users
Status: NEW ---    
Severity: enhancement    
Priority: P3    
Version: 3.6.1   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Add Surname as search field in completion
none
Autofill Name and Surname fields according to Display Name none

Description whats_up 2009-11-09 20:34:10 UTC
Hi there :)

while filling in a recipient using <tab> for mail address completion seems only work for the fields email and display name. Would be nice to consider first and last name also.

ciao Knut
Comment 1 Salvatore De Paolis 2009-11-11 12:12:11 UTC
Actually only the Surname field is missing.
The following patch fixes it.
For the other things it also does:

- if available, show first the Surname field in completion
The next patch also does that:
- autofill Surname and Name field according to Display Name

eg. Display Name: Foo Bar
    Name: Foo
    Surname: Bar
Comment 2 Salvatore De Paolis 2009-11-11 12:13:55 UTC
Created attachment 758 [details]
Add Surname as search field in completion
Comment 3 Salvatore De Paolis 2009-11-11 12:14:41 UTC
Created attachment 759 [details]
Autofill Name and Surname fields according to Display Name
Comment 4 Michael Pollmeier 2011-01-17 16:46:22 UTC
Many thanks Salvatore! 

Any idea why Salvatores patches are not yet in the main repo? I applied those locally and it works fine. 

Just in case anybody is working with git / github: I pushed these two patches to my github fork of claws-mail: 
https://github.com/mpollmeier/claws-mail

https://github.com/mpollmeier/claws-mail/commit/c8c21073b484726b1d5ef7373a1bbc06edb159b2

https://github.com/mpollmeier/claws-mail/commit/463b344848024814d49198332be27b93cde53baf

Cheers
Michael
Comment 5 P Fudd 2011-08-25 00:11:14 UTC
That's odd; I'm using Fedora14, and claws-mail-3.7.9-5.fc14.x86_64.rpm , and the firstname is not being considered.
Comment 6 Nathan Trapuzzano 2012-03-14 14:17:29 UTC
Can we please include this patch upstream?