Opened 9 years ago

Closed 9 years ago

Last modified 6 years ago

#1327 closed bug (fixed)

Pending members can send message

Reported by: jeanyves Owned by:
Priority: critical Milestone: Legacy Resolved
Component: BW Mail Keywords: contactmember message Pending
Cc:

Description

A not yet accepted can write a message to another member but this message is never sent because it is frozen.

The to do fix is that the member who wants to write the message should see a warning that he has to be accepted before.

Only members with members.Status='Active' should be able to send messages

Change History (9)

comment:1 Changed 9 years ago by jeanyves

  • follow_up changed from none to test

I setup a a fix on test, only Active member can contact another one, but even a pending member can reply to another message he has received (this could happen since very often the first people contacted are the one in the who is online window and they might be in Pending state)

comment:2 Changed 9 years ago by fake51

  • follow_up changed from test to test on alpha

I've improved the code and moved it to alpha for testing.

I'm not sure that the system will allow pending members to reply to messages - so the fix may need to be fixed.

comment:3 Changed 9 years ago by jeanyves

good Peter,

about reply: what I did is that I just blocked the composing on new message by not Active members (so reply is accessible for them), but you are right, it is not sufficient. Very probably the message will be frozen (same situation as the current one -without this fix- for new composed message).

Additional fix for the fix for reply case will be needed

comment:4 Changed 9 years ago by globetrotter_tt

I tested on alpha with user "testpending1327" and it's not possible to contact other memebers - so the fix worked. Could anybody send a message to "testpending1327" so that i can test if replying works.

comment:5 Changed 9 years ago by globetrotter_tt

  • follow_up changed from test on alpha to move to alpha

with JY's latest fix replying works on test now. move to alpha?

comment:6 Changed 9 years ago by fake51

  • follow_up changed from move to alpha to test on alpha

It's on alpha and part of the fix is on production. Should be checked on alpha, then moved to production

comment:7 Changed 9 years ago by globetrotter_tt

  • follow_up changed from test on alpha to release

seems there were no further problems on alpha -> move the rest to production

comment:8 Changed 9 years ago by globetrotter_tt

  • follow_up changed from release to none
  • Resolution set to fixed
  • Status changed from new to closed

comment:9 Changed 6 years ago by TimLoal

  • Milestone changed from unassigned to Legacy Resolved
Note: See TracTickets for help on using tickets.