Hello again. I am testing the pre-alpha4 of Advanced Forum now and found an incorrect interraction with Forum Access.
Advanced Forum is restyling the comment links by hook_link_alter (inserts some divs around links)
But then forum_access is using hook_preprocess_comment and calls module_invoke_all('link', 'comment', $variables['comment'], 0) there. The links are rebuild and styling is lost.
If I place a part of the Advanced Forum code afterwards I can get the styling back, but that is a weird solution.
I thought one could call module_invoke_all('link_alter',...) here but can't get the proper syntax.
A small discussion at AF: http://drupal.org/node/868942#comment-4010894

Comments

salvis’s picture

salvis
he/his
commented

Please try the -dev version of FA (and a recent version of Drupal).

__Sander__’s picture

__Sander__ commented
Status: Active » Fixed

Worked for me, thanks! Used the dev of 2011-Jan-07

salvis’s picture

salvis
he/his
commented
Category: bug » support

Thanks for letting us know.

meustrus’s picture

meustrus commented

I switched to the -dev version for this reason, and now I'm missing several of my links on comments. I only see the "edit" and "delete" buttons, and although I have permission to edit all comments I only see the "edit" button on my comments. I have Abuse and Vote Up/Down installed, both of which create comment links that aren't there anymore.

salvis’s picture

salvis
he/his
commented

You bring up two new issues that are completely unrelated to Advanced Forum, right?

Please open two new issues. For the edit/delete issue, double-check (and list) the permissions you have and the Permissions Information on the Forum administration page.

meustrus’s picture

meustrus commented

The reason it's here is because I don't know if AF had anything to do with it. I've double-checked the permissions in both places. It was fine before I installed -dev, and now it's not. No settings have changed.

salvis’s picture

salvis
he/his
commented

Disable AF temporarily to find out...

meustrus’s picture

meustrus commented

OK, when I disabled AF the problem was still there. Want me to start a new issue then?

salvis’s picture

salvis
he/his
commented

Yes, please, two new issues, one for edit/delete and one for third-party module links, because these are different issues.

Status: Fixed » Closed (fixed)

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