SuiteCRM 7.9.6 integration Mautic 2.10.1 Accounts duplicated

[rainolf]

What type of report is this:

Q	A
Bug report?

Description:

Tried to use the SugarCRM plugin to sync Accounts at first.
The settings are:
-> Triggered action push contacts to integration
-> Pull contacts from integration
-> Push contacts to this integration
-> Update blank values

The Account mapping field is everything needed but when i try to map also email address as follow:
Email Address <---> Company Email

the accounts gets duplicated

Once this mapping has been remove the sync works well.

Seems that the plugin cannot validate email address correctly and does not recognize the same account based on the email.

Tried with:
mautic:integration:fetchleads --integration=Sugarcrm
mautic:integration:fetchleads --integration=Sugarcrm --fetch-all
mautic:integration:synccontacts --integration=Sugarcrm

without success

Does anybody had the same problem and how to debug it?

Thanks

If a bug:

Q	A
Mautic version	-> 2.20.1
PHP version	-> PHP Version 7.0.19-1

Steps to reproduce:

1. Enable SugarCRM integration
2. Mapping even just required fields for Accounts and email address
3. Launch from console sync

Log errors:

Please check for related errors in the latest log file in [mautic root]/app/log/ and/or the web server's logs and post them here. Be sure to remove sensitive information if applicable.

No logs displayed and no errors in console

[stancel]

Hi @rainolf I experienced the same issue in Mautic 2.12.1. After several days I just setup a local environment and debugged through the code to find the issue. The problem is in the Mautic\LeadBundle\Helper\IdentifyCompanyHelper class and identifyLeadsCompany() method. Looks like it was introduced here #4742 (#4742) from this original commit (faff835). The problem is in an else if statement that tried to assign a company name based on the domain returned from MX records rather. If that returned a value then it skipped the last else if statement that would have tried to lookup the existing Account/Company contact by 'companyname'.

I have created a patch, tested it locally and it has passed. Going to create a pull request and attach it to this issue shortly.

[heathdutton]

Closing this as it will be (at least partially) addressed by 2.14.1.
If this is in error, please re-open after that release. Thanks!