Skip to content

Commit a062dc8

Browse files
authored
Merge pull request #48 from the-collab-lab/wc-gb-empty-duplicate-item-alert
add itemName variable back in
2 parents a799af8 + 764d456 commit a062dc8

File tree

1 file changed

+2
-19
lines changed

1 file changed

+2
-19
lines changed

src/components/AddItems.jsx

Lines changed: 2 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -17,28 +17,11 @@ export function AddItems({ items }) {
1717
async (event) => {
1818
event.preventDefault();
1919

20-
const itemName = event.target.elements['item-name'].value;
21-
const normalizedItemName = itemName
22-
.trim()
23-
.toLowerCase()
24-
.replace(/[&\/\\#, +$!,~%.'":*?<>{}]/g, '');
25-
if (items) {
26-
const currentItems = items.map((item) =>
27-
item.name
28-
.trim()
29-
.toLowerCase()
30-
.replace(/[&\/\\#, +$!,~%.'":*?<>{}]/g, ''),
31-
);
32-
if (currentItems.includes(normalizedItemName)) {
33-
alert('This item already exists in the list');
34-
event.target.reset();
35-
return;
36-
}
37-
}
38-
3920
const daysUntilNextPurchase =
4021
event.target.elements['purchase-date'].value;
4122

23+
const itemName = event.target.elements['item-name'].value;
24+
4225
try {
4326
if (itemName.trim() === '') {
4427
alert('Please add an item name.');

0 commit comments

Comments
 (0)