metosin/malli

malli.transform/-interceptor doesn’t support emptly-ish ?interceptor

Open

#596 aberto em 17 de dez. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Clojure (237 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (1.724 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

https://github.com/metosin/malli/blob/152af3a3f8d59cca37a597b2403111bafbdee6ee/src/malli/transform.cljc#L36-L42

Hi,

I’m hitting a corner case with malli transform. I’m still trying to figure out why I’m hitting that corner case and what I can do to avoid that, but I think there’s also a robustness improvement that’s possible in malli.

When the keep function returns an empty list, the reduce function calls the anonymous function on line 38 with no arguments, and this fails.

One possible fix would be to make the anonymous function multi-arity and make it return nil when called with no arguments. Another possible fix would be to check the return value of keep, and only proceed with reducing it if it is a non-empty-list.

Guia do colaborador

Direção de pesquisa
Examine as linhas 36-42 do arquivo src/malli/transform.cljc. A função anônima passada para reduce é chamada sem argumentos quando `keep` retorna uma lista vazia, causando uma falha. Duas correções possíveis são sugeridas: tornar a função anônima multi arity para retornar nil quando chamada sem argumentos, ou verificar o valor de retorno de `keep` e reduzir apenas se não estiver vazio. Revise o código ao redor e escolha a abordagem mais simples.
Pilha de tecnologia
None
Domain
backendapi
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
Clojure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65