I have two vocabularies assigned to my node. A Category non-tag vocabulary and a Tag tag vocabulary.

When new tags are added using community tag, it removes all my Category tags.

Comments

OneClickVentures’s picture

Priority: Normal » Critical
Status: Active » Needs review

This patch for the 5.x version fixes this version as well (line numbers are off, I'm sure):
http://drupal.org/node/159479#comment-581505

danielnolde’s picture

having the same problems. is this fixed in the 6.x-dev version?

danielnolde’s picture

patch seems to work for 6.x

dnewkerk’s picture

Subscribing... will test the patch as well and report back.
(no, it is not fixed in the 6.x-dev 2008-Aug-23 version yet)

owahab’s picture

Status: Needs review » Active

No patches here so moving it to active.

davidseth’s picture

Version: 6.x-1.0-beta1 » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new1.03 KB

Here is a proper patch for 6.x-1.x-dev.

andreiashu’s picture

Status: Needs review » Fixed

The patch from #6 provided by david is already included in the latest dev snapshot of the module.

bob_hirnlego’s picture

Status: Fixed » Needs work

Yes, the patch #6 is included but it shouldn't be.
In latest 6.x-1.x-dev release @ line 370 I read:

$result = db_query('SELECT n.tid, n.nid FROM {term_node} n LEFT JOIN {community_tags} c ON n.tid = c.tid AND n.nid = c.nid INNER JOIN {term_data} d ON n.tid = d.tid WHERE n.nid = %d AND c.nid IS NULL AND d.vid IN ('. $placeholders .')', $nid, $community_tagged);

which is wrong.
It seems to me that #6 diff should be the other way around.

loze’s picture

it is backwards.
I switched the +'s for -'s in #6 and it looks like it worked.

tiantian20007’s picture

Yes, the patch in #6 fix this bug. However, another bug come up : If the node is in the forum , new Tagging erases the forums' taxonomy ! The node will no longer in the forum after tagging.

fractile81’s picture

I also had this problem, and applying the reverse of the patch in #6 appears to have fixed the problem for me as well.

Quick question to everyone with this problem: were you only losing terms that were in your first vocabulary (vid 1)? Being curious, I'm looking at the old code to see why there was a problem to begin with. It looks like the $community_tagged array was being passed as the value for a single %d. It also appears that sites with more than one vocabulary configured for community tags may have not been working correctly. The opposite of what's in the patch in #6 (switch around the -'s and +'s) fixes both of these problems, as far as I can tell.

Tim Chan’s picture

Ya, I have the first vocabulary for categories and the second for community tags. I used the latest dev snapshot which includes the patch in #6. But the bug was still there i.e. the category became blank on node edit page. So I applied the reverse of the patch in #6 and it works. Weird.

WildBill’s picture

Sorry, it's not clear to me whether the latest dev version has fixed this problem. My understanding is no - that the latest dev version still requires patching so that Community Tags won't delete terms in an unrelated vocabulary.

This bug is critical and renders the module unusable for me. Could someone update the dev snapshot with a fixed version? Better yet, if we could also fix "Cannot delete tags using My Tags X" it would destroy what are in my opinion two seriously important bugs.

ankheg’s picture

For me #6 + #9 solved the problem.

marcoBauli’s picture

same here, #6+#9 made things work again.

@fractile81: yep, vocabulary 1 only here too

drupalok’s picture

so... is this gonna be in a release or do i have to patch?
thanks for this cool module!

jcmarco’s picture

Status: Needs work » Reviewed & tested by the community
StatusFileSize
new1.08 KB

I have tested last #6+#9 patch and works fine as the rest of people that already have reported it.
I attach the patch re-rolled in the right format.

entrigan’s picture

thanks jcmarco and david seth, I will commit this tomorrow.

entrigan’s picture

Status: Reviewed & tested by the community » Fixed

committed

Status: Fixed » Closed (fixed)

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