Skip to content

Commit 83392de

Browse files
authored
Merge pull request #829 from Project-MONAI/AC-2259
fix for silly mistake
2 parents 2045f5e + 52a423e commit 83392de

File tree

1 file changed

+18
-7
lines changed

1 file changed

+18
-7
lines changed

src/TaskManager/Plug-ins/Email/EmailPlugin.cs

+18-7
Original file line numberDiff line numberDiff line change
@@ -178,22 +178,33 @@ private async Task<Dictionary<string, List<string>>> AddRawMetaFromFile(Dictiona
178178

179179
foreach (var item in _includeMetadata)
180180
{
181-
var values = new List<string>();
182-
if (metadata.ContainsKey(item)) values = metadata[item];
183-
184-
DicomTag tag;
181+
DicomTag? tag = null;
185182
try
186183
{
187184
tag = DicomDictionary.Default[item];
188185
}
189186
catch (Exception)
190187
{
191-
tag = DicomTag.Parse(item);
188+
try
189+
{
190+
tag = DicomTag.Parse(item);
191+
}
192+
catch (Exception)
193+
{
194+
//empty on purpose
195+
}
192196
}
193197
if (tag is not null)
194198
{
195-
values.Add(dcmFile.Dataset.GetString(tag).Trim());
196-
metadata.Add(item, values);
199+
var value = dcmFile.Dataset.GetString(tag).Trim();
200+
if (metadata.ContainsKey(item))
201+
{
202+
metadata[item].Add(value);
203+
}
204+
else
205+
{
206+
metadata.Add(item, new List<string> { value });
207+
}
197208
}
198209
}
199210
}

0 commit comments

Comments
 (0)