sdepold/sequelize — pull request #315 by guersam (2012-11-06 12:16:17+00:00)
Title: Local options inherits global options properly

Hi,

It's revised version of pull request #312.

I found 'define' and 'sync' options in Sequelize object overwrite individual tables' options.

For example, when I set 'collate' as 'utf8_general_ci' in general and set 'utf8_bin' of a table, it's overwritten as 'utf8_general_ci'.

I think the right behavior is to allow overwriting global options by setting more specific ones.

In addition, since module 'Utils' contains underscore.js, I replaced Utils.merge with Utils._.extend, which is more general and already used in many more lines than Utils.merge is.

The seconds commit fixes #304 as discussed in #307.

Regards,
Jisoo