Backbone: v1.3.2 backbone.js und backbone-min.js sind nicht identisch

Erstellt am 22. März 2016 · 7Kommentare · Quelle: jashkenas/backbone

Die minimierte Version von 1.3.2 scheint nicht zu enthalten
https://github.com/jashkenas/backbone/commit/1db79ddd33f65e4ca93beb3174496cf148baebe5
während die unminfied Version tut.

Nicht minimiert

Events.once = function(name, callback, context) {
    // Map the event into a `{event: once}` object.
    var events = eventsApi(onceMap, {}, name, callback, _.bind(this.off, this));
    if (typeof name === 'string' && context == null) callback = void 0;
    return this.on(events, callback, context);
  };

Minimiert

l.once=function(t,e,r){
  var n=f(p,{},t,e,i.bind(this.off,this));
  return this.on(n,e,r)
}

@jridgewell @megawac

bug

Quelle

sarkasm

👍1

Alle 7 Kommentare

Ich habe überprüft, dass die diesbezüglichen Tests fehlschlagen, wenn Sie backbone.js in der Datei test / index.html durch backbone-min.js ersetzen

sarkasm am 22. März 2016

Seufzer. @megawac , kannst du eine v1.3.3 PR erstellen, da ich es scheinbar nicht richtig machen kann?

jridgewell am 22. März 2016

👍1

Ich habe gerade https://github.com/jashkenas/backbone/pull/3994 angeboten
Ich überlegte, die Datei index.html zu ändern, um die minimierte Version zu verwenden, war mir aber nicht sicher, ob alle vor dem Testen erstellen würden.

sarkasm am 22. März 2016

👍1

Wird es bald ein Marionetten-kompatibles Minified Backbone v1.3.x geben? noch keine afaik vorhanden

linus-amg am 25. März 2016

1.3.2 funktioniert mit Marionette 2.4.5 @elgubenis

megawac am 26. März 2016

😕1

Tatsächlich haben Mn 2.4.5-Verbundansichten ein Problem mit dem minimierten Backbone 1.3.2. So fand ich dieses Problem zunächst.

CollectionViews werden einmal gebunden
this.once('render', this._initialEvents);
https://github.com/marionettejs/backbone.marionette/blob/master/src/collection-view.js#L25

Zusammengesetzte Ansichten, die CollectionView erweitern
habe _initialEvents, die mit if (this.collection) {
https://github.com/marionettejs/backbone.marionette/blob/master/src/composite-view.js#L23
Zu diesem Zeitpunkt geht der this -Kontext verloren (aufgrund der Notwendigkeit eines Fixes https://github.com/jashkenas/backbone/commit/1db79ddd33f65e4ca93beb3174496cf148baebe5), daher ist diese Sammlung falsch und startet daher nicht angemessen auf die Sammlungsereignisse achten

Seien Sie also vorsichtig, wenn Sie CollectionViews von CompositeViews verwenden, für die vor dem Rendern eine Sammlung festgelegt wurde.

sarkasm am 27. März 2016

👍1

Okay, ja, wir müssen eine Version mit einer aktualisierten minimierten Datei erneut veröffentlichen, aber die backbone.js -Datei auf dem Master sollte mit 2.4.5 funktionieren. (Wir werden bald veröffentlichen)

megawac am 27. März 2016

War diese Seite hilfreich?

0 / 5 - 0 Bewertungen

Backbone: v1.3.2 backbone.js und backbone-min.js sind nicht identisch

Alle 7 Kommentare

Verwandte Themen