View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
18509Bug reportsImport/Exportpublic2022-11-23 20:452022-12-08 09:22
Reportergabrieljenik Assigned Toc_schmitz  
PrioritynoneSeveritypartial_block 
Status feedbackResolutionopen 
Product Version3.28.x 
Summary18509: Permissions are gone after survey export/import
Description

When importing/exporting a survey permissions are not ported.

  • User Group Permissions
  • UserName Permissions

Caught while reviewing 18460

Steps To Reproduce

Steps to reproduce

Assign a survey to a certain survey group.
Export survey as LSS file.
Import LSS file.
Check permissions

Expected result

Imported survey should still have same permissions if exists.
(If not exists, show a warning similar to how it is done for missing theme)

Actual result

Permissions are missing.

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)3.28.32
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMariaDB 10.1.48
Server OS (if known)
Webserver software & version (if known)
PHP VersionPHP 7.4

Relationships

Relationship GraphDependency Graph
related to 18460 assignedgabrieljenik Feature requests Assigned survey group is gone after survey export/import 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2022-11-24 12:09

developer   ~72885

Expected result

Imported survey should still have same permissions if exists AND USER WANT IT
If not exists, show a warning similar to how it is done for missing theme : ONLY IF USER WANT TO IMPORT PERMISSION
DenisChenu

DenisChenu

2022-11-25 10:12

developer   ~72890

Need same than SurveyGroup : don't add permission by default.

Same example than here : https://github.com/LimeSurvey/LimeSurvey/pull/2745#issuecomment-1327189281
gabrieljenik

gabrieljenik

2022-11-25 15:00

manager   ~72898

Carsten will review this as he is not that convinced about it
DenisChenu

DenisChenu

2022-11-25 15:15

developer   ~72901

Its different here: it's directly the Permission

@c_schmitz ?
DenisChenu

DenisChenu

2022-12-08 09:22

developer   ~73025

See related OLD commit when adding the option when copy survey
https://github.com/SondagesPro/LimeSurvey-SondagesPro/blob/3394d3d84ae811567e33e2fdafb2c02002290dae/docs/release_notes.txt#L808

+New feature: Permissions are now copied by default when copying a survey - also added an additional option to exclude it. (Carsten Schmitz)

The option to exclude import a settings must exist in my opinion.

Issue History

Date Modified Username Field Change
2022-11-23 20:45 gabrieljenik New Issue
2022-11-23 20:45 gabrieljenik Issue generated from: 18460
2022-11-23 20:45 gabrieljenik Relationship added related to 18460
2022-11-23 20:45 gabrieljenik Status new => confirmed
2022-11-24 12:09 DenisChenu Note Added: 72885
2022-11-24 12:09 DenisChenu Bug heat 0 => 2
2022-11-25 10:12 DenisChenu Note Added: 72890
2022-11-25 15:00 gabrieljenik Note Added: 72898
2022-11-25 15:00 gabrieljenik Bug heat 2 => 4
2022-11-25 15:15 DenisChenu Note Added: 72901
2022-12-07 19:42 gabrieljenik Assigned To => c_schmitz
2022-12-07 19:42 gabrieljenik Status confirmed => feedback
2022-12-08 09:22 DenisChenu Note Added: 73025