i think it is better to have the drupal (entity) object as argument in the export hooks (hook_salesforce_api_pre_export & hook_salesforce_api_post_export) instead of drupal (entity) id.

so the drupal object data can be used without having to rebuild the drupal object in the hooks.

CommentFileSizeAuthor
#1 entity-export-hooks-1378674-2.patch2.79 KBkostajh
#1 entity-export-hooks-1378674-1.patch2.79 KBkostajh

Comments

kostajh’s picture

kostajh commented
Status: Active » Needs review
StatusFileSize
new entity-export-hooks-1378674-1.patch2.79 KB
new entity-export-hooks-1378674-2.patch2.79 KB

This makes a lot of sense to me. Here is a patch that implements the change and updates the API docs.

kostajh’s picture

kostajh commented
Status: Needs review » Needs work

Patch in #1 needs to be updated to address SF Queue's usage of pre_export and post_export hooks. Also need to add a notice somewhere to developers that they will need to update their implementations of these hooks.

kostajh’s picture

kostajh commented
Status: Needs work » Closed (won't fix)