When browsing to the "source" and "destination" pages of a given migration, one gets the following notice (either directly on the page or in the logs) : "Notice: Undefined index: #description_display in template_preprocess_fieldset() (line 219 of core/includes/form.inc)" (also on line 220). Apparently, we only need to add this key to the fieldset definition.

This is of course a minor bug, but was already spotted a few years ago by mikeryan.

CommentFileSizeAuthor
#6 3272434-6-fixed-undefined-index-descriptiondisplay.patch1.51 KBtanmayk

Issue fork migrate_tools-3272434

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

FMB created an issue. See original summary.

FMB opened merge request !16

fmb’s picture

fmb commented
Status: Active » Needs review
sim_1’s picture

sim_1
she/her
commented
Status: Needs review » Reviewed & tested by the community

I ran into this issue as well. Big warning messages on the migration source/destination pages. Thanks for this patch! It worked to fix the issue. I don't have 6.x installed, but looking at the code I think it will be beneficial for both branches. Marking RTBC

kliker’s picture

kliker commented

I tested it with v6.0.0 and I am still getting error messages:

- "Warning: Undefined array key "#description_display" in template_preprocess_fieldset() (line 219 of core/includes/form.inc)."
- "Warning: Undefined array key "#description_display" in template_preprocess_fieldset() (line 220 of core/includes/form.inc)."

tanmayk’s picture

tanmayk
he/him
Primary language English
Location Maharashtra
commented
Version: 8.x-5.x-dev » 6.0.x-dev
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 3272434-6-fixed-undefined-index-descriptiondisplay.patch1.51 KB

Here is the patch against 6.0.x for composer builds.

kliker’s picture

kliker commented

Tested patch against v6.0.0. and there are no error messages.

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Needs review » Reviewed & tested by the community

Thanks!

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented

This working for me as well.

pcate’s picture

pcate commented

Ran into this error today. Patch applied cleaning and fixed issue. +1 RTBC

stijnhau’s picture

stijnhau commented

Patch is working perfectly

  • heddn committed e627b18 on 6.0.x 
    Issue #3272434 by FMB, tanmayk, sim_1, pfrenssen, heddn: Undefined index...
heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the fix and your contributions.

Status: Fixed » Closed (fixed)

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

heddn closed merge request !16