Replace the instances where the global $user object is used to the new current_user service.

Change record: global $user deprecated in favor of current_user service.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Palashvijay4O’s picture

Status: Active » Needs review
FileSize
9.92 KB

Please review !

Palashvijay4O’s picture

FileSize
3.02 KB

Sorry submitted a wrong patch . Here's the right one .

Palashvijay4O’s picture

FileSize
3.02 KB

Really very sorry for the wrong patch again. :(

Palashvijay4O’s picture

FileSize
3.49 KB

This should be the last one :(

Palashvijay4O’s picture

FileSize
3.56 KB

I found that I was working on some other branch .

pfrenssen’s picture

This looks very good, just one small remark:

+++ b/scheduler.module
@@ -5,6 +5,8 @@
+use \Drupal\Core\Session\AccountInterface;
+

Don't precede the namespace with a backslash.

pfrenssen’s picture

Status: Needs review » Fixed

Fixed minor remark from #6 on commit.

Committed to 8.x-1.x. Thanks!

Status: Fixed » Closed (fixed)

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