javascriptreactjstrimdynamic-forms
React dynamic form input fields typeError undefined trim()
I have a React form with dynamic input fields that a user can add and remove input fields. All inputs are required. If all input fields are filled and the form is submitted then i console.log() an array with the values, else empty input gets focused. The problem is that if i whitespace(backspace) inside input fields, the form can be submitted and i don't want this. How can i fix this? I tried to use the trim() method (const registriesIsValid = registryValues.every((registry)=>registry.name.trim() !== "" && registry.code.trim() !== "");) but im getting "typeError Cannot read properties of undefined (reading 'trim')".
Below my code without trim method.
App.js
import React, { useState, useRef } from "react";
import Wrapper from "./Wrapper";
import "./styles.css";
export default function App() {
const [registryValues, setRegistryValues] = useState([]);
const [registryValuesTouched, setRegistryValuesTouched] = useState(false);
const fieldRef = useRef();
// const registriesRef = useRef();
const registriesIsValid = registryValues.every(
(registry) => registry.name !== "" && registry.code !== ""
);
const registriesInputIsInvalid = !registriesIsValid;
let handleRegistryChange = (i, event) => {
setRegistryValuesTouched(true);
let newRegistryValues = [...registryValues];
if (event.target.name === "name") {
newRegistryValues[i].registry["name"] = event.target.value;
}
if (event.target.name === "code") {
newRegistryValues[i].code = event.target.value;
}
// newRegistryValues[i][event.target.name] = event.target.value;
setRegistryValues(newRegistryValues);
};
let addRegistryFields = (event) => {
event.preventDefault();
setRegistryValues([
...registryValues,
{ registry: { name: "" }, code: "" }
]);
};
let removeRegistryFields = (i, event) => {
event.preventDefault();
let newRegistryValues = [...registryValues];
newRegistryValues.splice(i, 1);
setRegistryValues(newRegistryValues);
};
const submitHandler = (event) => {
event.preventDefault();
setRegistryValuesTouched(true);
for (let elem of fieldRef.current.elements) {
if (elem.dataset.required && !elem.value) {
elem.focus();
return;
}
}
console.log(registryValues);
setRegistryValuesTouched(false);
};
return (
<Wrapper>
<form onSubmit={submitHandler} ref={fieldRef}>
<fieldset>
<legend>
<h3 className="govgr-heading-m margin-top">Registries</h3>
</legend>
<div
className={`${
registriesInputIsInvalid ? "govgr-form-group__error" : ""
}`}
>
{registriesInputIsInvalid && (
<p className="govgr-error-message">
<span className="govgr-visually-hidden">Λάθος:</span>All fields
are required.
</p>
)}
{registriesInputIsInvalid && (
<p className="govgr-error-message">
<span className="govgr-visually-hidden">Λάθος:</span>You must
fill in registry Name and registry Code.
</p>
)}
{registryValues.map((element, index) => (
<div key={index} className="flex-row registry-margin-bottom">
<div className="registry-flex-basis">
<div className="govgr-form-group">
<label
className="govgr-label govgr-!-font-weight-bold"
htmlFor="name"
>
Registry Name*
</label>
<input
className={`govgr-input govgr-!-width-three-quarter ${
element.registry["name"].trim() === ""
? "govgr-error-input"
: ""
}`}
type="text"
name="name"
data-required="true"
value={element.registry["name"] || ""}
onChange={(e) => handleRegistryChange(index, e)}
/>
</div>
<div className="govgr-form-group">
<label
className="govgr-label govgr-!-font-weight-bold"
htmlFor="code"
>
Registry Code*
</label>
<input
className={`govgr-input govgr-!-width-three-quarter ${
element.code.trim() === "" ? "govgr-error-input" : ""
}`}
type="text"
name="code"
data-required="true"
value={element.code || ""}
onChange={(e) => handleRegistryChange(index, e)}
/>
</div>
</div>
<button
className="govgr-btn govgr-btn-warning remove-registry"
onClick={(e) => removeRegistryFields(index, e)}
>
X
</button>
</div>
))}
</div>
</fieldset>
<button
className="govgr-btn govgr-btn-secondary button-registry"
onClick={addRegistryFields}
>
Add Registry
</button>
<button
className="govgr-btn govgr-btn-primary btn-center"
type="submit"
>
Save
</button>
</form>
</Wrapper>
);
}
Solution
I fixed it. Actually, inside my submitHandler i forgot to check (if !registriesIsValid){...} .
Also, my registry object has different structure. So i change validation to:
const registriesIsValid = registryValues.every((registry)=>registry.registry["name"].trim() !== "" && registry.code.trim() !== "");
App.js
import React, { useState, useRef } from "react";
import Wrapper from "./Wrapper";
import "./styles.css";
export default function App() {
const [registryValues, setRegistryValues] = useState([]);
const [registryValuesTouched, setRegistryValuesTouched] = useState(false);
const fieldRef = useRef();
// const registriesRef = useRef();
//1st change
const registriesIsValid = registryValues.every(
(registry) => registry.registry["name"].trim() !== "" && registry.code.trim() !== ""
);
const registriesInputIsInvalid = !registriesIsValid;
let handleRegistryChange = (i, event) => {
setRegistryValuesTouched(true);
let newRegistryValues = [...registryValues];
if (event.target.name === "name") {
newRegistryValues[i].registry["name"] = event.target.value;
}
if (event.target.name === "code") {
newRegistryValues[i].code = event.target.value;
}
// newRegistryValues[i][event.target.name] = event.target.value;
setRegistryValues(newRegistryValues);
};
let addRegistryFields = (event) => {
event.preventDefault();
setRegistryValues([
...registryValues,
{ registry: { name: "" }, code: "" }
]);
};
let removeRegistryFields = (i, event) => {
event.preventDefault();
let newRegistryValues = [...registryValues];
newRegistryValues.splice(i, 1);
setRegistryValues(newRegistryValues);
};
const submitHandler = (event) => {
event.preventDefault();
setRegistryValuesTouched(true);
//2nd change
if (!registriesIsValid) {
for (let elem of fieldRef.current.elements) {
if (elem.dataset.required && !elem.value.trim()) {
elem.focus();
return;
}
}
return;
}
console.log(registryValues);
setRegistryValuesTouched(false);
};
return (
<Wrapper>
<form onSubmit={submitHandler} ref={fieldRef}>
<fieldset>
<legend>
<h3 className="govgr-heading-m margin-top">Registries</h3>
</legend>
<div
className={`${
registriesInputIsInvalid ? "govgr-form-group__error" : ""
}`}
>
{registriesInputIsInvalid && (
<p className="govgr-error-message">
<span className="govgr-visually-hidden">Λάθος:</span>All fields
are required.
</p>
)}
{registriesInputIsInvalid && (
<p className="govgr-error-message">
<span className="govgr-visually-hidden">Λάθος:</span>You must
fill in registry Name and registry Code.
</p>
)}
{registryValues.map((element, index) => (
<div key={index} className="flex-row registry-margin-bottom">
<div className="registry-flex-basis">
<div className="govgr-form-group">
<label
className="govgr-label govgr-!-font-weight-bold"
htmlFor="name"
>
Registry Name*
</label>
<input
className={`govgr-input govgr-!-width-three-quarter ${
element.registry["name"].trim() === ""
? "govgr-error-input"
: ""
}`}
type="text"
name="name"
data-required="true"
value={element.registry["name"] || ""}
onChange={(e) => handleRegistryChange(index, e)}
/>
</div>
<div className="govgr-form-group">
<label
className="govgr-label govgr-!-font-weight-bold"
htmlFor="code"
>
Registry Code*
</label>
<input
className={`govgr-input govgr-!-width-three-quarter ${
element.code.trim() === "" ? "govgr-error-input" : ""
}`}
type="text"
name="code"
data-required="true"
value={element.code || ""}
onChange={(e) => handleRegistryChange(index, e)}
/>
</div>
</div>
<button
className="govgr-btn govgr-btn-warning remove-registry"
onClick={(e) => removeRegistryFields(index, e)}
>
X
</button>
</div>
))}
</div>
</fieldset>
<button
className="govgr-btn govgr-btn-secondary button-registry"
onClick={addRegistryFields}
>
Add Registry
</button>
<button
className="govgr-btn govgr-btn-primary btn-center"
type="submit"
>
Save
</button>
</form>
</Wrapper>
);
}
- Convert 64 bit Steam ID to 32 bit account ID
- Error: Absolute route path "/" nested under path "/app" is not valid
- How to search all JavaScript sources in Chrome Developer Tools?
- How to send a property from an array of object from a child component to the parent component?
- Toggle grid on draggable
- Supabase and Google Apps Script
- create a web app that sends a standard text message to a user submitted phone number
- Puppeteer , bringing back blank array
- fabric.js limit rotation to x degrees with rotation handle
- How do I recognize swipe events in React?
- How does jQuery make it so that its array-like collection objects are displayed like arrays in the browser console?
- Running a similar App Scripts code on Multiple Sheets
- Creating object with dynamic keys
- Sequentially Numbering Footnotes with React
- Shortens the data back to 5 characters if you put more that 5 in
- Reference to "this" is null in function called by redux-saga call
- Automatically Delete Files from Google Drive Older than 3 Days
- Html and css to add colorful codes
- Filter/Remove array object based on value, from nested JSON array object in JavaScript
- Argument of type 'string' is not assignable to parameter of type 'Auth'
- Algorithm to convert semi-transparent to solid color
- Format JavaScript date as yyyy-mm-dd
- How can I add an event for a one time click to a function?
- how can I disable everything inside a form using javascript/jquery?
- How to change screen and setState at the same time on first click in React Native
- Is there a way to tell if ReactElement renders "null"?
- Trigger Firebase Cloud Function on linkWithCredential()
- Event triggered when changing a user Firebase Authentication
- fabric.js-adding an event handler to an object doesn't work
- browser sessionStorage. share between tabs?