lukeed / lukeed/webpack-plugin-replace

Produces undefined if `values` is empty or undefined

オープン
#6 コメント 0 件 リアクション 8 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
JavaScript
スター
36
フォーク
8
PR マージ指標
30日以内にマージされた PR はありません

説明

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

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

置換用の正規表現を構築し、ファイルの内容に対して replace を呼び出す ReplacePlugin の実装から始めます。示されている設定で問題を再現し、空または undefined の values コレクションによって、undefined の出力が生成されるのではなく元の内容が変更されずに維持されることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, webpack
領域
build-system
issue の種類
バグ
難易度
1/5
見積もり時間
1時間未満
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。