Attached is a patch against the 4.6.0 version (which I believe is the same as the current CVS version).

This patch adds an implementation of hook_links, so that each posting on a site will have a "Add
to my buddylist" or a "Remove
to my buddylist>" link. I think this is an easier way to get people to use buddy lists, without forcing them to go digging through profiles. Note the number of "I can't find how to add buddies" issues.

Big change: I had to scrap buddylinks_goto_referrer in the add and remove methods. For some reason, it wasn't able to pick up the referrer on the main page. I just sent everyone to their profile page.

TODO: This should have an option to turn this on and off for roles with buddylist permission. I don't know much about how that would work, so I ignored it for now. This patch automatically ignores anonymous users, so they don't get a buddylist link.

Enjoy.

CommentFileSizeAuthor
buddylist.mfredrickson.patch1.65 KBmfredrickson

Comments

mfredrickson’s picture

mfredrickson commented

I just thought I'd follow up with some PHPTemplate code that does something similar. Again, the buddylist_goto_referrer is still broken, but this might be a better way to let users add people to buddy lists. I was modifying the lincoln's revenge theme:

 <?php if ($nodetype != 'page'): ?>
    <div class="info"><span class="submitted">Submitted by <?php print $name; ?>
<?php 
  global $user, $base_url;
  if ($user->uid == $node->uid || $user->uid == 0) {

  }
  else {
    $buddies = buddylist_get_buddies();
  
    $myquery = db_query("SELECT * FROM {users} WHERE UID = %d", $node->uid);
    $nodeuser = db_fetch_object($myquery);
    $buddybool = TRUE;
        
    foreach ($buddies as $buser) {
      if ($node->uid == $buser->uid) {
        //remove
        print " (<a href=\"$base_url/buddy/delete/$buser->uid\">Remove from buddylist</a>) ";
        $buddybool = FALSE;
      }
    }
    if ($buddybool) {
      print " (<a href=\"$base_url/buddy/add/$nodeuser->uid\">Add to buddylist</a>) ";
    }

  }
?>
Bèr Kessels’s picture

Bèr Kessels commented
Status: Needs review » Closed (duplicate)

http://drupal.org/node/18260 has a pending patch that allows the same. please review

Bèr Kessels’s picture

Bèr Kessels commented
Status: Closed (duplicate) » Needs review
mfredrickson’s picture

mfredrickson commented
Title: "Add/Remove" link for each posting » Deprecated
Status: Needs review » Closed (fixed)

I'm going to close this bug as I think it is deprecated by changes to format_name() in 4.7.

Since my last post to this issue, I had written a core patch against format_name to do this functionality, but with the changes out lined at:

http://drupal.org/node/22218#theme_username

I don't think that patching is necessary, and the functionality I want can be added through theme_username.