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.
| Comment | File | Size | Author |
|---|---|---|---|
| unnecessary__salesforce_call.patch | 1.03 KB | eojthebrave |
Comments
Comment #1
victorkane commentedA 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?
Comment #2
victorkane commentedChanging from bug to feature request
Comment #3
eojthebraveValid 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.
Comment #4
victorkane commentedSounds 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!
Comment #5
aaronbauman#702614: 5.x branch is no longer relevant. Let's drop it.