|
|
| Reporter | annighoefer | Assigned To | p_teichmann | |
|---|
| Priority | high | Severity | partial_block | |
|---|
| Status | closed | Resolution | fixed | |
|---|
| Product Version | 4.4.7 | |
|---|
| Fixed in Version | 4.5.x | |
|---|
This bug affects 1 person(s).
6
|
|
| Summary | 17126: encrypted questions do not work when multiple groups are used |
|---|
| Description | Encrypted data of encrypted question in database are encrypted again, if participant switch between groups with "previous" and "next" button or submit the survey. After that, data can't be decrypted and isn't readable in the backend under Responses (Screenshot). |
|---|
| Steps To Reproduce |
- add multiple groups and questions into groups (at least 2 groups)
- add encrypted Question in first group
- set Format to "Question by Question" or "Group by Group" under General Settings
- activate survey
- go to the survey
- fill out the questions
- submit the survey
- go to "Responses" in the backend
- the encrypted questions show encrypted value, the field can no longer be decrypted
|
|---|
| Additional Information | Every time there is a change between the groups, the function encryptSave () is called with the current response object. But the response object contains encrypted attributes and with encryptSave these are encrypted again. My quick & dirty solution is before the encryptSave function is called, the encrypted attributes are decrypted. (see patch) |
|---|
| Tags | encryption |
|---|
| Attached Files |
em_manager_helper.patch (1,903 bytes)
diff --git a/application/helpers/expressions/em_manager_helper.php b/application/helpers/expressions/em_manager_helper.php
index 25391f7d10..938a904b9b 100644
--- a/application/helpers/expressions/em_manager_helper.php
+++ b/application/helpers/expressions/em_manager_helper.php
@@ -5171,7 +5171,28 @@ class LimeExpressionManager
return;
}
if ($oResponse->submitdate == null || Survey::model()->findByPk($this->sid)->alloweditaftercompletion == 'Y') {
+
+ // -------------- BUG CODE IMPLEMENTATION DECRYPT FIRST ENCRYPTED FIELDS -------
+
+ // get all encrypted attributes of current survey
+ $encryptedAttributes = Response::getEncryptedAttributes($this->sid);
+
+ if (count($encryptedAttributes) > 0) {
+ $decryptedAttributes = array();
+
+ foreach ($encryptedAttributes as $key => $value) {
+ if (isset($oResponse->$value) && $oResponse->$value) {
+ // decrypt all encrypted values of current survey
+ $decryptedAttributes[$value] = $oResponse->decrypt($oResponse->$value);
+ }
+ }
+
+ // set decypted values to response object
+ $oResponse->setAttributes($decryptedAttributes, false);
+ }
+
$oResponse->setAttributes($aResponseAttributes, false);
+
if (!$oResponse->encryptSave()) {
$message = submitfailed('', print_r($oResponse->getErrors(), true)); // $response->getErrors() is array[string[]], then can not join
if (($this->debugLevel & LEM_DEBUG_VALIDATION_SUMMARY) == LEM_DEBUG_VALIDATION_SUMMARY) {
|
|---|
|
|
| Bug heat | 6 |
|---|
| Complete LimeSurvey version number (& build) | Release 4.4.9+210219 da4b348 |
|---|
| I will donate to the project if issue is resolved | No |
|---|
| Browser | |
|---|
| Database type & version | mysql Ver 8.0.23 |
|---|
| Server OS (if known) | |
|---|
| Webserver software & version (if known) | |
|---|
| PHP Version | PHP 7.4.15 |
|---|
|
|
