Would be great if there were some rules tie-ins for avatar approval, such as "avatar approved" and "avatar rejected".

Comments

dalad’s picture

Version: 6.x-1.x-dev » 6.x-1.1

Rules integration features in latest dev.
If you could take a look and tell me what kind of rules and actions you think that might be needed, I'll see if I can implement them.
Thank you

dalad’s picture

Version: 6.x-1.1 » 6.x-1.x-dev
sdsheridan’s picture

Version: 6.x-1.1 » 6.x-1.x-dev

Ah, great! Well, rule conditions that would be helpful would be in my mind:

  1. user uploaded new avatar
  2. user replaced avatar
  3. avatar approved by avatar moderator
  4. avatar rejected by avatar moderator
  5. user deleted avatar

so those can be combined with conditions like whether or not a user is a member of a particular role, or a CCK field having a particular value, etc.

And some actions that might be good could be

  1. automatically approve avatar
  2. automatically reject avatar

which could also be combined with other actions like sending a private message on approval, automatically approving avatars for pre-authorised groups, etc.

Does that help?

Shawn

sdsheridan’s picture

...and it looks pretty much like you've covered all that in the rules.inc, so that's great! :-) Looks good to me.

Shawn

dalad’s picture

Status: Active » Fixed

Ok. It stays in dev for further testing and possible updates

Status: Fixed » Closed (fixed)

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

BeatleMike’s picture

Status: Closed (fixed) » Needs work

I am using the rules in this Dev version, but when I set a rule:

"An Avatar is approved"

to do

"Add user role"

for

"User that had is avatar moderated"

and I choose a role

Nothing happens in the end result.

dalad’s picture

Status: Needs work » Fixed

This should now be fixed in both 6.x-1.x-dev and 7.x-1.x-dev

BeatleMike’s picture

Status: Fixed » Closed (fixed)

It works great now, thanks Dalad!

arianek’s picture

Title: Rules tie-in » Avatar Rules integration

updating with appropriate issue title, spammer got to it