This module is not compatible with Drupal 10 yet.
Fix deprecations using upgrade_status module and install and test in a plain Drupal 10
Fix:
- QueryInterface accessCheck is now required
- core_version_requirement to accept Drupal 10
- Fix prophetize deprecation and removal.
N/A
N/A
N/A
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | d10-compatibility-3364094-6.patch | 4.2 KB | sakthi_dev |
| #4 | 3364094-4-D10-compatibility.patch | 4.21 KB | isholgueras |
| #3 | 3364094-3-D10-compatibility.patch | 4.2 KB | isholgueras |
| #2 | 3364094-2-D10-compatibility.patch | 4.13 KB | isholgueras |
Show commandsStart 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
isholgueras commentedHere is the first patch.
In relation to the fix on tests, Sebastian suggest to fix the
$this->prophetizedeprecation by using thePropecyTraitfromphpspec/prophecy-phpunit. I've added it towidget_ingestion/composer.jsonas arequire-dev. Is this correct?Comment #3
isholgueras commentedUpgrade php requirement in composer.json to 8.1
Comment #4
isholgueras commentedThe required version of prohecy-phpunit was the 2.0-@dev
Comment #5
e0ipsoWhy are we skipping access checks here? I can't remember the exact context where this is called, but status is checked as TRUE so we maybe want access checks on.
I can't remember having to add a require-dev package for other Drupal 10 upgrades. Is it possible that the methods were added elsewhere?
Comment #6
sakthi_dev commentedCreated a patch based on #5. Please review.
Comment #8
e0ipsoComment #10
e0ipso