Hallo pathauto dev team,

I faced recently a quite serious mistake – pathauto creats the aliases to the already existing nodes. The mistake appears when there is the node having for example a name “kaktus” (nid = 123). If somebody tries to add new node calling “kaktus :)” (nid = 234) then the pathauto logic will remove “ :)” and try to create an alias that points to nid = 123. This is wrong, because the correct nid = 234.

In my conversion rules the module removes spaces, “:” and “)” signs.

In my case the right pathauto behavior would be to add at the end of alias digit 1, e.g. “kaktus1”.

Thank you in advance,
Siegfrid

Comments

dave reid’s picture

Status: Needs review » Postponed (maintainer needs more info)

That's what it should be doing and has been doing for a while.

Siegfrid’s picture

The conversion is working only for nodes having the same name, e.g. "kaktus" and nid = 123..128. Then each node will get it's own alias. But in my case user tries to add a new node having particular same name, i.e. "kaktus :)" nid = 234, and seems that conversion logic does not check after preprocessing (removing special signs) whether there is such alias or not for the new processed name - "kaktus" and nid = 234.

Hope I have clearly explained my case :)

greggles’s picture

Priority: Major » Normal

You explained your case, but I can't reproduce it.

I followed the steps exactly and the resulting alias was content/kaktus-0

Can you try again on a fresh system and write down every configuration step you take?

greggles’s picture

Title: pathauto creats the aliases to the already existing nodes » Pathauto creates the same alias for two nodes if the second has punctuation which gets removed

One other thought - perhaps this is caused by language? Do the two nodes have different languages on them?

Siegfrid’s picture

Yes, it is. I'm using transliteration module to convert kyrillic into latain.

Siegfrid’s picture

One more hint. I faced that the special sings like ":" and ")" only removes independing on conversion rule (i.e. change on default separator - "-").

As a temporary solution I decided to use additionally nid in the alias name.

mably’s picture

Issue summary: View changes
Status: Postponed (maintainer needs more info) » 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.