Skip to content

Refactor cache system #38

Description

@lcharette

The current UF cache package is overly complicated for nothing and inconsistent. For example, we create a new RedisStore, but have to call getInstance() to actually get a RedisStore.

Instead of creating singleton and fake app to create the RedisManager, it should probably be easier to directly implement our own CacheManager instead so we don't have to deal with Laravel app instance. Instead of defining RedisManager in $app it could probably be done in CacheManager directly.


TODO

  • Refactor cache package with our own CacheManager
  • Allows for multiple cache store to be used from the cache service
  • Add a cache Facade
  • Add array store to disable caching

Activity

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

Metadata

Metadata

Assignees

Labels

Projects

Relationships

None yet

Development

No branches or pull requests

Issue actions