nextcloud/server

Migrations generated via occ fail psalm validation

Open

#34.555 aberto em 12 de out. de 2022

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)PHP (4.865 forks)batch import
1. to developfeature: install and updategood first issueneeds reviewtechnical debt

Métricas do repositório

Stars
 (34.953 stars)
Métricas de merge de PR
 (Mesclagem média 17d 3h) (606 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Investigue o problema referenciado #34488 para contexto. Examine a classe OCP\Migration\SimpleMigrationStep para ver a dica de tipo atual para o método changeSchema. A correção envolve reverter a alteração que introduziu o tipo mais específico ou atualizar a classe base para aceitar um tipo de closure mais específico. Verifique o histórico do Git para o commit relevante.
Pilha de tecnologia
php
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitPHP
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70