Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o… #5610

Merged
Merged
Changes from 7 commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
7497eda
Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o…
stancel Jan 20, 2018
b0d6dfd
Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o…
stancel Jan 20, 2018
2793ef0
Fix for issue #5103 - correcting syntax/spaces for passing Travis CI …
stancel Jan 28, 2018
9ae6157
Fix for issue #5103 - correcting syntax/spaces for passing Travis CI …
stancel Jan 28, 2018
4097112
fix validation email exists
Noa83 Jan 25, 2018
f7ebf02
Pulling in addtional fix from Pull Request #5637
stancel Jan 28, 2018
94efbae
Pulling in addtional fix from Pull Request #5637
stancel Jan 28, 2018
f2e1847
Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o…
stancel Jan 20, 2018
6773371
Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o…
stancel Jan 20, 2018
20e6400
Fix for issue #5103 - correcting syntax/spaces for passing Travis CI …
stancel Jan 28, 2018
cc2226b
Fix for issue #5103 - correcting syntax/spaces for passing Travis CI …
stancel Jan 28, 2018
e60a2d4
fix validation email exists
Noa83 Jan 25, 2018
83d0340
Pulling in addtional fix from Pull Request #5637
stancel Jan 28, 2018
fb2d1e8
Pulling in addtional fix from Pull Request #5637
stancel Jan 28, 2018
36a9dfd
Merge branch 'duplicated-companies-created-fix' of github.com:Process…
stancel Apr 10, 2018
f670bc0
Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o…
stancel Jan 20, 2018
a76dccf
fix validation email exists
Noa83 Jan 25, 2018
b27073d
Pulling in addtional fix from Pull Request #5637
stancel Jan 28, 2018
d299726
Pulling in addtional fix from Pull Request #5637
stancel Jan 28, 2018
ea3ef85
Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o…
stancel Jan 20, 2018
e024649
Fix for issue #5103 - Company/Accounts being duplicated on CRM Sync o…
stancel Jan 20, 2018
de566d8
Fix for issue #5103 - correcting syntax/spaces for passing Travis CI …
stancel Jan 28, 2018
09293cb
Fix for issue #5103 - correcting syntax/spaces for passing Travis CI …
stancel Jan 28, 2018
d1391b1
fix validation email exists
Noa83 Jan 25, 2018
0178000
Rebasing code from 2.13.1 release into this branch/pull request
stancel May 31, 2018
4b2c7e0
Merge branch 'staging' into duplicated-companies-created-fix
heathdutton Sep 4, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 9 additions & 2 deletions app/bundles/LeadBundle/Helper/IdentifyCompanyHelper.php
Expand Up @@ -74,10 +74,14 @@ public static function findCompany(array $parameters, CompanyModel $companyModel

if (isset($parameters['company'])) {
$companyName = filter_var($parameters['company']);
} elseif (isset($parameters['email']) || isset($parameters['companyemail'])) {
$companyName = isset($parameters['email']) ? self::domainExists($parameters['email']) : self::domainExists($parameters['companyemail']);
} elseif (isset($parameters['companyname'])) {
$companyName = filter_var($parameters['companyname']);
} else {
// Do nothing here. Just adding this for proper syntax.
}

if (isset($parameters['email']) || isset($parameters['companyemail'])) {
$companyDomain = isset($parameters['email']) ? self::domainExists($parameters['email']) : self::domainExists($parameters['companyemail']);
}

if (empty($parameters['companywebsite']) && !empty($parameters['companyemail'])) {
Expand Down Expand Up @@ -129,6 +133,9 @@ public static function findCompany(array $parameters, CompanyModel $companyModel
*/
protected static function domainExists($email)
{
if (!strstr($email, '@')) { //not a valid email adress
return false;
}
list($user, $domain) = explode('@', $email);
$arr = dns_get_record($domain, DNS_MX);

Expand Down