Support from Acquia helps fund testing for Drupal Acquia logo

Comments

InternetDevels’s picture

Status: Active » Needs review
FileSize
782 bytes

Here is the patch.

Status: Needs review » Needs work
Issue tags: -CodeSprintCIS

The last submitted patch, drupal-tour_replace_user_access-2062031.patch, failed testing.

larowlan’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work
Issue tags: +CodeSprintCIS

The last submitted patch, drupal-tour_replace_user_access-2062031.patch, failed testing.

andypost’s picture

Component: user system » tour.module
+++ b/core/modules/tour/tour.module
@@ -73,7 +73,8 @@ function tour_library_info() {
+  $account = Drupal::request()->attributes->get('_account');

@@ -107,7 +108,8 @@ function tour_toolbar() {
+  $account = Drupal::request()->attributes->get('_account');

Use Drupal::currentUser() service https://drupal.org/node/2032447

rhm5000’s picture

Status: Needs work » Needs review
FileSize
832 bytes
684 bytes
andypost’s picture

Status: Needs review » Reviewed & tested by the community

Nice!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.