strange module behaviors when uploading image

What version of Term Reference Tree are you using?

I ask because I have an idea of what's causing the problem, but if I'm right it would be fixed in 1.9. The 2.x branch is for development only; it's not intended for production use and at the moment it doesn't do anything that 1.9 doesn't. If you're using 2.x (particularly an old version of it), please switch to 1.9 and let me know if the issue persists.

Hi BartK. I use Term Reference Tree 7.x-1.9 (and Drupal 7.8) and I confirm the issue dencho reported. When the node form first loads, I can open the tree normally. But if I select and upload an image and then move on to open the tree by clicking a "+" to go to an item on the lower level, the parent will open quickly but then close down, in a flash. If I remove the photo uploaded and go back to the tree and try to open a parent, it will work again, but, only after flashing one time, that is, open, close, open again. I uploaded and removed an image several times to see what happens - there will be several flashes before the parent finally opens increases, it seems. Also as dencho reports, if you first select tree items and then upload an image, things work, but not the other way around. Thanks.

I'm suspicious what's happening is that for some reason the drupal behavior that attaches the event that opens and closes the tree is running multiple times. I need to add some sort of check to the object to make sure it only happens once.

No, it's not your English. I just didn't read very carefully. I apologize. :)

I'll get the fix in shortly.

Hierarchical Select also causes this problem. And as posted previously this strange behavior only occurs after using another widget (HS in my case).

I can confirm this issue in the latest 1.x dev

This is occurring in combination with the video module upload as well. Patch applied with solution provided by @dencho.

works for me, can anyone else confirm?

Works for me as well.

Confirmed

Works, you could better use once() in Drupal 7 instead of .addClass() combined with :not() --> http://drupal.org/node/444344

There's the patch using D7 standard .once() .

#1559598: Patch to fix the collapse event click after another module (node reference) ajax call (with patch) closed as a duplicate to this. Out of all the patches the patch in #14 looks best to me.

Yes patch #14 works well for me too.

Can confirm that the patch works as expected.

As it is a simple change, I would hope that it gets commited soon.

Thanks!

We need this too. Pretty please drop a release?

confirm works, thanks for fix

Tested and committed #14 to 7.x-1.x. Thanks everyone!

http://drupalcode.org/project/term_reference_tree.git/commit/8c23c4f