Feeds integration: update constructor of FeedsTarget plugin to fix tests

Created on 9 September 2024, 5 months ago

Problem/Motivation

I see that after the release of Feeds 8.x-3.0-rc1 → , the FeedsPlugin "paragraphs" in this project did broke, because the constructor parameters of \Drupal\feeds\Feeds\Target\Text changed. These changed because of trying to be PHPStan compliant.

This causes the tests for Paragraphs to fail. Sorry for that.

Steps to reproduce

See the tests.

Proposed resolution

Update \Drupal\paragraphs\Feeds\Target\Paragraphs::__construct() and \Drupal\paragraphs\Feeds\Target\Paragraphs::create().

Remaining tasks

User interface changes

API changes

Data model changes

MR will follow.

📌 Task

Status

Needs review

Version

1.0

Component

Code

Created by

🇳🇱Netherlands megachriz

Live updates comments and jobs are added and updated live.

Merge Requests

!149Feeds integration: update constructor of FeedsTarget plugin to fix tests
Open
🇳🇱Netherlands megachriz
updated 3 months ago

Comments & Activities

Issue created by @megachriz
Merge request !149Updated constructor of FeedsTarget plugin 'paragraphs'. → (Open) created by megachriz
Pipeline finished with Success
5 months ago
Total: 412s
#277928
Pipeline finished with Success
5 months ago
Total: 390s
#277941
Issue was unassigned.
Status changed to Needs review 5 months ago9:54am 9 September 2024
Comment 5 months ago →
🇳🇱Netherlands megachriz
Paragraphs tests are passing again. 🙂
Comment 4 months ago →
adinan cenci

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024