Merge branch 'master' into fix/undefined-error

Author: jeremydaly

README.md

@@ -896,8 +896,10 @@ let key = {
 // Use the 'delete' method of MyEntity to delete the item from DynamoDB
 let result = await MyEntity.delete(
   key,
-  condition: { attr: 'date_modified' lt: '2020-01-01' },
-  returnValues: 'all_old'
+  {
+    condition: { attr: 'date_modified' lt: '2020-01-01' },
+    returnValues: 'all_old'
+  }
 )
 ```
 

src/__tests__/entity.put.unit.test.ts

@@ -88,6 +88,17 @@ const TestEntity4 = new Entity({
   table: TestTable3
 });
 
+const TestEntity5 = new Entity({
+  name: 'TestEntity5',
+  autoExecute: false,
+  attributes: {
+    pk: { partitionKey: true },
+    test_required_boolean: { type: 'boolean', required: true },
+    test_required_number: { type: 'number', required: true },
+  },
+  table: TestTable2
+})
+
 describe('put',()=>{
 
   it('creates basic item',() => {
@@ -291,6 +302,17 @@ describe('put',()=>{
     })).toThrow(`'test' is a required field`)
   })
 
+  it('puts 0 and false to required fields', () => {
+    let { Item } = TestEntity5.putParams({
+      pk: 'test-pk',
+      test_required_boolean: false,
+      test_required_number: 0
+    })
+
+    expect(Item.test_required_boolean).toBe(false)
+    expect(Item.test_required_number).toBe(0)
+  })
+
   it('formats a batch put response', async () => {
     let result = TestEntity.putBatch({ pk: 'x', sk: 'y' })
 

src/classes/Entity.ts

@@ -801,7 +801,7 @@ class Entity<
         && (!mapping.link || (mapping.link && mapping.save === true))
       ) {
         // If a number or a set and adding
-        if (['number','set'].includes(mapping.type) && data[field]?.$add) {
+        if (['number','set'].includes(mapping.type) && (data[field]?.$add !== undefined && data[field]?.$add !== null)) {
           ADD.push(`#${field} :${field}`)
           values[`:${field}`] = validateType(mapping,field,data[field].$add)
           // Add field to names
@@ -1109,8 +1109,7 @@ class Entity<
 
 
     // Check for required fields
-    Object.keys(required).forEach(field => 
-      required[field] !== undefined && !data[field] 
+    Object.keys(required).forEach(field => required[field] !== undefined && (data[field] === undefined || data[field] === null)
         && error(`'${field}${this.schema.attributes[field].alias ? `/${this.schema.attributes[field].alias}` : ''}' is a required field`)
     ) // end required field check