- Issue created by @japerry
- ๐ญ๐บHungary Balu Ertl Budapest ๐ช๐บ
Perhaps would be nice to clean up the situation around ๐ Drupal 10 compatibility issues Needs review and close it first, then continue with focusing on compatibility with Drupal 11.
- ๐ฎ๐ณIndia vishalkhode
vishalkhode โ made their first commit to this issueโs fork.
- Status changed to Needs review
6 months ago 4:27pm 26 May 2024 - ๐ฎ๐ณIndia vishalkhode
Hi @japerry
The CI is green now. All PHPUnit tests are passing. FYI: I've reverted the two commits: 12cf6068 & 94cdba03 as I don't think it's good idea to disable ConfigSchema checking. To fix this, I've added an event_subscriber to update config value at runtime.For Drupal 11, following two PHPUnit tests were failing:
1) Drupal\Tests\ctools_views\Functional\CToolsViewsBasicViewBlockTest::testHideFields Failed asserting that 0 matches expected 5. /builds/issue/ctools-3445547/modules/ctools_views/tests/src/Functional/CToolsViewsBasicViewBlockTest.php:250 2) Drupal\Tests\ctools_views\Functional\CToolsViewsBasicViewBlockTest::testOrderFields Failed asserting that 0 matches expected 4. /builds/issue/ctools-3445547/modules/ctools_views/tests/src/Functional/CToolsViewsBasicViewBlockTest.php:314 ERRORS!
When I debugged further, I found that the test was failing due to following schema validation check:
Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for block.block.views_block__ctools_views_test_view_block_fields with the following errors: block.block.views_block__ctools_views_test_view_block_fields:settings.fields.id missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.name missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.age missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.job missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.created missing schema, block.block.views_block__ctools_views_test_view_block_fields:settings.fields.name_1 missing schema in Drupal\Core\Config\Development\ConfigSchemaChecker->onConfigSave() (line 101 of core/lib/Drupal/Core/Config/Development/ConfigSchemaChecker.php).
I wasn't sure on how to fix it, so I temporary disable the schema validation check (same you did earlier). If you know, how to fix it properly, that would be great.
Thanks. - ๐บ๐ธUnited States japerry KVUO
Moving to 4.x as thats where we full support Drupal 11.
+1 to the event subscriber! Regarding the config, yah thats what mglaman suggested to do as well, that should be fine for testing.
- ๐บ๐ธUnited States japerry KVUO
reverted my removal of Drupal 9. 4.1.0 can be D9 compatible, and we can drop 9.5 later if the code requires it. Tests will only work in Drupal 10.2+, which is fine.
-
japerry โ
committed 2cc51b09 on 4.x
Issue #3445547: Drupal 11 Compatibility.
-
japerry โ
committed 2cc51b09 on 4.x
- Status changed to Fixed
5 months ago 9:43pm 11 June 2024 -
japerry โ
committed 2cc51b09 on 4.1.x
Issue #3445547: Drupal 11 Compatibility.
-
japerry โ
committed 2cc51b09 on 4.1.x
-
japerry โ
committed 02972410 on 8.x-3.x
Issue #3445547: Drupal 10.3 Compatibility.
-
japerry โ
committed 02972410 on 8.x-3.x
- ๐ฆ๐บAustralia dpi Perth, Australia
This issue caused a break in ๐ PageBlockDisplayVariant::__sleep() is no longer compatible with parent signature from ctools leading to PHP fatals RTBC . Mind reviewing that @japerry, since you're also a maintainer there.
Automatically closed - issue fixed for 2 weeks with no activity.