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.

CommentFileSizeAuthor
#1 amazon_test_form.patch5.73 KBPancho
good_news_test_failed.png26.83 KBPancho
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pancho’s picture

Pancho
Location UTC+2 🇪🇺 EU
CreditAttribution: Pancho commented
Status: Active » Fixed
FileSize
amazon_test_form.patch5.73 KB

Committed to 7.x: http://drupal.org/cvs?commit=499458

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.