|Issue tags:||Needs committer feedback|
Notes: This is not an issue for D7 any longer (see post #299 for more info). A -perhaps- related issue for pathauto 7.x is #1236030: interface language change the path of the node.
The original problem description mentioned the error "user warning: Duplicate entry '...' for key 2 query: INSERT INTO url_alias (src, dst, language) VALUES (...)". This warning has disappeared since Drupal 6.16**. Comments up to #52 are not valid any more.
The cause of this warning, outlined in #53 along with the first patch, also caused aliases to disappear in other nodes (with same path, but different language) -- as outlined in #103. This is what the issue is mainly about, at the moment.
After various incarnations of patches:
- pfournier posted a new patch in #188 which was closely examined (by various people, a.o. roderik).
- roderik expanded on his patch according to some new info I found, which resulted in the patch in #275 after close scrutiny by pfournier.
(copied from #302)
We can consider the patch well-researched.
The only 'unresolved' discussion is what the goal of the patch should be:
- only fixing the bug while taking care that as much behavior as possible is retained <- roderik's stance; patch in #275
- doing away with some illogical behavior at the same time. <- pfournier's stance; would need a little change to #275, basically scrapping an UPDATE statement.
This needs comment from the D6 core maintainer.
The difference is only in 'theoretical' edge cases; it doesn't have any effect on code in Drupal core or pathauto (or any known/reported contrib module).
(The mentioned "illogical behavior" was specified at some time, but the spec is clearly outdated.)
(** read #1106030: Node edit form risks deleting another node's path alias for the reason why the original error has disappeared.)