Add a return statement to MenuLinkManager::menuNameInUse and correct the...
Problem/MotivationThe return statement is missing in MenuLinkManager::menuNameInUse , in /core/lib/Drupal/Core/Menu/MenuLinkManager.php public function menuNameInUse($menu_name) {...
View ArticleAllow process plugins to stop further processing on a pipeline
Problem/MotivationThe MigrateSkipProcessException causes the processed value is set to NULL, no more plugins are used and the the value is not migrated. See...
View ArticleClik here to view.
Allow multiple field widgets to not use tabledrag
When fields allow multiple values (other than e.g. checkboxes), the widget is output in a table with tabledrag. This is functional but difficult to style to look consistent with other form elements....
View Article[PP-1] Expose Layout Builder data to REST and JSON:API
Problem/MotivationWhen accessing Layout Builder overrides via REST (i.e. when accessing an overridden entity's layout field), the contents of sections are empty.Proposed resolutionAdd a normalizer for...
View ArticleAdd support for negating user role condition for block visibility
Issue summary updated as of comment #32Problem/MotivationThe block module does not make provision for negating the user role visibility condition. That makes it impossible (via the UI) to make a block...
View ArticleRemove usages of book module from tests
Problem/MotivationRemove uses of book from from test, excluding Update tests.Steps to reproduce$ grep --exclude-dir={fixtures,book,node_modules,vendor,themes} -riw book core | grep Test | grep -vi...
View ArticleCannot save translated nodes after upgrading to 8.8 due to invalid path
Problem/MotivationHm. Perhaps I'm doing sth. wrong, but after upgrading a few sites to Drupal 8.8 I found that I cannot save newly translated nodes anymore. The error message is "Either the path...
View ArticleAutowireTest::testCoreServiceAliases should not check services that are abstract
Problem/MotivationIn #3406482: logger.channel.file and .contact definitions are not consistent with other loggers it became obvious that we have a bug in the ::testCoreServiceAliases test. When we...
View Articlelogger.channel.file and .contact definitions are not consistent with other...
Problem/MotivationJust randomly noticed. The logger channel for file is for some reason defined like this logger.channel.file: class: Drupal\Core\Logger\LoggerChannel factory: ['@logger.factory',...
View ArticleMove some Action plugins to Action module
Problem/MotivationIn the parent issue it was discovered that there are two tests that are not in the action module that are installing an Action. The two plugins are node_unpublish_by_keyword_action...
View ArticleUrl generator will add '=' after query parameter when value is an...
When user add any link with an internal link format like: /test?abcExpected behaviour: The output should be /test?abcActual behaviour: The output is /test?abc=Which have cause our js application failure.
View Articledocument the values in $ages in FieldTimeIntervalTest
Problem/MotivationThe $ages array should have comments to explain what the 3 different value are.The first is seconds, the other two are:> [, $formatted_value, $granularity] =...
View ArticleOptimize sessions reads immediately after a write
Problem/MotivationSteps to reproduceProposed resolutionRemaining tasksUser interface changesAPI changesData model changesRelease notes snippet
View ArticleIncorrect types in variables used in Drupal\Core\Field\Annotation\FieldType
Problem/MotivationHi!Two issues related with the Drupal/Core/Field/Annotation/FieldType.php file:1) After https://www.drupal.org/node/3375748 landed, FieldType can be grouped by "category", and if that...
View ArticleRemove book module from all core themes
Problem/MotivationThere are usages of book in demo_umami which need to be moved so the module can be deprecated.$ grep -ri book core/profiles/demo_umami/ | awk -F: '{print $1}' | sort -u | nl 1...
View ArticleDeprecated function: mb_strtolower(): Passing null to parameter #1 ($string)...
Problem/MotivationFor PHP 8.1, calling the mb_strtolower() function with a null argument returns a deprecation error:Deprecated function: mb_strtolower(): Passing null to parameter #1 ($string) of type...
View ArticleServerCommand command fails if not excecuted from Drupal root
Problem/Motivationphp core/scripts/drupal server will fail with the following error if it is not executed from Drupal's root directory:In Connection.php line 123: SQLSTATE[HY000] [14] unable to open...
View ArticleAdd label to Tableselect select all checkbox
The Select All checkbox currently has no label, which makes its styling hard. I propose adding an id generator .uniqueId() and an empty (or perhaps not empty?) label right after the checkbox, which...
View ArticleViews entity operations lack cacheability support, resulting in incorrect...
Problem/MotivationIn order to be able to cache the rendered output of views result rows, we need support for it when we render entity operation links, see the https://qa.drupal.org/pifr/test/1022293...
View ArticleAll core shipped image styles should include webp conversion
Problem/MotivationFollow-up from #3275557: Add webp image conversion to core's install profile's image style which didn't actually do what the issue summary suggested.We should add a webp image...
View Article