From d03a03ce76b2199c7dcbe7ad514c3096277396d9 Mon Sep 17 00:00:00 2001
From: Evren Ozkan <evren217@gmail.com>
Date: Tue, 21 Nov 2017 14:14:14 -0500
Subject: [PATCH] PWV-73 Revert commit b88c2f4 as it is moved - Indicate that
 study is not available only if its 'available' property is set to false
 explicitly

---
 .../components/timepointBrowser/item.html     |  3 --
 .../components/timepointBrowser/item.js       | 36 -------------------
 .../components/timepointBrowser/list.styl     | 10 ------
 .../client/components/browser/item.html       |  2 +-
 .../imports/client/components/browser/item.js |  4 +--
 5 files changed, 3 insertions(+), 52 deletions(-)

diff --git a/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.html b/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.html
index c7450bcca..9b1a4d81a 100644
--- a/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.html
+++ b/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.html
@@ -2,9 +2,6 @@
     <div class="timepoint-browser-item" data-id="{{this.timepoint.timepointId}}">
         <div class="timepoint-item p-y-2">
             <div class="timepoint-details clearfix">
-                <div class="timepoint-study-availability pull-left m-r-1" >
-                    <i class="fa fa-circle {{studyAvailability}}"></i>
-                </div>
                 <div class="timepoint-title pull-left">
                     {{this.timepointApi.title timepoint}}
                 </div>
diff --git a/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.js b/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.js
index c2736b6f2..a004c3eef 100644
--- a/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.js
+++ b/Packages/ohif-lesiontracker/client/components/timepointBrowser/item.js
@@ -13,7 +13,6 @@ Template.timepointBrowserItem.onCreated(() => {
     const hasStudiesData = !!(timepoint.studiesData && timepoint.studiesData.length);
     instance.summary = new ReactiveVar('');
     instance.studiesData = new ReactiveVar(hasStudiesData ? timepoint.studiesData : null);
-    instance.missingStudyUids = new ReactiveVar(null);
 
     const updateStudiesData = newDocument => {
         const newTimepoint = newDocument || timepointApi.timepoints.findOne({ timepointId });
@@ -58,45 +57,10 @@ Template.timepointBrowserItem.onCreated(() => {
         const studiesData = instance.studiesData.get();
         if (studiesData) {
             instance.setModalitiesSummary();
-
-            //  Determine missing timepoint studies
-            const missingStudyUids = [];
-            timepoint.studyInstanceUids.forEach((timepointStudyUid) => {
-                if (!studiesData.find(studyData => studyData.available && studyData.studyInstanceUid === timepointStudyUid)) {
-                    missingStudyUids.push(timepointStudyUid);
-                }
-            });
-            instance.missingStudyUids.set(missingStudyUids);
         }
     });
 });
 
-Template.timepointBrowserItem.helpers({
-    studyAvailability: () => {
-        const instance = Template.instance();
-        const { timepoint } = instance.data;
-        const missingStudyUids = instance.missingStudyUids && instance.missingStudyUids.get();
-
-        if (!missingStudyUids || !timepoint) {
-            //  Timepoint studies are not loaded/checked yet (may be indicated as disabled circle)
-            return "studies-not-ready";
-        }
-
-        if (missingStudyUids.length >= timepoint.studyInstanceUids.length) {
-            //  All timepoint studies are not available (may be indicated as empty circle)
-            return "studies-not-available";
-        }
-
-        if (missingStudyUids.length > 0) {
-            //  Timepoint studies are partially available (may be indicated as half-filled circle)
-            return "studies-partially-available";
-        }
-
-        //  All timepoint studies are available (may be indicated as filled circle)
-        return "studies-fully-available";
-    }
-});
-
 Template.timepointBrowserItem.events({
     'ohif.lesiontracker.timepoint.load .timepoint-item'(event, instance) {
         instance.loadStudies();
diff --git a/Packages/ohif-lesiontracker/client/components/timepointBrowser/list.styl b/Packages/ohif-lesiontracker/client/components/timepointBrowser/list.styl
index c0013baf0..653575aa3 100644
--- a/Packages/ohif-lesiontracker/client/components/timepointBrowser/list.styl
+++ b/Packages/ohif-lesiontracker/client/components/timepointBrowser/list.styl
@@ -78,16 +78,6 @@
     theme('color', '$textPrimaryColor')
     padding-top: 2px
 
-  .timepoint-study-availability
-    .studies-not-ready
-      color: gray
-    .studies-not-available
-      color: red
-    .studies-partially-available
-      color: yellow
-    .studies-fully-available
-      color: green
-
 .series-quick-switch .timepoint-browser-list
   padding: 0 10px
 
diff --git a/Packages/ohif-studies/imports/client/components/browser/item.html b/Packages/ohif-studies/imports/client/components/browser/item.html
index 6faeb6bd4..47ba98492 100644
--- a/Packages/ohif-studies/imports/client/components/browser/item.html
+++ b/Packages/ohif-studies/imports/client/components/browser/item.html
@@ -14,7 +14,7 @@
                             <div class="study-date">{{formatDA studyData.studyDate 'D-MMM-YYYY'}}</div>
                             <div class="study-description">{{studyData.studyDescription}}</div>
                         {{else}}
-                            <div class="study-availability">{{#if studyData.available}}Click to load{{else}}N/A{{/if}}</div>
+                            <div class="study-availability">{{#if eq studyData.available false}}N/A{{else}}Click to load{{/if}}</div>
                         {{/if}}
                     </div>
                 </div>
diff --git a/Packages/ohif-studies/imports/client/components/browser/item.js b/Packages/ohif-studies/imports/client/components/browser/item.js
index f2b508e2d..1b5118ee2 100644
--- a/Packages/ohif-studies/imports/client/components/browser/item.js
+++ b/Packages/ohif-studies/imports/client/components/browser/item.js
@@ -56,8 +56,8 @@ Template.studyBrowserItem.events({
 
         const { studyInformation } = instance.data;
 
-        //  Skip if study is not available
-        if (!studyInformation.available) {
+        //  Skip if study is not available (set to false explicitly)
+        if (studyInformation.available === false) {
             return;
         }