openhealthcare / elcid

elCID deployment for UCLH
http://elcid.openhealthcare.org.uk
Other
7 stars 3 forks source link

Search bug in elcid-opat #465

Closed GabPoll closed 9 years ago

GabPoll commented 9 years ago

this is a bigger issue than #464 .

464 error still there, but also, no other advanced search works.

Search for diagnosis, condition, condition ARDS. Finds 1 patient, but doesn't display it. However, it seems that can still download the hidden data.

davidmiller commented 9 years ago

Fix deployed to elcid-opat

GabPoll commented 9 years ago

agree fixed. One unusual result, which may be specific to the patient rather than the code - @davidmiller search for diagnosis - condition - contains - chickenpox. 3 entries for one patient. Odd.

Does not occur with other patients, so unless something obvious, i propose to close this issue and see if it arises again. That mock patient has been messed around with an awful lot!

davidmiller commented 9 years ago

@GabPoll The patient just happens to have three distinct episodes of chickenpox.

Unlikely but true!

(If you visit each row in the results you'll see that the URL is episode/$NUMBER - where number is the id of the episode)

I think that this is the correct behaviour!