SunCoastConnection / TheRealLibreEHR

SunCoast Connection - QA reporting LibreEHR system
Other
0 stars 0 forks source link

Reports / Clients / Clinical needs testing and repairs. #448

Open teryhill opened 5 years ago

teryhill commented 5 years ago

This report does not appear to be display the data based on the search criteria.

aethelwulffe commented 5 years ago

General EHR vs. MIPS issue. Does not mean it is low priority though.

Steps:

  1. Evaluate intended functionality of the report.
  2. If it seems like it is a useful thing, repair and enhance it.
  3. If it is of dubious use, remove it to a /boneyard repo.
aethelwulffe commented 5 years ago

My Test: image

aethelwulffe commented 5 years ago

I entered a wide date range and a age of 1-99. That is NOT saying that the results were related to the input criteria, just that the report at least ran. I imagine that there is a lot of digging that needs to be done to figure out what this report is doing in each case. Terry, what specific criteria did you run, and what let you know the results were not right?

Ngai-E commented 5 years ago

if you do a search for

  1. patient ID 2, you still get results of all patients instead of returning just one.
  2. Do a search female patients only, you still get results for male patients too.

I think all the search criteria buttons dont work as expected

muarachmann commented 5 years ago

Okay giving this a look and see that some variables are not declared within the scope of the function definition. Managed to fix the Report > Clients > Patient list & Report > Clients > Patient list Creation which all do not search with the criteria. @Trodrige i see that you worked on this, query looks okay but i see this $whr_stmt="where 1=1"; where u started the where statement and this causes it to always return all patients when i dump() or print_r() at any point in the codebase file. Now i dont know why i don't get them displayed.

$answer = array('result' => $result, 'row_id' => $row_id, 'img_id' => $img_id, 'k' => $k ); This seems to not return anything.

muarachmann commented 5 years ago

I think this is resolved @aethelwulffe