In drupal 7, "_contact_user_tab_access" was changed to "_contact_personal_tab_access." However views and ctools (at least) both have dependencies on the "_contact_user_tab_access" function and thus throw a fatal error if used in conjunction with this module.

Here's a patch to change the the function name back to "_contact_user_tab_access" so it will work with these.

CommentFileSizeAuthor
#1 _contact_user_tab_access.patch942 bytesjasonn1234

Comments

jasonn1234’s picture

jasonn1234 commented
StatusFileSize
new _contact_user_tab_access.patch942 bytes

Patch

fp’s picture

fp commented
Status: Needs review » Needs work

Looks good.

fp’s picture

fp commented
Status: Needs work » Needs review

Triggering the test bot.

Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
commented

Just out of curiosity, why are you patching a d6 backport of a d7 core feature to work with d7?

Nevermind - cranial/gluteal contact. I see this is fixin the backport.

jasonn1234’s picture

jasonn1234 commented

Bump... I see the test hasn't gone through, although you said that it looks good. Just wanting to keep this on your radar to get committed :)

fp’s picture

fp commented

@jasonn1234: have you also tested it? If it worked for you you can change the status to _reviewed and tested by the community_.

jasonn1234’s picture

jasonn1234 commented
Status: Needs review » Reviewed & tested by the community

Thank you sir. I just double-checked and yes, the patch gets applied successfully and the fix it provides works.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks committed just a shell function since I didn't want to rename the access callback function. This should work now.
http://drupal.org/cvs?commit=372950

Status: Fixed » Closed (fixed)

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

lefnire’s picture

lefnire commented

looks like this still may be an issue with latest drupal-7.x-dev?

lefnire’s picture

lefnire commented

never mind, didn't realize this was /project/contact rather than /project/drupal