Avoids saving an instance if a property is null and is marked as requ…

…ired (#142)
dresende · May 2, 2013 · 19492db · 19492db
1 parent 079fe7f
commit 19492db
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/lib/Instance.js b/lib/Instance.js
@@ -71,6 +71,16 @@ function Instance(opts) {
 			if (!opts.properties.hasOwnProperty(k)) continue;
 			if (opts.data[k] == null && opts.properties[k].hasOwnProperty("defaultValue")) {
 				opts.data[k] = opts.properties[k].defaultValue;
+			} else if (opts.properties[k].required && opts.data[k] == null) {
+				var err = new Error("required");
+				err.field = k;
+				err.value = opts.data[k];
+
+				Hook.trigger(instance, opts.hooks.afterSave, false);
+				if (typeof cb == "function") {
+					cb(err, instance);
+				}
+				return;
 			}
 		}