Replace _theme() calls by calls to \Drupal::theme()->render()

hi
i found another issue Replace all calls to _theme with calling \Drupal::theme()->render() i think it's a duplicate issue.
ok i've change 6 calls out of 7 callsof _theme() to \Drupal::theme()

+++ b/core/modules/system/src/Tests/Theme/ThemeTest.php
@@ -56,18 +56,18 @@ function testAttributeMerging() {
+    // theme_test_false is an implemented theme hook so \Drupal::theme() service container should return a
     // string, even though the theme function itself can return anything.

"service container" is this not just a service? Plus line wrapping.

      $elements['#children'] = _theme($wrapper_hook, $wrapper_elements);

This call in drupal_render() can be replaced too.

We should also have a followup to move _theme, template_preprocess and _template_preprocess_default_variables into one of the Theme services.

Ok here is the patch with the fix, hope it works :)

Committed e92eb2d and pushed to 8.0.x. Thanks!

Committed 4d82422 and pushed to 8.0.x. Thanks!

THANK YOU GUYS !!! I hope to contribute my two cents in #2340341 :)

+++ b/core/includes/common.inc
@@ -2906,7 +2906,7 @@ function drupal_render(&$elements, $is_recursive_call = FALSE) {
-      $elements['#children'] = _theme($wrapper_hook, $wrapper_elements);
+      $elements['#children'] = Drupal::theme()->render($wrapper_hook, $wrapper_elements);

Minor, but why is this one Drupal and not \Drupal?

#16 it's a mistake - I should have caught it. We need a small follow-up (in this issue) to address that - feel free to rtbc straight away.

OMG sorry about that ... my mistake, i don't know how to proceed when the patch is pushed, but here is the fix.

We just need a patch for that one line change, not all the changes that have already been committed. Thanks!

Here is :)

Great, thanks @omers!

Committed c58bca3 and pushed to 8.0.x. Thanks!