Closed (fixed)
Project:
Amazon Product Advertisement API
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Feb 2011 at 00:37 UTC
Updated:
28 Feb 2011 at 03:31 UTC
Some things are wrong with the asin field:
When an invalid ASIN is given, the following errors are thrown:
"Warning: htmlspecialchars() expects parameter 1 to be string, array given in check_plain() (line 1476 of /var/www/includes/bootstrap.inc).
No Amazon product with the ASIN "" could be located."
That's not exactly what we want.
Reason is that _asin_load_items() takes $items by reference and cryptically manipulates on it, even though it returns everything needed. As this helper function is being called only once and is overly abstracted, I removed it altogether.
Also, the form_set_error on the ASIN field does not work.
Finally, I refactored asin_field_formatter_view
| Comment | File | Size | Author |
|---|---|---|---|
| asin_fix_lookup.patch | 2.89 KB | pancho | |
| asin_field_error.png | 38.34 KB | pancho |
Comments
Comment #1
panchoCommitted to 7.x: http://drupal.org/cvs?commit=499450
Comment #2
rfaySo great for this module to be getting such excellent love.