Redifine the default 'id' field?

[cliftonc]

Hi,

Love your library, am definitely going to start using it, and as I figure out how it works hopefully contribute something back ;)

Quick q. - would it be possible to enable over-ride of the identifier? e.g. I want to retrofit this against an existing database, I can't do that if that existing database has a different naming convention for the identifier field.

Thanks!
Clifton

[sdepold]

OK here are some lines of testcode, which might be interesting for you:

https://github.com/sdepold/sequelize/blob/rewrite/test/Sequelize/define.js#L30
https://github.com/sdepold/sequelize/blob/rewrite/test/Sequelize/define.js#L36
https://github.com/sdepold/sequelize/blob/rewrite/test/Model/find-findAll-all.js#L63

At this very moment, the rewrite version allows you to specify an attribute as primary key using the flag primaryKey: true.
If one primaryKey is defined manually, sequelize will not add an id column. Please let me know, what you think and if that solves your problem.

[cliftonc]

Excellent - should do the trick, let me test it out and I'll get back to you.