Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
This PR will trigger a minor release when merged. |
| return; | ||
| } | ||
| // eslint-disable-next-line no-await-in-loop | ||
| results[i] = await mapper(array[i], i); |
Check warning
Code scanning / CodeQL
Superfluous trailing arguments Warning
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI about 11 hours ago
In general, this issue is fixed either by removing the unnecessary extra arguments from the call, or by updating the callee to declare and use those arguments. Here, the mapper function at line 198 only expects a single parameter (file) and does not use arguments, so the cleanest fix without changing behavior is to change mapWithConcurrency so that it only passes a single argument to mapper.
Concretely, in src/content/clone.cmd.js, inside mapWithConcurrency at line 53, change mapper(array[i], i) to mapper(array[i]). This ensures that all mappers receive only the element value, matching the existing call site at line 198. No additional imports, methods, or definitions are required; we are only changing the argument list of the mapper invocation.
| @@ -50,7 +50,7 @@ | ||
| return; | ||
| } | ||
| // eslint-disable-next-line no-await-in-loop | ||
| results[i] = await mapper(array[i], i); | ||
| results[i] = await mapper(array[i]); | ||
| } | ||
| } | ||
| const pool = Math.min(limit, array.length); |
2 participants
Allows to locally checkout the project content, make modifications, push back and serve locally. Useful for bulk editing.
Implement:
aem content clone: clone the da.live content of the project inside theaem-contentfolder (i.e. download locally)aem content status: show the locally modified filesaem content push: pushes locally modified files to the server (i.e. upload to server)aem content diff: compares the locally modified files with the ones on the serveraem content merge: merges locally the locally modified files and the ones on the serveraem-contentfolder is present,aem upserves content first from here (and injectshead.html)