raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-01-25 20:46:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Having a cleaner syntax a prefer using map() instead of forEach()

Browse Source
This commit is contained in:
Jean-Baptiste Bellet
2023-02-01 10:19:04 +01:00
parent 58ea3a10e8
commit 8926a3f08d
1 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
app/webpacker/controllers/bulk_actions_controller.js
View File

@@ -12,12 +12,9 @@ export default class extends ApplicationController {
// private
getOrdersIds() {
const order_ids = [];
document
.querySelectorAll("#listing_orders input[name='order_ids[]']:checked")
.forEach((checkbox) => {
order_ids.push(checkbox.value);
});
return order_ids;
const checkboxes = document.querySelectorAll(
"#listing_orders input[name='order_ids[]']:checked"
);
return Array.from(checkboxes).map((checkbox) => checkbox.value);
}
}