Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-28340

Advanced message search issue

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.0.3
          • Fix Version/s: 2.0.5, 2.1.2
          • Component/s: Messages
          • Labels:
            None
          • Environment:
            Linux debian Lenny
          • Database:
            Any
          • Testing Instructions:
            Hide

            To test you will need 3 users. Have user A message user B and user B message user C. Include something notable in the messages. Something that you can search for that will return both messages.

            Log in as user A. User A will need the capability to view other's messages. Logging in as admin is easiest.

            Go to the messaging screen and click "advanced". Select the "All messages from all users" option, enter the something notable from the messages you sent above. When results are displayed, click on context link of the message between your current user and user B. This should take you to a messaging screen that displays the current user on the left with the other user on the right.

            Click back to go back to your search. Click on the context link for the message between the other 2 users. This should take you to a messaging screen displaying any messages between the other 2 users.

            Show
            To test you will need 3 users. Have user A message user B and user B message user C. Include something notable in the messages. Something that you can search for that will return both messages. Log in as user A. User A will need the capability to view other's messages. Logging in as admin is easiest. Go to the messaging screen and click "advanced". Select the "All messages from all users" option, enter the something notable from the messages you sent above. When results are displayed, click on context link of the message between your current user and user B. This should take you to a messaging screen that displays the current user on the left with the other user on the right. Click back to go back to your search. Click on the context link for the message between the other 2 users. This should take you to a messaging screen displaying any messages between the other 2 users.
          • Affected Branches:
            MOODLE_20_STABLE
          • Fixed Branches:
            MOODLE_20_STABLE, MOODLE_21_STABLE
          • Pull from Repository:
          • Pull Master Branch:
            MDL-28340_advanced_search_other_users
          • Pull Master Diff URL:

            Description

            When searching all users as admin, you cannot link to message text unless you are a participant in the conversation.

              Gliffy Diagrams

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  andyjdavis Andrew Davis added a comment -

                  Adding master branch info. Will create other branches and cherry pick after peer review.

                  Show
                  andyjdavis Andrew Davis added a comment - Adding master branch info. Will create other branches and cherry pick after peer review.
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Looks good to me and works perfectly thanks Andrew

                  Show
                  samhemelryk Sam Hemelryk added a comment - Looks good to me and works perfectly thanks Andrew
                  Hide
                  Permalink
                  andyjdavis Andrew Davis added a comment -

                  Created branches. Submitting for integration.

                  Show
                  andyjdavis Andrew Davis added a comment - Created branches. Submitting for integration.
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Thanks this has been integrated now

                  Show
                  samhemelryk Sam Hemelryk added a comment - Thanks this has been integrated now
                  Hide
                  Permalink
                  nebgor Aparup Banerjee added a comment -

                  thanks Andrew, thats tested fine for me. I can now see the other users messaging screen all good.

                  Show
                  nebgor Aparup Banerjee added a comment - thanks Andrew, thats tested fine for me. I can now see the other users messaging screen all good.

                    People

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

                      Dates

                      • Created:
                        Updated:
                        Resolved:
                        Fix Release Date:
                        10/Oct/11