On 27 April 2013 00:32, Niv Sardi <notifications@github.com> wrote:
>
> In public/js/views/medialist.js:
>
> > @@ -86,17 +89,9 @@ window.MediaListView = function(options){
> >      };
> >
> >      this.save = function (newmodel) {
> > -        var set_piece_id = function(element, index, list) {
> > -            var newid = this.model.get('_id') + '-' + moment().valueOf();
> > -            element.set('_id', newid);
> > -        };
>
> that was more elegant...


But it was plain wrong. We argued a LOT about this and the consensus
was that Piece's ids should be uuids.

We ended up merging an incomplete or different branch. I cant recall now.