Skip to content

Commit

Permalink
EXPERIMENTAL: Remove seemingly unused code.
Browse files Browse the repository at this point in the history
I couldn't find any tests or documentation that use this code branch, and the
purpose isn't clear to me (why would you pass in a field object as a value?).
It's a small performance gain to just remove it, but if anyone depends on it,
it could be added back.
  • Loading branch information
mbrubeck committed Mar 31, 2009
1 parent cd833bc commit 6cd1f9e
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions src/active_record/main.js
Expand Up @@ -490,10 +490,6 @@ ActiveRecord = {
if(!this.constructor.fields[key].primaryKey)
{
var value = ActiveRecord.connection.fieldOut(this.constructor.fields[key],this.get(key));
if(Migrations.objectIsFieldDefinition(value))
{
value = value.value;
}
//don't supress notifications on set since these are the processed values
this.set(key,value);
}
Expand Down

0 comments on commit 6cd1f9e

Please sign in to comment.