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 from Repository:
        • Pull Master Branch:
          MDL-28568_noncontact_warning
        • Pull Master Diff URL:

          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.

            Gliffy Diagrams

              Issue Links

              Discovered while testing

              Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34406 Does messaging need to check a capbility before displaying user full name's?

              • Major - Major loss of function, incorrect output
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                Andrew Davis added a comment -

                Submitting for integration.

                Show
                Andrew Davis added a comment - Submitting for integration.
                Hide
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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: