bigskysoftware / bigskysoftware/_hyperscript

Bug: Make `add` command safe for null values

Offen
#269 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
3.8k
Forks
172
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

It seems like the `add` command does not accept null values, as in, I cannot say `add [@attribute=true] to `

Here's my example case: I have this hyperscript statement in a behavior

```hyperscript
add [@type="button"] to in me
```

In HTML, if this behavior is called on a node that does not contain any buttons, this breaks, complaining that ` in me` evaluates to null.

I think there are already other workarounds to use for this, for instance, the `tell` command should handle this better. But it would be nice for `add` to handle this gracefully, too.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die Implementierung des Befehls `add` und die Tests für Befehle zu finden, die Nullwerte erhalten. Reproduziere das gezeigte Verhalten mit einem fehlenden ``-Ziel, vergleiche es mit der Verarbeitung durch den Befehl `tell` und bestätige, dass `add` nicht mehr fehlschlägt, wenn sein Ziel zu null ausgewertet wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
frontend
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.