nextcloud/server

Migrations generated via occ fail psalm validation

Open

#34,555 创建于 2022年10月12日

在 GitHub 查看
 (3 评论) (0 反应) (0 负责人)PHP (4,865 fork)batch import
1. to developfeature: install and updategood first issueneeds reviewtechnical debt

仓库指标

Star
 (34,953 star)
PR 合并指标
 (平均合并 17天 3小时) (30 天内合并 606 个 PR)

描述

Actually such migrations fail psalm:

Error: ….php:42:56: MoreSpecificImplementedParamType: Argument 2 of …::changeSchema has the more specific type 'Closure():OCP\DB\ISchemaWrapper', expecting 'Closure' as defined by OCP\Migration\SimpleMigrationStep::changeSchema (see https://psalm.dev/140)

So I guess we have to revert it or update the base class as well?

Originally posted by @nickvergessen in https://github.com/nextcloud/server/issues/34488#issuecomment-1275904697

贡献者指南

研究方向
查阅引用的议题 #34488 以了解背景。检查 OCP\Migration\SimpleMigrationStep 类，查看 changeSchema 方法当前的类型提示。修复方案包括：要么回滚引入更具体类型的更改，要么更新基类以接受更具体的闭包类型。检查相关提交的 Git 历史记录。
技术栈
php
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
GitPHP
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70