Skip to content

[Bug] Fix progress bar for batched tag and rewrite-references jobs#3637

Merged
markus-moser merged 2 commits into
2025.4from
batch-tag-rewrite-ref-progress
Jun 3, 2026
Merged

[Bug] Fix progress bar for batched tag and rewrite-references jobs#3637
markus-moser merged 2 commits into
2025.4from
batch-tag-rewrite-ref-progress

Conversation

@markus-moser
Copy link
Copy Markdown
Contributor

Changes in this pull request

The backend now batches tag assign/replace and element reference rewrites into chunks of 500 (pimcore/studio-backend-bundle#1815), so their progress bars reset between batches — same root cause as the delete fix.

Wire AbstractBatchEditJob (covers studio_ee_job_patch_elements and studio_ee_job_rewrite_element_references) and TagAssignJob to BatchedStepProgressCalculator.

Additional info

Depends on pimcore/studio-backend-bundle#1815. No API/schema change.

markus-moser and others added 2 commits May 28, 2026 13:53
The backend now batches tag assign/replace and element reference rewrites
into chunks of 500, each step streaming progress 0–100. Both jobs used the
handler default ProgressFieldCalculator with DefaultStepTracker (no step
context), so the bar reset between batches.

Wire both AbstractBatchEditJob (covers studio_ee_job_patch_elements and
studio_ee_job_rewrite_element_references) and TagAssignJob to the existing
BatchedStepProgressCalculator.
@sonarqubecloud
Copy link
Copy Markdown

@markus-moser markus-moser added this to the 2025.4.5 milestone Jun 3, 2026
@markus-moser markus-moser marked this pull request as ready for review June 3, 2026 11:54
@markus-moser markus-moser merged commit 87db6e8 into 2025.4 Jun 3, 2026
1 check passed
@markus-moser markus-moser deleted the batch-tag-rewrite-ref-progress branch June 3, 2026 11:54
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant