Respect front page configuration

I tested this and it worked for me. It solved the issue, and I saw no regressions.

Patch #2 tested working.

Patch from #2 has been working nicely but looks like it needed re-rolling so here goes.

Or you could just leave the '/' in custom links right as it is defined by default. The node path will then be automatically excluded as duplicate (also the default setting).

Thanks for the patch! I feel however like this is too much of a change (in terms of compatibility with extending modules) for a niche use case that will not happen with default settings, hence marking as "won't fix".

@gbyte Just installed a clean Drupal 10 site and configured the defaults but got both the root as / and the node with its alias /homepage.
So I'm not really following why this is closed as more ppl have reported this.

Is there a test that we can run to make sure this won't happen?

Seems this patch needs a re-roll for 4.1.8

Re-roll.

This is still an issue, for me at least, on at least four sites I manage. I attached a screenshot of the issue. That shows how the front page is the first link in the XML sitemap, and uses a correct link, but the sitemap also includes a node/[nid] link to the home page, and it shouldn't.

d.stoyanov → made their first commit to this issue’s fork.

Re-roll.

d.stoyanov → changed the visibility of the branch 3264573-update-v4.2.1 to hidden.

d.stoyanov → closed merge request !106

Re-roll.

Rerolling patch.

Reroll patch.

Reroll patch for v 4.2.1

with ConfigFactoryInterface fixes.

Core issue 🐛 url() should return / when asked for the URL of the frontpage Needs work which should fix this problem has not been fixed yet - I hear you guys, I'll take another look at working around this in this module soon.

Patch in #22 was working perfect with 4.2.1 but it doesn't apply anymore on 4.2.2 and the initial problem is introduced again.

Just checked if it was possible to alter the patch for it to work again.

Looks like it's more work than I anticipated due to https://www.drupal.org/project/simple_sitemap/issues/3484114 📌 Clean up UrlGenerator classes Active

He's the patch working for 4.2.2

Just tested the new patch on 4.2.2, it works great.

Thanks a lot!