[OHI-1357] feat(static-wado): add support for case-insensitive searching

extensions/default/src/DicomWebDataSource/index.js

@@ -54,6 +54,7 @@ const metadataProvider = classes.MetadataProvider;
  * @param {boolean} dicomWebConfig.bulkDataURI - Whether to enable bulkDataURI
  * @param {function} dicomWebConfig.onConfiguration - Function that is called after the configuration is initialized
  * @param {boolean} dicomWebConfig.staticWado - Whether to use the static WADO client
+ * @param {object} dicomWebConfig.caseSensitive - Whether to use case sensitive filtering on results
  * @param {object} userAuthenticationService - User authentication service
  * @param {object} userAuthenticationService.getAuthorizationHeader - Function that returns the authorization header
  * @returns {object} - DICOM Web API object
@@ -107,6 +108,7 @@ function createDicomWebApi(dicomWebConfig, servicesManager) {
         url: dicomWebConfig.qidoRoot,
         staticWado: dicomWebConfig.staticWado,
         singlepart: dicomWebConfig.singlepart,
+        caseSensitive: dicomWebConfig.caseSensitive || {},
         headers: userAuthenticationService.getAuthorizationHeader(),
         errorInterceptor: errorHandler.getHTTPErrorHandler(),
       };
@@ -115,6 +117,7 @@ function createDicomWebApi(dicomWebConfig, servicesManager) {
         url: dicomWebConfig.wadoRoot,
         staticWado: dicomWebConfig.staticWado,
         singlepart: dicomWebConfig.singlepart,
+        caseSensitive: dicomWebConfig.caseSensitive || {},
         headers: userAuthenticationService.getAuthorizationHeader(),
         errorInterceptor: errorHandler.getHTTPErrorHandler(),
       };

extensions/default/src/DicomWebDataSource/utils/StaticWadoClient.ts

@@ -113,7 +113,15 @@ export default class StaticWadoClient extends api.DICOMwebClient {
     const lowerParams = this.toLowerParams(queryParams);
     const filtered = searchResult.filter(study => {
       for (const key of Object.keys(StaticWadoClient.studyFilterKeys)) {
-        if (!this.filterItem(key, lowerParams, study, StaticWadoClient.studyFilterKeys)) {
+        if (
+          !this.filterItem(
+            key,
+            lowerParams,
+            study,
+            StaticWadoClient.studyFilterKeys,
+            this.config.caseSensitive[key]
+          )
+        ) {
           return false;
         }
       }
@@ -156,19 +164,28 @@ export default class StaticWadoClient extends api.DICOMwebClient {
    *
    * @param {*} desired
    * @param {*} actual
+   * @param {boolean} caseSensitive
    * @returns true if the values match
    */
-  compareValues(desired, actual) {
+  compareValues(desired, actual, caseSensitive = true) {
     if (Array.isArray(desired)) {
-      return desired.find(item => this.compareValues(item, actual));
+      return desired.find(item => this.compareValues(item, actual, caseSensitive));
     }
     if (Array.isArray(actual)) {
-      return actual.find(actualItem => this.compareValues(desired, actualItem));
+      return actual.find(actualItem => this.compareValues(desired, actualItem, caseSensitive));
     }
     if (actual?.Alphabetic) {
       actual = actual.Alphabetic;
     }
     if (typeof actual == 'string') {
+      if (!caseSensitive) {
+        if (typeof desired === 'string') {
+          desired = desired.toLowerCase();
+        }
+        if (typeof actual === 'string') {
+          actual = actual.toLowerCase();
+        }
+      }
       if (actual.length === 0) {
         return true;
       }
@@ -208,9 +225,10 @@ export default class StaticWadoClient extends api.DICOMwebClient {
    * @param queryParams -
    * @param {*} study
    * @param {*} sourceFilterMap
+   * @param {boolean} caseSensitive
    * @returns
    */
-  filterItem(key: string, queryParams, study, sourceFilterMap) {
+  filterItem(key: string, queryParams, study, sourceFilterMap, caseSensitive = true) {
     const altKey = sourceFilterMap[key] || key;
     if (!queryParams) {
       return true;
@@ -227,7 +245,7 @@ export default class StaticWadoClient extends api.DICOMwebClient {
       return this.compareDateRange(testValue, valueElem.Value[0]);
     }
     const value = valueElem.Value;
-    return this.compareValues(testValue, value);
+    return this.compareValues(testValue, value, caseSensitive);
   }
 
   /** Converts the query parameters to lower case query parameters */

platform/app/public/config/default.js

@@ -52,6 +52,9 @@ window.config = {
         supportsFuzzyMatching: false,
         supportsWildcard: true,
         staticWado: true,
+        caseSensitive: {
+          patientname: false,
+        },
         singlepart: 'bulkdata,video',
         // whether the data source should use retrieveBulkData to grab metadata,
         // and in case of relative path, what would it be relative to, options

platform/docs/docs/configuration/configurationFiles.md

@@ -193,6 +193,12 @@ if auth headers are used, a preflight request is required.
 - `activateViewportBeforeInteraction`: (default to true), if set to false, tools can be used directly without the need to click and activate the viewport.
 - `autoPlayCine`: (default to false), if set to true, data sets with the DICOM frame time tag (i.e. (0018,1063)) will auto play when displayed
 - `addWindowLevelActionMenu`: (default to true), if set to false, the window level action menu item is NOT added to the viewport action corners
+- `caseSensitive`: an object with filter keys that should be case sensitive. By default, all filters are case sensitive. If you want to set a filter to be case insensitive, you can set it to false. Example:
+  ```js
+  caseSensitive: {
+    patientname: false,
+  }
+  ```
 - `dangerouslyUseDynamicConfig`: Dynamic config allows user to pass `configUrl` query string. This allows to load config without recompiling application. If the `configUrl` query string is passed, the worklist and modes will load from the referenced json rather than the default .env config. If there is no `configUrl` path provided, the default behaviour is used and there should not be any deviation from current user experience.<br/>
 Points to consider while using `dangerouslyUseDynamicConfig`:<br/>
   - User have to enable this feature by setting `dangerouslyUseDynamicConfig.enabled:true`. By default it is `false`.