Convert web tests to browser tests for syslog module
See #2735005: Convert all Simpletest web tests to BrowserTestBase (or UnitTestBase/KernelTestBase)
View ArticleMove rest/serialization module's "link manager" services to...
Problem/MotivationThis is awkward.#2758897: Move rest module's "link manager" services to serialization module landed on January 31. It moved the "link manager" services from the REST module to the...
View ArticleConvert all book web tests to BrowserTestBase
Child to #2735005: Convert all Simpletest web tests to BrowserTestBase (or UnitTestBase/KernelTestBase)Problem/MotivationProposed resolutionRemaining tasksUser interface changesAPI changesData model...
View ArticleViews admin summaries are double escaped
To reproduce:- create a view - add a page display - add a menu item under 'Page settings' and give it a title containing a ' - save the menu item dialog
View ArticleConvert web tests to browser tests for field_ui module
See #2735005: Convert all Simpletest web tests to BrowserTestBase (or UnitTestBase/KernelTestBase).
View ArticleSpaces shown before commas in publishing options
Problem/MotivationSteps to reproduce: 1. Add new content type at /admin/structure/types/add 2. Make sure some options are selected under "Publishing options"Actual result: Summary shows:"Published ,...
View ArticleDeprecate WebTestBase in favor of BrowserTestBase
Problem/MotivationBTB seems to be working well. Let's deprecate WTB so people start making new tests using BTB.Proposed resolutionDeprecate WTB in favor of BTB.Remaining tasksUser interface changesAPI...
View ArticleAdd the typed data manager as proper dependency.
Follow-up to #2488568: Add a TypedDataManagerInterface and use it for typed parametersProblem/MotivationThere are a few places in TypedData plugins where we are using the typed data manager from the...
View ArticleUse getDisplayName() for user names consistently
Problem/MotivationRealname module alters the display name, but at user/1234 the username is shown. This is incorrect.Tons of places in core that show the username, but need to show the users display...
View ArticleFix 'Drupal.Commenting.FunctionComment.ReturnTypeSpaces' coding...
Part of #2571965: [meta] Fix coding standards in core.ApproachWe are testing coding standards with PHP CodeSniffer, using the Drupal coding standards from the Coder module. Both of these packages are...
View ArticleAdd changelog for Drupal 8.3.0
Problem/MotivationNo changelog entry for 8.3 yet.Proposed resolutionAdd a changelog entry with sufficient items covering the changes. Look for and tag new issues with the 8.3.0 release notes...
View ArticleFix 'Drupal.Array.Array.CommaLastItem' coding standard
Part of #2571965: [meta] Fix coding standards in core.ApproachWe are testing coding standards with PHP CodeSniffer, using the Drupal coding standards from the Coder module. Both of these packages are...
View Article"email.validator" core service cannot be easily replaced
Problem/MotivationThe core mail validator service directly uses \Egulias\MailValidator\MailValidator class, without enforcing a specific interface to generalize the behaviour of such a service.If you...
View ArticleDo not copy over the owner and engine of a theme if the child theme uses a...
When you are using a different theme engine than php-template in your sub theme that extends a base theme that uses php-template, the system module copies the engine and owner of the base theme over in...
View ArticlePHP Fatal error: Nesting level too deep
I have spotted a PHP fatal error in the logs for a project I'm working on:PHP Fatal error: Nesting level too deep - recursive dependency? in docroot/vendor/twig/twig/lib/Twig/Environment.php(403) :...
View ArticleFor better URL we can use l()
Why we are using simple a href="' . base_path() . '?q=admin/config/development/maintenance" ? If we have enabled "Clean url" then also its seems...
View ArticleConvert web tests to browser tests for contact module
See #2735005: Convert all Simpletest web tests to BrowserTestBase (or UnitTestBase/KernelTestBase)
View ArticleRedesign the status report page
Problem/MotivationThe current status report is informative but not highlighting the crucial information really well. A lot of key information gets lost in the rows of green, blue and other color...
View ArticleMax width change to status page affects installer screens as well, but...
+++ b/core/themes/seven/css/theme/maintenance-page.css @@ -142,7 +142,7 @@ - max-width: 770px; This change affects install page as well.Screenshots before:...
View ArticleReplace 'node' terminology with 'content' in node...
Problem/MotivationThe Administer Nodes permission was renamed Administer Content, but "Role requires" text still refers to the former. Confusing. These descriptions could also be worded better.Proposed...
View Article