If I create a new model after having installed, I receive database errors about sails-permissions

[tjwebb]

Thanks for the report, I'll look into it.

[tjwebb]

What adapter are you using? It does attempt to create duplicate, but should just ignore the error.

btw, I think findOrCreate is going to be deprecated. Since this is a one-time operation, I'll just check for existence first

[tomhallam]

Using the stock mongodb adapter.

Findorcreate is far from perfect but I think it's valuable, can you link me
to deprecation thread?

~T
On 25 Feb 2015 20:29, "Travis Webb" [email protected] wrote:

What adapter are you using? It does attempt to create duplicate, but
should just ignore the error.

btw, I think findOrCreate is going to be deprecated

—
Reply to this email directly or view it on GitHub
#19 (comment)
.

[tjwebb]

balderdashy/waterline#843

Check the latest release. I check for existence before creating the models.

[heygrady]

Fresh install of [email protected] and [email protected]. Getting errors after restarting sails because the fixtures have already created all of my records. Using [email protected].

I was able to fix it by wrapping the create calls in a findOne.

In sails-permissions/config/fixtures/model.js

exports.createModels = function () {
  sails.log('Installing Models');

  var models = _.compact(_.map(sails.controllers, function (controller, name) {
    var model = sails.models[name];
    return model && model.globalId && model.identity && {
      name: model.globalId,
      identity: model.identity,
      attributes: _.omit(model.attributes, _.functions(model.attributes))
    };
  }));

  return Promise.map(models, function (model) {
    sails.log('creating', model.name);
    // return Model.create(model);
    // wrap the `create` in a `findOne` to check for existence
    return Model.findOne({ name: model.name }).then(function(existing) { return existing || Model.create(model); });
  });
};

In sails-permissions/config/fixtures/role.js

exports.create = function () {
  return Promise.all([
    Role.findOne({ name: 'admin' }).then(function(existing) { return existing || Role.create({ name: 'admin' }); }),
    Role.findOne({ name: 'registered' }).then(function(existing) { return existing || Role.create({ name: 'registered' }); }),
    Role.findOne({ name: 'public' }).then(function(existing) { return existing || Role.create({ name: 'public' }); })
  ]);
};

[tjwebb]

The unit tests use sails-disk so I missed this one for awhile