It appears that the form is broken again. The code has this comment above at line 754
// This is a total hack to work around changes made in the return value
// from uc_shipping_address_form(). That function needs to be fixed, but
// until then this should do the trick.

Then, this is the error message that comes up when I try to review shipment.
The page I am on is: admin/store/orders/4/ship/fedex_ground/1

Notice: Undefined index: phone in uc_fedex_fulfill_order_validate() (line 758 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 758 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: first_name in uc_fedex_fulfill_order_validate() (line 759 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 759 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: last_name in uc_fedex_fulfill_order_validate() (line 760 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 760 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: company in uc_fedex_fulfill_order_validate() (line 761 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 761 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: street1 in uc_fedex_fulfill_order_validate() (line 762 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 762 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: street2 in uc_fedex_fulfill_order_validate() (line 763 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 763 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: city in uc_fedex_fulfill_order_validate() (line 764 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 764 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: zone in uc_fedex_fulfill_order_validate() (line 765 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 765 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: country in uc_fedex_fulfill_order_validate() (line 766 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 766 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined index: postal_code in uc_fedex_fulfill_order_validate() (line 767 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 767 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_fulfill_order_validate() (line 768 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 38 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Warning: Attempt to assign property of non-object in uc_fedex_shipment_request() (line 39 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 101 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 107 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 108 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 112 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 113 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 114 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 115 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_shipment_request() (line 160 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Error in processing FedEx Shipping Request.
ERROR
ship
2471
shipper is required
shipper is required
shipper is required
Notice: Undefined property: stdClass::$CompletedShipmentDetail in uc_fedex_fulfill_order_validate() (line 814 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 814 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 814 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 819 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 831 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined property: stdClass::$CompletedShipmentDetail in uc_fedex_fulfill_order_validate() (line 835 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 835 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 840 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 854 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 857 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 858 of /usr/home/sitebase/www/sites/all/modules/uc_fedex/uc_fedex.ship.inc).

CommentFileSizeAuthor
#11 correct_origin_address-1268892-11.patch2.04 KBjerry

Comments

diggingrelic’s picture

diggingrelic commented

I wanted to add that I am using UberCart version 7.x-3.0-beta3 - I am going to update to version 7.x.3.0-beta4 and will update the status here.

diggingrelic’s picture

diggingrelic commented

Well, 7.x.3.0-beta4 does work however there are still some notice errors. Although, the module seems to function in the sense that you can get quotes and create shipping labels. I have not yet tested the address verification portion of the module. So, if I should post the new notice errors here, please comment and I will gladly do so.

tr’s picture

tr commented
Version: 7.x-2.0 » 7.x-2.x-dev
Priority: Major » Minor

If you fill out all the fields on the shipping form, no notices will be generated in the 7.x-2.x-dev version of this module running with Ubercart 7.x-3.x-dev. Those issues were fixed several months ago. The only remaining problem that I know about is that some of these notices will show if you fail to fill out one of the required fields on the shipping form. That is a problem with uc_shipping, as it does not properly designate required fields, allowing empty data to get through to uc_fedex rather than getting caught in the form validation stage.

rcharamella’s picture

rcharamella commented
Priority: Minor » Major

I'm getting an Authentication error as well errors with uc_fedex.ship.inc. All the uc_fedex.ship.inc errors seem to be form validate errors. I've filled in all fields on the form.

The front end quote system works fine with FedEx, no Authentication errors. Is this a bug?

Any help would be greatly appreciated as this site is supposed to launch on Monday.

Thanks in advance for your help

Here's the full error message:

Error in processing FedEx Shipping Request.
ERROR
prof
1000
Authentication Failed
Authentication Failed
Notice: Undefined property: stdClass::$CompletedShipmentDetail in uc_fedex_fulfill_order_validate() (line 814 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 814 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 814 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 819 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 831 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Undefined property: stdClass::$CompletedShipmentDetail in uc_fedex_fulfill_order_validate() (line 835 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 835 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 840 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 854 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 857 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).
Notice: Trying to get property of non-object in uc_fedex_fulfill_order_validate() (line 858 of /home/raretear/public_html/sites/all/modules/uc_fedex/uc_fedex.ship.inc).

tr’s picture

tr commented
Priority: Major » Minor

Authentication error means you don't have permission from FedEx to generate labels.

diggingrelic’s picture

diggingrelic commented
Status: Active » Closed (works as designed)
orangense’s picture

orangense commented
Status: Closed (works as designed) » Active

I am getting this exact same error, but I have followed the recommended solution in #4 by filling out all the fields and using the most recent dev version of each module. Does anyone have an idea on how I can fix this problem?

orangense’s picture

orangense commented

I was able to fix my problem by changing some of the key indexes. The actual $form_state values did not match the ones being referenced in uc_fedex.ship.inc on lines 748-762. Below is the change I made:

Before:

  // This is a total hack to work around changes made in the return value
  // from uc_shipping_address_form().  That function needs to be fixed, but
  // until then this should do the trick.
  $origin        = $form_state['values']['pickup_address'];
  $origin->phone = $form_state['values']['phone'];
  $origin->first_name = $form_state['values']['first_name'];
  $origin->last_name = $form_state['values']['last_name'];
  $origin->company = $form_state['values']['company'];
  $origin->street1 = $form_state['values']['street1'];
  $origin->street2 = $form_state['values']['street2'];
  $origin->city = $form_state['values']['city'];
  $origin->zone = $form_state['values']['zone'];
  $origin->country = $form_state['values']['country'];
  $origin->postal_code = $form_state['values']['postal_code'];
  $origin->email = $form_state['values']['pickup_email'];

After:

  // This is a total hack to work around changes made in the return value
  // from uc_shipping_address_form().  That function needs to be fixed, but
  // until then this should do the trick.
  $origin        = $form_state['values']['pickup_address']['pickup_address'];
  $origin->phone = $form_state['values']['pickup_address']['phone'];
  $origin->first_name = $form_state['values']['pickup_address']['first_name'];
  $origin->last_name = $form_state['values']['pickup_address']['last_name'];
  $origin->company = $form_state['values']['pickup_address']['company'];
  $origin->street1 = $form_state['values']['pickup_address']['street1'];
  $origin->street2 = $form_state['values']['pickup_address']['street2'];
  $origin->city = $form_state['values']['pickup_address']['city'];
  $origin->zone = $form_state['values']['pickup_address']['zone'];
  $origin->country = $form_state['values']['pickup_address']['country'];
  $origin->postal_code = $form_state['values']['pickup_address']['postal_code'];
  $origin->email = $form_state['values']['pickup_email'];
orangense’s picture

orangense commented
Issue summary: View changes

Spelling mistake

jerry’s picture

jerry commented
Related issues: +#1930528: UPS shipping fails with ShipFrom error

I'm seeing the same problems with address fields as above, very like this still-unfixed issue in the UPS shipping module. Fix is also the same.

jerry’s picture

jerry commented
Status: Active » Needs review
StatusFileSize
new correct_origin_address-1268892-11.patch2.04 KB

Patch for the change in the origin address fields attached.