If the 'groups path' is not set by modules implementing migrate_yaml_api, then the module seems to scan the module directory itself for YML files which might pick up the migration YML files. Since the format is different, there might be errors thrown.

Ideally, if groups path is not set, then migrate_yaml should not attempt to register groups at all. Similarly, if 'path' is not set, then migrate_yaml should bail (or log an error).

CommentFileSizeAuthor
#3 there_might_be_parsing-2789261-3.patch1.57 KBhussainweb
#2 there_might_be_parsing-2789261-2.patch1.47 KBhussainweb

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new there_might_be_parsing-2789261-2.patch1.47 KB

We'll check both 'path' and 'groups path' here.

hussainweb’s picture

hussainweb
Primary language English
commented
StatusFileSize
new there_might_be_parsing-2789261-3.patch1.57 KB

Rerolling after recent commits

hussainweb’s picture

hussainweb
Primary language English
commented
Title: There might be parsing errors if groups path is not set » Fix parsing errors when groups or migration path is not set
hussainweb’s picture

hussainweb
Primary language English
commented
Status: Needs review » Fixed

Fixed in b19524a

  • hussainweb committed b19524a on 7.x-1.x 
    Issue #2789261 by hussainweb: Fix parsing errors when groups or...

Status: Fixed » Closed (fixed)

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