Problem/Motivation
Given a form with multiple submit handlers, a submit handler which triggers a batch and other concurrent batches within itself will cause any submit handlers that are to execute after to have their files out of scope.

Proposed resolution:
Use the form state within the batch to load or require the files.

User interface changes
None

API changes
None

CommentFileSizeAuthor
#10 drupal-submit-handler-scope-2300367-10.patch1.49 KBjordandukart
#7 drupal-submit-handler-scope-2300367-7.patch1.49 KBjordandukart
#3 drupal-submit-handler-scope-2300367-2.patch1.4 KBjordandukart
#1 drupal-submit-handler-scope-2300367-1.patch1.14 KBjordandukart

Comments

jordandukart’s picture

jordandukart commented
StatusFileSize
new drupal-submit-handler-scope-2300367-1.patch1.14 KB

Status: Needs review » Needs work

The last submitted patch, 1: drupal-submit-handler-scope-2300367-1.patch, failed testing.

jordandukart’s picture

jordandukart commented
StatusFileSize
new drupal-submit-handler-scope-2300367-2.patch1.4 KB
jordandukart’s picture

jordandukart commented
Status: Needs work » Needs review
jordandukart’s picture

jordandukart commented
Related issues: +#1300928: Form submit handlers not called during batch processing if not already included

Status: Needs review » Needs work

The last submitted patch, 3: drupal-submit-handler-scope-2300367-2.patch, failed testing.

jordandukart’s picture

jordandukart commented
StatusFileSize
new drupal-submit-handler-scope-2300367-7.patch1.49 KB
jordandukart’s picture

jordandukart commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 7: drupal-submit-handler-scope-2300367-7.patch, failed testing.

jordandukart’s picture

jordandukart commented
StatusFileSize
new drupal-submit-handler-scope-2300367-10.patch1.49 KB
jordandukart’s picture

jordandukart commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 10: drupal-submit-handler-scope-2300367-10.patch, failed testing.

Status: Needs work » Needs review

JordanDukart queued 10: drupal-submit-handler-scope-2300367-10.patch for re-testing.

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.