Specific Character Set with MultiValue content fails on import

[Ed McDonagh]

get_values.safe_strings() is not able to cope with a multi-value specific character set value. And it is marked as probably unnecessary - the decoding is now done in the .decode() at the start.

Propose removing the function from the code.

[Ed McDonagh]

Removed the use of safe_strings and passing of the character set from rdsr. Refs #866.

→ <<cset 19c77808d67c>>

[Ed McDonagh]

Added MultiValue SpecificCharacterString and PerformingPhysicianName with escape character, and test of import. Refs #866. PersonName is copied from pydicom examples which are taken from David Clunie's examples.

→ <<cset 2e88dd5ffba7>>

[Ed McDonagh]

Refactored patient_attribute_modules into extract_common, tidying up whilst working on ref #866.

→ <<cset f0c6884c0959>>

[Ed McDonagh]

Sorted out import order for extract_common whilst working on ref #866.

→ <<cset 09620d4a34f7>>

[Ed McDonagh]

Removed SpecificCharacterSet variables from dx. Moved imports to top. Refs #866.

→ <<cset 58ac915b9315>>

[Ed McDonagh]

Removed SpecificCharacterSet variables from ct_philips. Moved imports to top. Refs #866.

→ <<cset 3ecb5c4056d6>>

[Ed McDonagh]

Running black on ref #866.

→ <<cset dc9c1a4a53b2>>

[Ed McDonagh]

Requesting pylint not to complain about import order for modles imports. Refs #866.

→ <<cset eaf9029f7c0f>>

[Ed McDonagh]

Added ref #866 to changes.

→ <<cset 6c69a7aaff87>>

[Ed McDonagh]

Removing redundant safe_strings function. Refs #866

→ <<cset 7e88e0848043>>

[Ed McDonagh]

Codacy - too many local variables. Refs #866

→ <<cset 1dfe059c6b61>>

[Ed McDonagh]

Codacy - switching imports around. Refs #866

→ <<cset 0263ba830fe0>>

[Ed McDonagh]

Codacy - might satisfy inconsistent-return-statements. Refs #866

→ <<cset cafde5183bcd>>

[Ed McDonagh]

Merged in issue866MultiValCharSet (pull request #429)

Fixes #866

→ <<cset 547bade5b9ef>>