don't update updatedAt attribute if timestamps is false

changelog.md

+# v1.3.6 #
+- [BUG] don't update an existing updatedAt-attribute if timestamps option for a DAO is false
+
 # v1.3.5 #
 # v1.3.5 #
-- [BUT] fixed missed DAO renaming in migrations (thanks to nov)
+- [BUG] fixed missed DAO renaming in migrations (thanks to nov)
 
 
 # v1.3.4 #
 # v1.3.4 #
 - [REFACTORING] renamed Model/ModelFactory/ModelFactoryManager to DAO/DAOFactory/DAOFactoryManager
 - [REFACTORING] renamed Model/ModelFactory/ModelFactoryManager to DAO/DAOFactory/DAOFactoryManager

lib/dao.js

@@ -83,7 +83,7 @@ module.exports = (function() {
   DAO.prototype.save = function() {
   DAO.prototype.save = function() {
     var updatedAtAttr = this.__options.underscored ? 'updated_at' : 'updatedAt'
     var updatedAtAttr = this.__options.underscored ? 'updated_at' : 'updatedAt'
 
 
-    if(this.hasOwnProperty(updatedAtAttr))
+    if(this.__options.timestamps && this.hasOwnProperty(updatedAtAttr))
       this[updatedAtAttr] = new Date()
       this[updatedAtAttr] = new Date()
 
 
     if(this.isNewRecord) {
     if(this.isNewRecord) {

spec/dao.spec.js

@@ -374,6 +374,30 @@ describe('DAO', function() {
             }, 10)
             }, 10)
           })
           })
         })
         })
+
+        describe('without timestamps option', function() {
+          var User2 = sequelize.define('User2', {
+            username: Sequelize.STRING,
+            updatedAt: Sequelize.DATE
+          }, {
+            timestamps: false
+          })
+
+          beforeEach(function() {
+            Helpers.async(function(done) {
+              User2.sync({ force: true }).success(done)
+            })
+          })
+
+          it("doesn't update the updatedAt column", function() {
+            Helpers.async(function(done) {
+              User2.create({ username: 'john doe' }).success(function(johnDoe) {
+                expect(johnDoe.updatedAt).toBeNull()
+                done()
+              })
+            })
+          })
+        })
       })
       })
 
 
       describe('updateAttributes', function() {
       describe('updateAttributes', function() {