cerebral-forms - npm Package Compare versions

Comparing version 1.0.0-alpha.1-alpha.ed1e4568 to 1.0.0-b-alpha.8dd85aae

lib/chains/changeField.js

@@ -21,3 +21,3 @@ 'use strict';
 
-exports.default = [(0, _operators.set)((0, _operators.state)(_templateObject, (0, _operators.input)(_templateObject2)), (0, _operators.input)(_templateObject3)), (0, _validateField2.default)()];
+exports.default = [(0, _operators.set)((0, _operators.state)(_templateObject, (0, _operators.input)(_templateObject2)), (0, _operators.input)(_templateObject3)), (0, _validateField2.default)((0, _operators.input)(_templateObject2))];
 //# sourceMappingURL=changeField.js.map

lib/factories/isValidForm.js

@@ -13,14 +13,12 @@ 'use strict';
 
-function isValidFormFactory(formPath) {
-  function isValidForm(_ref) {
-    var state = _ref.state,
-        path = _ref.path;
+function isValidFormFactory(formPathTemplate) {
+  function isValidForm(context) {
+    var formPath = typeof formPathTemplate === 'function' ? formPathTemplate(context).value : formPathTemplate;
+    var form = context.state.get(formPath);
 
-    var form = state.get(formPath);
-
     if ((0, _isValidForm2.default)(form)) {
-      return path.true();
+      return context.path.true();
     }
 
-    return path.false();
+    return context.path.false();
   }
@@ -27,0 +25,0 @@

lib/factories/resetForm.js

@@ -16,13 +16,15 @@ 'use strict';
   return Object.keys(form).reduce(function (newForm, key) {
-    if (Array.isArray(form[key])) {
-      newForm[key] = resetArray(form[key]);
-    } else if ('value' in form[key]) {
-      var newField = Object.keys(form[key]).reduce(function (newField, fKey) {
-        newField[fKey] = form[key][fKey];
-        return newField;
-      }, {});
-      newField.value = newField.defaultValue;
-      newForm[key] = (0, _configureField2.default)(form, newField);
-    } else {
-      newForm[key] = resetObject(form[key]);
+    if (form[key] === Object(form[key])) {
+      if (Array.isArray(form[key])) {
+        newForm[key] = resetArray(form[key]);
+      } else if ('value' in form[key]) {
+        var newField = Object.keys(form[key]).reduce(function (newField, fKey) {
+          newField[fKey] = form[key][fKey];
+          return newField;
+        }, {});
+        newField.value = newField.defaultValue;
+        newForm[key] = (0, _configureField2.default)(form, newField);
+      } else {
+        newForm[key] = resetObject(form[key]);
+      }
     }
@@ -41,9 +43,8 @@
 
-function resetFormFactory(formPath) {
-  function resetForm(_ref) {
-    var state = _ref.state;
+function resetFormFactory(formPathTemplate) {
+  function resetForm(context) {
+    var formPath = typeof formPathTemplate === 'function' ? formPathTemplate(context).value : formPathTemplate;
+    var form = context.state.get(formPath);
 
-    var form = state.get(formPath);
-
-    state.set(formPath, resetObject(form));
+    context.state.merge(formPath, resetObject(form));
   }
@@ -50,0 +51,0 @@

lib/factories/validateField.js

@@ -8,38 +8,17 @@ 'use strict';
 
-var _validate = require('../utils/validate');
+var _runValidation = require('../utils/runValidation');
 
-var _validate2 = _interopRequireDefault(_validate);
+var _runValidation2 = _interopRequireDefault(_runValidation);
 
-var _checkHasValue = require('../utils/checkHasValue');
-
-var _checkHasValue2 = _interopRequireDefault(_checkHasValue);
-
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-function runValidation(fieldPath, field, form) {
-  var hasValue = (0, _checkHasValue2.default)(form, field.value, field.isValueRules);
-  var result = (0, _validate2.default)(form, field.value, field.validationRules);
-  var isValid = result.isValid && (field.isRequired && hasValue || !field.isRequired);
-
-  return {
-    isValid: isValid,
-    isPristine: false,
-    hasValue: (0, _checkHasValue2.default)(form, field.value, field.isValueRules),
-    errorMessage: result.isValid ? null : field.errorMessages[result.failedRuleIndex]
-  };
-}
-
-function validateFieldFactory(path) {
-  function validateField(_ref) {
-    var input = _ref.input,
-        state = _ref.state;
-
-    var fieldPath = (path || input.field).split('.');
+function validateFieldFactory(pathTemplate) {
+  function validateField(context) {
+    var path = typeof pathTemplate === 'function' ? pathTemplate(context).value : pathTemplate;
+    var fieldPath = path.split('.');
     var formPath = fieldPath.slice().splice(0, fieldPath.length - 1);
-    var field = state.get(fieldPath);
-    var form = state.get(formPath);
-    var validationResult = runValidation(fieldPath, field, form);
+    var field = context.state.get(fieldPath);
+    var form = context.state.get(formPath);
+    var validationResult = (0, _runValidation2.default)(field, form);
 
-    state.merge(fieldPath, validationResult);
-
     var dependentFields = [];
@@ -52,17 +31,16 @@ if (Array.isArray(field.dependsOn)) {
 
-    var depententOfValidationResult = dependentFields.reduce(function (currentValidationResult, stringPath) {
+    var dependentOfValidationResult = dependentFields.reduce(function (currentValidationResult, stringPath) {
       var dependentFieldPath = stringPath.split('.');
       var dependentFormPath = dependentFieldPath.slice().splice(0, dependentFieldPath.length - 1);
-      var field = state.get(dependentFieldPath);
-      var form = state.get(dependentFormPath);
-
-      if (!form) {
+      var dependentField = context.state.get(dependentFieldPath);
+      var dependentForm = context.state.get(dependentFormPath);
+      if (!dependentForm || !dependentField) {
         throw new Error('The path ' + stringPath + ' used with "dependsOn" on field ' + fieldPath.join('.') + ' is not correct, please check it');
       }
 
-      var dependentValidationResult = runValidation(dependentFieldPath, field, form);
-      state.merge(dependentFieldPath, dependentValidationResult);
+      var dependentValidationResult = (0, _runValidation2.default)(dependentField, dependentForm);
+      context.state.merge(dependentFieldPath, dependentValidationResult);
 
       if (currentValidationResult.isValid && !dependentValidationResult.isValid) {
-        return dependentValidationResult;
+        return Object.assign(currentValidationResult, { isValid: false });
       }
@@ -73,3 +51,3 @@
 
-    state.merge(fieldPath, depententOfValidationResult);
+    context.state.merge(fieldPath, dependentOfValidationResult);
   }
@@ -76,0 +54,0 @@

lib/factories/validateForm.js

@@ -8,28 +8,24 @@ 'use strict';
 
-var _validate = require('../utils/validate');
+var _runValidation = require('../utils/runValidation');
 
-var _validate2 = _interopRequireDefault(_validate);
+var _runValidation2 = _interopRequireDefault(_runValidation);
 
-var _checkHasValue = require('../utils/checkHasValue');
-
-var _checkHasValue2 = _interopRequireDefault(_checkHasValue);
-
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-function validateFormFactory(passedFormPath) {
-  function validateForm(_ref) {
-    var input = _ref.input,
-        state = _ref.state;
+function validateFormFactory(passedFormPathTemplate) {
+  function validateForm(context) {
+    var passedFormPath = typeof passedFormPathTemplate === 'function' ? passedFormPathTemplate(context).value : passedFormPathTemplate;
+    var formPath = passedFormPath.split('.');
+    var currentPathValue = context.state.get(formPath);
 
-    var formPath = (passedFormPath || input.form).split('.');
-    var currentPathValue = state.get(formPath);
-
     function validateForm(path, form) {
       Object.keys(form).forEach(function (key) {
-        if (Array.isArray(form[key])) {
-          validateArray(path.concat(key), form[key]);
-        } else if ('value' in form[key]) {
-          doValidation(path.concat(key), form, key);
-        } else {
-          validateForm(path.concat(key), form[key]);
+        if (form[key] === Object(form[key])) {
+          if (Array.isArray(form[key])) {
+            validateArray(path.concat(key), form[key]);
+          } else if ('value' in form[key]) {
+            context.state.merge(path.concat(key), (0, _runValidation2.default)(form[key], form));
+          } else {
+            validateForm(path.concat(key), form[key]);
+          }
         }
@@ -45,16 +41,2 @@ });
 
-    function doValidation(path, form, key) {
-      var field = form[key];
-      var hasValue = (0, _checkHasValue2.default)(form, field.value, field.isValueRules);
-      var result = (0, _validate2.default)(form, field.value, field.validations);
-      var isValid = result.isValid && (field.isRequired && hasValue || !field.isRequired);
-
-      state.merge(path, {
-        isValid: isValid,
-        hasValue: hasValue,
-        errorMessage: isValid ? null : field.errorMessages[result.failedRuleIndex],
-        isPristine: false
-      });
-    }
-
     validateForm(formPath, currentPathValue);
@@ -61,0 +43,0 @@ }

lib/helpers/formToJSON.js

@@ -6,14 +6,13 @@ 'use strict';
 });
-
-var _typeof = typeof Symbol === "function" && typeof Symbol.iterator === "symbol" ? function (obj) { return typeof obj; } : function (obj) { return obj && typeof Symbol === "function" && obj.constructor === Symbol && obj !== Symbol.prototype ? "symbol" : typeof obj; };
-
 exports.default = formToJSON;
 function extractObject(object) {
   return Object.keys(object).reduce(function (newObject, key) {
-    if (Array.isArray(object[key])) {
-      newObject[key] = extractArray(object[key]);
-    } else if (object[key] && 'value' in object[key]) {
-      newObject[key] = object[key].value;
-    } else if (object[key] && _typeof(object[key]) === 'object') {
-      newObject[key] = extractObject(object[key]);
+    if (object[key] && object[key] === Object(object[key])) {
+      if (Array.isArray(object[key])) {
+        newObject[key] = extractArray(object[key]);
+      } else if ('value' in object[key]) {
+        newObject[key] = object[key].value;
+      } else {
+        newObject[key] = extractObject(object[key]);
+      }
     }
@@ -20,0 +19,0 @@

lib/helpers/getFormFields.js

@@ -22,3 +22,3 @@ 'use strict';
       return allFields;
-    } else if ('value' in object[key]) {
+    } else if (object[key] === Object(object[key]) && 'value' in object[key]) {
       allFields[currentPath.join('.')] = object[key];
@@ -25,0 +25,0 @@ currentPath.pop();

lib/index.js

@@ -101,3 +101,3 @@ 'use strict';
 function isValidForm(form) {
-  if (typeof form === 'string') {
+  if (typeof form === 'string' || typeof form === 'function') {
     return (0, _isValidForm4.default)(form);
@@ -104,0 +104,0 @@ }

lib/rules.js

@@ -6,2 +6,3 @@ 'use strict';
 });
+/* eslint-disable no-useless-escape */
 var rules = {
@@ -73,3 +74,3 @@ isExisty: function isExisty(value) {
   minLength: function minLength(value, form, length) {
-    return value.length >= length;
+    return !rules.isExisty(value) || rules.isEmpty(value) || value.length >= length;
   }
@@ -76,0 +77,0 @@ };

lib/utils/checkHasValue.js

@@ -8,28 +8,12 @@ 'use strict';
 
-var _rules = require('../rules');
+var _validate = require('./validate');
 
-var _rules2 = _interopRequireDefault(_rules);
+var _validate2 = _interopRequireDefault(_validate);
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-function parseKey(value) {
-  if (typeof value === 'string') {
-    var args = value.split(/:(.+)?/);
-
-    return args[1] ? { key: args[0], params: JSON.parse(args[1]) } : { key: value };
-  }
-
-  return { key: value };
-}
-
 function checkHasValue(form, value, isValueRules) {
-  return isValueRules.reduce(function (isValue, key) {
-    if (!isValue) {
-      return false;
-    }
-    var parsedKey = parseKey(key);
-
-    return _rules2.default[parsedKey.key](value, form, parsedKey.params);
-  }, true);
+  var result = (0, _validate2.default)(form, value, isValueRules);
+  return result.isValid;
 }
 //# sourceMappingURL=checkHasValue.js.map

lib/utils/configureField.js

@@ -8,15 +8,11 @@ 'use strict';
 
-var _checkHasValue = require('./checkHasValue');
+var _runValidation = require('./runValidation');
 
-var _checkHasValue2 = _interopRequireDefault(_checkHasValue);
+var _runValidation2 = _interopRequireDefault(_runValidation);
 
-var _validate = require('./validate');
-
-var _validate2 = _interopRequireDefault(_validate);
-
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function configureField(formData, field) {
-  // If not an actual field, just a namespace
-  if (!('value' in field)) {
+  // If not an actual field but a global property or just a namespace
+  if (field !== Object(field) || !('value' in field)) {
     return field;
@@ -30,7 +26,5 @@ }
   var validationRules = field.validationRules || null;
-  var errorMessages = field.errorMessages || [];
-  var hasValue = (0, _checkHasValue2.default)(formData, value, isValueRules);
-  var validationResult = (0, _validate2.default)(formData, value, validationRules);
+  var validationMessages = field.validationMessages || [];
+  var requiredMessage = field.requiredMessage || null;
 
-  field.value = value;
   field.defaultValue = defaultValue;
@@ -41,8 +35,12 @@ field.validationRules = validationRules;
   // has a value
-  field.isValid = hasValue && validationResult.isValid || !isRequired && !hasValue;
-  field.errorMessages = errorMessages;
-  field.errorMessage = validationResult.isValid ? null : errorMessages[validationResult.failedRuleIndex];
+  field.validationMessages = validationMessages;
+  field.requiredMessage = requiredMessage;
   field.isValueRules = isValueRules;
   field.isRequired = isRequired;
-  field.hasValue = hasValue;
+
+  var validationResult = (0, _runValidation2.default)(field, formData);
+
+  field.isValid = validationResult.isValid;
+  field.errorMessage = validationResult.errorMessage;
+  field.hasValue = validationResult.hasValue;
   field.isPristine = true;
@@ -49,0 +47,0 @@

lib/utils/validate.js

@@ -53,3 +53,3 @@ 'use strict';
           var rule = _rules2.default[key] || function () {
-            console.warn('Rule ' + key + ' is not found');
+            throw new Error('Rule ' + key + ' is not found');
           };
@@ -56,0 +56,0 @@

package.json

 {
   "name": "cerebral-forms",
-  "version": "1.0.0-alpha.1-alpha.ed1e4568",
+  "version": "1.0.0-b-alpha.8dd85aae",
   "description": "Signals, actions and state factories to create forms",
@@ -29,6 +29,6 @@ "main": "lib/index.js",
   "peerDependencies": {
-    "cerebral": "^2.0.0-alpha.4-alpha.ed1e4568"
+    "cerebral": "^2.0.0-b-alpha.8dd85aae"
   },
   "devDependencies": {
-    "cerebral": "^2.0.0-alpha.4-alpha.ed1e4568"
+    "cerebral": "^2.0.0-b-alpha.8dd85aae"
   },
@@ -35,0 +35,0 @@ "nyc": {

README.md

 # cerebral-forms
 Signals, actions and state factories to create forms
 
+## Install
+This project is still in alpha. To test alpha version check [instructions in monorepo](https://github.com/cerebral/cerebral/blob/master/README.md).
+
 ## API
@@ -10,2 +13,4 @@ Cerebral forms is basically a function that creates state needed to handle validation and an action factory for validating fields. It is simple in nature, but handles all the complexity that comes with forms.
 
+You can add any properties to a form where properties containing an object with a "value" property are identified as fields.
+
 ```js
@@ -24,6 +29,8 @@ import {form} from 'cerebral-forms'
         // Error messages mapped to same index as validation rule
-        errorMessages: ['Must be at least 3 characters long'],
+        validationMessages: ['Must be at least 3 characters long'],
         // When setting isRequired to true the field will be invalid if there
         // is no value. To determine if there is not value, check "isValueRules" below
         isRequired: false,
+        // Error message when field is required but has no value
+        requiredMessage: null,
         // Will only be valid if this other field is also valid.
@@ -62,3 +69,3 @@ // Point to a field in the model
         // Combine rules using an object
-        errorMessages: [{
+        validationRules: [{
           minLength: 3,
@@ -92,2 +99,44 @@ isAlpha: true
 
+#### Set a default value for the whole form
+You can set a default value for a property using a factory:
+```js
+import {form, getFormFields} from 'cerebral-forms'
+
+const MyFormFactory = (formObject) => {
+  const myForm = form(formObject)
+  const fields = getFormFields(myForm)
+
+  // You can also set some special properties for the whole form
+  newForm.showErrors = false
+
+  fields.forEach((field) => {
+    field.requiredMessage = field.requiredMessage || 'This field is required'
+    field.someProp = field.someProp || 'Some default'
+  })
+
+  return myForm
+}
+
+```
+
+#### Custom global props
+You can add custom props to the root to the form state.
+
+For example if you want to show validation errors only
+when submitting the form you can add a `showErrors` prop
+which you set true when validation fails during form submit.
+
+```js
+import {form} from 'cerebral-forms'
+
+export default function MyAction({state}) {
+  state.set('some.new.form', form({
+    name: {
+      value: '',
+    },
+    showErrors: false
+  }))
+}
+```
+
 ### field
@@ -146,3 +195,3 @@ To add a new field you simply merge a new form into the existing one.
           onChange={(event) => fieldChanged({
-            field: 'someModule.firstName',
+            field: 'someModule.form.firstName',
             value: event.target.value
@@ -161,2 +210,3 @@ })}
 ```js
+import {input} from 'cerebral/operators'
 import {validateField} from 'cerebral-forms'
@@ -166,3 +216,6 @@
   doThis,
+  // static
   validateField('path.to.form.field'),
+  // dynamic
+  validateField(input`fieldPath`),
   doThat
@@ -176,6 +229,10 @@ ]
 ```js
+import {input} from 'cerebral/operators'
 import {validateForm} from 'cerebral-forms'
 
 export default [
+  // static
   validateForm('path.to.form'),
+  // dynamic
+  validateForm(input`formPath`),
   isFormValid, {
@@ -196,2 +253,3 @@ true: [
 ```js
+import {input} from 'cerebral/operators'
 import {resetForm} from 'cerebral-forms'
@@ -201,3 +259,6 @@
   doThis,
+  // static
   resetForm('path.to.form'),
+  // dynamic
+  resetForm(input`formPath`),
   doThat
@@ -293,6 +354,10 @@ ]
 ```js
+import {input} from 'cerebral/operators'
 import {isValidForm} from 'cerebral-forms'
 
 export default [
-  isValidForm('path.to.form'), {
+  // static
+  isValidForm('path.to.form')
+  // dynamic
+  isValidForm(input`formPath`), {
     true: [],
@@ -310,3 +375,3 @@ false: []
 - **isUndefined** - Checks if undefined
-- **isEmpyString** - Checks if empty string
+- **isEmpty** - Checks if empty string
 - **isEmail** - Checks if valid email format
@@ -313,0 +378,0 @@ - **isUrl** - Checks if valid url format

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

218393

increased by93%

Number of lines in readme file

395

increased by19.7%

Worsened metrics

Number of package files

48

decreased by-20%

Lines of code

1126

decreased by-44.12%

No dependency changes