The current form just deletes aliases directly without a confirmation. We should have one.

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

As in modify default path.module's behavior for deleting an alias? I'm not sure that's worthy for Pathauto. Filing an issue against D7/D8's core path module seems more appropriate.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

This is about the page to bulk delete aliases that pathauto provides.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Issue tags: +bulk

Tagging all the bulk alias issues for #713238: RFC: Pathauto Bulk module.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Title: require confirmation before deleting aliases » Require confirmation before deleting aliases
Version: 6.x-1.x-dev » 7.x-1.x-dev
Component: Code » Bulk generation
Category: bug » feature

I think this is kinda silly. You're on a page who's explicit purpose is to delete aliases. Even more, all the checkboxes are unchecked, so you have to manually go through and check which URL types to be deleted.

There is even text on the page that says:

Note: there is no confirmation. Be sure of your action before clicking the "Delete aliases now!" button.
You may want to make a backup of the database and/or the url_alias table prior to using this feature.

I'm +1 to marking this as 'won't fix' or 'by design'.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

It was a request from Dries.

In general Drupal uses confirmation forms. Until we have a way to "undo" we should provide a confirmation form.

mably’s picture

mably commented
Issue summary: View changes
Status: Active » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.