View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
09498Bug reportsImport/Exportpublic2016-06-06 15:16
ReporterDenisChenu Assigned Toc_schmitz  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.05+ 
Target Version2.50.xFixed in Version2.50.x 
Summary09498: Error message when try to import a big LSA file can be an error
Description

When try to import a lsa file : we have the error : 'Please verify rigths on tmp'. But this can be OK but file can not be uploaded due to upload_max_filesize

Steps To Reproduce

Find a big LSA file (more than 2/4 mo) , try to import

Additional Information

2 solution : just add 'The file can be bigger than upload_max_filesize' Or do some test ?

(I just have this problem and search , search , search before thinking of upload max file size ;) )

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)150206
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionMysql 5
Server OS (if known)debian/linux
Webserver software & version (if known)apache
PHP VersionPHP Version 5.4.4

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2015-02-27 14:56

administrator   ~31730

Or you could just use getMaximumFileUploadSize() from common_helper ;)

DenisChenu

DenisChenu

2015-02-27 16:00

developer   ~31732

Yes, it's same :

Test if file size is biger : show the real error.
Don't test and show : maybe it's a problem with tmp, maybe it's a problem with file size ;).

2nd solution is really quickest ;)

c_schmitz

c_schmitz

2015-03-05 11:30

administrator   ~31767

Last edited: 2015-03-05 11:30

We should show an information about the maximum file size beside the upload function and adjust the error message to account for both cases.

c_schmitz

c_schmitz

2015-03-05 11:37

administrator   ~31768

Btw, you can't test the file size if you cannot upload it ;)

DenisChenu

DenisChenu

2015-03-05 16:23

developer   ~31775

Last edited: 2015-03-06 09:54

Yes, then only one solution :
Return : "Unable to upload the file : validate if tmp/upload is writeabla or if filesize is less than ..."

DenisChenu

DenisChenu

2015-05-11 03:37

developer   ~32156

Returned error is clear enough : http://php.net/manual/en/features.file-upload.errors.php

But surely better with http://www.yiiframework.com/doc/api/1.1/CUploadedFile and maybe a "settings" for ls3

DenisChenu

DenisChenu

2015-06-16 14:54

developer   ~32382

Have to create a new extension for better reusing : then for LS3.

c_schmitz

c_schmitz

2016-06-05 19:36

administrator   ~39230

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=19333

c_schmitz

c_schmitz

2016-06-05 19:53

administrator   ~39231

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=19334

c_schmitz

c_schmitz

2016-06-06 14:01

administrator   ~39237

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=19341

c_schmitz

c_schmitz

2016-06-06 15:16

administrator   ~39241

Version 2.50 Release 160606 released

Related Changesets

LimeSurvey: master af24ba70

2016-06-05 17:36:15

c_schmitz

Details Diff
Fixed issue 09498: Error message when try to import a big LSA file can be an error
Dev Partial fix - show max upload size in UI
Affected Issues
09498
mod - application/views/admin/labels/editlabel_view.php Diff File
mod - application/views/admin/survey/Question/importQuestion_view.php Diff File
mod - application/views/admin/survey/QuestionGroups/importGroup_view.php Diff File
mod - application/views/admin/survey/subview/tabImport_view.php Diff File
mod - application/views/admin/templates/templateeditorbar_view.php Diff File
mod - application/views/admin/templates/templatesummary_view.php Diff File

LimeSurvey: master 027e139e

2016-06-05 17:53:26

c_schmitz

Details Diff
Fixed issue 09498: Error message when try to import a big LSA file can be an error
Dev Partial fix - Proper error message when uploading a label set, that is too big.
Affected Issues
09498
mod - application/controllers/admin/labels.php Diff File

LimeSurvey: master 99f1262f

2016-06-06 09:36:14

c_schmitz

Details Diff
Fixed issue 09498: Wrong error message when trying to import a file that is too big Affected Issues
09498
mod - application/controllers/admin/participantsaction.php Diff File
mod - application/controllers/admin/questiongroups.php Diff File
mod - application/controllers/admin/questions.php Diff File
mod - application/controllers/admin/surveyadmin.php Diff File
mod - application/controllers/admin/tokens.php Diff File
mod - application/views/admin/participants/importCSV_view.php Diff File
mod - application/views/admin/survey/Question/importQuestion_view.php Diff File
mod - application/views/admin/survey/QuestionGroups/importGroup_view.php Diff File
mod - application/views/admin/survey/subview/tabImport_view.php Diff File
mod - application/views/admin/survey/surveybar_view.php Diff File

Issue History

Date Modified Username Field Change
2015-02-06 14:46 DenisChenu New Issue
2015-02-27 14:56 c_schmitz Note Added: 31730
2015-02-27 14:56 c_schmitz Assigned To => DenisChenu
2015-02-27 14:56 c_schmitz Status new => assigned
2015-02-27 16:00 DenisChenu Note Added: 31732
2015-03-05 11:30 c_schmitz Note Added: 31767
2015-03-05 11:30 c_schmitz Note Edited: 31767
2015-03-05 11:30 c_schmitz Assigned To DenisChenu => c_schmitz
2015-03-05 11:37 c_schmitz Note Added: 31768
2015-03-05 16:23 DenisChenu Note Added: 31775
2015-03-06 09:54 c_schmitz Note Edited: 31775
2015-05-11 03:35 DenisChenu Assigned To c_schmitz => DenisChenu
2015-05-11 03:37 DenisChenu Note Added: 32156
2015-06-16 14:54 DenisChenu Note Added: 32382
2015-06-16 14:54 DenisChenu Target Version => 3.0
2016-05-31 10:33 DenisChenu Assigned To DenisChenu =>
2016-06-03 13:25 LouisGac Sticky Issue No => Yes
2016-06-03 13:26 LouisGac Sticky Issue Yes => No
2016-06-04 14:22 DenisChenu Status assigned => new
2016-06-05 19:30 c_schmitz Target Version 3.0 => 2.5
2016-06-05 19:36 c_schmitz Changeset attached => LimeSurvey master af24ba70
2016-06-05 19:36 c_schmitz Note Added: 39230
2016-06-05 19:36 c_schmitz Assigned To => c_schmitz
2016-06-05 19:36 c_schmitz Resolution open => fixed
2016-06-05 19:53 c_schmitz Changeset attached => LimeSurvey master 027e139e
2016-06-05 19:53 c_schmitz Note Added: 39231
2016-06-06 14:01 c_schmitz Changeset attached => LimeSurvey master 99f1262f
2016-06-06 14:01 c_schmitz Note Added: 39237
2016-06-06 14:39 c_schmitz Status new => resolved
2016-06-06 14:39 c_schmitz Fixed in Version => 2.5
2016-06-06 15:16 c_schmitz Note Added: 39241
2016-06-06 15:16 c_schmitz Status resolved => closed