CrearAyT / mbc-playout

MBC Playout Server {mlt edition}
tic.inaes.gob.ar
2 stars 0 forks source link

KO Playlist: error luego de grabar un playlist #2

Open pardo-bsso opened 11 years ago

pardo-bsso commented 11 years ago

Elijo un playlist, modifico (o no) el contenido. Save. Elijo otras del resto una o varias veces, se muestran ok.

Vuelvo a elegir la original, se muestra vacia error en consola: Uncaught CollectionObservable: not a collection is unexpected vendor.js:37068

Si pico en cualquier otra la muestra ok.

josx commented 11 years ago

Crear una nueva graba OK

josx commented 11 years ago

El problema en KB lo hace al utilizar el kb.utils.wrappedObject(view_model) de abajo que no devuelve el modelo.

Y la excepción la tira cuando envía un modelo vacío a this.create_options.store.findOrReplace(model, this.create_options.creator, view_model);

CollectionObservable.prototype._onObservableArrayChange = function(models_or_view_models) {
    var collection, has_filters, model, models, observable, view_model, view_models, _i, _len,

...

      for (_i = 0, _len = models_or_view_models.length; _i < _len; _i++) {
        view_model = models_or_view_models[_i];
        model = kb.utils.wrappedObject(view_model);
        if (has_filters) {
          if (this._modelIsFiltered(model)) {
            continue;
          }
          view_models.push(view_model);
        }
        this.create_options.store.findOrReplace(model, this.create_options.creator, view_model);
        models.push(model);
      }
    }
pardo-bsso commented 11 years ago

Ahi subi algo temporal en 4158a1c2864d8d7ea333ba4a79c3431729a66f44

Hay que ver bien el tema de los modelos. Acá en mi base tengo listas que adentro tienen tanto un 'collection' como un 'models' con el mismo contenido (medias)

josx commented 11 years ago

Si para mi este bardo tiene que ver con eso. Con como tenemos los modelos creados.

pardo-bsso commented 11 years ago

Esta bien que haya elementos repetidos? en el original había un chequeo para que no pase.

josx commented 11 years ago

Para mi deberia poderse repetir medias dentro de un playlist.