just reverted getter idea for save, drop, etc.

lib/sequelize/model-definition.js

@@ -34,15 +34,15 @@ ModelDefinition.prototype.sync = function(options) {
       .on('failure', function() { self.emit('failure', self) })
   }
   
-  if(options.force) this.drop.on('success', function() { doQuery() })
+  if(options.force) this.drop().on('success', function() { doQuery() })
   else doQuery()
   
   return this
 }
 
-ModelDefinition.prototype.__defineGetter__('drop', function() {
+ModelDefinition.prototype.drop = function() {
   return this.query(QueryGenerator.dropTableQuery(this.tableName, this.id))
-})
+}
 
 ModelDefinition.prototype.__defineGetter__('all', function() {
   return this.query(QueryGenerator.selectQuery(this.tableName))

lib/sequelize/model.js

@@ -36,16 +36,16 @@ Model.prototype.query = function() {
   return s.query.apply(s, args)
 }
 
-Model.prototype.__defineGetter__('save', function() {
+Model.prototype.save = function() {
   if(this.isNewRecord)
     return this.query(QueryGenerator.insertQuery(this.definition.tableName, this.values))
   else
     return this.query(QueryGenerator.updateQuery(this.definition.tableName, this.values, this.id))
-})
+}
 
-Model.prototype.__defineGetter__('destroy', function() {
+Model.prototype.destroy = function() {
   return this.query(QueryGenerator.deleteQuery(this.definition.tableName, this.id))
-})
+}
 
 Model.prototype.__defineGetter__('isNewRecord', function() {
   return this.id == null

test/Model/sync.js → test/Model/sync+drop.js

@@ -18,5 +18,12 @@ module.exports = {
       bio: Sequelize.TEXT
     })
     User.sync().on('failure', beforeExit)
+  },
+  'drop should work': function(beforeExit) {
+    var User = sequelize.define('User', {
+      name: Sequelize.STRING,
+      bio: Sequelize.TEXT
+    })
+    User.drop().on('success', beforeExit)
   }
 }
\ No newline at end of file