Fix: Use ObservableCollection / avoid multiple calls

[BornToBeRoot]

Changes proposed in this pull request

• Re-select the network interface after a network change or configuration update.

Related issue(s)

• Fixes Network Interface - Reload after config change doesn't select last selected interface #2657

Copilot generated summary

Provide a Copilot generated summary of the changes in this pull request.

Copilot summary

This pull request includes several changes to the NetworkInterfaceViewModel class in the Source/NETworkManager/ViewModels/NetworkInterfaceViewModel.cs file. The main improvements are the transition from List to ObservableCollection for network interfaces, and the addition of checks to prevent multiple reloads of network interfaces.

Changes to data structures and initialization:

• Changed _networkInterfaces from List<NetworkInterfaceInfo> to ObservableCollection<NetworkInterfaceInfo> to better support dynamic updates in the UI.
• Updated the NetworkInterfaces property to use ObservableCollection<NetworkInterfaceInfo>.

Improvements to network interface loading:

• Modified LoadNetworkInterfaces to add network interfaces directly to the ObservableCollection instead of replacing the entire collection.
• Added a check in ReloadNetworkInterfaces to prevent multiple reloads if a reload is already in progress.
• Enhanced ReloadNetworkInterfaces to clear and repopulate the ObservableCollection on the UI thread to ensure thread safety and UI consistency.

To-Do

• Update changelog to reflect this changes

Contributing

By submitting this pull request, I confirm the following:

• I have read and understood the contributing guidelines and the code of conduct.
• I have have added my name, username or email to the contributors list or don't want to.
• The code or resource is compatible with the GNU General Public License v3.0.