feat(api): Further improve validations

Author: rstoenescu

ui/build/build.api.js

@@ -422,15 +422,19 @@ const astExceptions = {
 
 function validateArray (name, key, property, expected, propApi) {
   const apiVal = propApi[property]
+
   if (expected.length === 1 && expected[0] === apiVal) {
     return
   }
+
+  const expectedVal = expected.filter(t => t.startsWith('__') === false)
+
   if (
     !Array.isArray(apiVal) ||
-    apiVal.length !== expected.length ||
-    !expected.every(t => apiVal.includes(t))
+    apiVal.length !== expectedVal.length ||
+    !expectedVal.every(t => apiVal.includes(t))
   ) {
-    logError(`${name}: wrong definition for prop "${key}" on "${property}": expected ${expected} but found ${apiVal}`)
+    logError(`${name}: wrong definition for prop "${key}" on "${property}": expected ${expectedVal} but found ${apiVal}`)
     process.exit(1)
   }
 }
@@ -491,7 +495,7 @@ function fillAPI (apiType) {
                 process.exit(1)
               }
             }
-            if (definition.required && Boolean(definition.required) !== propApi.required) {
+            if (key !== 'value' && definition.required && Boolean(definition.required) !== propApi.required) {
               logError(`${name}: wrong definition for prop "${key}" on "required": expected "${definition.required}" but found "${propApi.required}"`)
               process.exit(1)
             }

ui/src/components/btn-toggle/QBtnToggle.json

@@ -10,8 +10,7 @@
       "type": "Any",
       "desc": "Model of the component; Either use this property (along with a listener for 'input' event) OR use v-model directive",
       "examples": [ "v-model=\"selected\"" ],
-      "category": "model",
-      "required": true
+      "category": "model"
     },
 
     "options": {

ui/src/components/option-group/QOptionGroup.json

@@ -4,8 +4,7 @@
       "extends": "value",
       "type": "Any",
       "examples": [ "v-model=\"group\"" ],
-      "category": "model",
-      "required": true
+      "category": "model"
     },
 
     "options": {

ui/src/components/popup-edit/QPopupEdit.json

@@ -4,8 +4,7 @@
       "extends": "value",
       "type": "Any",
       "examples": [ "v-model=\"myValue\"" ],
-      "category": "model",
-      "required": true
+      "category": "model"
     },
 
     "title": {

ui/src/components/rating/QRating.json

@@ -9,8 +9,7 @@
     "value": {
       "extends": "value",
       "type": "Number",
-      "examples": [ ":value=\"2\"" ],
-      "required": true
+      "examples": [ ":value=\"2\"" ]
     },
 
     "max": {

ui/src/components/virtual-scroll/QVirtualScroll.js

@@ -29,13 +29,8 @@ export default Vue.extend({
       default: () => []
     },
 
-    itemsFn: {
-      type: Function
-    },
-
-    itemsSize: {
-      type: Number
-    },
+    itemsFn: Function,
+    itemsSize: Number,
 
     scrollTarget: {
       default: void 0

ui/src/components/virtual-scroll/QVirtualScroll.json

@@ -22,11 +22,11 @@
     },
 
     "items-size": {
-      "type": [ "String", "Number"],
+      "type": "Number",
       "desc": "Number of available items in the list; Required and used only if 'itemsFn' is provided",
       "default": "void 0",
       "examples": [
-        "items-size=\"100000\"",
+        ":items-size=\"100000\"",
         ":items-size=\"500\""
       ],
       "category": "content"