19063: import of broken lsa file yields success message (but inactive "Go to survey" button)

This bug affects 1 person(s).

ID	Project	Category	View Status	Date Submitted	Last Update

19063	Bug reports	_ Unknown	public	2023-08-30 12:37	2023-09-25 10:55

Reporter	ritapas	Assigned To	tibor.pacalat
Priority	none	Severity	tweak
Status	closed	Resolution	fixed
Product Version	5.6.x

Summary	19063: import of broken lsa file yields success message (but inactive "Go to survey" button)
Description	if you import a broken or empty lsa file, you get a success message (with no numbers for questions etc.)
Steps To Reproduce	Steps to reproduce create a survey by importing a broken lsa file (i enclose one for your convenience) Expected result a warning message Actual result you get a success message (with no numbers for questions etc.)
Tags	No tags attached.
Attached Files	image.png (22,030 bytes) image.png (22,030 bytes) buttami.lsa (5 bytes) hello buttami.lsa (5 bytes)

Bug heat	8
Complete LimeSurvey version number (& build)	5.6.33 (build 230808)
I will donate to the project if issue is resolved	No
Browser
Database type & version	n/a
Server OS (if known)
Webserver software & version (if known)
PHP Version	php-fpm-7.4.33-1

User List	There are no users monitoring this issue.

gabrieljenik 2023-09-18 14:36 manager ~77129	Master: https://github.com/LimeSurvey/LimeSurvey/pull/3461

gabrieljenik 2023-09-18 16:31 manager ~77139	v5: https://github.com/LimeSurvey/LimeSurvey/pull/3463

tibor.pacalat 2023-09-22 17:41 administrator ~77274	Tested and merged.

guest 2023-09-22 17:55 viewer ~77284	Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35540

LimeBot 2023-09-25 10:55 administrator ~77299	Fixed in Release 6.2.9+230925

LimeSurvey: master d14242ff 2023-09-22 19:40 Gabriel Jenik Committer: GitHub Details Diff	Fixed issue 19063: import of broken lsa file yields success message (but inactive "Go to survey" button) (03461) Co-authored-by: Lapiu Dev <devgit@lapiu.biz>	Affected Issues 19063
	mod - application/helpers/admin/import_helper.php	Diff File
LimeSurvey: 5.x 1b5a1c04 2023-09-22 19:40 Gabriel Jenik Committer: GitHub Details Diff	Fixed issue 19063: import of broken lsa file yields success message (but inactive "Go to survey" button) (#3463) Co-authored-by: Lapiu Dev <devgit@lapiu.biz>	Affected Issues 19063
	mod - application/helpers/admin/import_helper.php	Diff File

Date Modified	Username	Field	Change
2023-08-30 12:37	ritapas	New Issue
2023-08-30 12:37	ritapas	File Added: image.png
2023-08-30 12:37	ritapas	File Added: buttami.lsa
2023-09-14 14:14	gabrieljenik	Assigned To	=> gabrieljenik
2023-09-14 14:14	gabrieljenik	Status	new => assigned
2023-09-18 14:36	gabrieljenik	Assigned To	gabrieljenik => DenisChenu
2023-09-18 14:36	gabrieljenik	Status	assigned => ready for code review
2023-09-18 14:36	gabrieljenik	Note Added: 77129
2023-09-18 14:36	gabrieljenik	Bug heat	0 => 2
2023-09-18 16:31	gabrieljenik	Note Added: 77139
2023-09-19 09:23	DenisChenu	Assigned To	DenisChenu => gabrieljenik
2023-09-19 09:23	DenisChenu	Status	ready for code review => ready for testing
2023-09-21 23:04	gabrieljenik	Assigned To	gabrieljenik => tibor.pacalat
2023-09-22 17:41	tibor.pacalat	Status	ready for testing => resolved
2023-09-22 17:41	tibor.pacalat	Resolution	open => fixed
2023-09-22 17:41	tibor.pacalat	Note Added: 77274
2023-09-22 17:41	tibor.pacalat	Bug heat	2 => 4
2023-09-22 17:55		Changeset attached	=> LimeSurvey 5.x 1b5a1c04
2023-09-22 17:55	guest	Note Added: 77284
2023-09-22 17:55	guest	Bug heat	4 => 6
2023-09-22 18:24		Changeset attached	=> LimeSurvey master d14242ff
2023-09-25 10:55	LimeBot	Note Added: 77299
2023-09-25 10:55	LimeBot	Status	resolved => closed
2023-09-25 10:55	LimeBot	Bug heat	6 => 8