Fix response not showing in summary mode

[hamza-vd]

IMPORTANT: Where possible all PRs must be linked to a Github issue

This PR addresses some issues mentioned in this comment. The 2 spcific fixes done are:

Forms saved as draft(even those not saved as draft and filled to the end) are loosing all the answers when you open as read only .

When a questionnaire response was saved earlier as draft and later on retrieved in summary mode. The answers to the questionnaire items were empty because it searched for an in-progress QR rather than the one with completed status. This has been fixed in this commit.

The read only forms (with a green tick) have the save as draft option.

I added a check in this commit to not to show save as draft option in Summary mode

Engineer Checklist

• I have written Unit tests for any new feature(s) and edge cases for bug fixes
• I have added any strings visible on UI components to the strings.xml file
• I have updated the CHANGELOG.md file for any notable changes to the codebase
• I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
• I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
• I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

• I have verified Unit tests have been written for any new feature(s) and edge cases
• I have verified any strings visible on UI components are in the strings.xml file
• I have verifed the CHANGELOG.md file has any notable changes to the codebase
• I have verified the solution has been implemented in a configurable and generic way for reuseable components
• I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 38.9%. Comparing base (d68886f) to head (f921158).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
...re/engine/util/extension/QuestionnaireExtension.kt	0.0%	0 Missing and 1 partial ⚠️
...re/quest/ui/questionnaire/QuestionnaireActivity.kt	0.0%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##              main   #3670   +/-   ##
=======================================
  Coverage     38.9%   38.9%           
  Complexity    1164    1164           
=======================================
  Files          284     284           
  Lines        15078   15078           
  Branches      2671    2671           
=======================================
  Hits          5872    5872           
+ Misses        8361    8360    -1     
- Partials       845     846    +1     

Flag	Coverage Δ
engine	62.5% <0.0%> (ø)
geowidget	16.0% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...re/engine/util/extension/QuestionnaireExtension.kt	55.3% <0.0%> (ø)
...re/quest/ui/questionnaire/QuestionnaireActivity.kt	6.1% <0.0%> (ø)

... and 1 file with indirect coverage changes

[FikriMilano]

Could you be more descriptive on what you're solving here?
And if it's possible, to add a test.

[hamza-vd]

Could you be more descriptive on what you're solving here? And if it's possible, to add a test.

Description updated and a test added

[FikriMilano]

Thanks!