updated patch in #1.
mnk-0210 → created an issue.
Updated patch from #3 with all the changes present in MR #1
mnk-0210 → created an issue.
Hi @thalles, i have tried applying patch again on vanilla Drupal 10(version: 10.1.6) freshly and I can apply patch with out any issues. PFA SS for the same.
Updated Patch in #2 to replace ckeditor as ckeditor has been replaced with ckeditor5 in Drupal core.
This patch is tested on Both Drupal 9.5.11 and Drupal 10.1.6 and is working as expected.
Updated above Patch in #13 to remove support of Drupal 8 as the updated functions do not support Drupal 8 and changed ckeditor to ckeditor5 as ckeditor has been replaced with ckeditor5 in Drupal core.
mnk-0210 → created an issue.
mnk-0210 → created an issue.
mnk-0210 → created an issue.
PFA the Screenshot for the same:
Even though the accessCheck is present in setup(), upgrade_status module is still providing deprecated warning. Please check the warning below:
Relying on entity queries to check access by default is deprecated in drupal:9.2.0 and an error will be thrown from drupal:10.0.0. Call \Drupal\Core\Entity\Query\QueryInterface::accessCheck() with TRUE or FALSE to specify whether access should be checked.
Added D10 in Drupal/core version of composer.json file.
Please review.
Changed variable declaration from Protected to Public in /tests/src/Functional/ModuleInstallUninstallWebTest.php file due to Fatal Error encountered in job: https://www.drupal.org/pift-ci-job/2457239 → .
Also, added D10 in core requirement.
Please Review.
Explicit access check is Mandatory for Drupal 10 and its missing in the module's MigrateExecutionFormTest testcase file. So we have added the access check.
Please review it.
I have added the changes on the MR. Please review.
I have created an issue: https://www.drupal.org/project/migrate_tools/issues/3349000 📌 Drupal 10 Compatibility Fixes Closed: won't fix for making the module D10 compatible, but its got blocked due to Spam Filter.
mnk-0210 → created an issue.