Rename `AttributeDictionary` to `AttributeValueDictionary` about ontopic-library HOT 2 CLOSED

This was resolved in 0b62fe0.

from ontopic-library.

This appears to be based on either an outdated issue or a faulty initial analysis.

If I recall, the initial issue was relevant to cshtml files, which rely heavily on classes in the ViewFeatures namespace (as the name suggests), and thus rely on that namespace being in scope. That said, the OnTopic.Attributes namespace is rarely needed in cshtml files since it pertains to the entity. And while AttributeDictionary is used by view models, it is exclusively used as a piece of scaffolding for the ITopicMappingService, and isn't publicly exposed via the view models, or accessed via the views. As a result, the scenarios where the two AttributeDictionary classes are used in the same context should be incredibly rare. (It does happen, but mostly when intercepting the view logic, as is done via TopicViewResult and TopicViewResultExecutor—which is a backend implementation detail, and not a common scenario for implementations).

It is conceivable that something has changed in how the .NET SDK handles global using directives with ASP.NET Core applications, thus resolving conflicts that previously existed (e.g., by exposing global using directives to cshtml files). That said, this isn't confirmed.

Regardless, this conflict doesn't appear to be an issue, and thus we should revert this work, preferring AttributeDictionary to AttributeValueDictionary. This has been done on the OnTopic Library (c0152cc) and the OnTopic Editor (be0b76c).

from ontopic-library.