Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need Int type inside Dicom.Data.DcmDataset.LoadDicomFieldValue for NumberOfStudyRelatedInstances #1

Closed
GoogleCodeExporter opened this issue May 7, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Do a C-Find and try to get either NumberOfStudyRelatedSeries or
NumberOfStudyRelatedInstances - they will return zero

What is the expected output? What do you see instead?
The numbers should not be zero


What version of the product are you using? On what operating system?
.9.5 on Windows, compiling in VS C# Express 2008

Please provide any additional information below.

Okay, I think I found a minor bug inside
Dicom.Data.DcmDataset.LoadDicomFieldValue. In the block starting around like
875, it checks if vtype matches several types, but there is no check for an
Int32 (which is my NumberOfStudyRelatedInstances).  So I added this:

                      } else if (vtype == typeof(Int32)) {
                           string temp = elem.GetValueString();
                           return System.Convert.ToInt32(temp, 10);

around line 884.  Works.  :-)

Best Regards,
Aerik 


Original issue reported on code.google.com by asyl...@gmail.com on 8 May 2008 at 10:53

@GoogleCodeExporter
Copy link
Author

Fixed(?) in SVN

Original comment by colby.di...@gmail.com on 23 Jul 2008 at 11:57

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant