Spinoff from #1702080: Introduce canonical FileStorage + (regular) cache

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Postponed (maintainer needs more info)

Can you please put something here explaining what this issue is for? What needs to be improved?

chx’s picture

chx commented
Status: Postponed (maintainer needs more info) » Active

catch asked:

   // We do not use drupal_static() here because we do not have a mechanism by
   // which to reinitialize the stored objects, so a drupal_static_reset() call

Is that actually true any more now you can pass in a completely new container?

My answer was:

The comment is about not using drupal_static -- if we were to use drupal_static here then after drupal_static_reset we would have just the emergency container and not the bundles based one from the Kernel.

And I suggested solving this here.

Crell’s picture

Crell commented
Issue summary: View changes
Status: Active » Closed (won't fix)

#2001206: Replace drupal_container() with Drupal::service()