lukeed / lukeed/webpack-plugin-replace

Produces undefined if `values` is empty or undefined

Offen
#6 0 Kommentare 8 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
36
Forks
8
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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]);
}
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne bei der ReplacePlugin-Implementierung, die den regulären Ausdruck für die Ersetzung erstellt und replace auf den Dateiinhalt anwendet. Reproduziere das Problem mit der gezeigten Konfiguration und verifiziere, dass eine leere oder nicht definierte values-Sammlung den ursprünglichen Inhalt unverändert lässt, anstatt eine undefined-Ausgabe zu erzeugen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, webpack
Bereich
build-system
Issue-Typ
Bug
Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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