On the admin/config/search/xmlsitemap/entities/settings configuration page, when any of the entity type checkboxes are checked, the only thing that appears is modules/xmlsitemap/templates/xmlsitemap-content-settings-table.html.twig (or similiar) which is configured as #theme setting for the $form['settings'][$entity_type_id] field, but that file doesn't exist.

CommentFileSizeAuthor
#4 2401665.xmlsitemap-content-settings-table-4.patch6.72 KBMichelle
#2 2401665.xmlsitemap-content-settings-table-2.patch6.69 KBAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stella’s picture

stella CreditAttribution: stella commented
Issue summary: View changes
Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
2401665.xmlsitemap-content-settings-table-2.patch6.69 KB

We no longer need a theme file now that the 'table' type exists

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Active » Needs review
Michelle’s picture

Michelle
she/her
Primary language English
Location Wisconsin, USA
CreditAttribution: Michelle at Mediacurrent commented
FileSize
2401665.xmlsitemap-content-settings-table-4.patch6.72 KB

Your patch didn't apply anymore so I manually applied it to the code and created a new patch. This applies and seems to work so that I can see and save the settings.

paulmckibben’s picture

paulmckibben
he/him
Primary language English
Location Atlanta, GA
CreditAttribution: paulmckibben commented
Status: Needs review » Reviewed & tested by the community

The patch applies cleanly and fixes the problem. I am able to configure the XML Sitemap Entities.

andrei.dincu’s picture

andrei.dincu CreditAttribution: andrei.dincu commented
Status: Reviewed & tested by the community » Fixed
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Fixed » Reviewed & tested by the community

@andrei.dincu: Please don't mark issues "fixed" until the patch is actually committed.

paulmckibben’s picture

paulmckibben
he/him
Primary language English
Location Atlanta, GA
CreditAttribution: paulmckibben commented
Status: Reviewed & tested by the community » Fixed

Thanks! The patch at #4 has been committed to the 8.x-1.x branch.

  • paulmckibben committed 55f00e8 on 8.x-1.x authored by Michelle 
    Issue #2401665 by andythorne, Michelle: xmlsitemap-content-settings-...

Status: Fixed » Closed (fixed)

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