default sorting of cases

[KrastyoYonkov]

Hello, how can I set the default sorting of cases to show the newest first and the oldest last?

[gunterze]

You can't set defaults, but the last selected sort order is persisted in the localStroage of the browser, so it will be preserved over user sessions, browser and archive restarts.

[gunterze]

The QIDO-RS services of the archive supports proprietary query parameter orderby=[-]{attributeID} to specify the sorting of the returned matches. E.g.

GET dcm4chee-arc/aets/DCM4CHEE/rs/studies?orderby=-StudyDate,-StudyTime

will return newest matching Studies first. I am not familiar with OHIF, so I don't know if you could configure it to append that query parameter on invoked QIDO-RS requests.

[gunterze]

Opened #4637

[andylinton]

Hi, I'm in the same situation, using OHIF in production with DCM4CHEE - wanting to sort by default with newest first. I am asking the OHIF development team if it is possible to use this query parameter.

[KrastyoYonkov]

Hi "/home/krastyo/Viewers/extensions/default/src/DicomWebDataSource/qido.js"

/**

• QIDO - Query based on ID for DICOM Objects
• search for studies, series and instances by patient ID, and receive their
• unique identifiers for further usage.
• Quick: https://www.dicomstandard.org/dicomweb/query-qido-rs/
• Standard: http://dicom.nema.org/medical/dicom/current/output/html/part18.html#sect_10.6
• Routes:
• ==========
• /studies?
• /studies/{studyInstanceUid}/series?
• /studies/{studyInstanceUid}/series/{seriesInstanceUid}/instances?
• Query Parameters:
• ================
• | KEY | VALUE |
• |------------------|--------------------|
• | {attributeId} | {value} |
• | includeField | {attribute} or all |
• | fuzzymatching | true OR false |
• | limit | {number} |
• | offset | {number} |
  */
  import { DICOMWeb, utils } from '@ohif/core';
  import { sortStudySeries } from '@ohif/core/src/utils/sortStudy';

const { getString, getName, getModalities } = DICOMWeb;

/**

• Parses resulting data from a QIDO call into a set of Study MetaData
• @param {Array} qidoStudies - An array of study objects. Each object contains a keys for DICOM tags.
• @param {object} qidoStudies[0].qidoStudy - An object where each key is the DICOM Tag group+element
• @param {object} qidoStudies[0].qidoStudy[dicomTag] - Optional object that represents DICOM Tag
• @param {string} qidoStudies[0].qidoStudy[dicomTag].vr - Value Representation
• @param {string[]} qidoStudies[0].qidoStudy[dicomTag].Value - Optional string array representation of the DICOM Tag's value
• @returns {Array} An array of Study MetaData objects
  */
  function processResults(qidoStudies) {
  if (!qidoStudies || !qidoStudies.length) {
  return [];
  }

const studies = [];

qidoStudies.forEach(qidoStudy =>
studies.push({
studyInstanceUid: getString(qidoStudy['0020000D']),
date: getString(qidoStudy['00080020']), // YYYYMMDD
time: getString(qidoStudy['00080030']), // HHmmss.SSS (24-hour, minutes, seconds, fractional seconds)
accession: getString(qidoStudy['00080050']) || '', // short string, probably a number?
mrn: getString(qidoStudy['00100020']) || '', // medicalRecordNumber
patientName: utils.formatPN(getName(qidoStudy['00100010'])) || '',
instances: Number(getString(qidoStudy['00201208'])) || 0, // number
description: getString(qidoStudy['00081030']) || '',
modalities: getString(getModalities(qidoStudy['00080060'], qidoStudy['00080061'])) || '',
})
);

return studies;
}

/**

• Parses resulting data from a QIDO call into a set of Study MetaData
• @param {Array} qidoSeries - An array of study objects. Each object contains a keys for DICOM tags.
• @param {object} qidoSeries[0].qidoSeries - An object where each key is the DICOM Tag group+element
• @param {object} qidoSeries[0].qidoSeries[dicomTag] - Optional object that represents DICOM Tag
• @param {string} qidoSeries[0].qidoSeries[dicomTag].vr - Value Representation
• @param {string[]} qidoSeries[0].qidoSeries[dicomTag].Value - Optional string array representation of the DICOM Tag's value
• @returns {Array} An array of Study MetaData objects
  */
  export function processSeriesResults(qidoSeries) {
  const series = [];

if (qidoSeries && qidoSeries.length) {
qidoSeries.forEach(qidoSeries =>
series.push({
studyInstanceUid: getString(qidoSeries['0020000D']),
seriesInstanceUid: getString(qidoSeries['0020000E']),
modality: getString(qidoSeries['00080060']),
seriesNumber: getString(qidoSeries['00200011']),
seriesDate: utils.formatDate(getString(qidoSeries['00080021'])),
numSeriesInstances: Number(getString(qidoSeries['00201209'])),
description: getString(qidoSeries['0008103E']),
})
);
}

sortStudySeries(series);

return series;
}

/**
*

• @param {object} dicomWebClient - Client similar to what's provided by dicomweb-client library
• @param {function} dicomWebClient.searchForStudies -
• @param {string} [studyInstanceUid]
• @param {string} [seriesInstanceUid]
• @param {string} [queryParamaters]
• @returns {Promise} - Promise that resolves results
  */
  async function search(dicomWebClient, studyInstanceUid, seriesInstanceUid, queryParameters) {
  // Add the orderby parameter to the query
  queryParameters = {
  ...queryParameters,
  orderby: '-StudyDate,-StudyTime', // Add sorting parameter
  };

let searchResult = await dicomWebClient.searchForStudies({
studyInstanceUid: undefined,
queryParams: queryParameters,
});

return searchResult;
}

/**
*

• @param {string} studyInstanceUID - ID of study to return a list of series for
• @returns {Promise} - Resolves SeriesMetadata[] in study
  */
  export function seriesInStudy(dicomWebClient, studyInstanceUID) {
  // Series Description
  // Already included?
  const commaSeparatedFields = ['0008103E', '00080021'].join(',');
  const queryParams = {
  includefield: commaSeparatedFields,
  };

return dicomWebClient.searchForSeries({ studyInstanceUID, queryParams });
}

export default function searchStudies(server, filter) {
const queryParams = getQIDOQueryParams(filter, server.qidoSupportsIncludeField);
const options = {
queryParams,
};

return dicomWeb.searchForStudies(options).then(resultDataToStudies);
}

/**

• Produces a QIDO URL given server details and a set of specified search filter
• items
• @param filter
• @param serverSupportsQIDOIncludeField
• @returns {string} The URL with encoded filter query data
  */
  function mapParams(params, options = {}) {
  if (!params) {
  return;
  }
  const commaSeparatedFields = [
  '00081030', // Study Description
  '00080060', // Modality
  // Add more fields here if you want them in the result
  ].join(',');

const useWildcard =
params?.disableWildcard !== undefined ? !params.disableWildcard : options.supportsWildcard;

const withWildcard = value => {
return useWildcard && value ? *${value}* : value;
};

const parameters = {
// Named
PatientName: withWildcard(params.patientName),
'00100020': withWildcard(params.patientId), // PatientID
AccessionNumber: withWildcard(params.accessionNumber),
StudyDescription: withWildcard(params.studyDescription),
ModalitiesInStudy: params.modalitiesInStudy,
limit: params.limit || 101,
offset: params.offset || 0,
fuzzymatching: options.supportsFuzzyMatching === true,
includefield: commaSeparatedFields, // include specific fields
};

// build the StudyDate range parameter
if (params.startDate && params.endDate) {
parameters.StudyDate = ${params.startDate}-${params.endDate};
} else if (params.startDate) {
const today = new Date();
const todayStr = ${today.getFullYear()}${String(today.getMonth() + 1).padStart(2, '0')}${String(today.getDate()).padStart(2, '0')};
parameters.StudyDate = ${params.startDate}-${todayStr};
} else if (params.endDate) {
parameters.StudyDate = 19700101-${params.endDate};
}

// Build the StudyInstanceUID parameter
if (params.studyInstanceUid) {
let studyUids = Array.isArray(params.studyInstanceUid) ? params.studyInstanceUid.join() : params.studyInstanceUid;
studyUids = studyUids.replace(/[^0-9.]+/g, '\');
parameters.StudyInstanceUID = studyUids;
}

// Add sorting parameter for studies
parameters.orderby = '-StudyDate,-StudyTime';

// Clean query params of undefined values.
const final = {};
Object.keys(parameters).forEach(key => {
if (parameters[key] !== undefined && parameters[key] !== '') {
final[key] = parameters[key];
}
});

return final;
}

export { mapParams, search, processResults };

[KrastyoYonkov]

// Add sorting parameter for studies
parameters.orderby = '-StudyDate,-StudyTime';

[KrastyoYonkov]

Hi, I found a solution, I'll write to you tomorrow. На вт, 14 яну 2025 г., 22:03 andylinton ***@***.***> написа:

…

Hi, I'm in the same situation, using OHIF in production with DCM4CHEE - wanting to sort by default with newest first. I am asking the OHIF development team if it is possible to use this query parameter. — Reply to this email directly, view it on GitHub <#4635 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BC6ZQ5WH22I45V4VCRMMXTD2KVUQBAVCNFSM6AAAAABUXWS756VHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTCOBTGU2TINA> . You are receiving this because you authored the thread.Message ID: ***@***.*** .com>

[andylinton]

I'm very interested to hear about your solution!

…

On Friday, 17 January 2025, KrastyoYonkov ***@***.***> wrote: Hi, I found a solution, I'll write to you tomorrow. На вт, 14 яну 2025 г., 22:03 andylinton ***@***.***> написа: > Hi, I'm in the same situation, using OHIF in production with DCM4CHEE - > wanting to sort by default with newest first. I am asking the OHIF > development team if it is possible to use this query parameter. > > — > Reply to this email directly, view it on GitHub > <#4635# discussioncomment-11835544>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ BC6ZQ5WH22I45V4VCRMMXTD2KVUQBAVCNFSM6AAAAABUXWS756VHI2DSMVQW IX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTCOBTGU2TINA> > . > You are receiving this because you authored the thread.Message ID: > ***@***.*** > .com> > — Reply to this email directly, view it on GitHub <#4635 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AWVZB5TEUQKKN5465BSZK2T2LBPD3AVCNFSM6AAAAABUXWS756VHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTCOBWGE2TCNY> . You are receiving this because you commented.Message ID: ***@***.*** .com>

[KrastyoYonkov]

"/home/krastyo/Viewers/extensions/default/src/DicomWebDataSource/qido.js"

/**

• QIDO - Query based on ID for DICOM Objects
• search for studies, series and instances by patient ID, and receive their
• unique identifiers for further usage.
• Quick: https://www.dicomstandard.org/dicomweb/query-qido-rs/
• Standard: http://dicom.nema.org/medical/dicom/current/output/html/part18.html#sect_10.6
• Routes:
• ==========
• /studies?
• /studies/{studyInstanceUid}/series?
• /studies/{studyInstanceUid}/series/{seriesInstanceUid}/instances?
• Query Parameters:
• ================
• | KEY | VALUE |
• |------------------|--------------------|
• | {attributeId} | {value} |
• | includeField | {attribute} or all |
• | fuzzymatching | true OR false |
• | limit | {number} |
• | offset | {number} |
  */
  import { DICOMWeb, utils } from '@ohif/core';
  import { sortStudySeries } from '@ohif/core/src/utils/sortStudy';

const { getString, getName, getModalities } = DICOMWeb;

/**

• Parses resulting data from a QIDO call into a set of Study MetaData
• @param {Array} qidoStudies - An array of study objects. Each object contains a keys for DICOM tags.
• @param {object} qidoStudies[0].qidoStudy - An object where each key is the DICOM Tag group+element
• @param {object} qidoStudies[0].qidoStudy[dicomTag] - Optional object that represents DICOM Tag
• @param {string} qidoStudies[0].qidoStudy[dicomTag].vr - Value Representation
• @param {string[]} qidoStudies[0].qidoStudy[dicomTag].Value - Optional string array representation of the DICOM Tag's value
• @returns {Array} An array of Study MetaData objects
  */
  function processResults(qidoStudies) {
  if (!qidoStudies || !qidoStudies.length) {
  return [];
  }

const studies = [];

qidoStudies.forEach(qidoStudy =>
studies.push({
studyInstanceUid: getString(qidoStudy['0020000D']),
date: getString(qidoStudy['00080020']), // YYYYMMDD
time: getString(qidoStudy['00080030']), // HHmmss.SSS (24-hour, minutes, seconds, fractional seconds)
accession: getString(qidoStudy['00080050']) || '', // short string, probably a number?
mrn: getString(qidoStudy['00100020']) || '', // medicalRecordNumber
patientName: utils.formatPN(getName(qidoStudy['00100010'])) || '',
instances: Number(getString(qidoStudy['00201208'])) || 0, // number
description: getString(qidoStudy['00081030']) || '',
modalities: getString(getModalities(qidoStudy['00080060'], qidoStudy['00080061'])) || '',
})
);

return studies;
}

/**

• Parses resulting data from a QIDO call into a set of Study MetaData
• @param {Array} qidoSeries - An array of study objects. Each object contains a keys for DICOM tags.
• @param {object} qidoSeries[0].qidoSeries - An object where each key is the DICOM Tag group+element
• @param {object} qidoSeries[0].qidoSeries[dicomTag] - Optional object that represents DICOM Tag
• @param {string} qidoSeries[0].qidoSeries[dicomTag].vr - Value Representation
• @param {string[]} qidoSeries[0].qidoSeries[dicomTag].Value - Optional string array representation of the DICOM Tag's value
• @returns {Array} An array of Study MetaData objects
  */
  export function processSeriesResults(qidoSeries) {
  const series = [];

if (qidoSeries && qidoSeries.length) {
qidoSeries.forEach(qidoSeries =>
series.push({
studyInstanceUid: getString(qidoSeries['0020000D']),
seriesInstanceUid: getString(qidoSeries['0020000E']),
modality: getString(qidoSeries['00080060']),
seriesNumber: getString(qidoSeries['00200011']),
seriesDate: utils.formatDate(getString(qidoSeries['00080021'])),
numSeriesInstances: Number(getString(qidoSeries['00201209'])),
description: getString(qidoSeries['0008103E']),
})
);
}

sortStudySeries(series);

return series;
}

/**
*

• @param {object} dicomWebClient - Client similar to what's provided by dicomweb-client library
• @param {function} dicomWebClient.searchForStudies -
• @param {string} [studyInstanceUid]
• @param {string} [seriesInstanceUid]
• @param {string} [queryParamaters]
• @returns {Promise} - Promise that resolves results
  */
  async function search(dicomWebClient, studyInstanceUid, seriesInstanceUid, queryParameters) {
  // Add the orderby parameter to the query
  queryParameters = {
  ...queryParameters,
  orderby: '-StudyDate,-StudyTime', // Add sorting parameter
  };

let searchResult = await dicomWebClient.searchForStudies({
studyInstanceUid: undefined,
queryParams: queryParameters,
});

return searchResult;
}

/**
*

• @param {string} studyInstanceUID - ID of study to return a list of series for
• @returns {Promise} - Resolves SeriesMetadata[] in study
  */
  export function seriesInStudy(dicomWebClient, studyInstanceUID) {
  // Series Description
  // Already included?
  const commaSeparatedFields = ['0008103E', '00080021'].join(',');
  const queryParams = {
  includefield: commaSeparatedFields,
  };

return dicomWebClient.searchForSeries({ studyInstanceUID, queryParams });
}

export default function searchStudies(server, filter) {
const queryParams = getQIDOQueryParams(filter, server.qidoSupportsIncludeField);
const options = {
queryParams,
};

return dicomWeb.searchForStudies(options).then(resultDataToStudies);
}

/**

• Produces a QIDO URL given server details and a set of specified search filter
• items
• @param filter
• @param serverSupportsQIDOIncludeField
• @returns {string} The URL with encoded filter query data
  */
  function mapParams(params, options = {}) {
  if (!params) {
  return;
  }
  const commaSeparatedFields = [
  '00081030', // Study Description
  '00080060', // Modality
  // Add more fields here if you want them in the result
  ].join(',');

const useWildcard =
params?.disableWildcard !== undefined ? !params.disableWildcard : options.supportsWildcard;

const withWildcard = value => {
return useWildcard && value ? *${value}* : value;
};

const parameters = {
// Named
PatientName: withWildcard(params.patientName),
'00100020': withWildcard(params.patientId), // PatientID
AccessionNumber: withWildcard(params.accessionNumber),
StudyDescription: withWildcard(params.studyDescription),
ModalitiesInStudy: params.modalitiesInStudy,
limit: params.limit || 101,
offset: params.offset || 0,
fuzzymatching: options.supportsFuzzyMatching === true,
includefield: commaSeparatedFields, // include specific fields
};

// build the StudyDate range parameter
if (params.startDate && params.endDate) {
parameters.StudyDate = ${params.startDate}-${params.endDate};
} else if (params.startDate) {
const today = new Date();
const todayStr = ${today.getFullYear()}${String(today.getMonth() + 1).padStart(2, '0')}${String(today.getDate()).padStart(2, '0')};
parameters.StudyDate = ${params.startDate}-${todayStr};
} else if (params.endDate) {
parameters.StudyDate = 19700101-${params.endDate};
}

// Build the StudyInstanceUID parameter
if (params.studyInstanceUid) {
let studyUids = Array.isArray(params.studyInstanceUid) ? params.studyInstanceUid.join() : params.studyInstanceUid;
studyUids = studyUids.replace(/[^0-9.]+/g, '\');
parameters.StudyInstanceUID = studyUids;
}

// Add sorting parameter for studies
parameters.orderby = '-StudyDate,-StudyTime';

// Clean query params of undefined values.
const final = {};
Object.keys(parameters).forEach(key => {
if (parameters[key] !== undefined && parameters[key] !== '') {
final[key] = parameters[key];
}
});

return final;
}

export { mapParams, search, processResults };

[andylinton]

Ok! Updating qido.js in my test worked for me. Thanks!
I will see if I can open an issue or communicate with the OHIF developers to make this option more easily available in OHIF.

[KrastyoYonkov]

I have another problem that I cannot solve, I want the time to be 24:00 hours and not AM/PM, I tried everything.

[andylinton]

Is that a dcm4chee issue or the way the OHIF viewer is rendering the times? The DICOM tag just saves times in a number format like this: 072302 so I think it is up to OHIF to display in 24hr time or AM/PM. I can recommended asking the question on the OHIF slack or GitHub