browserify / browserify/static-module

Nested static module replacements don't work

Offen
#16 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
bug
Vorherrschende Sprache
JavaScript
Sterne
75
Forks
22
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

I think an example is the easiest way to explain this.

``` js
var staticModule = require('static-module');

var rs = require('stream').Readable();
rs.push("var foo = require('foo');\n");
rs.push("window.x = foo(1, foo(2, 3, 4), foo(5, 6, 7));\n");
rs.push(null);

var sm = staticModule({
foo: function (x, y, z) {
return 'bar(' + x + ', ' + y + ', ' + z + ')'
}
});

rs.pipe(sm).pipe(process.stdout);

// expected: bar(1, bar(2, 3, 4), bar(5, 6, 7));
// actual: bar(1, bar(2, 3, 4), bar(5, 6, 7))bar(2, 3, 4)bar(5, 6, 7);
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Verwende das Beispiel zur Ersetzung von verschachteltem foo(...) und der Stream-Pipeline im Issue als Reproduktion. Vergleiche die erwartete und die tatsächliche Ausgabe und überprüfe anschließend, dass jede verschachtelte Ersetzung einmal ausgegeben wird und das abschließende Semikolon erhalten bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, node.js
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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