Problem/Motivation

The \Drupal\redirect_node\EventSubscriber\RedirectRedirectNodes throws the following exception when the restrict_by_ip module is installed.

It throws the following exception:

TypeError: Drupal\redirect_node\EventSubscriber\RedirectRedirectNodes::__construct(): Argument #1 ($currentUser) must be of type Drupal\Core\Session\AccountProxy, Drupal\restrict_by_ip\Session\AccountProxy given, called in /app/application/web/core/lib/Drupal/Component/DependencyInjection/Container.php on line 259 in Drupal\redirect_node\EventSubscriber\RedirectRedirectNodes->__construct() (line 44 of /app/application/web/modules/contrib/redirect_node/src/EventSubscriber/RedirectRedirectNodes.php)

Steps to reproduce

Install the restrict_by_ip module.

Proposed resolution

Use AccountProxyInterface rather than AccountProxy for the event subscriber parameter.

Remaining tasks

Provide issue fork/patch.

User interface changes

N/A

Issue fork redirect_node-3373166

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

codebymikey created an issue. See original summary.

codebymikey opened merge request !2

codebymikey’s picture

codebymikey commented
Issue summary: View changes
Status: Active » Needs review

  • codebymikey committed 6feffcaa on 8.x-1.x 
    Issue #3373166: Use AccountProxyInterface rather than AccountProxy
richgerdes’s picture

richgerdes
he/him
Primary language English
Location New Jersey, USA
commented
Status: Needs review » Fixed

@codebymikey,

Thanks for the mr. Looks great. Appreciate the work.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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