Added final changes as requested by sdepold

README.md

@@ -54,7 +54,8 @@ A very basic roadmap. Chances aren't too bad, that not mentioned things are impl
 - Support for update and delete calls for whole tables without previous loading of instances
 - Support for update and delete calls for whole tables without previous loading of instances
 - Eager loading of nested associations [#388](https://github.com/sdepold/sequelize/issues/388#issuecomment-12019099)
 - Eager loading of nested associations [#388](https://github.com/sdepold/sequelize/issues/388#issuecomment-12019099)
 - Model#delete
 - Model#delete
-- Validate a model before it gets saved. (Move validation of enum attribute value to validate method)
+- ~~Validate a model before it gets saved.~~ Implemented in [#601](https://github.com/sequelize/sequelize/pull/601), thanks to @durango
+- Move validation of enum attribute value to validate method
 - BLOB [#99](https://github.com/sequelize/sequelize/issues/99)
 - BLOB [#99](https://github.com/sequelize/sequelize/issues/99)
 - ~~Support for foreign keys~~ Implemented in [#595](https://github.com/sequelize/sequelize/pull/595), thanks to @optilude
 - ~~Support for foreign keys~~ Implemented in [#595](https://github.com/sequelize/sequelize/pull/595), thanks to @optilude
 
 

changelog.md

 # v1.7.0 #
 # v1.7.0 #
+- [FEATURE] Validate a model before it gets saved. [#601](https://github.com/sequelize/sequelize/pull/601), thanks to durango
 - [DEPENDENCIES] replaced underscore by lodash. [#954](https://github.com/sequelize/sequelize/pull/594). thanks to durango
 - [DEPENDENCIES] replaced underscore by lodash. [#954](https://github.com/sequelize/sequelize/pull/594). thanks to durango
 - [BUG] Fix string escape with postgresql on raw SQL queries. [#586](https://github.com/sequelize/sequelize/pull/586). thanks to zanamixx
 - [BUG] Fix string escape with postgresql on raw SQL queries. [#586](https://github.com/sequelize/sequelize/pull/586). thanks to zanamixx
 - [BUG] "order by" is now after "group by". [#585](https://github.com/sequelize/sequelize/pull/585). thanks to mekanics
 - [BUG] "order by" is now after "group by". [#585](https://github.com/sequelize/sequelize/pull/585). thanks to mekanics

lib/dao.js

@@ -218,9 +218,9 @@ module.exports = (function() {
     Utils._.each(self.values, function(value, field) {
     Utils._.each(self.values, function(value, field) {
 
 
       // if field has validators
       // if field has validators
-      var allowsNulls = (self.rawAttributes[field].allowNull && self.rawAttributes[field].allowNull === true && (value === null || value === undefined));
+      var hasAllowedNull = (self.rawAttributes[field].allowNull && self.rawAttributes[field].allowNull === true && (value === null || value === undefined));
 
 
-      if (self.validators.hasOwnProperty(field) && !allowsNulls) {
+      if (self.validators.hasOwnProperty(field) && !hasAllowedNull) {
         // for each validator
         // for each validator
         Utils._.each(self.validators[field], function(details, validatorType) {
         Utils._.each(self.validators[field], function(details, validatorType) {