Remove the special behavior of uid #1.
We have quite some special behavior for uid #1. This tends to confuse the user, and will do so even more now we have an admin role in core. Do all the special cases we designed around uid #1 really...
View ArticleRestore locale_test.js
Problem/MotivationAccidentally, we commited a transformed version of test file. The ES6 builder (Babel) removed some test cases from this file.Proposed resolutionRestore the file and only non-JS...
View ArticlePath alias validation and tests duplicative, inconsistent
Problem/MotivationThe validation for path aliases should be the same whether saving through the admin interface (admin/config/search/path) or node edit forms. As identified in #1847174: Path alias...
View ArticleSplit the internal page cache from the rest of the render cache
As suggested by @WimLeers on #2888838: Remove render caching?, the render cache contains both data for the internal page cache and for the rest of the render cache mechanism, with different patterns of...
View ArticleRemove render caching?
Problem/MotivationRender caching made sense when it was introduced in Drupal 7, and even when it was expanded to be used for many more things in Drupal 8.But since then, Dynamic Page Cache was added in...
View ArticleUpdate stylelint rule shorthand-property-no-redundant-values to be consistent...
Problem/MotivationFollow-up to #2865971: Use stylelint as opposed to csslint in core. Configure the shorthand-property-no-redundant-values to be consistent with...
View ArticleUpdate stylelint rule selector-type-no-unknown to be consistent with...
Problem/MotivationFollow-up to #2865971: Use stylelint as opposed to csslint in core. Configure the selector-type-no-unknown to be consistent with...
View ArticleContent Moderation decides to set a new revision as the default one way too...
Problem/MotivationContent Moderation sets the 'default revision' flag in \Drupal\content_moderation\Entity\Handler\ModerationHandler::onPresave() based on various logic checks in...
View ArticleRPC endpoint to reset user password
Problem/MotivationWe need an endpoint to allow users to trigger the reset password process.Proposed resolutionCreate a router method within Drupal\user\Controller\UserAuthenticationControllerRemaining...
View ArticleInstallation profiles do not support project:module format for dependencies
In #2205271: Project namespace for dependencies support was added to allow modules to list dependencies by prefixing their names with their project name, thus solving the age-old problem of module...
View ArticleAdd test coverage for admin/structure/block/demo
Problem/Motivationadmin/structure/block/demo has no test coverage.Beta phase evaluationReference: https://www.drupal.org/core/beta-changesIssue categoryTask because it adds tests. Issue priorityNormal...
View ArticleContentModerationState 'workflow' column empty for all non-default...
Problem/MotivationIf I run the multilingual moderation test and look at the database after, I get something like the following:mysql> select langcode,revision_id,workflow,moderation_state from...
View ArticleViews preview title is double escaped
When setting the title of a View to a string containing a '&', the & gets rendered as & in the Views Preview.So for example overriding the titles as such (where the title of the nid is...
View ArticleSubclasses of ResourceTestBase for non-entity resources are required to add...
Problem/MotivationResourceTestBase is not usable out of the box for tests of non-entity resources. There are a number of problems one encounters:$this->serializer is declared but never...
View Article[PP-12] Write EntityResourceTestBase subclasses for every other entity type.
This is a follow-up for #2737719: EntityResource: Provide comprehensive test coverage: for every entity type, every format, every method.It was deemed wiser to only apply this new test coverage to all...
View ArticleUpdate stylelint rule function-whitespace-after to be consistent with...
Problem/MotivationFollow-up to #2865971: Use stylelint as opposed to csslint in core. Configure the function-whitespace-after to be consistent with...
View Articleresponse.gzip is redundant and should be removed
Problem/MotivationThere's a config:system.performance response.gzip...which is defined in core/modules/system/config/schema/system.schema.yml and is mapped to the D6/7 variable page_compression e.g. in...
View ArticleMove buildStateConfigurationForm/buildTransitionConfigurationForm to...
Problem/MotivationIf our goal is to segregate our interfaces by workflow types that are configurable and workflows that are not, I think it makes sense to move the methods for transition/state forms to...
View ArticleUpdate stylelint rule declaration-block-no-shorthand-property-overrides to be...
Problem/MotivationFollow-up to #2865971: Use stylelint as opposed to csslint in core. Configure the declaration-block-no-shorthand-property-overrides to be consistent with...
View ArticleUpdate stylelint rule declaration-block-no-redundant-longhand-properties to...
Problem/MotivationFollow-up to #2865971: Use stylelint as opposed to csslint in core. Configure the declaration-block-no-redundant-longhand-properties to be consistent with...
View Article