Update form field ID's to always be safe
This commit is contained in:
parent
b1c82626d9
commit
f59e39931b
|
@ -3,6 +3,7 @@
|
||||||
import { setContext, getContext, onMount } from "svelte"
|
import { setContext, getContext, onMount } from "svelte"
|
||||||
import { writable, get } from "svelte/store"
|
import { writable, get } from "svelte/store"
|
||||||
import { createValidatorFromConstraints } from "./validation"
|
import { createValidatorFromConstraints } from "./validation"
|
||||||
|
import { generateID } from "../helpers"
|
||||||
|
|
||||||
export let datasource
|
export let datasource
|
||||||
export let theme
|
export let theme
|
||||||
|
@ -82,9 +83,7 @@
|
||||||
const makeFieldState = (field, defaultValue) => {
|
const makeFieldState = (field, defaultValue) => {
|
||||||
return writable({
|
return writable({
|
||||||
field,
|
field,
|
||||||
fieldId: `${Math.random()
|
fieldId: `id-${generateID()}-${field}`,
|
||||||
.toString(32)
|
|
||||||
.substr(2)}/${field}`,
|
|
||||||
value: initialValues[field] ?? defaultValue,
|
value: initialValues[field] ?? defaultValue,
|
||||||
error: null,
|
error: null,
|
||||||
valid: true,
|
valid: true,
|
||||||
|
|
|
@ -31,3 +31,35 @@ export const cssVars = (node, props) => {
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Generates a short random ID.
|
||||||
|
* This is "nanoid" but rollup was derping attempting to bundle it, so the
|
||||||
|
* source has just been extracted manually since it's tiny.
|
||||||
|
*/
|
||||||
|
export const generateID = (size = 21) => {
|
||||||
|
let id = ""
|
||||||
|
let bytes = crypto.getRandomValues(new Uint8Array(size))
|
||||||
|
|
||||||
|
// A compact alternative for `for (var i = 0; i < step; i++)`.
|
||||||
|
while (size--) {
|
||||||
|
// It is incorrect to use bytes exceeding the alphabet size.
|
||||||
|
// The following mask reduces the random byte in the 0-255 value
|
||||||
|
// range to the 0-63 value range. Therefore, adding hacks, such
|
||||||
|
// as empty string fallback or magic numbers, is unneccessary because
|
||||||
|
// the bitmask trims bytes down to the alphabet size.
|
||||||
|
let byte = bytes[size] & 63
|
||||||
|
if (byte < 36) {
|
||||||
|
// `0-9a-z`
|
||||||
|
id += byte.toString(36)
|
||||||
|
} else if (byte < 62) {
|
||||||
|
// `A-Z`
|
||||||
|
id += (byte - 26).toString(36).toUpperCase()
|
||||||
|
} else if (byte < 63) {
|
||||||
|
id += "_"
|
||||||
|
} else {
|
||||||
|
id += "-"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return id
|
||||||
|
}
|
||||||
|
|
|
@ -1768,6 +1768,11 @@ mri@^1.1.0:
|
||||||
resolved "https://registry.yarnpkg.com/mri/-/mri-1.1.6.tgz#49952e1044db21dbf90f6cd92bc9c9a777d415a6"
|
resolved "https://registry.yarnpkg.com/mri/-/mri-1.1.6.tgz#49952e1044db21dbf90f6cd92bc9c9a777d415a6"
|
||||||
integrity sha512-oi1b3MfbyGa7FJMP9GmLTttni5JoICpYBRlq+x5V16fZbLsnL9N3wFqqIm/nIG43FjUFkFh9Epzp/kzUGUnJxQ==
|
integrity sha512-oi1b3MfbyGa7FJMP9GmLTttni5JoICpYBRlq+x5V16fZbLsnL9N3wFqqIm/nIG43FjUFkFh9Epzp/kzUGUnJxQ==
|
||||||
|
|
||||||
|
nanoid@^3.1.20:
|
||||||
|
version "3.1.20"
|
||||||
|
resolved "https://registry.yarnpkg.com/nanoid/-/nanoid-3.1.20.tgz#badc263c6b1dcf14b71efaa85f6ab4c1d6cfc788"
|
||||||
|
integrity sha512-a1cQNyczgKbLX9jwbS/+d7W8fX/RfgYR7lVWwWOGIPNgK2m0MWvrGF6/m4kk6U3QcFMnZf3RIhL0v2Jgh/0Uxw==
|
||||||
|
|
||||||
node-releases@^1.1.60:
|
node-releases@^1.1.60:
|
||||||
version "1.1.60"
|
version "1.1.60"
|
||||||
resolved "https://registry.yarnpkg.com/node-releases/-/node-releases-1.1.60.tgz#6948bdfce8286f0b5d0e5a88e8384e954dfe7084"
|
resolved "https://registry.yarnpkg.com/node-releases/-/node-releases-1.1.60.tgz#6948bdfce8286f0b5d0e5a88e8384e954dfe7084"
|
||||||
|
|
Loading…
Reference in New Issue