Skip to content

Collection integration of Lazy-Loading, especially Topic.Children #119

Description

@JeremyCaney

Address collection-level issues introduced by lazy loading (#111). LoadState had been added to the generic KeyedTopicCollection<T>, which leaks a the "load state" concept onto non-child collections (e.g., metadata, content-type descriptors) that never need it.

Also, formalize documentation warning future ITopicRepository implementers about the risks of manually setting LoadState and Deferred incorrectly (i.e., data loss or spurious reloads).

  • Introduce a dedicated ChildTopicCollection for Topic.Children, and remove LoadState from the generic base
  • Documentation of risks on the various LoadState and Deferred setters

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Area: RepositoriesRelates to the `ITopicRepository` interface or one of its implementations.Priority: 3Severity 1: MinorStatus 5: CompleteTask is considered complete, and ready for deployment.Type: BugBehavior that is inconsistent with documented or expected behavior.Type: ImprovementImproves the functionality or interface of an existing feature.

Type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions