That's it - clean install, no contributed modules but Taxonomy Menu. When I delete a node tagged "white", menu item "white" disappears. I nave to uncheck "Syncronise changes to this vocabulary" to stop this. Same thing with the latest dev version.
Drupal - 6.14.

CommentFileSizeAuthor
#2 Fix-Menu-Corruption-v2.patch2.18 KBelprans
#1 0001-Fix-a-number-of-bugs-that-led-to-a-horrible-menu-cor.patch1.85 KBelprans

Comments

elprans’s picture

elprans commented
StatusFileSize
new 0001-Fix-a-number-of-bugs-that-led-to-a-horrible-menu-cor.patch1.85 KB

This is caused by a nasty bug in Taxonomy Menu synchronization code. The patch that fixed this for me is attached. This should also fix #514640: Duplication of terms.

elprans’s picture

elprans commented
StatusFileSize
new Fix-Menu-Corruption-v2.patch2.18 KB

Updated patch

indytechcook’s picture

indytechcook commented

Very good work elphrans. Thank you very much. It's tough to do hard core debugging with a full time job and 4 month old baby.

I agree with your statement about now running the rebuild every time a node is change. The only time it needs to run is if the option "show node count" or "hide empty terms" is enabled. Currently is always runs. I will create a new issue and a new patch for that.

Quick question. Is this patch for DRUPAL-6--2 branch for the DRUPAL-6--2-4-BETA1 release?

elprans’s picture

elprans commented

The patch is against DRUPAL-6--2-4-BETA1

indytechcook’s picture

indytechcook commented
Status: Active » Needs review

This has been commited. Please look for the dev release soon. If it fixes the issue, then please post here and I will add a new stable release.

Thanks again to elprans for fixing this long standing issues.

NaX’s picture

NaX commented

I think this patch fixed a nasty bug I ran into. http://drupal.org/node/383780
I can’t confirm if this patch fixed the problem but after rebuilding and clearing cache the errors went away.

I think another beta release is in order.

indytechcook’s picture

indytechcook commented
Status: Needs review » Fixed

This is part of 6.x-2.4-beta2. Since Nax reported it fixed the issue, setting to fixed.

Status: Fixed » Closed (fixed)

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