#374463: Alter comment links. is in D6.16.

This may allow simplifying FA considerably, and it may take care of

CommentFileSizeAuthor
#3 forum_access.733188.3.patch3.55 KBsalvis

Comments

salvis’s picture

salvis’s picture

The feature is still broken in D6.16, but my patch (#1 above) was committed and will probably show up in D6.17.

This should allow us to remove forum_access_preprocess_comment() in favor of forum_access_link_alter() (aka hook_link_alter()), which will hopefully cause less conflicts with other modules and which should be easier to implement.

salvis’s picture

Title: Investigate what the new comment link altering feature in D6.16 means for FA » Implement the D6.17 hook_link_alter() for comments
Version: 6.x-1.x-dev » 6.x-1.5
Priority: Normal » Major
Status: Active » Needs review
StatusFileSize
new3.55 KB

Starting with 6.17, Drupal passes the $comment along to hook_link_alter(), and this allows us to do without the very hackish hook_preprocess_comment().

The advantage of this patch over the one in #525392-37: Advanced Forum 1.x and Forum Access Incompatibility is that it deals only with making the links on the comments available. This means that all the other functionality, which is well-tested and stable, remains unchanged.

Please test this patch (against 6.x-1.5 or the current -dev) both with and without Advanced Forum and let us know what you find.

salvis’s picture

I'll have to cache the user_load() call. D6 doesn't.

salvis’s picture

I've committed this to the -dev version (give it up to 12h to be repackaged).

Please test this and let us know how it works, both for pre-D6.17 as well as for D6.17+, and also with or without Advanced Forum.

Note: This change only deals with whether the comment links are visible or not. It is completely safe, because it does not have any influence on the accessibility of the create, edit, and delete pages.

Fogg’s picture

subscribe

salvis’s picture

Status: Needs review » Fixed

Hear, hear...

Status: Fixed » Closed (fixed)

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