We need to add support for migrations from the Drupal 7 version of this module.

I'm just beginning to get a handle on Migrate, so I'm going to start trying to patch this up :)

CommentFileSizeAuthor
#8 2693695-8.patch785 bytesWim Leers
#4 d7_pathologic_filter_format.yml654 byteshass
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

willwh created an issue. See original summary.

hass’s picture

hass CreditAttribution: hass commented
Assigned: willwh » Unassigned
Priority: Normal » Major 
Filter pathologic could not be mapped to an existing filter plugin; defaulting to filter_null.
hass’s picture

hass CreditAttribution: hass commented 
Source ID 1: The configuration property filters.filter_null.settings.local_settings.protocol_style doesn't exist. (core\lib\Drupal\Core\Config\Schema\ArrayElement.php:76)
hass’s picture

hass CreditAttribution: hass commented
FileSize
d7_pathologic_filter_format.yml654 bytes
hass’s picture

hass CreditAttribution: hass commented
Status: Active » Needs review
bob.hinrichs’s picture

bob.hinrichs CreditAttribution: bob.hinrichs commented

Thank you. Imported via configuration single-import UI. However, the migration still fails with "Filter pathologic could not be mapped to an existing filter plugin; defaulting to filter_null."

sinasalek’s picture

sinasalek CreditAttribution: sinasalek as a volunteer and commented

Does not seem to work

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented
Issue tags: +migrate-d7-d8
FileSize
2693695-8.patch785 bytes
fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm at OSInet commented
Status: Needs review » Reviewed & tested by the community

Works flawlessly on 9.0.5.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented

Confirming RTBC! Any plan to commit this and create a new release?

  • dww committed cb080a7 on 8.x-1.x authored by Wim Leers 
    Issue #2693695 by Wim Leers, willwh, fgm, dww: Provide migration support
dww’s picture

dww
we/he/they
CreditAttribution: dww at TEN7 commented
Status: Reviewed & tested by the community » Fixed

Looks good. Didn't test it myself, but I trust @Wim Leers, @fgm and @Anybody. Thanks folks!

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented

Cool, thank you! :)

P.S.: do you have any plans for a new release? Perhaps a beta or even a stable release? :D With 8000 sites already using the D8|9 version of this module, despite it being in alpha, with not a whole lot of open bugs (https://www.drupal.org/project/issues/pathologic?categories=1&version=an...), that's probably an okay thing to do? 😊

dww’s picture

dww
we/he/they
CreditAttribution: dww at TEN7 commented

Definitely! Gonna tag something today. Was hoping to get #2418369: Internal URL handling (language prefixes, base://, ...) fixed, first...

Status: Fixed » Closed (fixed)

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