Home » Forums » Ubercart » Bug Reports

Shipping weight ignored on checkout page

Submitted by zmove on Thu, 11/22/2007 - 05:39
zmove
Posts: 554
 Joined: 08/13/2007
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer

Hi there,

I noticed a very minor problem related to shipping module.

The weight is ignored. In my case, I have the worldquote module and the store pickup module.

In the shipping quote setting I set -2 to worldquote and +2 to in-store pickup.

In the checkout page, the in-store pickup is still the first. I tried to invert the weight, no change...

This is a minor problem, but as lot of people don't read and take care of checkout pane, I'm afraid to see a lot of commande with the in-store pickup selected whereas the customer don't comes to check his product...
permalink #1 Thu, 11/22/2007 - 06:04
jorditr
Posts: 108
 Joined: 10/31/2007
Getting busy with the Ubercode.

There are also some issues with the flatrate calculation of the shipping cost which makes me think that the whole final step shipment cost could have some trouble.

permalink #2 Fri, 11/23/2007 - 16:53
zmove
Posts: 554
 Joined: 08/13/2007
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer

No feedback about that ? it is a bug of my installation or the weight defined in admin shipping quote setting are ignored on checkout page ?

permalink #3 Sat, 11/24/2007 - 08:27
jorditr
Posts: 108
 Joined: 10/31/2007
Getting busy with the Ubercode.

This is really important and a serious problem. I've been digging with the flatrate shipping module and it doesn't add the cost for the order in any way, it just calculates the cost for each product. I need to know how all this works to develop a module for my local courier companies.

I been looking carefully the uc_flatrate_shipping_method function and looking at the "hook_shipping_method" page on the development area on this site, and also comparing this function to the equivalent one on the uc_ups.module I finally tried to simplify it to have the most clear model, which I think is the next one:

<? php
function uc_flatrate_shipping_method(){
  $methods = array();
 
  $methods['flatrate'] = array(
    'id' => 'flatrate',
    'module' => 'uc_flatrate',
    'title' => t('Flat Rate per Product'),
    'enabled' => true,
    'quote' => array(
      'type' => 'small_package',
      'callback' => 'uc_flatrate_quote',
      'accessorials' => array(
        t('Shipping Rate'),
      ),
    ),
    'ship' => array(
      'type' => 'small_package',
      'callback' => 'uc_flatrate_quote_order',
    ),
    'weight' => 0,
  );
 
  return $methods;
}
?>

If that's true something is broken on the module uc_quote. There's no way to callback the uc_flatrate_quote_order function. The code here is too simple, this array is not so complex, so the problem must be on the uc_quote module, but it's too complex to me to see where it could be.

It's hard to believe that this is working on any real site without modification and we need a solution to start ours shops. Please, Lyle, Ryan, any idea?

permalink #4 Sat, 11/24/2007 - 08:46
jorditr
Posts: 108
 Joined: 10/31/2007
Getting busy with the Ubercode.

Could it be related to the uc_flatrate_configuration() function and maybe to not having proper or enough workflow_ng_use_action calls?

permalink #5 Sat, 11/24/2007 - 08:57
zmove
Posts: 554
 Joined: 08/13/2007
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer

I tried to change the weight of the worldquote shipping configuration because I didn't think it could be their weight that defined the order you see the shipping quote on checkout . But without success...

permalink #6 Mon, 11/26/2007 - 04:24
jorditr
Posts: 108
 Joined: 10/31/2007
Getting busy with the Ubercode.

(edited to not confuse things)

permalink #7 Mon, 11/26/2007 - 03:56
zmove
Posts: 554
 Joined: 08/13/2007
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer

I don't understand why you speak about flatrate calculation problems on this post, it's not his aim and it confuse post reader.

The problem in this issue is that the weight of the shipping method is not considered in the checkout page. I don't use the flatrate module so it's not a flatrate issue.

I still have the problem of the weight, which is very annoying because the store pickup is displayed first, and Most people choose this option and ask why, few days after, the product is not shipped....

So I really need an idea how to fix that, I tried to change weight in Workflow, shipping management, packaging..... no one fix this problem, in-store shipping cost is still displayed first...

permalink #8 Mon, 11/26/2007 - 04:23
jorditr
Posts: 108
 Joined: 10/31/2007
Getting busy with the Ubercode.

You're right, I'm very sorry zmove, I just thought that the problem was that the quote calculation fails at some point which maybe is similar and I tried to add more information because I also want that part fixed.

permalink #9 Mon, 11/26/2007 - 04:44
zmove
Posts: 554
 Joined: 08/13/2007
Bug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Internationalizationizer

Ok, It seems to be a storepickup problems that create a new shipping type.

In fact, I don't understand the interest to have different shipping type... (small package etc...)