Merge pull request tjinauyeung#84 from tjinauyeung/chore/update-deps

Author: Larry Botha

lib/components/Field.svelte

@@ -14,4 +14,5 @@
   value={$form[name]}
   on:change={handleChange}
   on:blur={handleChange}
-  {...$$props} />
+  {...$$props}
+/>

lib/components/Form.svelte

@@ -45,5 +45,6 @@
     {handleChange}
     {handleSubmit}
     {updateField}
-    {updateTouched} />
+    {updateTouched}
+  />
 </form>

lib/components/Select.svelte

@@ -12,6 +12,7 @@
   value={$form[name]}
   on:change={handleChange}
   on:blur={handleChange}
-  {...$$props}>
+  {...$$props}
+>
   <slot />
 </select>

lib/create-form.js

@@ -12,7 +12,7 @@ export const createForm = (config) => {
   let initialValues = config.initialValues || {};
 
   const validationSchema = config.validationSchema;
-  const validateFn = config.validate;
+  const validateFunction = config.validate;
   const onSubmit = config.onSubmit;
 
   const getInitial = {
@@ -73,10 +73,10 @@ export const createForm = (config) => {
         });
     }
 
-    if (validateFn) {
+    if (validateFunction) {
       isValidating.set(true);
       return Promise.resolve()
-        .then(() => validateFn({[field]: value}))
+        .then(() => validateFunction({[field]: value}))
         .then((errs) =>
           util.update(errors, field, !util.isNullish(errs) ? errs[field] : ''),
         )
@@ -101,19 +101,19 @@ export const createForm = (config) => {
     return updateValidateField(field, value);
   }
 
-  function handleSubmit(ev) {
-    if (ev && ev.preventDefault) {
-      ev.preventDefault();
+  function handleSubmit(event) {
+    if (event && event.preventDefault) {
+      event.preventDefault();
     }
 
     isSubmitting.set(true);
 
     return util.subscribeOnce(form).then((values) => {
-      if (typeof validateFn === 'function') {
+      if (typeof validateFunction === 'function') {
         isValidating.set(true);
 
         return Promise.resolve()
-          .then(() => validateFn(values))
+          .then(() => validateFunction(values))
           .then((error) => {
             if (util.isEmpty(error)) {
               clearErrorsAndSubmit(values);
@@ -137,7 +137,7 @@ export const createForm = (config) => {
               if (yupErrors && yupErrors.inner) {
                 const updatedErrors = getInitial.errors();
 
-                yupErrors.inner.forEach((error) =>
+                yupErrors.inner.map((error) =>
                   util.set(updatedErrors, error.path, error.message),
                 );
 

lib/util.js

@@ -29,9 +29,7 @@ function isEmpty(object) {
 function getValues(object) {
   let result = [];
   for (const [, value] of Object.entries(object)) {
-    result = result.concat(
-      typeof value === 'object' ? getValues(value) : value,
-    );
+    result = [...result, typeof value === 'object' ? getValues(value) : value];
   }
   return result;
 }
@@ -102,12 +100,16 @@ function set(object, path, value) {
 
   const result = path
     .slice(0, -1)
+    // TODO: replace this reduce with something more readable
+    // eslint-disable-next-line unicorn/no-array-reduce
     .reduce(
       (accumulator, key, index) =>
         new Object(accumulator[key]) === accumulator[key]
           ? accumulator[key]
           : (accumulator[key] =
-              Math.abs(path[index + 1]) >> 0 === +path[index + 1] ? [] : {}),
+              Math.trunc(Math.abs(path[index + 1])) === +path[index + 1]
+                ? []
+                : {}),
       object,
     );