dev/core#2549

[samuelsov]

Overview

c.f. https://lab.civicrm.org/dev/core/-/issues/2549

Before

Merging contact always merge websites even if the checkbox to copy is not checked

Comments

There is a special treatment for table with location that are found in CRM/Dedupe/Merger.php::getLocationBlockInfo

Even if the websites does not have a location, they are going through the same process.

Added civicrm_im as well which as the same issue.

Ideally locTables and getLocationBlockInfo should be linked somehow to avoid adding info to one without doing it for the other.

[civibot]

No issue was found matching the number given in the pull request title. Please check the issue number.

[civibot]

(Standard links)

• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[larssandergreen]

Could we add table to each array in getLocationBlockInfo and then just call getLocationBlockInfo and pull those out here?

[eileenmcnaughton]

You missed open ID :-)