PN tags interpretation as union between object and string breaks DIMSE functionality about dcmjs HOT 2 CLOSED

Thank you @dmlambo! Your explanation made me realize that it was easier than I thought! dcmjs-dimse is now using the latest dcmjs release and all unit tests are updated accordingly!

from dcmjs.

The type (json/part10) depends on whether you call toString or you call JSON.stringify on it. The former creates a part10 string (A^B==C), the latter a DICOM json string ({Alphabetic: 'A^B', Phoenetic='c'}). The tests should give good examples of this:

            jsonDataset.PerformingPhysicianName = { Alphabetic: "Doe^John" };

            expect(String(jsonDataset.PerformingPhysicianName)).toEqual(
                "Doe^John"
            );
            expect(JSON.stringify(jsonDataset.PerformingPhysicianName)).toEqual(
                '[{"Alphabetic":"Doe^John"}]'
            );

(This applies to denaturalized datasets as well.)

In terms of adding an option to change the behaviour, it seems like an anti-pattern. You'd probably be better off adding a fix on the client side. Can you point out the exact scenario where this breaks in dcmjs-dimse? I might be able to provide guidance on a fix.

from dcmjs.