Switch from false to null for and/or zero arguments

Author: TotalTechGeek

defaultMethods.js

@@ -230,7 +230,7 @@ const defaultMethods = {
     [Sync]: (data, buildState) => isSyncDeep(data, buildState.engine, buildState),
     method: (arr, context, above, engine) => {
       if (!Array.isArray(arr)) throw INVALID_ARGUMENTS
-      if (!arr.length) return false
+      if (!arr.length) return null
 
       let item
       for (let i = 0; i < arr.length; i++) {
@@ -242,7 +242,7 @@ const defaultMethods = {
     },
     asyncMethod: async (arr, _1, _2, engine) => {
       if (!Array.isArray(arr)) throw INVALID_ARGUMENTS
-      if (!arr.length) return false
+      if (!arr.length) return null
 
       let item
       for (let i = 0; i < arr.length; i++) {
@@ -256,7 +256,7 @@ const defaultMethods = {
     compile: (data, buildState) => {
       let res = buildState.compile``
       if (Array.isArray(data)) {
-        if (!data.length) return buildState.compile`false`
+        if (!data.length) return buildState.compile`null`
         for (let i = 0; i < data.length; i++) res = buildState.compile`${res} engine.truthy(prev = ${data[i]}) ? prev : `
         res = buildState.compile`${res} prev`
         return res
@@ -388,7 +388,7 @@ const defaultMethods = {
     [Sync]: (data, buildState) => isSyncDeep(data, buildState.engine, buildState),
     method: (arr, context, above, engine) => {
       if (!Array.isArray(arr)) throw INVALID_ARGUMENTS
-      if (!arr.length) return false
+      if (!arr.length) return null
 
       let item
       for (let i = 0; i < arr.length; i++) {
@@ -399,7 +399,7 @@ const defaultMethods = {
     },
     asyncMethod: async (arr, _1, _2, engine) => {
       if (!Array.isArray(arr)) throw INVALID_ARGUMENTS
-      if (!arr.length) return false
+      if (!arr.length) return null
       let item
       for (let i = 0; i < arr.length; i++) {
         item = await engine.run(arr[i], _1, { above: _2 })
@@ -412,7 +412,7 @@ const defaultMethods = {
     compile: (data, buildState) => {
       let res = buildState.compile``
       if (Array.isArray(data)) {
-        if (!data.length) return buildState.compile`false`
+        if (!data.length) return buildState.compile`null`
         for (let i = 0; i < data.length; i++) res = buildState.compile`${res} !engine.truthy(prev = ${data[i]}) ? prev : `
         res = buildState.compile`${res} prev`
         return res

suites/control/and.json

@@ -124,7 +124,7 @@
         "description": "And with no arguments should return false",
         "rule": { "and": [] },
         "data": null,
-        "result": false
+        "result": null
     },
     "Valid Arguments Checks",
     {

suites/control/if.json

@@ -15,7 +15,7 @@
     "Null check",
     {
         "description": "null is falsy",
-        "rule": { "if": [[], "apple", "banana"] },
+        "rule": { "if": [null, "apple", "banana"] },
         "data": null,
         "result": "banana"
     },
@@ -78,19 +78,19 @@
         "result": null 
     },
     {
-        "description": "If no then is defined when true, it'll just return null.",
+        "description": "If no 'then' is defined, it should return the first argument (true)",
         "rule": { "if": [true] },
         "data": null,
         "result": true
     },
     {
-        "description": "If no fallback is defined when falsy, it'll just return null.",
+        "description": "If no 'then' is defined, it should return the first argument (false)",
         "rule": { "if": [false] },
         "data": null,
         "result": false
     },
     {
-        "description": "If no then is defined when truthy, it'll just return null.",
+        "description": "If no 'then' is defined, it should return the first argument (apple string)",
         "rule": { "if": ["apple"] },
         "data": null,
         "result": "apple"

suites/control/or.json

@@ -124,7 +124,7 @@
         "description": "Or with no arguments should return false",
         "rule": { "or": [] },
         "data": null,
-        "result": false
+        "result": null
     },
     "Valid Arguments Checks",
     {