Added initial setup of test with comments for explanation:- Test is committed in a WorkInProgress category as the timebox limit was reached and the test could not be made to passRelated to Issue [MWDAM-1529]
Fixed ThrowIfArgumentNull to give proper message.Fixed use of message CsvHeaderNullOrEmpty (it was not found because name was different).Improved tests to see if expected message was given for header and splitter.Related to Issue [MWDAM-1556]
Reverted revision(s) 2602 from DamClients/DamUI/trunk/src/DamClientsLibrary/Deltares.Dam.Data/Translations.xml: change of names accidentally lead to RequiredRequired in translation ids, I changed that back.Related to Issue [MWDAM-1521]
Cleanup code due to SonarCube analysis- Use ArgumentNullException i.s.o. NullReferenceException (implement ThrowHelper)- Move strings to resourcesRelated to Issue [MWDAM-1556]
Improved handling of import errors for TimeDataSeries. Program no longer crashes, the error message now shows the relevant information.Made sure that inner exception message is shown when program crashes.Related to Issue [MWDAM-1457]
Fixed update logic of the cross-section view:- View wouldn't update when characteristic points were assigned or unassigned from the property control of a surface line. Subscribing to the OnAfterChange events coming from a CharacteristicPoint AND calling the LoadLocation() solves this problem.Related to Issue [MWDAM-1529]