nextcloud/server

Migrations generated via occ fail psalm validation

Aperta

#34.555 aperta il 12 ott 2022

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)PHP (4865 fork)batch import
1. to developfeature: install and updategood first issueneeds reviewtechnical debt

Metriche repository

Star
 (34.953 stelle)
Metriche merge PR
 (Merge medio 20g) (627 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il problema di riferimento #34488 per il contesto. Controlla la classe OCP\Migration\SimpleMigrationStep per vedere l'attuale hint di tipo per il metodo changeSchema. La correzione comporta o il ripristino della modifica che ha introdotto il tipo più specifico o l'aggiornamento della classe base per accettare un tipo di chiusura più specifico. Controlla la cronologia Git per il commit rilevante.
Tech stack
php
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitPHP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70