Properly instantiate a queue in _batch_queue()

Created on 3 December 2022, about 2 years ago
Updated 9 February 2023, almost 2 years ago

Problem/Motivation

Trying to pass new $time argument to queue object for #3123216-48: Replace non-test usages of \Drupal::time() with IoC injection β†’ but Memory does not need database connection argument and $name argument also unused.

Proposed resolution

As the only 2 classes expected, see _batch_populate_queue(): 'Drupal\Core\Queue\Batch' : 'Drupal\Core\Queue\BatchMemory' use to check parent class

Remaining tasks

- find better approach
- create MR
- add tests

User interface changes

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Needs work

Version

10.1 ✨

Component
BatchΒ  β†’

Last updated 4 days ago

Created by

πŸ‡«πŸ‡·France andypost

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024