Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 293129

move config_context_enter to a container-aware service

$
0
0

Problem/Motivation

+++ b/core/includes/config.incundefined
@@ -99,6 +108,54 @@ function config($name) {
+function config_context_enter($context_name) {
+  if (drupal_container()->has($context_name)) {
+    $context = drupal_container()->get($context_name);

Could move to a container-aware service.

Proposed resolution

Move to a container-aware service.

Remaining tasks

  • implement the move

User interface changes

No UI changes.

API changes

Yes?

Original report by @catch

Follow up for #1763640-207: Introduce config context to make original config and different overrides accessible and #212

+++ b/core/includes/config.incundefined
@@ -99,6 +108,54 @@ function config($name) {
+function config_context_enter($context_name) {
+  if (drupal_container()->has($context_name)) {
+    $context = drupal_container()->get($context_name);

@catch:

This looks like it could move to a container-aware service?

#212
@alexpott:

@catch said on irc that moving config_context_enter() to a container-aware service could be explored in a later patch.

Viewing all articles
Browse latest Browse all 293129

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>