Use ContainerAwareTrait
Updated: Comment 0Problem/MotivationCurrently we do have a couple of container aware services which provided $container as well as setContainer().Proposed resolutionLet's use the ContainerAwareTrait...
View ArticleRename \Drupal\Core\Path\Path to \Drupal\Core\Path\AliasStorage
Updated: Comment #0Problem/MotivationSome classes in Path subsystem have strange naming conventions. We should re-think this and make naming more clear and logic.This issue is dependent on #2199381:...
View ArticleViews handlers should not be initialized during route building
Updated: Comment #NProblem/MotivationSteps to reproduce: Go to /admin/structure/views Click "Enable" for one of the disabled viewsExpected After a throbber, the view will move to the "enabled" table...
View ArticleStep 2: Wrap the existing logic of accessing menu links into a plugin manager
Updated: Comment 0Problem/MotivationOnce #2226903: Step 1: Move static menu links to yml files is in we should try to wrap all the current code which is rendering menu links etc. to not use the content...
View ArticleClik here to view.
Incorrect error message when changing the outline of a book page
saveBookLink() in \Drupal\book\BookManager should return an array on success but doesn't return anything. This causes submit() in \Drupal\book\Form\BookOutlineForm to think an error has occurred on...
View Article[regression] Support route parameters for *info.yml files'"configure" route name
Updated: Comment #NProblem/Motivation*.info.yml files support the configure property, which can contain the name of a route to which a link can be generated for use on the modules page. However, route...
View Article[META] - Split book module from menu link
Updated: Comment #1Problem/MotivationThe re-use of the {menu_links} table by book module has always been a bit of an awkward implementation. I (pwolanin) wanted to build book on tops of the menu link...
View ArticleRoute rebuilding is not guaranteed to finish in time for the next request
Updated: Comment #NProblem/MotivationWhen some code needs to rebuild the router, it calls setRebuildNeeded() instead of calling rebuild() directly, in order to prevent multiple rebuilds in one request....
View ArticledrupalSettings.pathPrefix does not bear language identifier
Should we expect that in drupalSettings.path.pathPrefix we will have the language code, in case we have a multilingual installation with URL language detection enabled, and we are on a page with a...
View Article.htaccess protections do not work on Apache 2.4 without mod_access_compat
In Apache 2.4, the "Order" becomes deprecatedWith Apache 2.4 the old Order get's deprecated in favor of Require.If hosters by default have Access compat, then Order keeps working. Otherwise however,...
View ArticleConvert element_info() to a service
Updated: Comment #NProblem/Motivation@TODOProposed resolution@TODORemaining tasks@TODOUser interface changes@TODOAPI changes@TODO
View Articleimage.module - Convert theme_ functions to Twig
Issue #1898420 by shanethehat, claudiu.cristea, mondrake, DamienMcKenna, c4rl, thedavidmeister, Albert Volkman: Image.module - Convert theme_ functions to Twig.TaskUse Twig instead of...
View ArticleRemove globals on template_preprocess_pager
template_preprocess_pager use two global: $pager_page_array and $pager_total.Can we remove those global? I was making a theme for D8 and having to manipulate global variable on a processors was feeling...
View ArticleConvert pager.inc to a Pager component
I do not know how this would fit in the timeline :( but looking at pager.inc overall I thought it may be worth to build a component for the helper functions and to move the pager theme preprocessing to...
View ArticleOn Ajax-enabled forms with a pager, pager is disappearing on form validation...
Updated: Comment #11Problem/MotivationWhen a form has a pager in it, and an element is Ajax enabled, if the form gets a validation error and/or the form is reloaded out of an Ajax request, the pager is...
View ArticleRegression: Entity definitions miss a language entity key
While working on #1966436: Default *content* entity languages are not set for entities created with the API we figured that Drupal 8 misses the 'language' entity key, as it got introduced for Drupal 7...
View ArticleUpdate hook_help for content_translation module
Background: This issue is part of the task to update the hook_help texts of the Drupal 8 modules:#1908570: [meta] Update or create hook_help() texts for D8 core modulesTasks: - review / write the...
View ArticleAdd a ParamConverter for upcasting TypedData objects
I've been playing with this around in my head for the past couple of days and after discussions with Kris, Sam, Larry, etc. I came back to the conclusion that we -really- need this. And before the end...
View ArticleRemove all references to/implementations of hook_schema_0()
Apparently #1018602: Move entity system to a module sneaked in this hook but we don't have docs to explain what it does.This was originally a critical upgrade path issue due to the unpredictability of...
View ArticleVarious calls to exit() in core completely break the HttpKernel...
ProblemIn various places throughout core, code is manually sending a response and calls exit().This completely breaks the HttpKernel request/response flow, which primarily means that Kernel::TERMINATE...
View Article