Closed (duplicate)
Project:
Ubercart
Version:
7.x-3.1
Component:
Shipping
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Aug 2012 at 10:38 UTC
Updated:
16 Oct 2013 at 19:00 UTC
The uc_ups_fulfill_order_validate() function uses $response_obj->data without checking whether the data property is set. It would be better to check it and show an error message if there is no data (for example in case of network errors).
$response_obj = drupal_http_request(variable_get('uc_ups_connection_address', ... );
$response = new SimpleXMLElement($response_obj->data);
Comments
Comment #1
gabor_h commentedSame happens in uc_ups_confirm_shipment_submit()
Comment #2
longwaveDuplicate of #2109531: Shipping quotes fail without error handling which has a patch for USPS at least.