[ListBaseComponent] Fix for default selection options

[Tyme-Bleyaert]

Pull Request

📖 Description

Replace item?.Tostring() to determine current selected value as this would result in NamespaceName.MyClass for most cases.
Now uses a hash key as the default and not the GetHashCode for security.

🎫 Issues

Fix #4128

✅ Checklist

General

• I have added tests for my changes.
• I have tested my changes.
• I have updated the project documentation to reflect my changes.
• I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

• I have added a new component
• I have added Unit Tests for my new component
• I have modified an existing component
• I have validated the Unit Tests for an existing component

[dvoituron]

Give me a few days to verify these changes. One initial note: You cannot change the unit tests. If that is required, it is a breaking change.

[Tyme-Bleyaert]

Give me a few days to verify these changes. One initial note: You cannot change the unit tests. If that is required, it is a breaking change.

No rush at all 😊
You’re right that changing tests typically implies a breaking change, but in this case, most of the tests I updated were either checking ToString() outputs or performing *.verified.razor.html comparisons with hardcoded option values. So while it could be a breaking change, I’d expect it to only affect very specific edge cases.

I’m also happy to discuss this further over a Teams call if that would help.

[Tyme-Bleyaert]

Just realized something, when working with a list of strings the reference will be different while the value might be the same.
I need to address that issue first. So i marked the pr as draft again.

[Tyme-Bleyaert]

Ok, that’s better. I was making it way harder than the issue actually was. 😐
Why write many lines when deleting one does the trick.