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.

CommentFileSizeAuthor
#1 1351392_allow_drupal_alter_on_advanced_forum.patch1.04 KBaidanlis

Comments

aidanlis’s picture

aidanlis commented
Status: Active » Needs review
StatusFileSize
new 1351392_allow_drupal_alter_on_advanced_forum.patch1.04 KB

Here's a patch :)

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

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

aidanlis’s picture

aidanlis commented

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.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Ah, that makes sense.

Thanks,

Michelle

mcdruid’s picture

mcdruid commented
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.

troky’s picture

troky commented

7.x patched, too.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Thanks both of you!

brandy.brown’s picture

brandy.brown commented
Status: Fixed » Closed (fixed)
mcdruid’s picture

mcdruid commented
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.

Status: Fixed » Closed (fixed)

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