I'm not sure what actually needs to be done, but I'll create an issue to start off discussion. I'd imagine a views field for imining users is a must.

Comments

pahariwalla’s picture

This is a good idea @Miraploy. I'm kind of new to Drupal, but I really get the idea here about Views and will definitely develop this this feature. I know it's pretty easy to load and get resutls from a view, so maybe all I need to do is allow the IM Config to select "Use View X" instead of "use UR relationship type X". (It may be also be a good idea to allow multiple UR relationship types too? )

My original idea was to expand from just UR to other "friend" modules, like Buddy, and Firendlist, etc. and also expose a hook to allow developers to override the friend list with any logic they like. But I think that integrating with Views is the easiest to implement as well as the most comprehensive and flexible method.

p.s. Based on your comments on @Vako's "Feature Requests" issue, I really appreciate your appreciation for doing it the "Drupal Way". I look forward to more of your words of wisdom !!

Thanks

pahariwalla’s picture

Status: Active » Patch (to be ported)

The module is now free from the grip of User Relationships! No longer required, as you can now create a view to feed the IM Friend list instead of relying on a User Relationship Type (although it is still available to use)

Just make a view based on users, and make sure to include the user name in the fields (uid gets stuck into the view automatically)

I've committed the changes to CVS and am hoping that the next cron run will add these changes to the development snapshot tarball.

I'll add a new comment when that happens, and also update the project page to let everyone know.

Great idea @Miraploy

pahariwalla’s picture

Looks like the new tarball has these mods.

pahariwalla’s picture

Status: Patch (to be ported) » Fixed
Flying Drupalist’s picture

Great, I'll be testing this as soon as I get my new server set up. :)

Status: Fixed » Closed (fixed)

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