fixed #7922: "Export results" suggests that unanswered multiple choic…

…e questions as having been answered
LimeSurvey · Jun 17, 2013 · d38a1af · d38a1af
1 parent 9270e1e
commit d38a1af
Showing 1 changed file with 8 additions and 13 deletions.
diff --git a/application/helpers/admin/exportresults_helper.php b/application/helpers/admin/exportresults_helper.php
@@ -626,20 +626,15 @@ public function getFullAnswer($fieldName, $answerCode, Translator $translator, $
                 }
                 else
                 {
-                    switch ($answerCode)
+                    if ($answerCode == 'Y') 
                     {
-                        case 'Y':
-                            $fullAnswer = $translator->translate('Yes', $sLanguageCode);
-                            break;
-
-                        case 'N':
-                        case '':
-                            $fullAnswer = $translator->translate('No', $sLanguageCode);
-                            break;
-
-                        default:
-                            //echo "\n -- Branch 2 --";
-                            $fullAnswer = $answerCode;
+                        $fullAnswer = $translator->translate('Yes', $sLanguageCode);
+                    } 
+                    elseif ($answerCode == 'N' || $answerCode === '')   // Strict check for empty string to find null values
+                    {
+                        $fullAnswer = $translator->translate('No', $sLanguageCode);
+                    } else {
+                        $fullAnswer = $translator->translate('N/A', $sLanguageCode);
                     }
                 }
                 break;