Removes an unnecessary call to Salesforce when displaying the admin/content/salesforce/contact page. Currently the module queries Salesforce to retrieve the contacts account_id even if it's already been stored locally. This should speed up loading of this page as it will now only query Salesforce if it can't find a local account_id for the contact.

CommentFileSizeAuthor
unnecessary__salesforce_call.patch1.03 KBeojthebrave

Comments

victorkane’s picture

A question on this, before proceeding: in many use cases, user will often be modifying contact info within their Salesforce account online, and not necessarily within the Drupal app only. So the calls to salesforce will not be "unnecessary".

Looks to me like we need this to be configurable somehow; so I am not convinced this patch should be part of the default installation.

Discussion?

victorkane’s picture

Category: bug » feature

Changing from bug to feature request

eojthebrave’s picture

Valid point. I hadn't thought about contacts changing from one account to another.

I'm currently working on getting things setup to allow Drupal to automatically updated a local contact's profile if their profile has been updated in Salesforce. Maybe during that sync/check we can updated the account_id as well.

victorkane’s picture

Sounds very interesting. We need to make a thorough design based on a whole series of use cases and the best design to implement them, tho.

What is troubling me is the absence of input from others who have expressed interest in the module. Oh well, patience!

aaronbauman’s picture

Status: Needs review » Closed (fixed)