http://drupal.org/node/132442#signature

I thought I was clever enough to fix this myself... I guess not... Sorry...

Pobster

Comments

Michelle’s picture

Title: Advanced_forum strips signature from comments under 6.x... » Adjust forum post template to account for signature change
Assigned: Unassigned » Michelle

Thanks for pointing this out. I forgot this changed.

Note to self: don't forget to account for signature_forum module.

Michelle

Michelle’s picture

Status: Active » Fixed

If using signature_forum in 5.x, I suggest this template to take advantage of the css:

<div class="user-signature">%s</div>

Michelle

[Edit: forgot the code tags.]

pobster’s picture

Status: Fixed » Needs review

Okay... Using 6.x (released 13th March 2008);

Add this to advanced_forum.module around line 190, mine looks like this;

  // Username
  $variables['name_raw'] =  theme('username', $account);
  $variables['name'] =  '<div class="username">' .$variables['name_raw'] . '</div>';

  // Signature
  $variables['signature'] = $account->signature;

  // Avatar
  $variables['picture'] = theme('user_picture', $account);

Then add the signature to themes/advforum/advf-forum-post.tpl.php (around line 65), so it looks like this;

      <div class="content">
        <?php print $content ?>
        <?php print $signature; ?>
      </div>

Sorry couldn't be bothered to make a proper patch... One thing though - couldn't work out how to get the signature to appear on the 'first' post - only on the comments after it? I'm sure it's something simple, I didn't try looking very hard tbh!

Pobster

pobster’s picture

Oops sorry I completely glossed over your reply! Mind you, your comments were for signature_forum and 5.x, whereas the missing signatures I'm talking about are from core in 6.x as the api has changed (signatures are dynamic now and not just appended to the comment when it's posted like they were before 6.x...)

Pobster

Michelle’s picture

Status: Needs review » Fixed

My comment was just a note to people using signature forum in 5.x that they can take advantage of the CSS in the fix using that template. This issue is already fixed, so setting it back to that.

The signatures not appearing on the node is a separate issue that I need to think about how best to handle.

Michelle

Michelle’s picture

Ok, also fixed the not being on forum nodes while I was at it.

Michelle

Anonymous’s picture

Status: Fixed » Closed (fixed)

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