Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#1487894 closed Bugs (wontfix)

Recipient address formatting problem

Reported by: deltatech Owned by:
Priority: 5 Milestone: later
Component: Addressbook Version: 0.5.2
Severity: normal Keywords: recipient format google addressbook
Cc:

Description

When using autofill from a Google Contacts list, the address formatting is incorrect. This because of how Google formats the display name.

The way google formats these is like this example...
Display Name: John Doe (john.doe@…)
Name: John doe
Email: john.doe@…

The problem is that when rc formats the email address it looks like this...

John Doe (john.doe@…) <john.doe@…>

That looks pretty stupid. Proposed solutions...

  1. use the first and last name instead of the display name to form the recipient/cc/bcc lines.
  1. use preg_replace or other method to remove the (john.doe@…)

I am running 0.5.2 on CentOS 5.6 with Apache 2.2.3

Change History (4)

comment:1 Changed 2 years ago by deltatech

Here is an easy fix for this...

in program/include/main.inc on line 1094 add this...

$name = preg_replace('/\([\d!#$%&\'*+.\/=?_`a-z{|}~^-]++@[\d.a-z-]+\)/i','',$name);
Last edited 2 years ago by deltatech (previous) (diff)

comment:2 Changed 2 years ago by alec

  • Resolution set to wontfix
  • Status changed from new to closed

You're using some plugin then? So, this should be handled by plugin, I think.

comment:3 Changed 2 years ago by deltatech

The fix for the problem was made to /roundcube/program/include/main.inc. I didn't think that had anything to do with a plugin.

It isn't just google contacts that has the problem although that is where it it is the biggest problem. Imported contacts that came from google have the same thing. So even without the plugin, this fix is necessary. This fixes both.

comment:4 Changed 2 years ago by alec

The fix is just wrong. I don't want to remove parentheses from display name.

Note: See TracTickets for help on using tickets.