-
-
Notifications
You must be signed in to change notification settings - Fork 361
Refactor cache system #838
Copy link
Copy link
Open
Labels
architectureRelated to the framework architectureRelated to the framework architecturestandards and best practicesCoding guidelinesCoding guidelines
Milestone
Metadata
Metadata
Assignees
Labels
architectureRelated to the framework architectureRelated to the framework architecturestandards and best practicesCoding guidelinesCoding guidelines
Type
Projects
Status
Not Started
The current UF cache package is overly complicated for nothing and inconsistent. For example, we create a new
RedisStore, but have to callgetInstance()to actually get aRedisStore.Instead of creating singleton and fake app to create the
RedisManager, it should probably be easier to directly implement our ownCacheManagerinstead so we don't have to deal with Laravel app instance. Instead of definingRedisManagerin $app it could probably be done in CacheManager directly.TODO
CacheManagerarraystore to disable caching