Add owner to the BlockContent entity type
Problem/MotivationBlock content entities do not have an owner (uid) field, they do have a revision_user field. All other core entity types have owner fields, block content should have one too!Proposed...
View Article[D7] GitLab CI tests for MariaDB currently not working
Problem/MotivationSee the parent issue. This has now been fixed so we should be able to support MariaDB.Steps to reproduceMariaDB is not currently being tested via GitlabCI.Proposed resolutionSupport...
View Article[D7] Test-only job shouldn't require constant rebases to detect which...
Problem/MotivationThis is D7 backport of: #3401971: Test-only job shouldn't require constant rebases to detect which files were changed.I think it is worth to change this for D7 as well, to have the...
View ArticleUpdatePathTestBase::prepareSettings writes entity_update_batch_size incorrectly
Problem/MotivationDiscovered here https://git.drupalcode.org/project/drupal/-/merge_requests/5500#note_235984In UpdatePathTestBase::prepareSettings it tries to override entity_update_batch_size to 1 so...
View ArticleWhen setting `NotNull` constraint on type: required_label, two validation...
Problem/MotivationRan into this while working on #3364109: [PP-2] Configuration schema & required values: add test coverage for `nullable: true` validation support.#3341682: New config schema data...
View ArticleSupport setting service parameters via environment variables
Problem/MotivationThe symfony container allows to define service parameters from environment variables. (Symfony documentation on configuration and Environment Variable Processors) However this does...
View ArticleUse the API to set up languages in tests that are not specifically testing...
Problem/MotivationMultiple tests are using the language configuration forms to change the configuration. Except for a few exceptions that are actually testing those forms, for all the rest that is not...
View ArticlePlan issue index to provide straightforward way for reviewers to point to...
Problem/MotivationWhen someone is reviewing an issue in the Needs Review Issue Queue there maybe some recommended steps. Having an example of a ticket that accomplishes those recommended steps can be...
View Article[PP-1] Make NodeType config entities fully validatable
Problem/MotivationNode type config entities are mostly validatable using the typed config system, but not quite. There is at least one attribute (preview_mode) that needs a constraint or two on it....
View ArticleFocus should be set to current password in case of validation errors for...
Problem/MotivationIf a user wants to change the password he/she needs to enter the current password as well in the user edit form. The current password undergoes the form validation and return with the...
View ArticleClik here to view.
Deprecated core themes cannot be disabled on the "Appearance" tab...
Problem/MotivationAfter upgrading a website to version 9.5.0, the following warning appeared in the site's Status report:Deprecated themes enabled Deprecated themes found: Classy, Stable.My site is...
View ArticleClik here to view.
Allow user to add display modes from respective field UI's.
Problem/MotivationCurrently the users cannot add/create a new form/display mode while entity creation flow.The view display mode has a link Manage view modes which redirects to the view display mode...
View ArticleEnable AdjacentListsSupport plugin
Problem/MotivationThe CKEditor 5 list plugin will merge two adjacent lists into one by default. This can be adverse behavior for users who are pasting in lists that are separated and delineated by some...
View ArticleFork Symfony's ContainerAwareTrait and ContainerAwareInterface into core
Problem/MotivationIn #3392616-5: Update to Symfony 6.4 we've found out that the Symfony\Component\DependencyInjection\ContainerAwareTrait and...
View ArticleReplace views_add_contextual_links() in views.module with service
Problem/MotivationFollow up to #2476947: Convert "title" page element into a block, we had to namespace views_add_contextual_links() in ViewPageControllerTest which was a copy of the same thing done in...
View ArticleEditor entity class should document that it expects a 1-1 relationship with...
Problem/MotivationLooking at the entity_load() function and the places that call it, there is an assumption that an editor entity has the same entity ID as the filter format entity it is for.In other...
View ArticleEntityRepository::getTranslationFromContext() function forcibly adds default...
Problem/MotivationWe have a site that actively uses fallbacks and some fallback chains have the default Entity language disabled. For example, we have an Entity with French (fr) as default language,...
View ArticlePagination does not work correctly for comment fields when using Layout Builder
Problem/MotivationPagination does not work correctly for comment fields that are rendered using Layout Builder.Everything is displayed correctly, but the links to pages in the pager are not formed...
View Article[upstream] [Style] [GHS] <br class> does not work
ckeditor5 strips classes from something like `<br class="mobile-break">`. This is problematic switching from ckeditor4. It can cause data loss on save when the WYSIWYG strips out critical classes...
View ArticleEntity reference field View output is not used for selected entity display
Follow-up for #2174633: View output is not used for entityreference options: This covers the display on the form of already selected items.When an entity relation field displays options via a view the...
View Article