- @dhruvmittal opened merge request.
dhruv.mittal → changed the visibility of the branch 3284318-valueerror-xmlreaderopen-argument to hidden.
I mistakenly rerolled onto 6.0.x branch so for the same I've created new clean branch
Apologies
Let's continue fixing issue introduced in https://www.drupal.org/project/migrate_plus/issues/3009345 → , since
For migrations that have no urls configuration, like here:
-
https://www.drupal.org/project/wordpress_migrate/issues/3284316 →
- https://git.drupalcode.org/project/wordpress_migrate/-/merge_requests/4/...
Url::construct() doesn't guarantee correct empty configuration for sourceUrls field.
public function __construct(array $configuration, $plugin_id, $plugin_definition, MigrationInterface $migration) {
if (!is_array($configuration['urls'])) {
$configuration['urls'] = [$configuration['urls']];
}
parent::__construct($configuration, $plugin_id, $plugin_definition, $migration);
$this->sourceUrls = $configuration['urls'];
}
Create migration configuration with plugin: url but without urls
source:
plugin: url
data_fetcher_plugin: http
data_parser_plugin: xml
...
process:
...
Provide empty array whe urls configuration is empty.
N/A
N/A
N/A
Needs review
5.0
Plugins
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
dhruv.mittal → changed the visibility of the branch 3284318-valueerror-xmlreaderopen-argument to hidden.
I mistakenly rerolled onto 6.0.x branch so for the same I've created new clean branch
Apologies