make sure that sources is either TODO or empty

Author: Remi-Gau

.gitignore

@@ -12,6 +12,7 @@ lib/JSONio
 coverage_html
 coverage.xml
 test_report.log
+mapping.md
 
 ## MATLAB / OCTAVE gitignore template
 

src/spm_2_bids.m

@@ -149,8 +149,15 @@
     json = bids.derivatives_json(bf.filename);
 
     content = json.content;
+
     content.RawSources = identify_rawsources(file, verbose);
-    content.Sources = identify_sources(file, map, verbose);
+
+    Sources = identify_sources(file, map, verbose);
+    if ~isempty(Sources)
+        content.Sources = Sources;
+    else
+        content = rmfield(content, 'Sources');
+    end
 
     if isfield(bf.entities, 'space') && strcmp(bf.entities.space, 'IXI549Space')
         content.SpatialReference  = struct('IXI549Space', ...

src/utils/identify_sources.m

@@ -120,10 +120,12 @@
                 %                 'meanu'
                 %                 'meanua'
                 %                };
+                sources = 'TODO';
                 return
 
             else
                 % no idea
+                sources = 'TODO';
                 return
 
             end
@@ -136,6 +138,7 @@
 
     sources{1, 1} = fullfile(bf.bids_path, new_filename);
 
+    % for normalized images
     if add_deformation_field
 
         % for anatomical data we assume that

tests/test_spm_2_bids_metadata.m

@@ -8,6 +8,20 @@
     initTestSuite;
 end
 
+function test_spm_2_bids_metadata_source_must_be_empty()
+
+    file = 'msub-01_T1w.nii';
+
+    [~, ~, json] = spm_2_bids(file, [], false);
+
+    assertEqual(fieldnames(json), {'filename'; 'content'});
+    assertEqual(fieldnames(json.content), {'Description'; 'RawSources'; 'SpatialReference'});
+    assertEqual(json.content.RawSources{1}, 'sub-01/sub-01_T1w.nii');
+
+    bids.util.jsonencode(json.filename, json.content);
+
+end
+
 function test_spm_2_bids_metadata_anat()
 
     file = 'wmsub-01_T1w.nii';