refactor: generate unique names/urls locally

[danielpeintner]

Yet another approach... (alternative for #1352)

Advantages I see:

• each binding can choose the way it likes to create urls if there are thing name clashes. This is anyway a bit of a problematic case since no one can rely on the strategy since it depends on the order a thing has been added.
• no need for a helpers method
• no need to pass (and create) a set around with the existing names so far

fixes #1351

[danielpeintner]

@relu91 doesn't like adding "_" since it may become hard to see how man underscores have been used.
I tend to agree with that statement and I will rework the PR.

[relu91]

Ok for going with this direction, hoping then, in the future, we will have a more consistent way to generate URLs of afforndances! Thank you @danielpeintner !!!

[JKRhb]

I am also fine with the direction :) Maybe we can refactor the code at some point in the future and move the "sluggification" to a central location to ensure a consistent naming scheme, but for now, this looks good to me :)

Edit: Or rather: Integrate the name clash avoidance into a common sluggification function.

[JKRhb]

I am also fine with the direction :) Maybe we can refactor the code at some point in the future and move the "sluggification" to a central location to ensure a consistent naming scheme, but for now, this looks good to me :)

Edit: Or rather: Integrate the name clash avoidance into a common sluggification function.

Ah, sorry, I just noticed that that was actually the alternative in the other PR 😅 I have to apologize, I am only slowly getting back into the current set of issues and PRs at the moment. So I think avoiding duplication might still be worth considering in the future, but for now the current approach should work pretty well.

[danielpeintner]

Thank you for your comments!