 |
Do you have the same thing with VV export/import ? |
 |
I hadn't tried it yet, I don't have this error with VV export/import. I still get it just by importing the entire questionnaire with the answers with the .lsa file |
|
|
news? |
|
|
@gabrieljenik I exported and imported the survey and then looked at the answers. They are indeed missing single quote. Is this maybe because we are sanitizing those strings somehow? Screenshot 2024-11-13 at 17.54.26.png (44,018 bytes) |
 |
I will review and be back. |
|
|
https://github.com/LimeSurvey/LimeSurvey/pull/4056 SurveyDynamic's insertRecords method was being used, which effectively has a sanitization, but it's not clear to me why... Didn't dare to take it out. |
|
|
@gabrieljenik some tests are failing on your branch, can you please take a look at that? |
|
|
The automatic tests for this ticket were failing not because we introduced an error but because the fix impacted the import process (and fixed it). The failing test was trying to import a survey for reviewing stats. Seems there was some manual editing on the file because in the other responses it has missing nodes (as if they hadn't been exported because they were null) and in the last one it has all the nodes empty, as if they had data and had been cleaned to simulate that they weren't answered. We created a new branch based on master with an automatic test to show the issue: That test is now green on this PR. PS: There is a red on the PR automatic test because of an issue on master regarding the versioning. |
|
|
There are a lot of test to fix |
|
|
PR for fixing tests: |
|
|
Fixed in Release 6.9.0+241218 |
- Anonymous
