Skip to content

Update ContactEntityMapper.cs#322

Open
filippobottega wants to merge 1 commit intoaluxnimm:masterfrom
filippobottega:patch-2
Open

Update ContactEntityMapper.cs#322
filippobottega wants to merge 1 commit intoaluxnimm:masterfrom
filippobottega:patch-2

Conversation

@filippobottega
Copy link
Copy Markdown

I propose to change target.Inner.FullName to target.Inner.Title as ContactEntityMapper.cs line 257:

...
if (!string.IsNullOrEmpty(source.Inner.FileAs))
target.Title = source.Inner.FileAs;
...

I propose to change target.Inner.FullName to target.Inner.Title as ContactEntityMapper.cs line 257:

...
      if (!string.IsNullOrEmpty(source.Inner.FileAs))
        target.Title = source.Inner.FileAs;
...
@filippobottega
Copy link
Copy Markdown
Author

The problem si that FileAs should be mapped with contact Name in case the name will be empty.
The same if FormatName is defined and Name is empty. FormatName should be mapped with Name in Outllok to prevent Outlook from parsing FormatName and splitting the value into multiple parts using space as character separator.

The field Title is not the correct field to use because in Android address book the Title is not displayed.
Name is displayed correctly.
The rule to sync Outlook with Google should be to map FileAs to FormatName and if the FileAs is a generic description or the name of a company, the value should be assigned to the Name field of Name entity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant