BAH-1236 | Bug fix. Modified query to display only preferred patients…

[SooryaKumaranC-tw]

… in active tab after merging

Co-authored-by: Kavitha S [email protected]

[gsluthra]

Why DELETE and INSERT? And not instead just UPDATE?

[SooryaKumaranC-tw]

Why DELETE and INSERT? And not instead just UPDATE?

Ohh we thought we have to update the whole table. Okay we will just update that file and push them again.

[SooryaKumaranC-tw]

@gsluthra we have modified the file with the update statement.

[gsluthra]

Just for reference, these are the older SQL queries that were there before this change was made:

bahmni-core/bahmnicore-omod/src/main/resources/V1_97_PatientSearchSql.sql

Line 14 in 12edb7d

VALUES ('emrapi.sqlSearch.activePatients',

[SooryaKumaranC-tw]

Different Tabs tested in this flow are:

1. Active
2. Programs
3. My Patients
4. OPD-1
5. PatientsByLocation

Note:

1. All tab is working fine before itself. So we did not change anything there.
2. We removed activeVisitUuid column from the select statement because we have to display, only unique patient cards irrespective of number of active visits.

Scenario's Tested after merging 2 or 3 patients:

1. With 1 active visits:
   Before: There were duplicate patient cards in each tab (Active, Program, My Patients....)
   After: Only one preferred patient card is shown in all the tabs.

   Upon clicking the card it will redirect to respective dashboard. In patient dashboard we can see all visits under visit tab.

2. With 2 or more active visits:
   Before: There were duplicate patient cards in each tab (Active, Program, My Patients....)
   After: Only one preferred patient card is shown in all the tabs.

   Upon clicking the card it will redirect to respective dashboard. In patient dashboard we can see all visits under visit tab.

3. With no active visits, no patients cards are displayed.

[gsluthra]

Thanks! sounds good.

[angshu]

can we please break the SQL file into 3 different ones please and also create liquibase changesets that are specific to a specific property?

[SooryaKumaranC-tw]

Okay, sure.

[angshu]

can you please use 3 different changesets as suggested above? also please make sure that the changeset clearly says which is for which property

[SooryaKumaranC-tw]

Okay