fixed removal of unused reference attribute

lib/associations/has-many.js

@@ -10,12 +10,13 @@ module.exports = (function() {
     this.target = targetDAO
     this.target = targetDAO
     this.options = options
     this.options = options
     this.useJunctionTable = this.options.useJunctionTable === undefined ? true : this.options.useJunctionTable
     this.useJunctionTable = this.options.useJunctionTable === undefined ? true : this.options.useJunctionTable
-    this.isSelfAssociation = (this.source.tableName == this.target.tableName)
+    this.isSelfAssociation = (this.source.tableName === this.target.tableName)
 
 
-    this.associationAccessor = this.combinedName = this.options.joinTableName || Utils.combineTableNames(
+    var combinedTableName = Utils.combineTableNames(
       this.source.tableName,
       this.source.tableName,
       this.isSelfAssociation ? (this.options.as || this.target.tableName) : this.target.tableName
       this.isSelfAssociation ? (this.options.as || this.target.tableName) : this.target.tableName
     )
     )
+    this.associationAccessor = this.combinedName = (this.options.joinTableName || combinedTableName)
 
 
     var as = (this.options.as || Utils.pluralize(this.target.tableName))
     var as = (this.options.as || Utils.pluralize(this.target.tableName))
 
 
@@ -43,7 +44,7 @@ module.exports = (function() {
         this.foreignIdentifier = Utils._.underscoredIf((this.options.as || this.target.tableName) + 'Id', this.options.underscored)
         this.foreignIdentifier = Utils._.underscoredIf((this.options.as || this.target.tableName) + 'Id', this.options.underscored)
       } else {
       } else {
         this.foreignIdentifier = this.target.associations[this.associationAccessor].identifier
         this.foreignIdentifier = this.target.associations[this.associationAccessor].identifier
-        delete this.source.attributes[this.foreignIdentifier]
+        delete this.source.rawAttributes[this.foreignIdentifier]
       }
       }
 
 
       // define a new model, which connects the models
       // define a new model, which connects the models

spec/associations/has-many.spec.js

@@ -122,5 +122,18 @@ describe('Associations', function() {
         })
         })
       })
       })
     })
     })
+
+    describe('many to many', function() {
+      it("removes the reference id, which was added in the first place", function() {
+        var User = sequelize.define('User', { username: Sequelize.STRING })
+          , Task = sequelize.define('Task', { title: Sequelize.STRING })
+
+        User.hasMany(Task)
+        expect(Task.attributes.UserId).toBeDefined()
+
+        Task.hasMany(User)
+        expect(Task.attributes.UserId).not.toBeDefined()
+      })
+    })
   })
   })
 })
 })