passing empty bundle instead of null to trigger the careplan when we …

…don't have any bundle resource (#2431)
opensrp · Jun 8, 2023 · 74107a4 · 74107a4
1 parent e839cf7
commit 74107a4
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/...src/main/java/org/smartregister/fhircore/quest/ui/questionnaire/QuestionnaireViewModel.kt b/...src/main/java/org/smartregister/fhircore/quest/ui/questionnaire/QuestionnaireViewModel.kt
@@ -269,7 +269,7 @@ constructor(
         performExtraction(questionnaireResponse, questionnaireConfig, questionnaire, bundle)
       } else {
         saveQuestionnaireResponse(questionnaire, questionnaireResponse)
-        performExtraction(questionnaireResponse, questionnaireConfig, questionnaire, bundle = null)
+        performExtraction(questionnaireResponse, questionnaireConfig, questionnaire)
       }
       viewModelScope.launch(dispatcherProvider.main()) { extractionProgress.postValue(true) }
     }
@@ -282,12 +282,10 @@ constructor(
     questionnaireResponse: QuestionnaireResponse,
     questionnaireConfig: QuestionnaireConfig,
     questionnaire: Questionnaire,
-    bundle: Bundle?
+    bundle: Bundle = Bundle()
   ) {
-    if (bundle?.entry?.isNotEmpty() == true) {
-      extractCqlOutput(questionnaire, questionnaireResponse, bundle)
-      extractCarePlan(questionnaireResponse, bundle, questionnaireConfig)
-    }
+    extractCqlOutput(questionnaire, questionnaireResponse, bundle)
+    extractCarePlan(questionnaireResponse, bundle, questionnaireConfig)
   }
 
   fun savePartialQuestionnaireResponse(
@@ -380,7 +378,11 @@ constructor(
     bundle: Bundle?
   ) {
     withContext(dispatcherProvider.default()) {
-      val data = bundle ?: Bundle().apply { addEntry().apply { resource = questionnaireResponse } }
+      val data =
+        Bundle().apply {
+          bundle?.entry?.map { this.addEntry(it) }
+          addEntry().resource = questionnaireResponse
+        }
       questionnaire
         .cqfLibraryIds()
         .map {