Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.2.1
-
Component/s: Messages
-
Labels:
-
Testing Instructions:
For this test you will need 2 users and 2 browsers (firefox and chrome for example)
- log in as a user (message recipient) and check that your messaging preferences are "popup" for personal messages while both online and offline.
- go to any messaging screen. Viewing a conversation, doing a message search etc.
- In your second browser log in as a 2nd user. Send the first user a message.
- As the first user refresh the messaging screen. No popup should appear.
- Still as the message recipient go to another page, /my for example. A popup should now appear in the bottom right of the screen.
Note that message popups will not be shown if a popup was shown in the past 2 minutes. If you get a message popup unrelated to this test or similar then go and get a cup of tea and start over
For this test you will need 2 users and 2 browsers (firefox and chrome for example) log in as a user (message recipient) and check that your messaging preferences are "popup" for personal messages while both online and offline. go to any messaging screen. Viewing a conversation, doing a message search etc. In your second browser log in as a 2nd user. Send the first user a message. As the first user refresh the messaging screen. No popup should appear. Still as the message recipient go to another page, /my for example. A popup should now appear in the bottom right of the screen. Note that message popups will not be shown if a popup was shown in the past 2 minutes. If you get a message popup unrelated to this test or similar then go and get a cup of tea and start over -
Affected Branches:MOODLE_22_STABLE
-
Fixed Branches:MOODLE_21_STABLE, MOODLE_22_STABLE
-
Pull from Repository:
-
Pull Master Branch:
MDL-31432_messaging_popup -
Pull Master Diff URL:
Description
When a message is sent to me through Moodle I receive an email (while offline) containing a link to take me to the message, so I can respond. When I click the link I am taken to the messaging page, but I am also presented with a pop-up message telling me about the message I am currently viewing. Apart from being annoying it is also confusing as there are two options on the pop-up: Go to messaging (I'm already there) or ignore. This pop-up message should not appear.
Gliffy Diagrams
Attachments
Issue Links
- has been marked as being related by
-
-
- Closed
-
Activity
- All
- Comments
- Work Log
- History
- Activity
- Links Hierarchy
I think that this is probably the simplest solution to the problem. Although it is worth noting that other popup notifications could be received other than notification of messages (e.g. grades).
I would probably put a comment explaining why you are disabling the notifications as it may not be clear to others.
Im not sure what popup notifications you mean. The get/set_popup_notification_allowed() pair only govern message popups although notifications of assignment submissions, assignment grading etc are all "messages".
I've added a comment explaining why we're turning off popups on the messaging screen.
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
Hi Andrew,
What branches does this need to end up on?
Cheers
Sam
Oops. It should go into master, 2.2 and 2.1. creating branches now.
Adding branches
Thanks Andrew this has been integrated now
This works as expected!
Passing!
Thanks
Some changes to Moodle should be milestones in the project by themselves.
This is not the case and your fix is not so important, but your collaboration is highly appreciated, thanks!
Closing as fixed, ciao
Ive added a potential fix. This just prevents any messaging popups while the user is within the messaging system. The popups are actually generated by lib/moodlelib.php ie not the messaging system itself so I decided against more targeted popup prevention as that would require rummaging around in the message system to compare message IDs. Plus I think it makes sense to simply prevent them. The messaging pages display a count of new messages you've received so further prompting probably isnt necessary.