Merge pull request #53 from jelhan/fixes-52

Fixes #52
balderdashy · Jun 6, 2016 · 149a481 · mikermcneil · Aug 9, 2016 · 149a481
2 parents de8ed6f + 064c744
commit 149a481
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/lib/database.js b/lib/database.js
@@ -627,6 +627,11 @@ Database.prototype.enforceUniqueness = function(collectionName, values, pkValueO
       // (they shouldn't have been stored anyway)
       if (_.isUndefined(values[attrName])) continue;
 
+      // Ignore uniqueness check on null values
+      // null should be ignored in unique constraint following SQL:2011 standard
+      // even so some DBMS (eg. Microsoft SQL Server) are not standard compliant
+      if (_.isNull(values[attrName])) continue;
+
       // If `pkValueOfFirstRecordInResultSet` was provided, this must be an update,
       // and we're updating the record with this pk value.
       // e.g. if I'm updating record w/ id=3 to have email='foo@foo.com',