move caching logic from workspace to interceptor - Githubissues

kukeiko / entity-space

MIT License

5 stars 1 forks source link

move caching logic from workspace to interceptor #210

Closed kukeiko closed 11 months ago

kukeiko commented 1 year ago

What

Instead of the EntityWorkspace handling caching logic, move it to interceptors

Why

consistency with having data loading logic inside interceptors
reduce responsibility of EntityWorkspace (possibly moving towards making it some kind of configurable container component only)
should help simplifying data loading logic
i think it opens up some doors for future features - just have a gut feeling, nothing i could conretely document yet though

How

Currently looking at the following idea: create two new interceptors LoadFromCacheInterceptor & WriteToCacheInterceptor

LoadFromCacheInterceptor is the first interceptor to run, and
WriteToCacheInterceptor is the last interceptor to run

Both interceptors would have a reference to some IEntityDatabase to load from / write to.