Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
When upgrading to version 8.x-2.0beta4 I found that the `FeedTypeTamperManager` has mismatched dependency in `__construct` function
Steps to reproduce
- Install the `8.x-2.0beta4` version by command
composer require 'drupal/feeds_tamper:^2.0@beta'
- Take a look to the file `docroot/modules/contrib/feeds_tamper/src/FeedTypeTamperManager.php`
- See the error `Undefined type 'Drupal\Component\DependencyInjection\ContainerInterface'.`
Proposed resolution
- Correct dependency from
use Drupal\Component\DependencyInjection\ContainerInterface;
↓
use Symfony\Component\DependencyInjection\ContainerInterface;
Remaining tasks
- N/A
User interface changes
- N/A
API changes
- N/A
Data model changes
- N/A
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | mismatched-dependencies-3479271.patch | 1.09 KB | ananya.k |
| #2 | Screenshot 2024-10-08 at 12.01.45.png | 291.32 KB | khanh vuong |
Issue fork feeds_tamper-3479271
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:
changes
Comments
Comment #2
khanh vuong commentedComment #3
ananya.k commentedI checked, and there are the same dependencies in other files as well. I have attached the patch. Please review it once.
Comment #6
megachrizPatches are unfortunately no longer evaluated by the testbot, so I moved your code changes to a Merge Request instead.
Comment #8
megachrizI merged the code!
Comment #10
ananya.k commented