dcmjs-org / dcmjs

Javascript implementation of DICOM manipulation
https://dcmjs.netlify.com/
MIT License
286 stars 107 forks source link

anonymizer uses incorrect tag names #345

Open magurevitch opened 1 year ago

magurevitch commented 1 year ago

On some tag names in tagNamesToEmpty in dcmjs/src/anonymizer.js, some tags are called out as singular when they should be plural, or are otherwise incorrect

ReferringPhysicianTelephoneNumbers (has: ReferringPhysicianPhoneNumbers) PhysiciansOfRecord (has: PhysicianOfRecord) NameOfPhysiciansReadingStudy (has: NameOfPhysicianReadingStudy) OperatorsName (has: OperatorName) AdmittingDiagnosesDescription (has: AdmittingDiagnosisDescription) Allergies (has: ContrastAllergies) ResponsiblePersonRole (does not have)

pieper commented 1 year ago

Thanks for pointing this out - would you be able to provide a pull request with the fixes?