Support from Acquia helps fund testing for Drupal Acquia logo

Comments

m1r1k’s picture

Status: Active » Needs review
FileSize
4.69 KB

Here is a patch

Status: Needs review » Needs work
m1r1k’s picture

Status: Needs work » Needs review
FileSize
619 bytes
4.26 KB

Ok, this piece of code depends on #2076411: Remove the request scope from the current user service

+    $account = \Drupal::currentUser() ?: $user;
-    $account = \Drupal::request()->attributes->get('_account') ?: $user;

so lest move to separate commit.
Here is new patch

Status: Needs review » Needs work

The last submitted patch, user-use-current_user-instead-of-_account-2095667-3.patch, failed testing.

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.9 was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 is now available and sites should prepare to upgrade to 8.2.0.

Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

dpi’s picture

Issue summary: View changes
Status: Needs work » Closed (cannot reproduce)

There are no references to '_account' in core.

Outdated issue.