updating APIs, removing obsolete API usage, fixing build warnings

[jiri-malec]

Motivation

• Fixes issue 86198608
• Removing obsolete API, upgrading API references, removing additional build warnings

Checklist

• Code follows coding conventions held in this repo
• Automated tests have been added
• Tests are passing
• Docs have been updated (if applicable)
• Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

If manual testing is required, what are the steps?

[Rumec]

Look at my proposition, please.

If you have any questions, don't hesitate to ask!

[Rumec]

Do we really want to cast allowedScopes to List in the lambda? Why don't we use it like this:

List<string> allowedScopes = ["Kentico.", "DancingGoat.General.", "DancingGoat.LandingPage." ];

return GetWidgetsIdentifiers()
    .Where(id => allowedScopes.Exists(scope => id.StartsWith(scope, StringComparison.OrdinalIgnoreCase)))
    .ToArray();```

[Rumec]

Also, wouldn't be HashSet more suitable?

[jiri-malec]

Yes, good point. Since a HashSet has constant-time complexity for lookups, your solution would be more efficient.

[seangwright]

Just a heads up, I did a lot of this work already in #34

There is some project structured/config that was missing that I also added.

[jiri-malec]

Just a heads up, I did a lot of this work already in #34

There is some project structured/config that was missing that I also added.

Thank you for pointing that out! From my perspective, your merge request addresses all the issues I was targeting with this one. Therefore, it might make sense to close this MR and proceed with #34 instead. @Rumec

[jiri-malec]

All issues were already resolved. Closing this PR