14701: upload files - duplicate alert message - LimeSurvey bugs and feature requests

This issue affects 1 person(s).

ID	Project	Category	View Status	Date Submitted	Last Update
14701	Bug reports	Survey taking	public	2019-03-28 14:28	2019-05-29 16:39

Reporter	ritapas	Assigned To	DenisChenu
Priority	low	Severity	tweak
Status	closed	Resolution	fixed
Product Version	3.15.x

Summary	14701: upload files - duplicate alert message
Description	When you upload a file to a "file upload" question which sets a maximum of 1 file to be uploaded, you get the same message twice, in the <p class="alert alert-success"><span class="fa fa-check"></span> The maximum number of files has been uploaded. You may return back to survey. and in the <div class="uploadstatus" id="uploadstatus">The maximum number of files has been uploaded. You may return back to survey.</div>
Steps To Reproduce	Create a "file upload" question which sets a maximum of 1 file to be uploaded, execute or preview the survey and upload a file to it.
Tags	No tags attached.
Attached Files

Bug heat	8
Complete LimeSurvey version number (& build)	Version 3.15.5+181115
I will donate to the project if issue is resolved	No
Browser
Database type & version	not relevant
Server OS (if known)
Webserver software & version (if known)
PHP Version	not relevant

User List	There are no users monitoring this issue.

cdorin 2019-03-29 16:52 reporter ~51204	Hmm, if (filecount >= maxfiles) to be changed to if (filecount > maxfiles) in uploader.js?

DenisChenu 2019-03-29 18:31 developer ~51208	I can't really reproduce on last GIT (but no See screenrecord I test with 2 files : «The maximum etc … » was shown : seems OK too simplescreenrecorder-2019-03-29_18.27.19.webm (1,130,408 bytes) simplescreenrecorder-2019-03-29_18.27.19.webm (1,130,408 bytes)

DenisChenu 2019-03-30 20:07 developer ~51226	Can not reproduce

ritapas 2019-04-01 09:51 reporter ~51232	Hello Denis, you actually did that. Please see the enclosed picture: the same message appears twice. duplicateMessage.PNG (179,107 bytes)

DenisChenu 2019-04-01 09:54 developer ~51233 Last edited: 2019-04-01 09:55	Oh yes , OK :) since the second one are just here … didn't see it ;)

DenisChenu 2019-05-07 17:40 developer ~51814	https://github.com/LimeSurvey/LimeSurvey/commit/4d1f42b5280717f1130942a73abc66f7c3663d8b https://github.com/LimeSurvey/LimeSurvey/commit/62fa3a075a322c4f6e5387469978b651f8157e8e

DenisChenu 2019-05-29 16:12 developer ~52190	Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=28918

c_schmitz 2019-05-29 16:39 administrator ~52219	Version 3.17.4+190529 released

Date Modified	Username	Field	Change
2019-03-28 14:28	ritapas	New Issue
2019-03-29 16:52	cdorin	Priority	none => low
2019-03-29 16:52	cdorin	Status	new => confirmed
2019-03-29 16:52	cdorin	Description Updated
2019-03-29 16:52	cdorin	Note Added: 51204
2019-03-29 17:46	DenisChenu	Assigned To	=> DenisChenu
2019-03-29 17:46	DenisChenu	Status	confirmed => assigned
2019-03-29 18:31	DenisChenu	File Added: simplescreenrecorder-2019-03-29_18.27.19.webm
2019-03-29 18:31	DenisChenu	Note Added: 51208
2019-03-30 20:07	DenisChenu	Status	assigned => feedback
2019-03-30 20:07	DenisChenu	Note Added: 51226
2019-04-01 09:51	ritapas	File Added: duplicateMessage.PNG
2019-04-01 09:51	ritapas	Note Added: 51232
2019-04-01 09:51	ritapas	Status	feedback => assigned
2019-04-01 09:54	DenisChenu	Note Added: 51233
2019-04-01 09:55	DenisChenu	Note Edited: 51233
2019-05-07 17:40	DenisChenu	Status	assigned => resolved
2019-05-07 17:40	DenisChenu	Resolution	open => fixed
2019-05-07 17:40	DenisChenu	Note Added: 51814
2019-05-07 17:52	DenisChenu	Relationship added	related to 14858
2019-05-29 16:12	DenisChenu	Changeset attached	=> LimeSurvey master 4d1f42b5
2019-05-29 16:12	DenisChenu	Note Added: 52190
2019-05-29 16:39	c_schmitz	Note Added: 52219
2019-05-29 16:39	c_schmitz	Status	resolved => closed

LimeSurvey: master 4d1f42b5 2019-05-07 19:38 DenisChenu Details Diff	Fixed issue 14701: upload files - duplicate alert message		Affected Issues 14701
	mod - assets/scripts/uploader.js		Diff File