[8.x] Add query builder map method #36193
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a new
map()
method to theQuery/Builder
andEloquent/Builder
classes (via theBuildsQueries
trait). This method is similar to theeach()
query builder method, where it automatically chunks over the results. Here's how to use it:The primary motivation here is to make it easier to chunk through query results while also performing a map. Previously, this was quite verbose to do (and required a temporary variable):
For what it's worth, I probably would have preferred the
$count
argument first (before the$callback
), like thechunk()
method, but I did it this way to match theeach()
method.