Error on migration: some files contain invalid YAML

Drupal\Component\Discovery\DiscoveryException: The /var/www/nch/modules/wordpress_migrate/migrations/._wordpress_authors.yml contains invalid YAML in Drupal\Component\Discovery\YamlDirectoryDiscovery->findAll() (line 90 of core/lib/Drupal/Component/Discovery/YamlDirectoryDiscovery.php).

This is due to an unquoted string containing an @ character exists in some migration files. These strings should be in quotes.

CommentFileSizeAuthor
#3 error_on_migrate-2832551-3.patch1.16 KBmaccath
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bloomt created an issue. See original summary.

maccath’s picture

maccath CreditAttribution: maccath at Deeson commented
Issue summary: View changes
maccath’s picture

maccath CreditAttribution: maccath at Deeson commented
Status: Active » Needs review
FileSize
error_on_migrate-2832551-3.patch1.16 KB
ronald_istos’s picture

ronald_istos CreditAttribution: ronald_istos at Deeson commented
Status: Needs review » Reviewed & tested by the community

Can confirm that this fixes the issue.

HongPong’s picture

HongPong CreditAttribution: HongPong at kor group commented

Also confirming this works. I rolled this patch and the other longstanding approved ones on the issue queue here, if you'd like to try it: https://github.com/HongPong/wordpress_migrate
I would request this get added.

imyaro’s picture

imyaro CreditAttribution: imyaro at DrupalJedi commented

+1 RTBC
Long time following this issue to remove this patch from the project.

  • HongPong committed 9afd98f on 8.x-3.x authored by maccath 
    Issue #2832551 by maccath: Error on migrate /wordpress_migrate/...
HongPong’s picture

HongPong CreditAttribution: HongPong at kor group commented
Status: Reviewed & tested by the community » Fixed

Rolled in now. Thank you! :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.