The declaration of Drupal\migrate\Plugin\MigrateDestinationInterface::fields() does not match its doc block:
/**
* ...
* @return array
* - Keys: machine names of the fields
* - Values: Human-friendly descriptions of the fields.
*
* @throws \Drupal\migrate\MigrateException
* Thrown when the destination plugin is not configured correctly.
*/
public function fields();
Add the return-type declaration:
public function fields(): array;
Make the same change in the core destination plugins. Some of these currently have no return statement (so they return NULL instead of []).
First deprecate any implementation that does not declare and return an array.
Any implementation of Drupal\migrate\Plugin\MigrateDestinationInterface::fields() must declare its return type and return an array.
None.
N/A
Active
11.0 🔥
migration system
Postponed on 📌 Enable existing interfaces to add return type hints with a deprecation message for implementors Active .
@joachim: Thanks for pointing me to that issue. Maybe we will not need this issue, since it will be a small part of one of the related issues. For now, I am linking the issues, if only so that this one can be closed when it becomes obsolete.