Create any path alias already ported to D8 so this issue is to split out the delete functionality and port that to D8

Comments

Anonymous’s picture

Title: Port "Delete any URL alias (path_alias)" to D8 » Port "Delete any URL alias (path_alias)" action to D8
Anonymous’s picture

Status: Active » Needs review

Have created a pull request over on https://github.com/fago/rules/pull/105 along with the following discussion:

The original D7 function was the create or delete URL alias and thus had two inputs - source and alias. If the source was missing but an alias provided it would delete the alias, and if the alias was missing and a source provided then it would delete aliases associated with that source.

Now that the create and delete functions are split, the delete only needs one input, i.e. what you want to delete. This initial patch provides the code to delete a single url alias as the class name suggests.

I believe further discussion is needed to decide how this functionality is going to work in 8, i.e. can we still pass it a source and delete all relevant aliases (how do we check whether it's source or alias?) - can we send it an array of sources and/or aliases, etc.

So, for the moment, this is a start.

fago’s picture

Status: Needs review » Needs work

yeah, we already discussed that we want separate delete actions. However, I'm unsure how we should handle delete by path or alias. Maybe we can discuss this at todays call.

klausi’s picture

Status: Needs work » Fixed

this has been merged.

Status: Fixed » Closed (fixed)

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

dasjo’s picture

dasjo’s picture