FIX-2009 Update logCurveInfo properties

[jaroslavbliznak]

Fixes

This pull request fixes #2009

Description

Type of change

• Bugfix
• New feature (non-breaking change which adds functionality)
• Enhancement of existing functionality
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Impacted Areas in Application

• Frontend
• API
• WITSML
• Other (please describe)

Checklist:

Communication

• I have made corresponding changes to the documentation
• PR affects application security

Code quality

• I have self-reviewed my code
• No new warnings are generated

Test coverage

• New code is covered by passing tests

Further comments

[janmarius]

I added suggestions and comments on minor issues. During testing, I also noticed changing the index curve was possible. Is this expected behavior?

[jaroslavbliznak]

I have fixed the modification if it is the index curve. In this case, it is not possible to rename the mnemonic.

[janmarius]

I am unable to change the name of the mnemonic (error shown in image below). Additionally, it is still possible to modify the unit of the index curve. Should this be allowed?

[jaroslavbliznak]

This is an error with overlapping indexes during editing, and it only occurs on the test server for Petrolink (https://eqwitsml.demo.petrolink.net/Petrovault/WITSML/WMLS.asmx). Asbjørn has already reported a message about it to them. It should work fine on other servers. The unit should be editable for the index curve as well.

[janmarius]

Reviewer checklist

• Read issue and PR
• Pulled branch and manually tested
• Verify that PR resolves issue
• Reviewed the code