Problem/Motivation

I got this error: TypeError: Argument 1 passed to Drupal\simple_sitemap_engines\Form\FormHelper::displayEntitySettings() must be of the type array, null given, called in /xxx/modules/contrib/simple_sitemap/modules/simple_sitemap_engines/simple_sitemap_engines.module on line 53 in Drupal\simple_sitemap_engines\Form\FormHelper->displayEntitySettings() (line 35 of /xxx/modules/contrib/simple_sitemap/modules/simple_sitemap_engines/src/Form/FormHelper.php)

Steps to reproduce

I created a new node type in a new site. The sitemap settings have been created for google and indexnow. I enabled the sitemap settings in the new node type. I added an image field. Next I tried to delete the node type and got this error.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

PROMES created an issue. See original summary.

gbyte’s picture

Version: 4.1.1 » 4.x-dev
Assigned: Unassigned » gbyte

I think I encountered it somewhere in the wild recently - will fix it quickly.

  • gbyte committed 9b7f6df on 4.x
    Issue #3268899 by PROMES, gbyte: FormHelper::displayEntitySettings()...
gbyte’s picture

Status: Active » Fixed

Fixed in dev.

Status: Fixed » Closed (fixed)

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

afagioli’s picture

In the meantime, the workaround is:
temporary disable involved entity at /admin/config/search/simplesitemap/entities

kieran.cott’s picture

I have reproduced @gbyte's solution in comment #3 as a patch.

gbyte’s picture

@kieran.cott

I have reproduced @gbyte's solution in comment #3 as a patch.

No you haven't, your patch does something entirely different. Also there is no need for that - it's been merged and has been available in the most recent version of the module.

gbyte’s picture

kieran.cott’s picture

it's been merged and has been available in the most recent version of the module.

Many thanks