Fold `MoveFromCacheTrait` into `OC\Files\Cache\Cache`
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
找到 MoveFromCacheTrait 和 OC\Files\Cache\Cache,然后检查该 trait 的使用位置以及其 fallback 的调用方式。在将 fallback 移动到 Cache 之前,先阅读该类及其相关测试;当该类拥有该行为且相关缓存测试通过时,视为完成。
由索引模型根据 Issue 内容生成。
描述
🧹 item
MoveFromCacheTrait currently appears to be an implementation detail of OC\Files\Cache\Cache rather than a reusable trait.
Since it is only used in production by Cache, I lean toward folding the fallback into the class itself to make ownership and behavior clearer.
I was initially confused about its use case, since no production code uses the trait other than Cache (which I wouldn’t have expected to need the fallback), but then I realized that the source caches that may use this fallback still ultimately flow through Cache, which is the one responsible for deciding whether to use the fallback or not.
Originally posted by @joshtrichards in https://github.com/nextcloud/server/issues/59916#issuecomment-4322195789
- 主要语言
- PHP
- 星标
- 37k
- 派生
- 5.3k
- 平均合并
- 2 天 1 小时
- 30 天内合并 PR
- 723
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
nextcloud/server 的其他 Issue
-
34-feedback
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
enhancement feature: TaskProcessing good first issue
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
0. Needs triage 35-feedback bug
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
0. Needs triage 35-feedback bug
难度 1/5 1 小时以内 新手友好度 92/100
维护者通常 1 天内回复
-
0. Needs triage 35-feedback bug
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 90/100
coollabsio/shoutrrr#190 ·
-
Bug Enhancement Performance
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
Feature Status: Needs Triage
难度 2/5 1-3 小时 新手友好度 73/100
维护者通常 1 天内回复
-
frontend low-priority
难度 2/5 1-3 小时 新手友好度 77/100
mplodowski/dynamicpdf-plugin#336 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
AdvancedCustomFields/acf#1044 ·