Page MenuHomePhabricator

Remove old deprecated property names of DICOM information
Open, NormalPublic

Description

We still generate the deprecated properties with DICOM information like "modality" or "dicom.series.*". I think they where marked and communicated long enough as deprecated.

Therefore I think we should rework the DICOMReader to not store the properties with the old names.

The backwards compatibility in the rest of the mitk code should stay to be able to handle old scene files and co.

Event Timeline

floca triaged this task as Normal priority.Wed, Feb 12, 11:50 AM
floca created this task.

FYI: When changing/removing properties there's always the chance that old mitk scene files are suddenly incompatible as properties are stored in them. If that's an issue, property aliases may be a solution for a legacy mapping between property names.

floca added a comment.Wed, Feb 12, 3:33 PM

I know. I was just talking about removing the code from our DICOMReader, that gernerates these deprecated properties. The rest could stay as is, so scene files should should not be feel an impact. And we also have the mapping code in place.