Double // in file paths
Problem/MotivationThis is a follow up to #2925899: MigrateUpgradeImportBatch does not use source_private_file_path & source_base_path correctly, making it impossible to have public & private...
View Article"reset password" page ignores the user's language selection
Problem/MotivationThis issues discussion has now changed it's focus to determine the users expectation on what language should the content of recovery email is when multiple language is setup. (email...
View ArticleRemove unused $admin_user variable in FieldEntityOperationsTest.php, views...
Problem/MotivationIn core/modules/views/tests/src/Functional/Handler/FieldEntityOperationsTest.php, $admin_user variable was unused.
View ArticleAssert the current path on Views AJAX requests is not set with two leading...
In the AJAX request, Views passes the current path (retrieved in views_views_pre_render()) in the page request as the view_path parameter – for example, /admin/content. As nearly everywhere in Drupal...
View ArticlePassword confirm field does not inherit #attributes attribute
Use Case: password_confirm form field type isn't inheriting the attributesWhen a custom form is to be developed that has password and confirm password fields, that need to follow a pattern.Based on the...
View ArticlePassword confirm field type should have support for #maxlength attribute
Use Case: password_confirm form field type isn't usable when there needs to be max character limitWhen a custom form is to be developed that has password and confirm password fields, with a restriction...
View ArticleEntityTestAccessControlHandler allows viewing an unpublished entity
Problem/MotivationWhile working on #2879087: [PP-1] Use comment access handler instead of hardcoding permissions , I hit this issue trying to write a test that uses a publishable entity. I've picked up...
View Articlefield_read_fields() omits indexes from hook_field_schema
I have a custom module that calls field_update_field(), but that function fails with exceptions like "DatabaseSchemaObjectExistsException: Cannot add index field_sunflower_geo_lat to table...
View ArticleMove Olivero's preload.twig documentation to @file and include variables...
From @lauriii in #216 in #3111409: Add new Olivero frontend theme to Drupal 9.1 core as beta+++ b/core/themes/olivero/templates/includes/preload.twig @@ -0,0 +1,7 @@ +{# + Preload the fonts for the...
View Articleindex.php randomly appears in friendly URLs
Please note that this issue affects more than one version of Drupal, and my selection of the path.module was only a guess at what might be responsible. It might be noteworthy to mention that we are...
View ArticleCannot administer menu item/link if it points to an unpublished node
Steps to reproduce:Create an unpublished node.Go to menu adminCreate a menu link pointing to the node.Try to find the menu link you just created, good luck.
View ArticleUndocumented title variable in feed-icon.html.twig
Problem/MotivationThe feed-icon.html.twig template uses a top-level variable called title. This is originally defined in core/includes/theme.inc:'feed_icon' => [ 'variables' => ['url' => NULL,...
View Article[Code Review] Provide a BEM methodology to comments.pcss.css partial
Problem/MotivationThese changes are coming from the feedback that we've received from the patch submission that we've made on 9/23/2020.+++ b/core/themes/olivero/css/components/comments.pcss.css @@...
View ArticleFix Claro details caret
Problem/MotivationThe Details caret is misaligned after changes were made for the Extend page. We still need the effects from the Extend page commit on the interior details elements, but we need the...
View ArticleClik here to view.
Olivero homepage has an empty title block div with an empty H1
The homepage has an empty block title with an empty h1.Steps to reproduceInstall drupal and create contentVisit the homepage.Notice that the homepage block exists and has an empty h1
View ArticleRemove duplicate suggestions block--block-content.html.twig
Problem/MotivationTheme suggestions for a custom block type generate a duplicate of block--block-content.html.twig.FILE NAME SUGGESTIONS: * block--example.html.twig *...
View ArticleClik here to view.
Status page should link to D8/9 information for public files
Problem/MotivationOn /admin/reports/status page, if the "PUBLIC FILES DIRECTORY" settings are not proper it shows a message Not fully protected See https://www.drupal.org/SA-CORE-2013-003 for...
View ArticleClik here to view.
"Add another item" field button is displayed as a modal action
Problem/MotivationWhen a form pops up in a modal window, the form includes a multi-value field and the Claro theme is in use, the "Add Another" buttons are added to the footer of the modal instead of...
View Article