Automatic syncing can not be changed after fieldmap-creation

Corrected description: There is no way to change the "sync to salesforce automatically" checkbox once a field map has been created. The checkbox should be available on the fieldmap-edit form in order to enable/disable automatic syncing.

I think it might be better to move that checkbox into the fieldmap/#/edit page altogether, instead of having it in both the add and edit pages, since it doesn't seem to me that knowing whether you want to sync the data is necessary to create the fieldmap.

legion80, agreed.

Attached is a patch to correct this issue.

It's made against salesforce-6.x-2.x-dev, from Sep-01-2009 (latest available for download as of this writing). Something along these lines has been on my todo list for a while. It follows the method described in comment #2.

Thanks!

Wanted to verify that this works. When can we see this patch pushed to the -dev branch?

I had intended to mark the issue "needs review", and not "RTBC" in comment #5.

legion, it's not clear whether you tested this patch or not, or what – specifically – you tested with the patch. please clarify and feel free to change the issue status to RTBC if you think that is valid.

I have verified that the UI checkbox has moved from the "fieldmap add" page to the "fieldmap edit" page. I can also verify that changing the value of the checkbox properly adjusts the "automatic" column in the fieldmap SQL table.

Has anyone else tried the patch? I personally was wondering at point any given patch can move to RTBC--as in, how large "C" needs to be before it is approved.

Great! For simple patches one good review by a confident drupal dev is enough. More reviews are generally needed respective of the complexity of the bug/patch. Basically, if you think that it's ready to be committed, AND you think that the maintainer/s will probably agree with you, then you can mark it RTBC.

As a maintainer, I'd need to either;

• see one more good review (as thorough as yours but trying to test/check other aspects of it), OR
• review it myself

before I commit it.

Since I barely have time to address the list of issues marked RTBC for salesforce, let alone the patches needing review or work, I'll leave this on "needs review" for now.

Thanks for your review though, much appreciated!

I tested and verified legion80's results from comment #8

• verified checkbox moved from the "add" screen to the "edit" screen
• verified existing fieldmaps "automatic" value is populated correctly
• verified new fieldmaps "automatic" value defaults to 0
• verified saving the "edit" screen properly updates the "automatic" value and properly saves other fieldmap data

This is good to go - let's get it in!