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.

CommentFileSizeAuthor
#1 1001464.image_attach.views-attach-handler-multiple.patch1.66 KBjoachim

Comments

joachim’s picture

joachim commented
Version: 6.x-1.0 » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new 1001464.image_attach.views-attach-handler-multiple.patch1.66 KB

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

Can you try this patch please?

ice5nake’s picture

ice5nake commented

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

joachim commented

Yes please!

ice5nake’s picture

ice5nake commented

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

joachim commented

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

ice5nake’s picture

ice5nake commented

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

joachim commented

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

joachim commented
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

joachim commented
Issue tags: -views
Scott J’s picture

Scott J commented

Sorry wrong thread.

Status: Fixed » Closed (fixed)

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