Skip to content
Snippets Groups Projects

Store the values of the $element variable in the sandbox to let it survive to...

All threads resolved!

Store the values of the $element variable in the sandbox to let it survive to the next iteration of the upgrade function Closes #3436569

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Klaas Eikelboom
  • jensschuppe resolved all threads

    resolved all threads

  • Haven't tested, but those changes make sense.

    This was indeed tricky, because there's iteration over all webforms, and over all submissions. I tried to base the batch on submissions and dynamically re-fill them during the batch processing. But you're right, re-filling should not be done with multiple webforms at a time.

    Re-producing would involve a webform with less than <batch_size> submissions.

  • jensschuppe approved this merge request

    approved this merge request

  • Author Maintainer

    Thanks for the feedback. I will do the merge

  • Please register or sign in to reply
    Loading