The admin test form doesn't use a validation handler to check the entered asin. Therefore no form_set_error() is thrown on the asin field. Rather, a curious drupal_set_message of type "status" implies "Good news: The test failed."
I also refactored some code in order to improve the user experience a bit.
Finally, I separated out the lengthy error handling from _amazon_item_batch_lookup_from_web(), in order to keep the main functionality easy to grasp.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | amazon_test_form.patch | 5.73 KB | pancho |
| good_news_test_failed.png | 26.83 KB | pancho |
Comments
Comment #1
panchoCommitted to 7.x: http://drupal.org/cvs?commit=499458