Problem/Motivation
Provide Drupal Migrate support for updating from Drupal 7.
Proposed resolution
Adds migration template and migrate source and destination plugins necessary to migrate variables matching honeypot_%
as configuration.
Each variable merges into the existing honeypot configuration object.
Form ids are statically-mapped to their equivalents supported by honeypot in 2.1.x.
Remaining tasks
- Figure out core contact module migration mess.
- Review expected values for form ids to make sure they are what they should be.
Release notes snippet
Issue fork honeypot-2401647
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #1
heddnLinking in #2906878: [Meta] Support for D7 -> D9 contrib migrate
Comment #2
geerlingguy CreditAttribution: geerlingguy at Midwestern Mac, LLC commented@heddn - thanks!
Comment #3
Wim Leers@geerlingguy If you can give some guidance on the changes in configuration between D7 and D8, I'm willing to write the migration.
Comment #4
geerlingguy CreditAttribution: geerlingguy at Midwestern Mac, LLC commented@Wim - Thanks for the offer!
So, Honeypot has a variety of settings (stored in variables table) that are more or less hardcoded in D7 (and D6 I think), like:
Then there are
honeypot_form_*
settings which are dynamic—these are basically a list of all the forms Honeypot is configured to protect. As an example, here's the list of these settings on my personal site:And for Drupal 8, the config schema should be up to date: https://git.drupalcode.org/project/honeypot/blob/8.x-1.x/config/schema/h...
I believe the
honeypot_form_*
settings all go intohoneypot.settings.form_settings
; see how they're structured in the Config form here: https://git.drupalcode.org/project/honeypot/blob/8.x-1.x/src%2FControlle...Comment #5
TR CreditAttribution: TR commentedNew features should go into the most-current version of this module first, then be backported if there is community interest.
Comment #6
TR CreditAttribution: TR commentedComment #9
mradcliffeThis a work-in-progress to start migrating variables into the honeypot configuration object. It probably needs a lot of work.
It needs a test case + fixture for when all forms isn't selected and further assertions for the fixture.
Setting to Needs work.
Comment #10
drumm(updating to resolve Drupal.org indexing issue, please disregard)
Comment #11
mradcliffeProbably ready for some sanity check review and manual testing.
Comment #12
mradcliffeRemoving old issue summary and adding up-to-date issue summary.