#374463: Alter comment links. is in D6.16.
This may allow simplifying FA considerably, and it may take care of
- #525392: Advanced Forum 1.x and Forum Access Incompatibility
- other compatibility issues with Advanced Forum
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | forum_access.733188.3.patch | 3.55 KB | salvis |
Comments
Comment #1
salvis#374463-47: Alter comment links.
Comment #2
salvisThe 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.
Comment #3
salvisStarting 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.
Comment #4
salvisI'll have to cache the user_load() call. D6 doesn't.
Comment #5
salvisI'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.
Comment #6
Fogg commentedsubscribe
Comment #7
salvisHear, hear...