Don't show Salesforce tab on drupal objects that don't have fieldmaps
Bevan - April 25, 2009 - 05:23
| Project: | Salesforce |
| Version: | 6.x-2.x-dev |
| Component: | User interface |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | reviewed & tested by the community |
Description
Node types that don't have fieldmaps, still have a salesforce tab. See the attachment. This is redundant. Hide it by returning FALSE in the hook_menu() item's access callback.
This is a usability bug.
| Attachment | Size |
|---|---|
| About us | sf.lcl_.png | 48.04 KB |
#1
I patched this but have not tested yet, since I am reserving the passwordExpired state of my testing accounts to develop/test that patch first.
#2
I've applied this and have done a bit of cursory testing - the tab does not show up for objects without a fieldmap, and does when they do. The manual export/import buttons still seem to be working. I will keep this patch in place as I am testing my setup for creating objects automatically as well. Is there any specific functionality that you would like me to take a look at for this patch?
So excited about this module, by the way - it will be quite helpful for our organization.
#3
Is it possible to get a new patch for this? I tried to apply it to the latest with #476978: Implement "Object Setup" Feature and #536292: Automatic syncing can not be changed after fieldmap-creation and it didn't work, and I'm unsure how to apply the diff code manually based on what I'm seeing in the patch.
#4
Patch #1 works great for me.
I added similar functionality for sf_user and removed an erroneous edit to salesforce_api.admin.inc
re comment #3 from legion80:
I was able to apply the patch from #1 to the latest version of 6-2-dev, and you should be able to do the same for this version of the patch. Since this patch does not require any other patches, including the 2 you mentioned, you're best bet is to work from a fresh copy from CVS.
#5