From: Eduardo DAVILA <davila@localhost.localdomain>
Date: Sat, 26 Oct 2019 09:29:39 +0000 (+0200)
Subject: #3309 BBTK Bug New Normal  -  Bug gdcmvtk - GetXCoherentInfoGdcmReader std - FilesFro... 
X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=commitdiff_plain;h=4ae4c8a57a71c86165a9889e2569e249197f48c0;p=bbtk.git

#3309 BBTK Bug New Normal  -  Bug gdcmvtk - GetXCoherentInfoGdcmReader std - FilesFromDirectory
---

diff --git a/packages/gdcmvtk/src/bbgdcmvtkGetXCoherentInfoGdcmReader.cxx b/packages/gdcmvtk/src/bbgdcmvtkGetXCoherentInfoGdcmReader.cxx
index e553a67..e09b38a 100644
--- a/packages/gdcmvtk/src/bbgdcmvtkGetXCoherentInfoGdcmReader.cxx
+++ b/packages/gdcmvtk/src/bbgdcmvtkGetXCoherentInfoGdcmReader.cxx
@@ -144,6 +144,12 @@ void GetXCoherentInfoGdcmReader::Process()
    // Should only contain one!
    l = sh->GetFirstSingleSerieUIDFileSet();
 
+	if (l==NULL)
+	{	
+		printf("EED GetXCoherentInfoGdcmReader::Process Warnning not coherent list/file \n");
+		return;
+	}
+
    double zspacing = 0.;
    sh->OrderFileList(l); // this one should compute the *actual* Z Spacing!
    zspacing 		= sh->GetZSpacing();
@@ -168,7 +174,6 @@ void GetXCoherentInfoGdcmReader::Process()
    	for ( ; iitt != l->end(); ++iitt)
    	{
 		MapInfoDicom mapinfodicom;
-
 	  	for (iTag=0; iTag<sizeDicomTagsVector; iTag++)
 		{
 			key = bbGetInputDicomTags()[iTag];
@@ -208,7 +213,6 @@ void GetXCoherentInfoGdcmReader::Process()
  // Read the *first* image file (a SET of file names is given as input) 
 	gdcm::Reader *read = new gdcm::Reader();
 	reader->SetFileName( bbGetInputIn()[0].c_str() );
-  
 	bool res = read->Read();  
   	if ( !res )
    	{
@@ -217,8 +221,6 @@ void GetXCoherentInfoGdcmReader::Process()
       	return;
    	}
     
-
-
  // Get info from the first image file
    const gdcm::File &f = read->GetFile();
    int i;
@@ -229,17 +231,17 @@ void GetXCoherentInfoGdcmReader::Process()
    gdcm::Element<gdcm::VR::DS,gdcm::VM::VM6> iop;
    iop.Read( ss );
    for(i=0; i< 6; i++)
-	{
+   {
       	v_iop.push_back((float)(iop[i]));
-	}
+   }
    bbSetOutputIOP(v_iop );
  
-  std::vector<double> v_ipp;
+   std::vector<double> v_ipp;
    const gdcm::DataElement &deIpp = f.GetDataSet().GetDataElement(gdcm::Tag((0x0020,0x0032)));
    deIpp.GetValue().Print(ss);
    gdcm::Element<gdcm::VR::DS,gdcm::VM::VM3> ipp;
    ipp.Read( ss );
-	for(i=0; i< 3; i++)
+   for(i=0; i< 3; i++)
 	{
 		v_ipp.push_back((float)(ipp[i]));
 	} // for 3
@@ -287,7 +289,7 @@ void GetXCoherentInfoGdcmReader::Process()
 
 void GetXCoherentInfoGdcmReader::bbUserSetDefaultValues()
 {
-	reader=NULL;
+   reader=NULL;
    std::vector<std::string> init;
    init.push_back("");
    bbSetInputIn(init);