[MDL-65464] Quick reply allows multiple posts - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 3.7
Fix Version/s: 3.7
Component/s: Forum
Labels:
- ci

Affected Branches:
MOODLE_37_STABLE
Fixed Branches:
MOODLE_37_STABLE
Epic Link:
Forum facelift
Pull from Repository:
https://github.com/ryanwyllie/moodle.git
Pull Master Branch:
~~MDL-65464~~-master
Pull Master Diff URL:
https://github.com/moodle/moodle/compare/master...ryanwyllie:MDL-65464-master
Testing Instructions:
Hide

Log in as admin

Create a site with a course, a forum, and a discussion in the forum

View the discussion

Click "Reply" on one of the posts in the discussion to open the in page reply form

Turn on the network connection throttling in your browser. The slower the network the better.

This is how you do it in Chrome

Type in a message to post

Click the "Submit" button multiple times

CONFRIM your post is only added to the page once

Reload the page

CONFIRM that your post is still only shown once
Show
Log in as admin Create a site with a course, a forum, and a discussion in the forum View the discussion Click "Reply" on one of the posts in the discussion to open the in page reply form Turn on the network connection throttling in your browser. The slower the network the better. This is how you do it in Chrome Type in a message to post Click the "Submit" button multiple times CONFRIM your post is only added to the page once Reload the page CONFIRM that your post is still only shown once

Description

On a slow connection it is easily possible to click Submit button multiple times and generate a slew of extra posts.

See attached video.

Attachments

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

image-2019-05-06-17-14-29-110.png
131 kB
06/May/19 5:14 PM
multiple-replies.mov
11.25 MB
02/May/19 8:32 AM

Activity

People

Assignee:: Ryan Wyllie

Reporter:: Andrew Lyons

Peer reviewer:: Peter Dias

Integrator:: David Monllaó

Tester:: Janelle Barcega

Participants:: Andrew Lyons, CiBoT, David Monllaó, Janelle Barcega, Peter Dias, Ryan Wyllie

Component watchers:: Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie

Votes:: 0 Vote for this issue

Watchers:: 5 Start watching this issue

Dates

Created:: 02/May/19 8:31 AM

Updated:: 09/May/19 12:13 AM

Resolved:: 09/May/19 12:13 AM

Fix Release Date:: 20/May/19

Time Tracking

Estimated:

Not Specified

Remaining:

0m

Logged:

2h 10m