Maybe I'm missing something, but I don't see why \Drupal\simple_sitemap\Form\FormHelper::processForm() needs to continue if no entity was found.

\Drupal\simple_sitemap\Form\FormHelper::negotiateSettings() does a lot of quite extensive processing, like a config entity query, but the supports call will then just check if NULL is an enabled entity type.

CommentFileSizeAuthor
#2 simple_sitemap-3162869-2.patch512 bytesBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Active » Needs review
FileSize
simple_sitemap-3162869-2.patch512 bytes

  • acdbedd committed on 8.x-3.x 
    Issue #3162869 by Berdir, gbyte: Skip redundant processing of forms
gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and at gbyte commented
Title: Don't attempt to process forms if no entity can be found » Skip redundant processing of forms
Status: Needs review » Fixed

Thanks, as always a good find (how did you notice?). I altered your logic to still use supports(), but skipping the redundant code.

Status: Fixed » Closed (fixed)

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