Closed (fixed)
Project:
Amazon Store
Version:
7.x-1.0-beta3
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
26 Feb 2011 at 22:48 UTC
Updated:
13 Mar 2011 at 20:51 UTC
Jump to comment: Most recent file
Version is 7.x-1.0-beta5, not 3 as shown above...
When trying to add any item to the shopping cart this error appears:
An AJAX HTTP error occurred.
HTTP Result Code: 200
Debugging information follows.
Path: /system/ajax
StatusText: OK
ResponseText:
Fatal error: Unsupported operand types in /home/pcsuppor/public_html/sites/all/modules/amazon_store/amazon_store.module on line 576
Try for yourself at www.pc-support.uk.com
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | amazon_store_fixcartcreds.patch | 861 bytes | pancho |
| #1 | Screen shot 2011-02-26 at 22.48.51.jpg | 290.36 KB | pcsupport |
Comments
Comment #1
pcsupport commentedScreenshot attached.
Comment #2
panchoSorry, this bug slipped in with #1068600: Remove MergeCart parameter and rewrite management of cart credentials. Thanks for bringing it up!
The attached one-liner should fix it though - does it work for you?
@Randy: It might be necessary to create a 7.x-1.0-beta3a release. Or should we collect some more fixes and followup with a beta4 in a week or two?
Comment #3
pcsupport commentedFixed - Perfect! :)
Thank you very much
Comment #4
panchoCommitted to 7.x-1: http://drupalcode.org/project/amazon_store.git/commit/9f81f9c
Comment #5
panchobetter title
Comment #6
rfay@Pancho, I agree we need a release pretty soon to fix this. Otherwise it will be reported again and again. Your call.
Comment #7
panchoSo I'm gonna release a new beta tonight. I might roll in one or two more fixes and some codestyle stuff, but you can bet I will doublecheck everything this time! :)
Comment #8
rfayFiguring out a way to do simpletest is the real answer to (most of) this kind of regression. I think that's a key step for Amazon and Amazon Store. #1007272: Tests for Amazon module
Thanks for the great work.
Comment #9
panchoYeah, tests is something we desperately need to avoid such stupid little regressions.
I think, the 7.x-2 branch should be equipped with tests from the very beginning.