Fix 'Drupal.Commenting.InlineComment.SpacingAfter' 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 ArticleFixing docblocks for core/lib/Drupal/Core/Entity
Issue referred from https://www.drupal.org/node/2599524#comment-10490444+++ b/core/lib/Drupal/Core/Entity/Query/Sql/Tables.php @@ -235,7 +235,9 @@ public function isFieldCaseSensitive($field_name) { @@...
View ArticleInclude 'composer' directory in phpcs scans
Problem/MotivationRecently, a new top-level directory composer was introduced. This directory holds sources for code that does not need to be packaged in the drupal/core subtree split, and therefore...
View Article[meta] Fix coding standards in core
You can check the current state of coding standards compliance using phpcs and coder from the command line:$ cd /path/to/drupal/root/ $ composer install $ ./vendor/bin/phpcs -p -s...
View ArticleFix 'Drupal.Commenting.DocComment.LongFullStop' 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 ArticleClik here to view.
Design for Range input
Problem/MotivationDuring testing of some patches relating to incline from error stlying in Seven, we have noticed that the '#type' => 'range' Form API element has inconsistent styling accross...
View ArticleBug in Content Type
When I am going to Structures then Content Types and then under Article Operations and from manage fields, I am selecting to delete the content type. Suppose I change my mind in the confirm delete page...
View ArticleAdd hook_post_config_import_NAME after config import
Problem/MotivationDrupal core has a hook_update_N() mechanism with which developers can fix the database. But often we also want to abuse it to deploy other changes. This can lead to the situation that...
View ArticleCSRF check always fails for users without a session
Problem/MotivationAny route using the _csrf_token requirement doesn't work for users without session because the CSRF checker fails as the CSRF seed is not stored anywhere. Proposed resolutionOnly add...
View ArticleRemove pathauto version check from 9.0, no incompatible version is Drupal 9...
The check has some problems, like not being able to detect dev versions/git checkouts without additional modules. And it serves no purpose in 9.0 plus,
View ArticleClik here to view.
Class active-trail not added to li element when linking to front page
D8 explanation:I've installed bootstrap 8.x-3.x on a multilanguage site, but in the default main menu the home link is never active, when I'm in the home page the claass="is-active" won't appear. This...
View ArticleClik here to view.
Toolbar rendering creates flickering
Problem/MotivationThe toolbar creates a visual flickering when initialised. This happens on every page-load, both in vertical and horizontal orientation.There are 4 possible...
View ArticleRevamp descriptions of items in handler listings
Problem/MotivationText in the Views interface doesn't follow suggestions and guidelines documented in User interface guidelines: Interface text. Most of this originates from the desire to have...
View ArticleViews Sql Server OFFSET problem
When I click on next in the pager. Drupal\Core\Database\DatabaseExceptionWrapper: Exception in Webforms RI[webforms_1]: SQLSTATE[42000]: [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Incorrect...
View ArticleClik here to view.
Missing padding in Views Add button popup's notice message
Problem/MotivationWhen adding new field in view, the notification message misses left-paddingSteps to reproduce:Set Claro as default admin themeAdd a new view. Choose display FieldsAdd a new field - a...
View ArticleDo not track viewing history for unsaved entities
Problem/Motivationhistory_node_view_alter() tracks the viewing history for all nodes that are viewed in the full view mode.That means also previews of unsaved nodes are tracked.Proposed...
View ArticleExposed filter using checkbox always checked (e.g. history is-new)
I'm trying to create a Views filter plugin which shows a checkbox, and seeing some odd behaviour:1. My own custom code works with submitting the form: I correctly get 'key=1' in the URL query when I...
View Articleextend Block Visibility to Taxonomy and Views
The Block Visability doesn't reflect the entire Drupal 8 capability. It misses the ability to place blocks in context of Taxonomy Vocabulary and Terms. It also misses the ability to place Blocks in...
View ArticleOverriding already overridden libraries-override requires knowledge of...
Problem/MotivationA bug in libraries-override #2451411: Add libraries-override to themes' *.info.yml makes it difficult to override a library or library asset that has already been overridden. You have...
View ArticleFix SQLite variable limit
Updated: Comment #6Problem/MotivationSQLite has a variable limit of 999 which is not configurable and which makes this database driver unusable for medium sized sites. See #2028713: Menu router does...
View Article