facebook/prop-types

`oneOfType()` does not seem to allow all types specified

Open

#388 aberto em 20 de out. de 2022

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (4.379 stars) (389 forks)batch import
bughelp wanted

Description

I am running into an issue where oneOfType() is passed in types of shape and string but is only allowing string seemingly.

Reproduced here (using a simplified version of types I have encountered on a project I am working on), https://jsfiddle.net/liamness/1du5jvwt/, with the warning:

Failed prop type: Invalid prop `schema[1]` supplied to `ComponentName`, expected one of type [string].

Guia do colaborador

Pilha de tecnologia
javascriptreact
Domain
frontend
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 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
JavaScriptReact prop types
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75
Direção de pesquisa
Investigate the implementation of oneOfType in the prop types source code (likely in src/index.js or similar). Use the provided JSFiddle reproduction to understand the issue: when passing a shape and string type, only string is validated. Look for how oneOfType iterates over types and checks validity. Check if there is a bug in the type checking loop that causes early exit or incorrect validation. Consider submitting a fix that ensures all specified types are properly validated.