wrong shipping class allowed if over 5 items

Hi, @slservice33, sorry, I’m not sure I understand well… please, do the following:

1. Take screenshots of the shipping settings (the table rules), and share the images with any service like: https://imgbb.com/

2. Activate the log system (at bottom), go cart, force recalculate by adding/removing some product.

3. Go back to shipping class settings and open the new log. Paste it here.

Thanks,
Carles Martin
wp-centrics support team

• This reply was modified 3 years, 7 months ago by wp-centrics.

this log is for when its correct

– 5 items which are in a class which allow 1st class post to appear
– 1 item which is not in the class to allow 1st class post to appear

CORRECT = 1st class post does not appear as option

Starting Fish and Ships (free) calculation, for method: [1st class]. Instance_id: [3], Local time: [2021-07-08 01:02:16]
Fish and Ships version: [1.2.9], WP: [5.7.2], WC: [5.4.1], Multilingual: [NO], Multicurrency: [NO]
 
Cart contents:
– Ceramic Clipper Blade Sharpening (A5 Blades) (5)
– Lister Liberty LITHIUM Charger (1)
Rule calculations are folded. Click on each to see the details, or [open all]:
Rule #1
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0], max_comp: [less], max: [0.1]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 5) accumulated: 0.495
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.695
    Currently matching products (accumulated checkings result):
      all > > items: 0
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
  – No product matches for this rule
Rule #2
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.1], max_comp: [less], max: [0.25]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 5) accumulated: 0.495
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.695
    Currently matching products (accumulated checkings result):
      all > > items: 0
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
  – No product matches for this rule
Rule #3
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.25], max_comp: [less], max: [0.5]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 5) accumulated: 0.495
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.695
    Currently matching products (accumulated checkings result):
      all > > items: 0
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
  – No product matches for this rule
Rule #4
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.5], max_comp: [less], max: [1]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 5) accumulated: 0.495
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.695
    Currently matching products (accumulated checkings result):
      all > > items: 2
        . Ceramic Clipper Blade Sharpening (A5 Blades) (5)
        . Lister Liberty LITHIUM Charger (1)
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
      . Ungrouped, checking: #3506 Ceramic Clipper Blade Sharpening (A5 Blades), result: [TRUE]
      . Ungrouped, checking: #1093 Lister Liberty LITHIUM Charger, result: [FALSE]
    Currently matching products (accumulated checkings result):
      all > > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (5)
      Non-grouped > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (5)
    All match checking must be reevaluated for rule #4
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.5], max_comp: [less], max: [1]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 5) accumulated: 0.495
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
  – No product matches for this rule
 
Free shipping not allowed for this method.
Method not applicable
Usage on calculation: Memory: [21.4KB], DB queries: [0], Time elapsed: [0.0014 sec.]

screen shot of 1st class shipping rule: https://ibb.co/MSvwFRT

screen shot of basket with

1st class does not appear as option – https://ibb.co/s1jL56R
5 items shipping rule allowed
1 item shipping rule not allowed

1st class does appear as option – https://ibb.co/bPV8NYQ
6 item shipping rule allowed
1 item shipping rule not allowed

In rule #4 when there is 5 items allowed and 1 item not allowed
– No product matches for this rule`

In rule #4 when there is 6 items allowed and 1 item not allowed
– Calculated rule #4 cost: ￡3.70

Thanks for your help ??

Hi, one question:

If there are on cart one or more products NOT in shipping class “Large letter size” the shipping method must be not offered?

If this is how you expect to work, simply put a rule on the top:

WHERE products NOT IN shipping class “Large letter size” => add a special action “Abort shipping method”. You can leave price as 0 (never will be applied).

…and remove the shipping class filtering for the rest of rules

=====

However, seems there is some issue on Fish and Ships calculation. Please, can do you add also the log of the bad calculation?

It help us a lot to make a better plugin ??

Thanks,
Carles.

• This reply was modified 3 years, 7 months ago by wp-centrics.

Hi, thats great I’ve done that.

There does seem to be some issues when higher numbers of different classes are in the basket.

But overall its a super plugin for woocommerce and very good for calculating accurate postage costs. Even if the wrong option does appear its only ￡1 less so. no big deal.

Thanks for your help ??

the log when the postage is wrong is below, looks like something goes wrong in rule 4:

Starting Fish and Ships (free) calculation, for method: [1st class]. Instance_id: [3], Local time: [2021-07-08 01:11:10]
Fish and Ships version: [1.2.9], WP: [5.7.2], WC: [5.4.1], Multilingual: [NO], Multicurrency: [NO]
 
Cart contents:
– Ceramic Clipper Blade Sharpening (A5 Blades) (6)
– Lister Liberty LITHIUM Charger (1)
Rule calculations are folded. Click on each to see the details, or [open all]:
Rule #1
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0], max_comp: [less], max: [0.1]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 6) accumulated: 0.594
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.794
    Currently matching products (accumulated checkings result):
      all > > items: 0
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
  – No product matches for this rule
Rule #2
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.1], max_comp: [less], max: [0.25]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 6) accumulated: 0.594
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.794
    Currently matching products (accumulated checkings result):
      all > > items: 0
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
  – No product matches for this rule
Rule #3
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.25], max_comp: [less], max: [0.5]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 6) accumulated: 0.594
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.794
    Currently matching products (accumulated checkings result):
      all > > items: 0
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 0
      Non-grouped > items: 0
  – No product matches for this rule
Rule #4
    Logical operator: [AND]
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.5], max_comp: [less], max: [1]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 6) accumulated: 0.594
        + #1093 Lister Liberty LITHIUM Charger (0.2 * 1) accumulated: 0.794
    Currently matching products (accumulated checkings result):
      all > > items: 2
        . Ceramic Clipper Blade Sharpening (A5 Blades) (6)
        . Lister Liberty LITHIUM Charger (1)
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
      . Ungrouped, checking: #3506 Ceramic Clipper Blade Sharpening (A5 Blades), result: [TRUE]
      . Ungrouped, checking: #1093 Lister Liberty LITHIUM Charger, result: [FALSE]
    Currently matching products (accumulated checkings result):
      all > > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (6)
      Non-grouped > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (6)
    All match checking must be reevaluated for rule #4
    Check matching selection. Method: [by-weight], Group-by: [all], min_comp: [ge], min: [0.5], max_comp: [less], max: [1]
      Grouped [all] calculating: [by-weight]
        + #3506 Ceramic Clipper Blade Sharpening (A5 Blades) (0.099 * 6) accumulated: 0.594
    Currently matching products (accumulated checkings result):
      all > > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (6)
      Non-grouped > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (6)
    Check matching selection. Method: [in-class], Group-by: [none] (This method can’t be grouped), classes: [175]
    Currently matching products (accumulated checkings result):
      all > > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (6)
      Non-grouped > items: 1
        . Ceramic Clipper Blade Sharpening (A5 Blades) (6)
  – Calculated rule #4 cost: ￡3.70
 
FINAL COST: ￡3.70 [non-taxable]
Usage on calculation: Memory: [24.7KB], DB queries: [0], Time elapsed: [0.0016 sec.]