Skip to content
Snippets Groups Projects

[#3483996] Replace service proxies with service closures

Closed [#3483996] Replace service proxies with service closures
1 unresolved thread
Closed Kim Pepper requested to merge issue/drupal-3483996:3483996-replace-lazy-service into 11.x
1 unresolved thread

Closes #3483996

Merge request reports

Code Quality is loading
Test summary results are being parsed
Metrics reports are loading

Closed by catchcatch 1 week ago (Jun 2, 2025 10:09am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Kent Richards added 1 commit

    added 1 commit

    Compare with previous version

  • Kent Richards added 1 commit

    added 1 commit

    • b3cb2a93 - Issue #3483996: Add empty update function to rebuild service container

    Compare with previous version

  • 38 36 /**
    39 37 * Constructs a new automated cron runner.
    40 38 *
    41 * @param \Drupal\Core\CronInterface $cron
    39 * @param \Drupal\Core\CronInterface|\Closure $cron
    42 40 * The cron service.
    43 41 * @param \Drupal\Core\Config\ConfigFactoryInterface $config_factory
    44 42 * The config factory.
    45 43 * @param \Drupal\Core\State\StateInterface $state
    46 44 * The state key-value store service.
    47 45 */
    48 public function __construct(CronInterface $cron, ConfigFactoryInterface $config_factory, StateInterface $state) {
    49 $this->cron = $cron;
    46 public function __construct(CronInterface|\Closure $cron, ConfigFactoryInterface $config_factory, StateInterface $state) {
    47 if ($cron instanceof CronInterface) {
    48 @trigger_error('Passing the $cron argument as an instance of type \Drupal\Core\CronInterface to ' . __METHOD__ . '() is deprecated in drupal:11.1.0 and type \Closure is required in drupal:12.0.0. Pass type \Closure instead. See https://www.drupal.org/node/3484001', E_USER_DEPRECATED);
    • If we're keeping deprecation, version number needs updating.

      Suggested change
      Applied
      48 @trigger_error('Passing the $cron argument as an instance of type \Drupal\Core\CronInterface to ' . __METHOD__ . '() is deprecated in drupal:11.1.0 and type \Closure is required in drupal:12.0.0. Pass type \Closure instead. See https://www.drupal.org/node/3484001', E_USER_DEPRECATED);
      48 @trigger_error('Passing the $cron argument as an instance of type \Drupal\Core\CronInterface to ' . __METHOD__ . '() is deprecated in drupal:11.2.0 and type \Closure is required in drupal:12.0.0. Pass type \Closure instead. See https://www.drupal.org/node/3484001', E_USER_DEPRECATED);
    • Kent Richards changed this line in version 5 of the diff

      changed this line in version 5 of the diff

    • Please register or sign in to reply
  • Kent Richards added 1 commit

    added 1 commit

    • 04e60f40 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Kent Richards added 1 commit

    added 1 commit

    Compare with previous version

  • Kent Richards added 1 commit

    added 1 commit

    Compare with previous version

  • godotislate added 716 commits

    added 716 commits

    Compare with previous version

  • godotislate added 1 commit

    added 1 commit

    Compare with previous version

  • Adam Bramley added 200 commits

    added 200 commits

    Compare with previous version

  • closed

  • Please register or sign in to reply
    Loading