fix(model): bulkCreate now sets isNewRecord: false on returned instances, closes #4522

changelog.md

 # Next
-- [FIXED] Calling set with dot.separated key on a JSON/JSONB attribute will not flag the entire object as changed [#4379](https://github.com/sequelize/sequelize/pull/4379)
 - [ADDED] Expose Association constructor as `Sequelize.Association`
+- [FIXED] Calling set with dot.separated key on a JSON/JSONB attribute will not flag the entire object as changed [#4379](https://github.com/sequelize/sequelize/pull/4379)
+- [FIXED] instances returned from `bulkCreate` now has `isNewRecord: false` and should be updateable if using `returning: true` with dialects that support it.
 
 # 3.9.0
 - [ADDED] beforeRestore/afterRestore hooks [#4371](https://github.com/sequelize/sequelize/issues/4371)

lib/model.js

@@ -2137,6 +2137,7 @@ Model.prototype.bulkCreate = function(records, options) {
         if (Array.isArray(results)) {
           results.forEach(function (result, i) {
             instances[i].set(self.primaryKeyAttribute, result[self.rawAttributes[self.primaryKeyAttribute].field], {raw: true});
+            instances[i].isNewRecord = false;
           });
         }
         return results;

test/integration/model/create.test.js

@@ -1450,6 +1450,21 @@ describe(Support.getTestDialectTeaser('Model'), function() {
       });
     });
 
+    it('should set isNewRecord = false', function() {
+      var self = this
+        , data = [{ username: 'Peter', secretValue: '42', uniqueName: '1' },
+                  { username: 'Paul', secretValue: '23', uniqueName: '2'}];
+
+      return this.User.bulkCreate(data).then(function() {
+        return self.User.findAll({order: 'id'}).then(function(users) {
+          expect(users.length).to.equal(2);
+          users.forEach(function (user) {
+            expect(user.isNewRecord).to.equal(false);
+          });
+        });
+      });
+    });
+
     it('saves data with single quote', function() {
       var self = this
         , quote = "Single'Quote"