View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
18265Bug reportsResponse browsingpublic2022-07-19 21:532022-07-20 19:07
Reportergabrieljenik Assigned To 
PrioritynoneSeverityminor 
Status confirmedResolutionopen 
Product Version3.28.x 
Summary18265: beforeDataEntryImport is not called consistently
Description

When importing responses from a table, the 'beforeDataEntryImport' event is executed before each response is saved, while when importing a CSV it is executed AFTER saving it.

Steps To Reproduce

Steps to reproduce

(Replace this text with detailed step-by-step instructions on how to reproduce the issue)

Expected result

(Write here what you expected to happen)

Actual result

(Write here what happened instead)

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)3.x
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMysql
Server OS (if known)
Webserver software & version (if known)
PHP Version7

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2022-07-20 11:38

developer   ~71050

  1. This event add more and more time when importing VV file …
  2. This event are not documented in https://manual.limesurvey.org/Plugin_events
  3. We already have https://manual.limesurvey.org/Plugin_events#Dynamic_model_events_.28New_in_3.15.1_.29

Then we have 2 events for the same action (with before)

Personally : i manually disable this one on a fork to get some microseconds on each line imported (whan you have 10 000 lines : it' really improve).

Can we delete it ?
gabrieljenik

gabrieljenik

2022-07-20 15:46

manager   ~71059

I wouldn't remove it. Backward compatibility on one side.
On the other, I think is good to have it something specific for import.
I would say need to be enhanced: Adding import type, row source, ...
DenisChenu

DenisChenu

2022-07-20 19:07

developer   ~71060

VV file (and SQL import) take a long long time when SQL server is on another machine.
…

Since it's not documented : it's not in API  …

Issue History

Date Modified Username Field Change
2022-07-19 21:53 gabrieljenik New Issue
2022-07-19 21:53 gabrieljenik Issue generated from: 18264
2022-07-19 21:53 gabrieljenik Status new => confirmed
2022-07-20 11:38 DenisChenu Note Added: 71050
2022-07-20 11:38 DenisChenu Bug heat 0 => 2
2022-07-20 15:46 gabrieljenik Note Added: 71059
2022-07-20 15:46 gabrieljenik Bug heat 2 => 4
2022-07-20 19:07 DenisChenu Note Added: 71060