Closed (fixed)
Project:
Amazon associate tools
Version:
5.x-1.5
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
19 Aug 2007 at 18:17 UTC
Updated:
14 Jul 2012 at 19:36 UTC
Jump to comment: Most recent file
Comments
Comment #1
Chad_Dupuis commentedI've also recreated this issue on a clean 5.x install to eliminate the possibility that the upgrade would have caused this behavior. Any help would be appreciated....
Comment #2
ambereyes commentedI have the same issue. I tried changing the weight of the module to 5, no good.
Comment #3
ambereyes commentedI fixed it.
I added a weight to the related links array in the amazon nodeapi hook.
If I have some time tomorrow I will try and create a patch ... but if you are faster .. by all means go for it.
Comment #4
ambereyes commentedOoops left out some code ...
Comment #5
Chad_Dupuis commentedThank you ambereyes -- works like a charm and I would have never found that! I had also tried changing the weight in the system table for amazon.module to no avail - I'm not sure why that wouldn't work... Oh well!
Comment #6
Chad_Dupuis commentedI've attached the patch that fixes this issue, hopefully this can get rolled in to a release. Accordingly, I've marked the priority critical and moved it to bug reports....
Comment #7
csc4 commentedPatch seems to work well - is there any chance of getting this committed to a full release?
Comment #8
jo1ene commented+1 Works for me
Comment #9
ramper commentedJust another reminder of the real power of Drupal - it's the support from the community. I thank the module creator, as well as the user (s) who raised the issue. I was faced with the same situation where the related content was showing up at the top and I couldn't figure out a way to fix it -so here I come and found the solution. Thank you.
Comment #10
ramper commentedHi, I am not creating a separate issue as this seems related to the original issue here: The patch seems to work, although initially it was working only for certain user roles - I enabled 'search amazon' for all roles, and then it seems to work for anonymous users as well, though I don't understand the connection. But here is one thing I would still like fixed, if possible:
On the teaser, the related products show up along with the text, on the front page. I have the 'excerpt' module and often use custom teasers, but there is no way to get rid of the amazon related products from showing up in the teasers - any suggestions, or anything I am overlooking? I am still a relative newbie, BTW, and I'm playing with my Linux sandbox.
Comment #11
ramper commentedI found the 'Retease' module, installed it and rebuilt the teasers - so now the amazon related products do not show up on the teasers anymore.
Comment #12
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #13
ambereyes commentedThis issue was prematurely closed due to Ramper mistakenly changing the status.
Has this patch been included in the current release?
Comment #14
ambereyes commentedOh and he also changed the issue. Arrgh!
Comment #15
MGParisi commentedThe patch has been included in my release. How this will be handled is upto the drupal staff and the original author. I would love to just post the new moduel!
Comment #16
mcurry commented*subscribe*
Comment #17
MGParisi commentedfixed, should be in Dev
Comment #18
MGParisi commentedmarked as fixed
Comment #19
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.