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.