Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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