🔧 Hot Fix : Correctly append PlanDefinition reference in CarePlan

[f-odhiambo]

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

Fixes #3383

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 75.86207% with 7 lines in your changes missing coverage. Please review.

Project coverage is 28.3%. Comparing base (5d5454b) to head (be6aeeb).

Additional details and impacted files

@@           Coverage Diff           @@
##              main   #3385   +/-   ##
=======================================
  Coverage     28.2%   28.3%           
- Complexity     713     716    +3     
=======================================
  Files          266     266           
  Lines        12737   12760   +23     
  Branches      2264    2265    +1     
=======================================
+ Hits          3603    3621   +18     
- Misses        8661    8666    +5     
  Partials       473     473           

Flag	Coverage Δ
engine	65.0% <75.8%> (+<0.1%)	⬆️
geowidget	19.8% <ø> (ø)
quest	4.8% <ø> (ø)

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

Files	Coverage Δ
...ster/fhircore/engine/task/FhirCarePlanGenerator.kt	78.1% <75.8%> (+<0.1%)	⬆️

[ndegwamartin]

Also in line with the internal conversation we can mark this method here with with the @Deprecated annotation cc @pld

fhircore/android/engine/src/main/java/org/smartregister/fhircore/engine/task/FhirCarePlanGenerator.kt

Line 171 in 70a8bc1

private suspend fun liteApplyPlanDefinitionOnPatient(

You can add this:

  @Deprecated(
    message = "This function is deprecated, use workflowCarePlanGenerator.applyPlanDefinitionOnPatient() instead",
    replaceWith = ReplaceWith("workflowCarePlanGenerator.applyPlanDefinitionOnPatient()"),
    level = DeprecationLevel.WARNING
  )
  private suspend fun liteApplyPlanDefinitionOnPatient( ...

[f-odhiambo]

Resolved comments from PR rewview