Dev: don't save empty value for not visible attribute

Dev: still didn't work
LimeSurvey · Oct 27, 2018 · 3b6d830 · 3b6d830
1 parent 29649c8
commit 3b6d830
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/application/controllers/admin/database.php b/application/controllers/admin/database.php
@@ -569,7 +569,7 @@ private function actionUpdateQuestion($iSurveyID)
         $aLanguages = array_merge(array(Survey::model()->findByPk($iSurveyID)->language), Survey::model()->findByPk($iSurveyID)->additionalLanguages);
         foreach ($validAttributes as $validAttribute) {
             /* Readonly attribute and visible==false: disable save */
-            if( $validAttribute['readonly'] || ( $validAttribute['readonly_when_active'] && Survey::model()->findByPk($iSurveyID)->getIsActive() ) ) {
+            if( !$validAttribute['visible'] || $validAttribute['readonly'] || ( $validAttribute['readonly_when_active'] && Survey::model()->findByPk($iSurveyID)->getIsActive() ) ) {
                 continue;
             }
             if ($validAttribute['i18n']) {