Improve status message on alias create/update

Let's see if tests pass or anything needs to be amended there.

Please note that DrupalCI and all patch testing will be turned off on July 1, 2024, so generally speaking, it's probably better to create a patch and Merge Request with the Gitlab integration, to be better prepared in advance of the eventual phase out.

Where can the message shown in the issue be reproduced? Is it related to the Pathauto module or Path? I only see this status message when I create or modify a URL Alias.

Tests are failing.

Tests are passing again.

Code review of MR #123

The two commits improve the status message displayed when pathauto creates or updates a URL alias, making it more user-friendly for content editors.

Changes

AliasStorageHelper — Two improvements to the status messages:

• The word "alias" is replaced with "path", which is more accessible to non-technical users.
• The alias is now rendered as a clickable link using Link::fromTextAndUrl(), allowing editors to quickly navigate to the new path. The placeholder changes from %alias (escaped text) to @alias (raw markup) so the link HTML is rendered correctly.

PathautoSettingsFormWebTest — The two pageTextContains assertions are updated from "Created new alias" to "Created new path" to match the new wording.

The fix is correct and straightforward.

@minirobot

The alias is now rendered as a clickable link using Link::fromTextAndUrl(), allowing editors to quickly navigate to the new path. The placeholder changes from %alias (escaped text) to @alias (raw markup) so the link HTML is rendered correctly.

isn't that incorrect?
@ and % are both sanitzied, but % highlights the string in <em> which makes sense here.

See https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Component%21Rend...

Fixed that small hallucination 😉

The field in the UI is called "URL alias", I think if anything we should use the same, not path.

admin/config/search/path/add also uses URL Alias and System path, using path here is in the middle of these two terms and makes it worse, not better.

These messages are I think also off by default and few editors will see them in practice.

Did you test that it works with the link like this? I guess it will still be escaped? (Just looking at the code). Maybe better split the text and the link in the message?

And maybe the text should better be:

Created new path alias %alias ...

?
Maybe you could post a screenshot from the result?

Are you ok with "path alias" @berdir or should we close it as "Won't fix"?

I guess #19 was a crosspost, just one minute after me. IMHO it's either "URL Alias" or won't fix, don't care too much which it is, it's hidden in a verbose logging setting, I don't know how many have this one but I guess not many, I think most editors/sites don't need this information at all.

@anybody, what do you prefer? "Created new URL alias %alias..." or "Won't fix"?

You choose 😉

@mably I agree with berdir, but I like

Created new URL alias %alias

and it will make things better, so let's do it.

It's done.