View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
19274 | Bug reports | Central participant database | public | 2023-11-23 13:07 | 2024-11-19 18:14 |
Reporter | Mazi | Assigned To | tibor.pacalat | ||
Priority | none | Severity | block | ||
Status | closed | Resolution | fixed | ||
Product Version | 5.6.x | ||||
Summary | 19274: Attribute encryption can not be turned OFF, it can only be turned on | ||||
Description | At the Attribute overview, the status at column "encrypted" can only be switched to ON. The change gets saved on click (see 18799 for report UI issue). | ||||
Steps To Reproduce | Steps to reproduceGo to the attribute page at the CPDB. Expected resultStatus OFF needs to be saved. Actual resultOnly status ON gets saved properly. | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 12 | ||||
Complete LimeSurvey version number (& build) | Version 5.6.46+231121 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | MySQL | ||||
Server OS (if known) | Ubuntu 20 | ||||
Webserver software & version (if known) | Apache 2.0 | ||||
PHP Version | 8.1 | ||||
related to | 18926 | closed | gabrieljenik | CPDB participant Actions menu items don't work |
Couldn't reproduce it. When I set it to Yes, the existing data in the DB is saved and encrypted. When I set it to No, the existing data in the DB is saved and decrypted. |
|
The problem was caused by having copied CPDB tables directly from 3.x to 5.x. That caused DB updates to not be executed. Thanks @gabrieljenik for the great help. This ticket can be closed. |
|
I am not sure those are included in the update scrtips. Then, when asking to encrypt, the data was not encrypted but still the attribute marked as encrypted. Another problem, when failed, it failed silently, as the exception was not shown, neither frienly error informed. |
|
There are issues. Still, de-assigning myself from it as to allow priorization from @tibor.pacalat |
|
Master: https://github.com/LimeSurvey/LimeSurvey/pull/3670 During the update from 3 to 5, the “encrypted” and “core_attribute” fields of the existing attributes are not set. So in this part the variables do not have “N”, but ““ (see attachment) We updated the code so that it does not check “N” but !=”Y”. Maybe even add isEncrypted and isCoreAttribute methods to the model. |
|
@tibor.pacalat: This is very sensistive as it involves encryption. |
|
I'm OK with this solution. What for token and response ? We check != 'N' or == 'Y' ? |
|
Tested and merged. |
|
Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35969 |
|
Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35968 |
|
Fixed in Release 6.4.3+240122 |
|
Fixed in Release 5.6.52+240123 |
|
LimeSurvey: master 9d045ca3 2024-01-16 16:58 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 19274: Attribute encryption can not be turned OFF, it can only be turned on (#3670) Co-authored-by: lapiudevgit <devgit@lapiu.biz> |
Affected Issues 19274 |
|
mod - application/controllers/admin/ParticipantsAction.php | Diff File | ||
mod - application/models/ParticipantAttributeName.php | Diff File | ||
LimeSurvey: master 9d045ca3 2024-01-16 16:58 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 19274: Attribute encryption can not be turned OFF, it can only be turned on (#3670) Co-authored-by: lapiudevgit <devgit@lapiu.biz> |
Affected Issues 19274 |
|
mod - application/controllers/admin/ParticipantsAction.php | Diff File | ||
mod - application/models/ParticipantAttributeName.php | Diff File | ||
LimeSurvey: 5.x be215e4c 2024-01-16 17:06 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 19274: Attribute encryption can not be turned OFF, it can only be turned on (#3671) Co-authored-by: lapiudevgit <devgit@lapiu.biz> |
Affected Issues 19274 |
|
mod - application/controllers/admin/ParticipantsAction.php | Diff File | ||
mod - application/models/ParticipantAttributeName.php | Diff File | ||
LimeSurvey: 5.x be215e4c 2024-01-16 17:06 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 19274: Attribute encryption can not be turned OFF, it can only be turned on (#3671) Co-authored-by: lapiudevgit <devgit@lapiu.biz> |
Affected Issues 19274 |
|
mod - application/controllers/admin/ParticipantsAction.php | Diff File | ||
mod - application/models/ParticipantAttributeName.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2023-11-23 13:07 | Mazi | New Issue | |
2023-11-23 13:07 | Mazi | File Added: image.png | |
2023-11-27 23:34 | gabrieljenik | Assigned To | => gabrieljenik |
2023-11-27 23:34 | gabrieljenik | Status | new => assigned |
2023-11-29 14:57 | Mazi | Relationship added | related to 18926 |
2023-11-29 15:56 | gabrieljenik | Status | assigned => feedback |
2023-11-29 15:56 | gabrieljenik | Note Added: 78804 | |
2023-11-29 15:56 | gabrieljenik | Bug heat | 0 => 2 |
2023-11-29 15:56 | gabrieljenik | Note Added: 78805 | |
2023-11-29 15:56 | gabrieljenik | File Added: Peek_2023-11-29_08-21.gif | |
2023-11-29 15:56 | gabrieljenik | Note Edited: 78804 | |
2023-11-29 22:52 | Mazi | Note Added: 78813 | |
2023-11-29 22:52 | Mazi | Bug heat | 2 => 4 |
2023-11-29 22:52 | Mazi | Status | feedback => assigned |
2023-11-30 14:10 | gabrieljenik | Note Added: 78831 | |
2023-11-30 14:11 | gabrieljenik | Assigned To | gabrieljenik => |
2023-11-30 14:11 | gabrieljenik | Status | assigned => confirmed |
2023-11-30 14:11 | gabrieljenik | Note Added: 78832 | |
2023-12-18 16:48 | gabrieljenik | Note Added: 79033 | |
2023-12-18 16:48 | gabrieljenik | File Added: image-2.png | |
2023-12-18 16:48 | gabrieljenik | Assigned To | => DenisChenu |
2023-12-18 16:48 | gabrieljenik | Status | confirmed => ready for code review |
2023-12-18 16:49 | gabrieljenik | Note Added: 79034 | |
2023-12-18 17:33 | DenisChenu | Assigned To | DenisChenu => tibor.pacalat |
2023-12-18 17:33 | DenisChenu | Status | ready for code review => ready for testing |
2023-12-18 17:33 | DenisChenu | Note Added: 79036 | |
2023-12-18 17:33 | DenisChenu | Bug heat | 4 => 6 |
2024-01-16 16:09 | tibor.pacalat | Status | ready for testing => resolved |
2024-01-16 16:09 | tibor.pacalat | Resolution | open => fixed |
2024-01-16 16:09 | tibor.pacalat | Note Added: 79185 | |
2024-01-16 16:09 | tibor.pacalat | Bug heat | 6 => 8 |
2024-01-16 16:22 | Changeset attached | => LimeSurvey master 9d045ca3 | |
2024-01-16 16:22 | Changeset attached | => LimeSurvey 5.x be215e4c | |
2024-01-16 16:22 | guest | Note Added: 79186 | |
2024-01-16 16:22 | guest | Bug heat | 8 => 10 |
2024-01-16 16:22 | Changeset attached | => LimeSurvey master 9d045ca3 | |
2024-01-16 16:22 | Changeset attached | => LimeSurvey 5.x be215e4c | |
2024-01-16 16:22 | guest | Note Added: 79187 | |
2024-01-22 12:45 | LimeBot | Note Added: 79265 | |
2024-01-22 12:45 | guest | Bug heat | 10 => 12 |
2024-01-22 12:50 | LimeBot | Note Added: 79267 | |
2024-11-19 18:14 | c_schmitz | Status | resolved => closed |