Remove calls to deprecated global $user from language.module

Name is not a tag.

This looks very straightforward and all are in global functions, so global invocation of Drupal::* is appropriate.

This patch and all of its ilk are postponed on at least a couple of upstream issues right now. See the referenced meta issue.

#2062151: Create a current user service to ensure that current account is always available landed. This should now use the current_user service.

Rerolling after #2062151: Create a current user service to ensure that current account is always available. Also depends on #2076411: Remove the request scope from the current user service

for easier tracking

Is Drupal::currentUser() the best current way to access this? Also, I think Drupal:: should be \Drupal:: no?

Drupal::currentUser() is an official way to get current userSession object now, also \Drupal:: is used on OO code, not in functional as I know. Did I miss something?

Right, superb, let's get this in then!

Let's postpone this on #2076411: Remove the request scope from the current user service as the patch includes it :)

For easy tracking

#7: language-remove-global-user-from-language-module-2061929-7.patch queued for re-testing.

.

Re-roll

Change status

Looks simple and good.

+++ b/core/modules/language/language.module
@@ -450,7 +450,7 @@ function language_get_default_langcode($entity_type, $bundle) {
+      $user = Drupal::currentUser();

We should prefix Drupal as \Drupal for this: #2053489: Standardize on \Drupal throughout core ?

Yep.

Committed f5a948f and pushed to 8.x. Thanks!