Automatic LNode removals and adding LNode interface

[danyill]

Describe the bug

We expect to use LNode elements in the Substation section with references to the Protection 1 and Protection 2 systems.

We also expect to use separate files for the Protection 1 and Protection 2 system due to incompatibilities with ICTs.

Currently, OpenSCD silently and automatically removes LNode entries which do not have a valid iedName attribute.

Additionally when unreferenced LNodes exist, OpenSCD opens the new LNode screen at an incorrect initial starting point.

To Reproduce

Steps to reproduce the behavior:

The attached file is an extracted Substation section without some of the IEDs and their instantiated LNodes:

Prot1_Test_File_With_Substation.scd.zip

1. Open the attached file.
2. Go to the BusPhysConnection for one of the buses within the Substation editor.
3. Click on the Add LNode menu item from the drop down menu:
4. Click the three dots and select, "Add reference to existing logical node"
5. Notice that the screen is already partway through the wizarding process (we have jumped the IED selection process).
6. Return to the IED selection screen.
7. Select any IED and then any logical node and then select Save
8. The existing logical nodes are removed:

Expected behavior

1. When a reference is added to an existing logical node the starting screen should be the IED selection screen.
2. Invalid or incorrect references to IEDs in the LNode section should not be automatically removed. The SSD file may be managed without instantiated IEDs.

Screenshots

See above.

Additional Information

See

• IEC 61850-6 Ed 2.1, 9.2.6, 9.2.7
• IEC 61850-7-1 Ed 2.1, Annex B

[trusz]

As discussed this seems to be two separate bugs:

• we start the wizard at a strange point
• we remove stuff we did not even wanted to touch

[JakobVogelsang]

@danyill The file you have pushed does not have any Substation section. It is hard for me to reproduce.

[danyill]

Sorry @JakobVogelsang I attached an incorrect file. Please use the attached:

TPMulticastNaming-Test-2Bus-Prot2 (1).scd.zip

[github-actions[bot]]

Hello there,

Thank you for opening this issue! We appreciate your interest in our project. However, it seems that this issue hasn't had any activity for a while. To ensure that our issue tracker remains organized and efficient, we occasionally review and address stale issues.

If you believe this issue is still relevant and requires attention, please provide any additional context, updates, or details that might help us understand the problem better. Feel free to continue the conversation here.

If the issue is no longer relevant, you can simply close it. If you're uncertain, you can always reopen it later.

Remember, our project thrives on community contributions, and your input matters. We're here to collaborate and improve. Thank you for being part of this journey!