Skip to content
Snippets Groups Projects

Issue #3443205: Remove deprecated code from migration modules

Closed Issue #3443205: Remove deprecated code from migration modules
1 unresolved thread
1 unresolved thread

Closes #3443205

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
92 79 PrivateTempStoreFactory $tempstore_private,
93 80 MigrationState $migrationState,
94 81 ConfigFactoryInterface $config_factory,
95 ModuleExtensionList|ModuleHandlerInterface $module_extension_list,
96 protected ?TimeInterface $time = NULL,
82 protected ModuleExtensionList $moduleExtensionList,
83 protected TimeInterface $time,
97 84 ) {
98 85 parent::__construct($config_factory, $migration_plugin_manager, $state, $tempstore_private);
99 86 $this->migrationState = $migrationState;
100 if ($this->time === NULL) {
101 @trigger_error('Calling ' . __METHOD__ . ' without the $time argument is deprecated in drupal:10.3.0 and it will be required in drupal:11.0.0. See https://www.drupal.org/node/3112298', E_USER_DEPRECATED);
102 $this->time = \Drupal::service('datetime.time');
103 }
104 if ($module_extension_list instanceof ModuleHandlerInterface) {
105 @trigger_error('Calling ' . __METHOD__ . '() with the $module_extension_list argument as ModuleHandlerInterface is deprecated in drupal:10.3.0 and will be required in drupal:12.0.0. See https://www.drupal.org/node/3310017', E_USER_DEPRECATED);
  • Maintainer

    Removing this is correct - it should have been deprecated for removal in 11.0.0 in the first place, but we need a 10.3.0 MR here to fix the message there too.

  • Please register or sign in to reply
  • closed

  • Please register or sign in to reply
    Loading