Download & Extend
Advanced Forum » Issues

Add drupal_alter to forum_get_forums

Posted by aidanlis on November 24, 2011 at 9:02am
Project:Advanced Forum
Version:6.x-2.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

We need to hide the last_reply sometimes when a user wishes to cloak their email. It would be awesome to have a drupal_alter at the end of forum_get_forums so we don't have to patch AF.

Login or register to post comments

Comments

#1

Posted by aidanlis on November 24, 2011 at 9:54am
Status:active» needs review

Here's a patch :)

AttachmentSize
1351392_allow_drupal_alter_on_advanced_forum.patch 1.04 KB

#2

Posted by Michelle on February 20, 2012 at 3:35pm

This seems reasonable. What is the purpose of the other line being added in the patch?

#3

Posted by aidanlis on February 21, 2012 at 11:54am

The last_post object gets overwritten with a string and I needed access to the raw data, seemed reasonable to leave the raw data there in case other people need it.

#4

Posted by Michelle on February 21, 2012 at 7:03pm

Ah, that makes sense.

Thanks,

Michelle

#5

Posted by mcdruid on February 28, 2012 at 10:22pm
Status:needs review» fixed

Good idea - thanks aidanlis.

Committed to 6.x-2.x branch.

I added a comment to explain $forum->last_post_obj as per #3.

#6

Posted by troky on February 29, 2012 at 9:47am

7.x patched, too.

#7

Posted by Michelle on February 29, 2012 at 2:44pm

Thanks both of you!

#8

Posted by brandy.brown on March 5, 2012 at 5:16pm
Status:fixed» closed (fixed)

#9

Posted by mcdruid on March 5, 2012 at 5:35pm
Status:closed (fixed)» fixed

Thanks for helping with the housekeeping brandy.brown, but we usually leave tickets 'fixed' and let the system close them, which it does automatically after 2 weeks if nobody has posted any follow-ups.

#10

Posted by System Message on March 19, 2012 at 5:40pm
Status:fixed» closed (fixed)

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

nobody click here