Make sure model definition is built when attaching to a DS

lib/datasource.js

@@ -412,9 +412,17 @@ DataSource.prototype.attach = function (ModelCtor) {
     // Already attached to the data source
     return;
   }
-  var properties = ModelCtor.dataSource.definitions[ModelCtor.modelName].properties;
-  var settings = ModelCtor.dataSource.definitions[ModelCtor.modelName].settings;
   var className = ModelCtor.modelName;
+  var properties = ModelCtor.dataSource.definitions[className].properties;
+  var settings = ModelCtor.dataSource.definitions[className].settings;
+
+  // redefine the dataSource
+  ModelCtor.dataSource = this;
+  // add to def
+  var def = new ModelDefinition(this, className, properties, settings);
+  def.build();
+  this.definitions[className] = def;
+  this.models[className] = ModelCtor;
 
   this.mixin(ModelCtor);
 
@@ -427,15 +435,6 @@ DataSource.prototype.attach = function (ModelCtor) {
     });
   }
 
-  // redefine the dataSource
-  hiddenProperty(ModelCtor, 'dataSource', this);
-  ModelCtor.dataSource = this;
-  
-  // add to def
-  this.definitions[className] = new ModelDefinition(this, className, properties, settings);
-  
-  this.models[className] = ModelCtor;
-  
   return this;
 };
 

lib/model-definition.js

@@ -41,6 +41,7 @@ function ModelDefinition(modelBuilder, name, properties, settings) {
     }
     this.associations = [];
     this.properties = null;
+    this.build();
 }
 
 util.inherits(ModelDefinition, EventEmitter);