@beseif-solutions/prow-core 0.3.1 → 0.3.2

package/dist/minified-utils/fields.d.ts

@@ -38,6 +38,7 @@ type StringInputField<Extend = Record<never, never>> = {
     max?: number;
     regexp?: string;
     pattern?: `email` | `uri`;
+    allow_empty?: boolean;
 } & (SelectField<string, Extend> | CustomDisplayTextField | IDETextField | {});
 type NumberInputField<Extend = Record<never, never>> = {
     type: `number`;

package/dist/minified-utils/fields.js

@@ -79,6 +79,7 @@ const stringInputFieldSchema = () => joi_1.default.object({
     max: joi_1.default.number(),
     regexp: joi_1.default.string(),
     pattern: joi_1.default.string().valid(`email`, `uri`),
+    allow_empty: joi_1.default.bool().default(false),
 })
     .concat(joi_1.default.object({
     format: joi_1.default.string().valid(`select`, `textarea`, `rich-text`, `markdown`, `code`),
@@ -164,6 +165,7 @@ const fileSchema = () => joi_1.default.object({
 const fileReplacer = (field) => ({
     key: field.key,
     type: `dict`,
+    unknown: true,
     items: [
         {
             key: `name`,
@@ -256,10 +258,10 @@ const getSimpleFieldSchema = (field) => {
             if (field.length) {
                 typeSchema = typeSchema.length(field.length);
             }
-            if (field.min) {
+            if (field.min > 0) {
                 typeSchema = typeSchema.min(field.min);
             }
-            else {
+            else if (field.allow_empty) {
                 typeSchema = typeSchema.allow(``);
             }
             if (field.max) {
@@ -474,12 +476,10 @@ const recursive_schema = async (field, data, options) => {
                 let genericItemSchema;
                 if (noArrayField.type === `dict` && noArrayField.items) {
                     const items = [];
-                    if (lodash_1.default.isObject(resolved)) {
-                        for (const item of noArrayField.items) {
-                            const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: relativeKey, context: options.context });
-                            if (recursive) {
-                                items.push({ key: item.key, schema: recursive.schema, field: recursive.field });
-                            }
+                    for (const item of noArrayField.items) {
+                        const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: `${relativeKey}[0]`, context: options.context });
+                        if (recursive) {
+                            items.push({ key: item.key, schema: recursive.schema, field: recursive.field });
                         }
                     }
                     genericItemSchema = getObjectFieldSchema(noArrayField, items);
@@ -489,16 +489,15 @@ const recursive_schema = async (field, data, options) => {
                     genericItemSchema = getSimpleFieldSchema(noArrayField);
                 }
                 calculatedSchema = getItemsArrayFieldSchema(newField, [genericItemSchema]);
+                lodash_1.default.unset(data, relativeKey);
             }
         }
         else if (newField.type === `dict` && newField.items) {
             const items = [];
-            if (lodash_1.default.isObject(resolved)) {
-                for (const item of newField.items) {
-                    const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: relativeKey, context: options.context });
-                    if (recursive) {
-                        items.push({ key: item.key, schema: recursive.schema, field: recursive.field });
-                    }
+            for (const item of newField.items) {
+                const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: relativeKey, context: options.context });
+                if (recursive) {
+                    items.push({ key: item.key, schema: recursive.schema, field: recursive.field });
                 }
             }
             calculatedSchema = getObjectFieldSchema(newField, items);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@beseif-solutions/prow-core",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Prow core functionalities and classes",
   "main": "dist/index.js",
   "scripts": {

package/src/minified-utils/fields.ts

@@ -59,6 +59,7 @@ type StringInputField<Extend = Record<never, never>> = {
   max?: number,
   regexp?: string,
   pattern?: `email` | `uri`,
+  allow_empty?: boolean,
 } & (
     | SelectField<string, Extend>
     | CustomDisplayTextField
@@ -275,6 +276,7 @@ const stringInputFieldSchema = () =>
     max: Joi.number(),
     regexp: Joi.string(),
     pattern: Joi.string().valid(`email`, `uri`),
+    allow_empty: Joi.bool().default(false),
   })
     .concat(
       Joi.object({
@@ -375,6 +377,7 @@ const fileSchema = () => Joi.object({
 const fileReplacer = (field: Common & FileInputField): Field => ({
   key: field.key,
   type: `dict`,
+  unknown: true,
   items: [
     {
       key: `name`,
@@ -474,9 +477,9 @@ const getSimpleFieldSchema = (field: Field) => {
     case `string`:
       typeSchema = Joi.string();
       if (field.length) { typeSchema = (typeSchema as StringSchema).length(field.length); }
-      if (field.min) {
+      if (field.min > 0) {
         typeSchema = (typeSchema as StringSchema).min(field.min);
-      } else {
+      } else if (field.allow_empty) {
         typeSchema = typeSchema.allow(``);
       }
       if (field.max) { typeSchema = (typeSchema as StringSchema).max(field.max); }
@@ -682,11 +685,9 @@ const recursive_schema = async (field: Field, data: Record<string, any>, options
         if (noArrayField.type === `dict` && noArrayField.items) {
           const items: { key: string, schema: Joi.Schema, field: Field }[] = [];
 
-          if (_.isObject(resolved)) {
-            for (const item of noArrayField.items) {
-              const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: relativeKey, context: options.context });
-              if (recursive) { items.push({ key: item.key, schema: recursive.schema, field: recursive.field }); }
-            }
+          for (const item of noArrayField.items) {
+            const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: `${relativeKey}[0]`, context: options.context });
+            if (recursive) { items.push({ key: item.key, schema: recursive.schema, field: recursive.field }); }
           }
 
           genericItemSchema = getObjectFieldSchema(noArrayField, items);
@@ -696,15 +697,16 @@ const recursive_schema = async (field: Field, data: Record<string, any>, options
         }
 
         calculatedSchema = getItemsArrayFieldSchema(newField, [genericItemSchema]);
+
+        // unset value if it wasn't resolved
+        _.unset(data, relativeKey);
       }
     } else if (newField.type === `dict` && newField.items) {
       const items: { key: string, schema: Joi.Schema, field: Field }[] = [];
 
-      if (_.isObject(resolved)) {
-        for (const item of newField.items) {
-          const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: relativeKey, context: options.context });
-          if (recursive) { items.push({ key: item.key, schema: recursive.schema, field: recursive.field }); }
-        }
+      for (const item of newField.items) {
+        const recursive = await recursive_schema(item, data, { alter: options.alter && true, relative: relativeKey, context: options.context });
+        if (recursive) { items.push({ key: item.key, schema: recursive.schema, field: recursive.field }); }
       }
 
       calculatedSchema = getObjectFieldSchema(newField, items);