lukeed / lukeed/webpack-plugin-replace

Produces undefined if `values` is empty or undefined

Aperta
#6 0 commenti 8 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Lingua principale
JavaScript
Stelle
36
Fork
8
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

If values is not specified result is always a bunch of undefined

new ReplacePlugin({
  include: /serviceManager/,
  patterns: [
    {
      regex: /var IS_WEB = [^;]+;/,
      value: 'var IS_WEB = true;'
    }
  ]
})

Calls on file content:

mod._source._value.replace(/(?:)/g, k => vals[k]);

Can be easily fixed:

if(keys.length) {
  mod._source._value.replace(REGEXP, k => vals[k]);
}

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Parti dall'implementazione di ReplacePlugin, che costruisce l'espressione regolare di sostituzione e invoca replace sul contenuto del file. Riproduci il problema con la configurazione mostrata e verifica che una raccolta di values vuota o undefined lasci invariato il contenuto originale invece di produrre un output undefined.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript, webpack
Ambito
build-system
Tipo di issue
Bug
Difficoltà
1/5
Tempo stimato
Meno di un'ora
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.