Good day, thanks for the module, it really makes up forum to look better in many aspects.

I have noticed in my Drupal 5.10 + AF that the links "send PM" and "Add/remove to/from Buddylist" appear in the post of guests. Since they are not reasoned there, one can not write message or add user which is not registered I removed them by editing advf-author-pane.tpl.php. It is very easy, who needs do the same:

replace this:

    <?php if (!empty($privatemsg)): ?>
      <div class="author-pane-icon"><?php print $privatemsg; ?></div>
    <?php endif; ?>

    <?php if (!empty($buddylist)): ?>
      <div class="author-pane-icon"><?php print $buddylist; ?></div>
    <?php endif; ?>

with:

    <?php if (!empty($privatemsg) && !empty($account_id)): ?>
      <div class="author-pane-icon"><?php print $privatemsg; ?></div>
    <?php endif; ?>

    <?php if (!empty($buddylist) && !empty($account_id)): ?>
      <div class="author-pane-icon"><?php print $buddylist; ?></div>
    <?php endif; ?>

The same issue with the "Member ID" and "Joined", these labels are shown up, although their values are empty for the not registered user. So, here we can do similar check for the $account_id of post's author.

Comments

transliteration’s picture

For the "Member ID" and "Joined" one can use e.g. following...

replase this:

    <div class="author-pane-line author-id">
      <span class="author-pane-label"><?php print t('Member'); ?>:</span> <?php print $account_id; ?>
    </div>

    <div class="author-pane-line author-joined">
      <span class="author-pane-label"><?php print t('Joined'); ?>:</span> <?php print $joined; ?>
    </div>

with this:

  <?php if (!empty($account_id)): ?>
    <div class="author-pane-line author-id">
      <span class="author-pane-label"><?php print t('Member'); ?>:</span> <?php print $account_id; ?>
    </div>

    <div class="author-pane-line author-joined">
      <span class="author-pane-label"><?php print t('Joined'); ?>:</span> <?php print $joined; ?>
    </div>
  <?php endif; ?>
Anonymous’s picture

Status: Fixed » Closed (fixed)

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

michelle’s picture

Just happened to see this issue... Don't mark issues as fixed when you post them. When I look at the queue, I exclude fixed ones. At any rate, I noticed this same problem and fixed it myself a while ago so it is, actually, fixed.

Michelle