- 🇺🇸United States Mile23 Seattle, WA
I suggest this should be more formally part of the Queue API (rather than cron), under
Drupal\Core\Queue
namespace and the service namedqueue.processor
.Also +1 on #11.4.
A recurring issue is that the Drush queue worker doesn't match the logic for queues processed by cron, leading to subtle bugs.
https://github.com/drush-ops/drush/issues/4032
Let's refactor queue processing out from the Cron class so it can be reused by other modules or Drush, without relying on cron itself.
Needs work
10.1 ✨
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
I suggest this should be more formally part of the Queue API (rather than cron), under Drupal\Core\Queue
namespace and the service named queue.processor
.
Also +1 on #11.4.