Problem/Motivation

The FeedsTarget plugins need to be converted to use attributes instead of annotation. Feeds already added support for attributes in #3564248: Add support for Attributes.

This will also help on #3598136: Automated Drupal 12 compatibility fixes for feeds 3.x-dev. Currently tests on Drupal 12 do not complete because of the many deprecations warnings. A lot of these are:

Using @FeedsTarget annotation for plugin with ID x is deprecated and is removed from drupal:13.0.0. Use a Drupal\feeds\Attribute\FeedsTarget attribute instead.

So doing the conversion now would help in getting the tests to finish on Drupal 12. And it also would remove noise from the test report.

Issue fork feeds-3616226

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

megachriz created an issue. See original summary.

megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Active » Needs review

megachriz opened merge request !261

  • megachriz committed d7e6c488 on 8.x-3.x 
    task: #3616226 Convert annotations to attributes in FeedsTarget plugins...
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Needs review » Fixed

Merged!

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

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

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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