This is probably related to EA-131 ticket. My broad level questions are
Is the “EmrConceptSearchController” only serves to search “Diagnosis Concepts”? This line of code seems to find concept sources that are specific to diagnosis concept.
Should the concept search be limited to a single concept source? If it is so we decided, then fine. still the “concept source” is hard coded to “ICD-10-WHO” here. Why so? I would assume that the name of the source is defined through some global property. I can consider “ICD-10-WHO” to be the default, but why not giving a way to override the default ? Btw, ICD 11 has been in the draft for sometime.
- Even if all the above are somewhat acceptable, the call will break at this line, if the “concept sources” are not found!
- currently throws a Null pointer exception if a system does not define “ICD-10-WHO” concept source.
- either send an empty list from “emrApiProperties.getConceptSourcesForDiagnosisSearch()”
- or modify EmrConceptSearchController .createListResponse() to handle appropriately for empty/null sources.