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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aidanlis’s picture

Status: Active » Needs review
FileSize
1.04 KB

Here's a patch :)

Michelle’s picture

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

aidanlis’s picture

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

Ah, that makes sense.

Thanks,

Michelle

mcdruid’s picture

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

7.x patched, too.

Michelle’s picture

Thanks both of you!

brandy.brown’s picture

Status: Fixed » Closed (fixed)
mcdruid’s picture

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.