Privatemsg integration

Well, it's been removed from AF in the dev so as of the next alpha, which should be within a week, people will have to manually add this file to privatemsg. It's up to you when you add it. It's just a simple file drop in so it's not horrible for users to have to do. I just wanted to get the ball rolling because I'd like the transition to be complete by the time I release 1.0, hopefully in November some time.

Thanks,

Michelle

I guess it depends. Do you expect privatemsg is going to want to integrate with many other modules? If it's only AF/APK that you're going to integrate with, then it seems like a lot of fuss over a very small function. In fact, the function could be made a lot smaller if you made an API function something like:

function privatmsg_get_link($account1, $account2) {
  // check if $account1 has access to send $account2 a message and that $account2 is accepting them. If so, fill in $link with a relative link. If not, leave it empty.

  return $link;
}

Then it just becomes:

function privatemsg_preprocess_author_pane(&$variables) {
  global $user;
   
  $link = privatmsg_get_link($user, $variables['account']);

  if ($link) {
    $variables['privatemsg'] = 
      l(theme('image', "$variables['image_path']/private-message.png", 
              t('Send private message'), t('Send private message'), NULL, TRUE), $link, array(), NULL, NULL, FALSE, TRUE); 
       
    $variables['privatemsg_link'] = l(t('Send PM'), $link);
   }
}

Just off the top of my head... No checking for typos.

At any rate, like I said on IRC, you can stick the function wherever you want. If you don't stick it anywhere at all, then your users will have to add it to your module themselves if they want to have privatemsg functionality added to their forums and user profiles.

Michelle

AF/APK doesn't care where the hook lives as long as it's accessable. It just needs to go somewhere because, as of the alpha I released last night, privatemsg no longer works with AF without users having to hack the privatemsg module to add the hook themselves.

Michelle

Yes, I pulled those files out of alpha 13 where they were working fine. You should be able to just drop the file into your project directory or pull the hook out of the file and put it whereever you want. For D6, it's just the normal template preprocess function handled by core. For D5, AF emulates the D6 theme system to grab the preprocesses.

Michelle

nbz - Yes, but people on this thread are reporting that the user titles function I'm using doesn't work in D6. If that function has changed, I'll need to make a new file for D6.

Michelle

Erm... #18 was supposed to go on the user titles issue. Not sure if it's me that got confused or Drupal. LOL! Probably had more than one open in tabs.

Edit: Actually, maybe it was supposed to be in answer to #17 but I was just mixing issues up. So, um, to answer #17, yes, unless litwol changes the function it uses.

Michelle

Re-opening this since I've just been made aware the 5.x branch never got the file. I am attaching the version that is in use on my 5.x site with privatemsg 5.x-3.0.

Michelle

Pulling this back to my queue since litwol decided not to add it after all so it needs to go in AP.

Michelle

Oops, yes, good catch, nbz.

Michelle

Ok, committed to Author Pane.

Michelle