Changes between Version 1 and Version 2 of Ticket #28473, comment 8
- Timestamp:
- 08/03/2025 12:07:49 AM (6 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #28473, comment 8
v1 v2 11 11 }}} 12 12 13 Second, apart from this, I don't love that iterator. Instead of exploding the $headers first, and then running this, we could do both actions in the same run. I would have chosen a `foreach` because it's more visual.13 Second, apart from this, I don't love that iterator. Instead of exploding the $headers first, and then running this, we could do both actions in the same run. The for is Ok, but I would have chosen a `foreach` because it's easier to follow. 14 14 15 15 With this review, I'm looking forward to seeing those changes in code. The unit test refresh looks good.