One field is "Image attach: Attached images Attached images" and the other is "(Venue) Image attach: Attached images Attached images"

You might noticed that one is through a View Relationship. In this case which ever one is ordered first is the one that gets displayed. The image that is displayed is displayed in both places.

#1 1001464.image_attach.views-attach-handler-multiple.patch1.66 KBjoachim
PASSED: [[SimpleTest]]: [MySQL] 253 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


joachim’s picture

Version: 6.x-1.0 » 6.x-1.x-dev
Status: Active » Needs review
1.66 KB
PASSED: [[SimpleTest]]: [MySQL] 253 pass(es). View

Wow, that's a really cool bug! :D

Can you try this patch please?

ice5nake’s picture

Just got back to the office after an extend holiday break. I noticed there has been some releases since you made the patch. Does this patch still need tested?

joachim’s picture

Yes please!

ice5nake’s picture

This patch works as expected on the version of 6.x-1.x-dev that was out when you made the patch. Thanks for the fix!

I tested the patch on 6.x-1.1 and no luck there. It doesn't work on the newer 6.x-1.x-dev that was released on 12/22/2010 either. Bummer.

Not sure why because the patch seems to have applied itself properly. I didn't do a version diff. I am clueless as to why it didn't work on the newer version of the module.

joachim’s picture

Do you mean that the patch applies, but doesn't fix the problem?

ice5nake’s picture

Yeah. Although when I wiped my image module directory and did a fresh install the patch worked on 1.1.

I'm guessing it was some funkiness on how drush was updating my install that was causing my problem.

joachim’s picture

So the patch fixes the problem on an old dev but not on the current dev? That's really weird -- I don't think anything has changed in that part of the module for months.

Could you try again with the CURRENT dev version please? That's the only thing that matters -- 1.1 and old devs are irrelevant.

joachim’s picture

Status: Needs review » Fixed

I'm satisfied that the patch fixes the problem. Committing to git.

Issue #1001464 by joachim: Fixed multiple instances of the Views attached image handler clobbering its own prior data.

joachim’s picture

Issue tags: -views
Scott J’s picture

Sorry wrong thread.

Status: Fixed » Closed (fixed)

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