To recreate: send a message to someone who accepts messages from someone who doesn't accept messages. The person who receives the message should try to reply.

Comments

fool2’s picture

fool2 commented

What do you think we should do? I think it would be wise to do the access check when generating the link to the reply. So instead of reply it should say "This user is not accepting messages"

This is also probably an error in code that does not return the correct thing for the menu callback. I could be wrong. I'll take a look tonight.

robertdouglass’s picture

robertdouglass commented

I agree with you on both counts. Thanks for investigating.

mindless’s picture

mindless commented

When I tried this I got "{user} does not accept private messages." message at the top.. however, I do agree it is better to find out they don't accept replies before you write your whole message and click "Send".

mindless’s picture

mindless commented
Assigned: Unassigned » mindless
Status: Active » Fixed

HEAD and DRUPAL-4-7 now show a msg instead of reply link if sender does not accept messages.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)