Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.I recently upgraded Amazon Store to 6.x-2.1-rc2 and had some major problems with the way Amazon Store pages were displayed in Firefox and Chrome -- pages displayed correctly in IE7.
I'm using the Marinelli theme and, in Firefox and Chrome, the top of the page displayed correctly but the body section was truncated, resulting in most of the Amazon Store content being displayed in the page background area below the theme's footer.
No other pages in the site are causing problems. When I reverted to Amazon Store 6.x-2.0-rc2 the problem went away.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | amazon_store.item_detail_html_d7_1010484_12.patch | 3.6 KB | rfay |
| #9 | amazon_store.item_detail_html_d6_1010484_09.patch | 2.6 KB | rfay |
| #8 | amazon_store.item_detail_html_d6_1010484_08.patch | 2.59 KB | rfay |
| #6 | amazon_store.fix_invalid_html.patch | 3.19 KB | the_g_bomb |
Comments
Comment #1
rfayThanks for the report on this.
I'm a little surprised you'd be upgrading to 6.x-2.1-rc2, when 6.x-2.1 is the current version. I suspect you mean you upgraded to 6.x-2.1? If not, please report back after upgrading to 2.1.
In 6.x-2.1 there was a significant theme-related change with #597362: Printing multiple product description in one page load results in fatal error. It's possible that that's what happened to you. You should check carefully whether you have a copy of an amazon_store_details_panel.tpl.php in your theme directory, as it will need to be redone because of that change. Please take a look. After you remove it from your theme directory you'll need to rebuild the cache so the theme registry gets rebuilt.
-Randy
Comment #2
clarenza commentedHi Randy
You're right, it was 6.x-2.1 that I upgraded to, not 2.1-rc2. I have just tried 2.1 again, same problem.
I have done a search on the file amazon_store_details_panel.tpl.php, it only shows up in the modules / amazon_store directory, nowhere else.
Not totally sure what you mean by rebuilding the caches. I have flushed all caches and updated the database and that makes no difference.
Again I have reverted to 2.0-rc2 and the problem no longer exhibits.
Rob
Comment #3
rfayI assume you're using Marinelli 6.x-2.96?
Comment #4
rfayI can confirm this with 6.x-2.1 of Amazon Store and Marinelli 6.x-2.96. I imagine it's some shared CSS class or something.
Comment #5
clarenza commentedHi Randy, yes you're correct, I'm using the latest version of the Marinelli theme, 6.x-2.96. Do you think I should pursue it with Marinelli or is it an Amazon Store issue? Are there any reasons not to use Amazon_Store 6.x-2.0rc? Seems to be working OK. Rob
Comment #6
the_g_bomb commentedNot sure if this is related, I have discovered a couple of issues regarding the validity of the html produced by the module in certain situations and also noticed a couple of missing
I wasn't sure if it was just my hacked module files, but having found this, could someone test the following patch to see how it reacts to solving this issue?
If it doesn't help, then I'll create an issue my self and see what kind of response.
Comment #7
the_g_bomb commentedComment #8
rfayThanks, @the_g_bomb, that was the ticket, and resolved this problem. Had to adjust the patch as here.
Comment #9
rfayMade a mistake by not bringing over the new path stuff.
Comment #10
rfayCommitted to DRUPAL-6--2
https://drupal.org/cvs?commit=491310
@clarenza, please try out the dev release when it next gets built (in less than 12 hours)
Comment #11
rfayStill needs to be done on D7
Comment #12
rfayHere's the D7 version. It validates (except the iframe for customer reviews, which isn't preferred by the validator). Hoping that does the job. (Still works with D7 marinelli).
Comment #13
rfayCommitted to D7 HEAD: https://drupal.org/cvs?commit=491376