Problem/Motivation

If I'm enabling Trash functionality for Content, I *more than likely* want to enable Trash for URL aliases, otherwise it can result in URL aliases sticking around even for trashed content, and if an editor goes to create a new path with the same desired alias, they end up getting an alias with the -0 suffix on it. How can we guide users to enable both Content and URL aliases at the same time? I checked multiple projects I've been on that have the module enabled, and none of them had enabled Trash support for URL aliases, but they likely should have given the issues I knew they ran into (and didn't realize it was the Trash module).

Proposed resolution

TBD

Remaining tasks

TBD

User interface changes

TBD

Issue fork trash-3570653

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

dave reid created an issue. See original summary.

amateescu made their first commit to this issue’s fork.

amateescu opened merge request !104

amateescu’s picture

amateescu commented
Status: Active » Needs review

I checked multiple projects I've been on that have the module enabled, and none of them had enabled Trash support for URL aliases

That's probably because support for URL aliases has been added fairly recently :)

In any case, let's enable them by default on module installation, and provide a helpful message on the module's configuration form.

  • amateescu committed c2b23b78 on 3.1.x 
    feat: #3570653 Really easy to forget to turn on Trash for URL aliases...

  • amateescu committed fd142d14 on 3.x 
    feat: #3570653 Really easy to forget to turn on Trash for URL aliases...
amateescu’s picture

amateescu commented
Status: Needs review » Fixed

Merged :)

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.

dave reid’s picture

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

That's probably because support for URL aliases has been added fairly recently :)

Well that explains that. Yeah, turning it on by default makes sense to me, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.