Problem/Motivation

Use dependency injection for $user_settings = \Drupal::config('user.settings');

Proposed resolution

  /**
   * RegistrationLinkAccessCheck constructor.
   *
   * @param ConfigFactory $configFactory
   *   The Config Factory service.
   */
  public function __construct(ConfigFactory $configFactory) {
    $this->configFactory = $configFactory->get('user.settings');
  }

Remaining tasks

  1. Create patch
  2. Review/RTBC
  3. Commit

User interface changes

- None

API changes

- None

CommentFileSizeAuthor
#2 3091039-2.patch2.23 KBi-trokhanenko
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

i-trokhanenko created an issue. See original summary.

i-trokhanenko’s picture

Assigned: i-trokhanenko » Unassigned
Status: Active » Needs review
FileSize
2.23 KB

Please review!

i-trokhanenko’s picture

Issue summary: View changes
Nikolay Khomich’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! #2 works fine for me

i-trokhanenko’s picture

Assigned: Unassigned » i-trokhanenko
Issue summary: View changes

i-trokhanenko’s picture

Assigned: i-trokhanenko » Unassigned
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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