Problem/Motivation

Deprecate the migrate field plugins used for migrating legacy Drupal sites.
There are 24 Migrate Drupal field plugins.

Steps to reproduce

Proposed resolution

Deprecate in 11.3.0 for removal in 12.0.0

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#16 3502749-nr-bot_ocqdl3qe.txt91 bytesneeds-review-queue-bot
#13 3502749-nr-bot_ey4zsfur.txt91 bytesneeds-review-queue-bot

Issue fork drupal-3502749

Command icon 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:

About issue forks

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Issue summary: View changes
quietone’s picture

quietone commented
Status: Active » Postponed
quietone’s picture

quietone commented
Issue summary: View changes
quietone’s picture

quietone commented
Issue summary: View changes
benjifisher’s picture

benjifisher
he/him or they/them
Primary language English
Location Boston area
commented
Status: Postponed » Active

Updated from discussion on the Migrate video call today. benjifisher, heddn, mikelutz and quietone were present. #3518542: [meeting] Migrate Meeting 2025-04-24 2100Z

quietone opened merge request !12567

quietone’s picture

quietone commented

Starting over. Will deprecating FieldPluginBase be sufficient, similar to DrupalSqlBase for the source plugins?

heddn made their first commit to this issue’s fork.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Active » Needs review

Re #8: let's try that. It gives things the greatest visibility.

heddn’s picture

heddn
Primary language English
Location Illinois
commented

Its green.

quietone’s picture

quietone commented

REmoved some '@group legacy' because of the deprecation is in .deprecation-ignore.txt. That keeps this consistent with the way the source plugins are deprecated.

This looks fine to me.

needs-review-queue-bot’s picture

needs-review-queue-bot commented
Status: Needs review » Needs work
StatusFileSize
new 3502749-nr-bot_ey4zsfur.txt91 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

quietone’s picture

quietone commented
Status: Needs work » Needs review

I added @phpstan-ignore classConstant.deprecatedClass to two test files to fix reported errors from the use of #[CoversClass( )] with a deprecated class.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Using CR as a guide https://www.drupal.org/node/3533566 matches the MR which appear to be good deprecations.

needs-review-queue-bot’s picture

needs-review-queue-bot commented
Status: Reviewed & tested by the community » Needs work
StatusFileSize
new 3502749-nr-bot_ocqdl3qe.txt91 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

rpayanm made their first commit to this issue’s fork.

quietone’s picture

quietone commented
Status: Needs work » Needs review

To fix the rebase errors I generated the phpstan baseline and MigrateFieldPluginManagerTest.php.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Seems like a good rebase.

catch closed merge request !12567

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x, thanks!

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

  • catch committed 83af5abe on 11.x 
    Issue #3502749 by quietone, heddn: Deprecate migrate field plugins
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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