Uploaded image for project: '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
          • Status: Closed
          • Priority: 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

                Attachments

                  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
                    salvetore 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
                    salvetore 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
                    nebgor 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
                    nebgor 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
                    andyjdavis 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
                    andyjdavis 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
                    andyjdavis Andrew Davis added a comment -

                    Adding testing instructions and putting this up for peer review.

                    Show
                    andyjdavis Andrew Davis added a comment - Adding testing instructions and putting this up for peer review.
                    Hide
                    Permalink
                    ankit_frenz 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_frenz 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
                    andyjdavis 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
                    andyjdavis 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
                    andyjdavis 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
                    andyjdavis 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_frenz Ankit Agarwal added a comment -

                    Hi Andrew,
                    Looks good to me.
                    Thanks

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

                    Submitting for integration.

                    Show
                    andyjdavis Andrew Davis added a comment - Submitting for integration.
                    Hide
                    Permalink
                    nebgor 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
                    nebgor 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
                    samhemelryk Sam Hemelryk added a comment -

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

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

                    Works Great,

                    Thanks for fixing this Andrew.

                    Show
                    rajeshtaneja Rajesh Taneja added a comment - Works Great, Thanks for fixing this Andrew.
                    Hide
                    Permalink
                    poltawski 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
                    poltawski 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:
                          Fix Release Date:
                          10/Sep/12