sf_node_export() when it loads a node uses node_load($nid);, this should be node_load($node);.

When it saves the salesforce id it uses

salesforce_api_id_save('node', $nid, $response->id, $fieldmap);

This should be

salesforce_api_id_save('node', $node->nid, $response->id, $fieldmap);

($nid should be $node->nid)

CommentFileSizeAuthor
#3 395648.patch1.87 KBBevan
#1 sf_node.module.patch849 bytesaaronbauman

Comments

aaronbauman’s picture

aaronbauman
he/him
commented
Assigned: Unassigned » aaronbauman
Status: Active » Needs review
StatusFileSize
new sf_node.module.patch849 bytes
al.ex’s picture

al.ex commented

I can confirm both the bug and the patch as working.

Bevan’s picture

Bevan commented
StatusFileSize
new 395648.patch1.87 KB

I found these changes broke node-create. All CCK fields were deleted from the node and values NULL were saved to SF feilds. I made one additional change to fix this. See the attached patch. It applies from the CVS root of this project, not the salesforce_api subdirectory.

This needs more testing.

Bevan’s picture

Bevan commented
Status: Needs review » Fixed

Committed to DRUPAL-6--2. Further testing would be ideal before release. Please update from CVS and check this works.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.