yahoo / yahoo/serialize-javascript
The regExp of IS_PURE_FUNCTION & IS_ARROW_FUNCTION seem to be extra???
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 2.9k
- Fork
- 215
- Merge medio
- 1g 12h
- PR unite (30g)
- 2
Descrizione
source code :
var IS_PURE_FUNCTION = /function.?(/;
var IS_ARROW_FUNCTION = /.?=>.*?/;
// pure functions, example: {key: function() {}}
if(IS_PURE_FUNCTION.test(serializedFn)) {
return serializedFn;
}
// arrow functions, example: arg1 => arg1+5
if(IS_ARROW_FUNCTION.test(serializedFn)) {
return serializedFn;
}
Q:
when I use serialize like:
{
name: "aa",
attrname: "bb",
render(item, row) {
return JSON.parse(item) .map((it) => it.name).join("");
},
}
the regExp will catch this and cause problem, not when it's commented.
then I test the normal way like this:
{
name: "aa",
attrname: "bb",
render: (item, row) => {
return JSON.parse(item) .map((it) => it.name).join("");
},
}
and:
{
name: "aa",
attrname: "bb",
render: function(item, row) {
return JSON.parse(item) .map((it) => it.name).join("");
},
}
without the regExps, these are also correct.
could you please explain me?
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia tracciando dove sono definiti IS_PURE_FUNCTION e IS_ARROW_FUNCTION e dove vengono applicati a serializedFn, quindi confronta i tre esempi di rendering nell’issue. Determina se le funzioni freccia annidate vengono classificate erroneamente e documenta il comportamento previsto o un test riproducibile che fallisce.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100