Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.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.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 1001464.image_attach.views-attach-handler-multiple.patch | 1.66 KB | joachim |
Comments
Comment #1
joachim CreditAttribution: joachim commentedWow, that's a really cool bug! :D
Can you try this patch please?
Comment #2
ice5nake CreditAttribution: ice5nake commentedJust 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?
Comment #3
joachim CreditAttribution: joachim commentedYes please!
Comment #4
ice5nake CreditAttribution: ice5nake commentedThis 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.
Comment #5
joachim CreditAttribution: joachim commentedDo you mean that the patch applies, but doesn't fix the problem?
Comment #6
ice5nake CreditAttribution: ice5nake commentedYeah. 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.
Comment #7
joachim CreditAttribution: joachim commentedSo 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.
Comment #8
joachim CreditAttribution: joachim commentedI'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.
Comment #9
joachim CreditAttribution: joachim commentedComment #10
Scott J CreditAttribution: Scott J commentedSorry wrong thread.