# HG changeset patch
# User amazy
# Date 1567614202 -7200
# Node ID 4bced7d1ec2065ed7dc0f62a070591c60dc512fd
# Parent  6b61ba8359641bc9a02d34870dc85a89d9dbcb80
in /ordered-slices route, ignore instances without position/normal/seriesIndex

diff -r 6b61ba835964 -r 4bced7d1ec20 NEWS
--- a/NEWS	Tue Sep 03 10:34:26 2019 +0200
+++ b/NEWS	Wed Sep 04 18:23:22 2019 +0200
@@ -11,7 +11,7 @@
 * Fix generation of "SOP Instance UID" on split and merge
 * Orthanc Explorer: include the url search params into HTTP headers to the Rest API to ease usage of the Authorization plugin
   Note that only the 'token', 'auth-token' & 'authorization' search params are transmitted into HTTP headers.
-
+* in /ordered-slices route, ignore instances without position/normal/seriesIndex
 
 Version 1.5.7 (2019-06-25)
 ==========================
diff -r 6b61ba835964 -r 4bced7d1ec20 OrthancServer/SliceOrdering.cpp
--- a/OrthancServer/SliceOrdering.cpp	Tue Sep 03 10:34:26 2019 +0200
+++ b/OrthancServer/SliceOrdering.cpp	Wed Sep 04 18:23:22 2019 +0200
@@ -310,7 +310,11 @@
     for (std::list<std::string>::const_iterator
            it = instancesId.begin(); it != instancesId.end(); ++it)
     {
-      instances_.push_back(new Instance(index_, *it));
+      std::auto_ptr<Instance> instance(new Instance(index_, *it));
+      if (instance->HasPosition() || instance->HasNormal() || instance->HasIndexInSeries())
+      {
+        instances_.push_back(instance.release());
+      }
     }
   }