Hi,

I noticed a problem with the pathauto and node clone modules:

1) I create a new node. Pathauto gives it the path /events/new-event. So far so good.
2) I clone this node. Title stays the same, body / content changes. Now pathauto again gives the path /events/new-event to this node. Looking in the admin - path list (admin/build/path/list), I now see one path leading to two nodes.
3) I clone this node a second, third, ... time. Title stays the same, body / content changes. Now pathauto gives other paths: /events/new-event-0, /events/new-event-1, ...

The problem is there only for the first time I clone a node. I have not noticed this problem before.

My settings:
Update action: Do nothing. Leave the old alias intact.
Pathauto 6.x-1.2
Node clone 6.x-1.0
Debian Etch, MySQL

Comments

greggles’s picture

That is odd. I imagine that it's somewhat because of the update action, but really this seems like a bug. It must be prefilling the node form in a way that makes pathauto think that the alias is an existing alias the first time or something?

vaafri2’s picture

There is one other thing I just noticed:

Option 1:
The original node was created under Drupal 5. If I clone this node (now under Drupal 6), the new node gets the same path.
In the admin - path - list I see the following:
(alias) /events/new-event | (system) node/50463 | (language) All
(alias) /events/new-event | (system) node/83912 | (language) German

Option 2:
The original node was created under Drupal 6. If I clone this node, the new node gets a new path.
In the admin - path list I see the following:
(alias) /events/new-event2 | (system) node/83913 | (language) German
(alias) /events/new-event2-0 | (system) node/83914 | (language) German

Perhaps the language - differences in Option 1 are the reasons for this problem?

greggles’s picture

Yeah, it could definitely be the language thing. I think there's a bug report about that already.

dave reid’s picture

Status: Active » Closed (duplicate)

I think the recent language patches that have been committed to pathauto should fix this. Likely it was #593048: _pathauto_alias_exists handles language-neutral aliases wrong.