Use forEach instead of map

We’re not returning anything so map is not needed
elastic · Sep 1, 2020 · 435677f · 435677f
1 parent 04a2e8d
commit 435677f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...plugins/enterprise_search/public/applications/shared/react_router_helpers/format_route.ts b/...plugins/enterprise_search/public/applications/shared/react_router_helpers/format_route.ts
@@ -29,7 +29,7 @@ const reTokens = /<(.*?)>/g;
 export const formatRoute = (routePath: string, params: ParamsObject) => {
   const tokens = {} as { [key: string]: string };
 
-  Object.entries(params).map(([paramName, paramValue]) => {
+  Object.entries(params).forEach(([paramName, paramValue]) => {
     const paramRegex = new RegExp('(/|\\(|\\)|^):' + paramName + '(/|\\)|\\(|$)');
     routePath = routePath.replace(paramRegex, (_, g1, g2) => {
       tokens[paramName] = encodeURIComponent(paramValue);