When upgrading a D7 site (not a D6 site) to D8.5-rc1 I get this error message and cannot move forward.

Resolve the issue below to continue the upgrade.
The d6_path_redirect plugin must define the source_module property.

CommentFileSizeAuthor
#2 2947642-2.redirect.d6pathredirect-plugin-must-define-the-sourcemodule-property.patch3.55 KBmikelutz

Comments

hass created an issue. See original summary.

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
commented
Status: Active » Needs review
StatusFileSize
new 2947642-2.redirect.d6pathredirect-plugin-must-define-the-sourcemodule-property.patch3.55 KB

The enforcement of the source_module annotation was added in Drupal 8.5. The attached patch updates the source plugins and test fixture to work in 8.5

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
commented
Related issues: +#2908282: Throw exception for source plugins without a source_module property

Adding core issue queue link. This will break drupal migrations on Drupal 8.5 if the redirect module is installed (not just redirect migrations, since the exception is thrown when loading the set of migrations for a drupal migration). Hopefully this can be committed before 8.5 comes out this week.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Needs review » Reviewed & tested by the community

Quick fix for https://www.drupal.org/node/2911881

hass’s picture

hass commented

Thanks. Error is gone.

  • Berdir committed 070f177 on 8.x-1.x authored by mikelutz 
    Issue #2947642 by mikelutz: d6_path_redirect plugin must define the...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Berdir credited joelpittet.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Status: Fixed » Closed (fixed)

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

dww’s picture

dww
we/he/they
commented

Note: this actually broke the d6 case. See #2955400: d6_path_redirect has to define the *correct* source_module annotation