Issue [#3252386]

This adds support for attributes in PHP 8.1 - i.e we need nested attrbiutes.

changed target branch from 9.4.x to 10.0.x

added 1 commit

• 0cd95365 - Initial work with working block plugins discovered via attributes

Compare with previous version

changed title from Revert "Issue #3251936 by Gábor Hojtsy, Mixologic, longwave: Make DrupalCI... to Issue [#3252386]

changed the description

added 3 commits

• 0cd95365...3eb8a344 - 2 commits from branch project:10.0.x
• eb45d871 - Initial work with working block plugins discovered via attributes

Compare with previous version

added 2 commits

• 3236f7e0 - Add note about performance
• 8c6068c8 - Layout Builder code has a hidden dependency on Block Content!

Compare with previous version

added 2 commits

• 241e9e34 - Move more thngs to core
• c0982e12 - Preserve BC interface

Compare with previous version

added 1 commit

• 3a225526 - Better move to core for block_content access stuff

Compare with previous version

added 1 commit

• 3407b10a - Move classes about to maintain BC

Compare with previous version

added 14 commits

• 3407b10a...638c8bdb - 5 commits from branch project:10.0.x
• e124dded - Initial work with working block plugins discovered via attributes
• fe910dba - Add note about performance
• f6d95d25 - Layout Builder code has a hidden dependency on Block Content!
• b37b2a47 - Move more thngs to core
• 9d3ff3c2 - Preserve BC interface
• d4a5a30b - Better move to core for block_content access stuff
• 075b4e90 - Move classes about to maintain BC
• a8442227 - revert the block content / layout builder changes
• 7ebcdeb2 - Use alternate fix from #3255804 - it is way smaller and less impact

Compare with previous version

added 1 commit

• 8ae84954 - Prove we can coverter the InlineBlock tricky plugin to PHP attribute discovery

Compare with previous version

added 62 commits

• 8ae84954...3db8f620 - 58 commits from branch project:10.0.x
• bb79515d - Merge branch '10.0.x' into 3252386
• 88e4efeb - Do some more block conversions
• e0a0fa0e - Fix NodeBlockFunctionalTest
• fb43a6f2 - Convert all the blocks

Compare with previous version

added 1 commit

• d2bf2e2b - Fix SystemMessagesBlock

Compare with previous version

Should this not be 8.0.0 or higher? As PHP 8.0 is the minimum required version for D10.

Nope. If we've going to use PHP Attributes we need to change the minimum PHP to PHP 8.1 - see comments on issue already. The issue is we need to be able to create objects within our attribute declaration to support translatable strings and context definitions.

Thank you your explanation! This thread can be marked as resolved.

changed this line in version 29 of the diff

resolved all threads

added 5 commits

• d2bf2e2b...2f1c7166 - 2 commits from branch project:10.0.x
• ead73f2b - Merge branch '10.0.x' into 3252386
• f2098b24 - Merge branch '10.0.x' into 3252386
• cc1110e6 - admin_label is not always defined

Compare with previous version

added 1 commit

• a518c125 - Allow constraints to be added to context definitions via the constructor

Compare with previous version

added 1 commit

• 090dffca - Fix LanguageBlock

Compare with previous version

added 1 commit

• 87a79b30 - Use AttributeDiscoveryWithAnnotations for optimised discovery instead of a decorator

Compare with previous version

added 2 commits

• 7eaad6fa - Convert action plugins to attributes
• 3018a740 - Use fibers to hopefully isolate plugin discovery

Compare with previous version