Merge pull request #4 from mcollina/fix-required-field

Fix required field

Author: mcollina

README.md

@@ -69,13 +69,13 @@ And nested ones, too.
 
 #### Specific use cases
 
-| Instance   | Serialized as                               |
-| -----------|---------------------------------------------|
+| Instance   | Serialized as                |
+| -----------|------------------------------|
 | `Date`     | `string` via `toISOString()` |
-| `RegExp`   | `string`                                    |
+| `RegExp`   | `string`                     |
 
 #### Required
-You can set specific fields of an object as `required` in your schema, by adding `required: true` inside the key properties.  
+You can set specific fields of an object as required in your schema, by adding the field name inside the `required` array in your schema.  
 Example:
 ```javascript
 const schema = {
@@ -86,10 +86,10 @@ const schema = {
       type: 'string'
     },
     mail: {
-      type: 'string',
-      required: true
+      type: 'string'
     }
-  }
+  },
+  required: ['mail']
 }
 ```
 If the object to stringify has not the required field(s), `fast-json-stringify` will throw an error.

example.js

@@ -16,13 +16,13 @@ const stringify = fastJson({
       type: 'integer'
     },
     now: {
-      type: 'string',
-      required: true
+      type: 'string'
     },
     reg: {
       type: 'string'
     }
-  }
+  },
+  required: ['now']
 })
 
 console.log(stringify({

index.js

@@ -158,7 +158,7 @@ function buildObject (schema, code, name) {
       `
     }
 
-    if (schema.properties[key].required) {
+    if (schema.required && schema.required.indexOf(key) !== -1) {
       code += `
       } else {
         throw new Error('${key} is required!')

test.js

@@ -263,13 +263,13 @@ test('object with required field', (t) => {
     type: 'object',
     properties: {
       str: {
-        type: 'string',
-        required: true
+        type: 'string'
       },
       num: {
         type: 'integer'
       }
-    }
+    },
+    required: ['str']
   }
   const stringify = build(schema)