Moodle
  1. Moodle
  2. MDL-28568

warning needed when sending message to non-contact but have 'do not receive messages from non-contacts' setting on.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.2.5, 2.3.2
    • Component/s: Messages
    • Labels:
    • Testing Instructions:
      Hide

      For this test you'll need two users, user A and user B.

      1) User A should have 'Prevent non-contacts from messaging me' ticked in their messaging preferences.
      2) Go to user A's messages.
      3) If user B is a contact just click on their name to display their message history. If user B is not a contact, search for them and add them as a contact.
      4) There should be no warning above the new message box.
      5) Remove user B as a contact.
      6) You should now get a warning above the new message box telling you that the other user will not be able to reply.

      7) Log in as user B.
      8) View the message history with user A.
      9) You should get a warning saying that you cannot message user A and no new message box.

      Show
      For this test you'll need two users, user A and user B. 1) User A should have 'Prevent non-contacts from messaging me' ticked in their messaging preferences. 2) Go to user A's messages. 3) If user B is a contact just click on their name to display their message history. If user B is not a contact, search for them and add them as a contact. 4) There should be no warning above the new message box. 5) Remove user B as a contact. 6) You should now get a warning above the new message box telling you that the other user will not be able to reply. 7) Log in as user B. 8) View the message history with user A. 9) You should get a warning saying that you cannot message user A and no new message box.
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-28568_noncontact_warning
    • Rank:
      18273

      Description

      on moodle.org , i've received a msg from someone who obviously isn't very sure of whats going on in moodle.

      He seems to have set his settings to no receive messages from non contacts.
      He in turn has also sent me some messages a few times.

      There should be a warning that explains there is no way for the messages recipient to reply. also how to turn it off if needed.

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

          A message such as...

          '''Recipient unable to reply'''
          The recipient of this message will not be able to reply because you have blocked messages from users who are not in your contact list".

          ...or perhaps just "You are a numbskull"

          Show
          Michael de Raadt added a comment - A message such as... '''Recipient unable to reply''' The recipient of this message will not be able to reply because you have blocked messages from users who are not in your contact list". ...or perhaps just "You are a numbskull"
          Hide
          Aparup Banerjee added a comment -

          lol, something along those lines Michael!

          but we should never allow this state - it can be so confusing to a user.

          Or perhaps we should simply deny (with a kinder 'numbskull' message) sending of message if any side (from or to ) has the other blocked.

          Show
          Aparup Banerjee added a comment - lol, something along those lines Michael! but we should never allow this state - it can be so confusing to a user. Or perhaps we should simply deny (with a kinder 'numbskull' message) sending of message if any side (from or to ) has the other blocked.
          Hide
          Andrew Davis added a comment -

          Adding a branch with a possible solution. It also includes a little bit of an enhancement to another warning the message page can display.

          Show
          Andrew Davis added a comment - Adding a branch with a possible solution. It also includes a little bit of an enhancement to another warning the message page can display.
          Hide
          Andrew Davis added a comment -

          Adding testing instructions and putting this up for peer review.

          Show
          Andrew Davis added a comment - Adding testing instructions and putting this up for peer review.
          Hide
          Ankit Agarwal added a comment -

          Hi Andrew,
          The patch looks good. I have a few small queries:-

          1. fullname($user2), don't we need to make sure if the user has permission to view full names before doing that?
          2. array('id'=>'messagewarning') should be array('id' => 'messagewarning') (spaces)
          3. I am not sure but is $user1blocknoncontacts, okay as per our naming guidelines?
          4. you missed a "y" in "they" in the comment 'Otherwise they may wonder why the never get a reply'

          Rest looks great.
          Thanks

          Show
          Ankit Agarwal added a comment - Hi Andrew, The patch looks good. I have a few small queries:- fullname($user2), don't we need to make sure if the user has permission to view full names before doing that? array('id'=>'messagewarning') should be array('id' => 'messagewarning') (spaces) I am not sure but is $user1blocknoncontacts, okay as per our naming guidelines? you missed a "y" in "they" in the comment 'Otherwise they may wonder why the never get a reply' Rest looks great. Thanks
          Hide
          Andrew Davis added a comment -

          Ive fixed all of that with the exception of #1. Ive raised MDL-34406 to look into that. We use fullname(), possibly incorrectly, in a few places around the messaging system.

          Show
          Andrew Davis added a comment - Ive fixed all of that with the exception of #1. Ive raised MDL-34406 to look into that. We use fullname(), possibly incorrectly, in a few places around the messaging system.
          Hide
          Andrew Davis added a comment -

          Adding branches. I've put that new MDL in the next sprint. Putting this up for peer review again.

          Show
          Andrew Davis added a comment - Adding branches. I've put that new MDL in the next sprint. Putting this up for peer review again.
          Hide
          Ankit Agarwal added a comment -

          Hi Andrew,
          Looks good to me.
          Thanks

          Show
          Ankit Agarwal added a comment - Hi Andrew, Looks good to me. Thanks
          Hide
          Andrew Davis added a comment -

          Submitting for integration.

          Show
          Andrew Davis added a comment - Submitting for integration.
          Hide
          Aparup Banerjee added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Aparup Banerjee added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Sam Hemelryk added a comment -

          Thanks Andrew, all looked spot on and has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Andrew, all looked spot on and has been integrated now
          Hide
          Rajesh Taneja added a comment -

          Works Great,

          Thanks for fixing this Andrew.

          Show
          Rajesh Taneja added a comment - Works Great, Thanks for fixing this Andrew.
          Hide
          Dan Poltawski added a comment -

          asko, Дзякуй, ধন্যবাদ, Благодаря, Gràcies, 感谢, 謝謝, Hvala, Díky, Tak, Bedankt, Tänan, متشکریم, Salamat, Kiitokset, Merci, Grazas, Danke, Ευχαριστώ, આભાર, תודה, धन्यवाद, Köszönjük, Takk fyrir, Terima Kasih, Grazie, ありがとうございます, Рахмет, សូមអរគុណ, 감사합니다, gratiās, Pateicamies, Ačiū, Благодарам, Tēnā koa, Kia Ora Rawa Atu, आभारी आहोत, Талархал, Takk, Dziękuję, Obrigado, Mulţumesc, Engraziel, Спасибо, Fa'afetai, Хвала, Hvala, ස්තූතියි, Vďaka, Hvala, Mahadsanid, Thanks, Gracias, Tack, Salamat, நன்றி, నెనరులు, ขอบคุณค่ะ!

          Your work has made it into this weeks Moodle release! There are no gold medals available this week - but millions around the world will benefit. Thank you!

          Show
          Dan Poltawski added a comment - asko, Дзякуй, ধন্যবাদ, Благодаря, Gràcies, 感谢, 謝謝, Hvala, Díky, Tak, Bedankt, Tänan, متشکریم, Salamat, Kiitokset, Merci, Grazas, Danke, Ευχαριστώ, આભાર, תודה, धन्यवाद, Köszönjük, Takk fyrir, Terima Kasih, Grazie, ありがとうございます, Рахмет, សូមអរគុណ, 감사합니다, gratiās, Pateicamies, Ačiū, Благодарам, Tēnā koa, Kia Ora Rawa Atu, आभारी आहोत, Талархал, Takk, Dziękuję, Obrigado, Mulţumesc, Engraziel, Спасибо, Fa'afetai, Хвала, Hvala, ස්තූතියි, Vďaka, Hvala, Mahadsanid, Thanks, Gracias, Tack, Salamat, நன்றி, నెనరులు, ขอบคุณค่ะ! Your work has made it into this weeks Moodle release! There are no gold medals available this week - but millions around the world will benefit. Thank you!

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: