content_moderation_update_8700 Fails
When i am updating from 8.7.12 to 8.8.5 this hook update fails. function content_moderation_update_8700() { $definition_update_manager = \Drupal::entityDefinitionUpdateManager(); $entity_type =...
View ArticleAdd type declaration to Html::escape() and...
Html::escape() and FormattableMarkup::placeholderEscape() needs a type declaration to prevent it from passing invalid values to htmlspecialchars(). This can only be done in Drupal 10 since it requires...
View ArticleQuery condition 'users_field_data.mail IN ()' cannot be empty
I get an exception when I make a curl request to /user/password with invalid parameters. Ex: curl -X POST --form name%5B0%5D="" --form form_id=user_pass --...
View ArticleNew installation 8.9.1 crashes upon language selection
When installing from scratch (with softaculous), a new installation of Drupal 8.9.1 crashes after saving a new default language (in my case Dutch). As from then, all admin links report error-500....
View ArticleImpossible to delete menu
There is no way to delete a menu (even if the menu doesn't contains any item) If I try to reach "/admin/structure/menu/manage/{menu}/delete" I got "access denied"
View ArticleCreating database on install breaks BrowserTestBase cleanup
\Drupal\Core\Database\Driver\mysql\Install\Tasks::connect will attempt to create the database if it doesn't already exist. To do this, it rebuilds the connection info without the database, connects and...
View ArticleInvalid argument in LibraryDiscoveryParser->buildByExtension()
I get following warning in recent log report whenever i do update.php on my drupal8.I am in migration process.Warning: Invalid argument supplied for foreach() in...
View ArticleMake JSONAPI documentation even better
https://www.drupal.org/docs/8/modules/jsonapi/what-jsonapi-doesnt-do tells you how to log in with a user / password without basic auth.The sub pages of...
View ArticleWildcard block visibility settings resolve too soon
First off - This may be expected behaviour so I'm happy to be wrong and close this issue, the behaviour just seemed wrong to me.The issue is that wildcards on block visibility settings seem to resolve...
View ArticlePlugin Metadata Translation and Sanitization
It is unclear to me at this point exactly how we need to be approaching this topic as a whole. Currently the only example I know of Drupal\block\Tests\BlockTitleXSSTest::testXSSInTitle() which is...
View ArticleClik here to view.
jQuery UI resize handle is hidden
Problem/MotivationProblem/MotivationThe sidebar for adding blocks and sections in layout builder has a jQuery resize option, but it is not usable because the z-index is too low.Steps to...
View Article[Symfony 6.1] Replace deprecationListenerTrait with PHPUnitBridge ignoreFile
Postponed on:#3261266: Remove deprecated code from the testing framework (base classes, listeners, etc)Codebase to be updated to use Symfony 6.1, #3275864: [META] Symfony 6.1...
View ArticleSqlBase::prepareQuery() should be called also on count
Problem/MotivationSuppose that in a source plugin, extending from SqlBase, you want to override SqlBase::prepareQuery() and use that to add additional tags or meta-data to the query. You don't do that...
View ArticleImprove count() for ContentEntity source plugin
Problem/MotivationRetrieving counts of source data when the source data can include translations is troublesome. It isn't possible to just call `$this->query()->count()->execute();` That won't...
View ArticleRemove BC layers from the extension system
Problem/MotivationThere are a few BC layers in constructors in the extension system that were added in Drupal 9, these can be removed in Drupal 10.Steps to reproduceProposed resolutionFind and remove...
View ArticleRemove dead code from ContentTranslationController
Problem/MotivationLine 189 of ContentTranslationController.php has a test which cannot possibly have any other value than FALSE. It serves no purpose but to confuse the reader of the code.Steps to...
View ArticleClik here to view.
Olivero: Progress indicator percentage label does not have proper spacing
Within Olivero, the progress indicator bar percentage label doesn't look quite right. You can see this using the CD_Tools module and at https://tugboat-aqrmztryfqsezpvnghut1cszck2wwasr.tugboat.qa/progress
View ArticleClik here to view.
Olivero main/user account menu layout struggles with long menus
Problem/MotivationFollow-up from Olivero has 'main' and 'secondary' menus, the obvious choice is to put the 'navigation' menu in main and 'user account' menu in secondary.When you have between 4-6...
View ArticleUse settings.local.php if available
Problem/MotivationDrupal 8 propose to use a settings.local.php file to allow project to disconnect general settings ( in settings.php file) and local settings. In hightly scripted environnement, we...
View ArticleFilter tests should not rely on Classy
Problem/MotivationBecause of #3083275: [meta] Update tests that rely on Classy to not rely on it anymore and Classy being deprecated in Drupal 9 + removed in Drupal 10,: Tests that aren't specifically...
View Article