metosin/malli

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

Open

#596 geöffnet am 17. Dez. 2021

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Clojure (237 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (1.724 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Betrachten Sie die Zeilen 36-42 in der Datei src/malli/transform.cljc. Die an reduce übergebene anonyme Funktion wird ohne Argumente aufgerufen, wenn `keep` eine leere Liste zurückgibt, was zu einem Fehler führt. Zwei mögliche Lösungen werden vorgeschlagen: Machen Sie die anonyme Funktion multi arity, sodass sie nil zurückgibt, wenn sie ohne Argumente aufgerufen wird, oder überprüfen Sie den Rückgabewert von `keep` und reduzieren Sie nur, wenn er nicht leer ist. Überprüfen Sie den umgebenden Code und wählen Sie den einfacheren Ansatz.
Tech Stack
Keine
Domain
backendapi
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Clojure
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65