Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions extensions/default/src/DicomWebDataSource/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ export type DicomWebConfig = {
staticWado?: boolean;
/** User authentication service */
userAuthenticationService: Record<string, unknown>;
/** Case sensitivity configuration */
caseSensitive?: Record<string, boolean>;
};

export type BulkDataURIConfig = {
Expand Down Expand Up @@ -147,6 +149,7 @@ function createDicomWebApi(dicomWebConfig: DicomWebConfig, servicesManager) {

qidoConfig = {
url: dicomWebConfig.qidoRoot,
caseSensitive: dicomWebConfig.caseSensitive || {},
staticWado: dicomWebConfig.staticWado,
singlepart: dicomWebConfig.singlepart,
headers: userAuthenticationService.getAuthorizationHeader(),
Expand All @@ -155,6 +158,7 @@ function createDicomWebApi(dicomWebConfig: DicomWebConfig, servicesManager) {

wadoConfig = {
url: dicomWebConfig.wadoRoot,
caseSensitive: dicomWebConfig.caseSensitive || {},
staticWado: dicomWebConfig.staticWado,
singlepart: dicomWebConfig.singlepart,
headers: userAuthenticationService.getAuthorizationHeader(),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -156,18 +156,27 @@ 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) {
desired = desired.toLowerCase();
actual = actual.toLowerCase();
}
}

if (typeof actual == 'string') {
if (actual.length === 0) {
return true;
Expand Down Expand Up @@ -227,7 +236,9 @@ export default class StaticWadoClient extends api.DICOMwebClient {
return this.compareDateRange(testValue, valueElem.Value[0]);
}
const value = valueElem.Value;
IbrahimCSAE marked this conversation as resolved.
Show resolved Hide resolved
return this.compareValues(testValue, value);
const caseSensitive = this.config.caseSensitive[key];

return this.compareValues(testValue, value, caseSensitive);
}

/** Converts the query parameters to lower case query parameters */
Expand Down
6 changes: 6 additions & 0 deletions platform/app/public/config/default.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,12 @@ window.config = {
supportsFuzzyMatching: false,
supportsWildcard: true,
staticWado: true,
caseSensitive: {
patientname: false,
studydescription: false,
accessionnumber: false,
'00100020': 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
Expand Down
12 changes: 12 additions & 0 deletions platform/app/public/config/e2e.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,12 @@ window.config = {
relativeResolution: 'studies',
transform: url => url.replace('/pixeldata.mp4', '/index.mp4'),
},
caseSensitive: {
patientname: false,
studydescription: false,
accessionnumber: false,
'00100020': false,
},
},
},
{
Expand Down Expand Up @@ -118,6 +124,12 @@ window.config = {
relativeResolution: 'studies',
transform: url => url.replace('/pixeldata.mp4', '/rendered'),
},
caseSensitive: {
patientname: false,
studydescription: false,
accessionnumber: false,
'00100020': false,
},
},
},

Expand Down
9 changes: 9 additions & 0 deletions platform/docs/docs/configuration/configurationFiles.md
Original file line number Diff line number Diff line change
Expand Up @@ -193,6 +193,15 @@ 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, only works on STATIC-WADO backends. 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,
studydescription: false,
accessionnumber: false,
'00100020': 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`.
Expand Down
Loading