Problem/Motivation

PHP Notice "Notice: Undefined offset: 1 in Drupal\login_redirect_per_role\Form\RedirectURLSettingsForm->validateForm() (line 121 of modules/contrib/login_redirect_per_role/src/Form/RedirectURLSettingsForm.php)."

Proposed resolution

Add isset() to avoid notice

Remaining tasks

Supply patch

User interface changes

None

API changes

None

Data model changes

None

Release notes snippet

Fixed php notice on settings page.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

scott_euser created an issue. See original summary.

scott_euser’s picture

Status: Active » Needs review
FileSize
1.17 KB
dakruchko’s picture

Please provide steps to reproduce. I am not facing with this issue

joco_sp’s picture

Status: Needs review » Reviewed & tested by the community

The steps are:
1. install module,
2. add the links like scott_euser shows in the image,
3. click on Save configuration,
4. error occurs.

I wanted to write a patch, but scott_euser already did it, so I applied it and it is working. I tested and applied it on the 8.x-1.2 version of the module, but I see that the same code is on the dev version, so it should also work on it.

Antonnavi’s picture

Status: Reviewed & tested by the community » Fixed
pratik.mehta19’s picture

Status: Fixed » Closed (fixed)