Remove non-schema properties on reload

1602 · Dec 11, 2011 · 44267a3 · 44267a3
1 parent 877e309
commit 44267a3
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/lib/abstract-class.js b/lib/abstract-class.js
@@ -406,6 +406,9 @@ AbstractClass.prototype.reload = function (cb) {
 AbstractClass.prototype.reset = function () {
     var obj = this;
     Object.keys(obj).forEach(function (k) {
+        if (k !== 'id' && !obj.constructor.schema.definitions[obj.constructor.modelName].properties[k]) {
+            delete obj[k];
+        }
         if (obj.propertyChanged(k)) {
             obj[k] = obj[k + '_was'];
         }

diff --git a/test/common_test.js b/test/common_test.js
@@ -179,6 +179,20 @@ function testOrm(schema) {
         });
     });
 
+    it('should save only schema-defined field in database', function (test) {
+        Post.create({title: '1602', nonSchemaField: 'some value'}, function (err, post) {
+            test.ok(!post.nonSchemaField);
+            post.a = 1;
+            post.save(function () {
+                test.ok(post.a);
+                post.reload(function (err, psto) {
+                    test.ok(!post.a);
+                    test.done();
+                });
+            });
+        });
+    });
+
     it('should not create new instances for the same object', function (test) {
         var title = 'Initial title';
         Post.create({ title: title }, function (err, post) {