othmane/ohif-viewer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix a bug by sorting the studies by study date and time instead of only study date in prior function (to add prior indicator)

Browse Source
This commit is contained in:
Evren Ozkan 2016-11-14 21:19:52 -05:00
parent b0f9426015
commit c511edd419
1 changed files with 13 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
Packages/viewerbase/client/components/viewer/viewportOverlay/viewportOverlay.js
View File

@ -135,6 +135,10 @@ function getImage(viewportIndex) {
return enabledElement.image;
}
function formatDateTime(date, time) {
return `${date} ${time}`;
}
Template.viewportOverlay.helpers({
wwwc: function() {
Session.get('CornerstoneImageRendered' + this.viewportIndex);
@ -306,19 +310,18 @@ Template.viewportOverlay.helpers({
}
// Make sure there are more than two studies loaded in the viewer
//
var viewportStudies = ViewerStudies.find();
if (viewportStudies.count() < 2) {
return;
}
// Here we sort the collection in ascending order by study date, so
// that we can obtain the oldest study as the first element of the array
//
// TODO= Find out if we should encode studyDate as a Date in the ViewerStudies Collection
var viewportStudies = ViewerStudies.find({}, {
sort: {
studyDate: 1
}
var viewportStudiesArray = _.sortBy(viewportStudies.fetch(), function(study) {
return formatDateTime(study.studyDate, study.studyTime);
});
if (viewportStudies.count() < 2) {
return;
}
// Get study data
var study = cornerstoneTools.metaData.get('study', this.imageId);
@ -326,8 +329,8 @@ Template.viewportOverlay.helpers({
return;
}
var oldestStudy = viewportStudies.fetch()[0];
if ((study.studyDate + " " + study.studyTime) <= (oldestStudy.studyDate + " " + oldestStudy.studyTime)) {
var oldestStudy = viewportStudiesArray[0];
if (formatDateTime(study.studyDate, study.studyTime) <= formatDateTime(oldestStudy.studyDate, oldestStudy.studyTime)) {
return 'Prior';
}
}