Closed (fixed)
Project:
Commerce Physical Product
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
17 Nov 2011 at 16:56 UTC
Updated:
13 Dec 2014 at 01:21 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rupertj commentedI've just noticed that some functions in the version of commerce_physical I just pulled down duplicate some code I added, but here's the patch anyway :)
I'll give it a bit of a rework so there's no duplicated functionality, but I'd still be interested in what people think otherwise.
Comment #2
rupertj commentedNew patch. This one's better, in that it uses the existing functions in commerce_physical, like commerce_physical_order_weight(), and adds some corresponding functions for getting dimension info.
Functionality-wise, the rules conditions are the same as before.
Comment #3
ñull commentedUnfortunately I don't see the new conditions appear. What could be the cause? Can you give me more information about the environment so that I might reproduce
Comment #4
kiwimind commented#2 worked great for me. Might be an idea to include this in the module.
I'm working on a sliding scale of shipping by weight.
My 0-500g rule has a single condition set - weight < 0.5 kg.
My 500-1000g rule has 2 conditions: weight < 1kg and NOT weight < 0.5kg.
Works a treat.
My only suggestion would be to allow users to select which unit they wanted the value to be in. It seems set at kg, unless that's picking up on units I've got set in my store.
Thanks Ru!
Comment #5
webroru commented+1
and #3 +1
can anybody give more info about calculating total weight?
Comment #6
kiwimind commentedThe total weight is calculated when you use the condition of Commerce Physical > Order's total weight is under a chosen limit.
Comment #7
webroru commentedBut I have no condition of Commerce Physical in the Rules. Maybe I'm wrong to apply a patch? I attached the patched file. Please check whether all okay with it.
When I apply patch #2 it display this notice:
Perhaps the problem is that I do not have this file - commerce_physical.rules.inc?
Comment #8
webroru commentedOk. I understand. I should manualy create this file.
Comment #9
webroru commentedDoes anyone know how to get a variable with a value weight of the order?
Comment #10
Jason Dean commentedrupertj you are a star - this works great :)
thanks!
Comment #11
rogermcallen commentedSolved! I have to set up the rules in the "configure component" of each shipping services! Now seems to work!
The rules will be:
Rule 1: order total weight < 1kg
Rule 2: order total weight < 10kg, AND, NOT order total weight < 1.1kg
Thanks a lot again for the patch!
Comment #12
ñull commentedJust reporting that patch works like promised. the rules.inc file should be in the same folder and because of faulty patching it ended up in a folder called b. The conditions became available and are sufficient for their purpose.
When can we consider this "reviewed and tested"?
Comment #13
Patricec commentedPatch #2 works great for me. Many thanks.
Comment #14
rupertj commentedLet's see if we can get Ryan's attention ;)
Comment #15
ñull commentedbump,
When is this going to be at least submitted to dev?
Comment #16
savingstrangers commentedPatch worked PERFECTLY for me, thanks!
Anyone have a decent method of converting the kg to ounces? Any makeshift way of doing this would be great, my products are in ounces in my store, so I created ranges of shipping flat rate prices, and it's not the most friendly thing to convert my ranges to kilograms rather than convert the patch to be using ounces. Thanks!
Comment #17
duaelfrHi !
#2 seems to work perfectly but here is a new patch which :
[Take care ! The rules created with the previous patch won't continue to work with this one.]
becomes
Comment #18
duaelfrWow !
Someone told me that a big part of the patch was missing ! (enjoy git diff...)
Here is the complete one.
Enjoy
Comment #19
poniesCommerce ups calls commerce_physical_order_volume. I added it back in using commerce_physical_order_dimensions to get the dimensions and sanitize the units then returned the $volume array.
Comment #20
bradhawkins commented@ponies can you explain a bit more? I follow what you're saying except for the implementation. Did you alter commerce_ups or the above patch?
Comment #21
bradhawkins commentedAlso, I'm just wondering why was the volume function removed in the first place? Since Commerce UPS needs it to work properly and I'm guessing there's a decent number of users for UPS.
Comment #22
Anonymous (not verified) commentedI can confirm that #18 works - thanks very much!
My feedback would be that the naming is slightly misleading: calling the condition "Order's total weight is under a chosen limit" makes it seem like the comparison would be "less than" whereas actually it's "less than or equal to".
Also it is slightly awkward to implement weight bands e.g. 0-10kg, 10-20kg etc, as you have to do 2 conditions: one for "NOT total weight is less than 10kg" and "total weight is less than 20kg". Now I don't know much about Rules, but would it be possible to instead let the user do a normal "Data comparison" condition, and feed in the order total weight as the input variable? That would make it much simpler for the user.
Comment #23
muhammad.tanweer commentedHello DuaelFr,
Thanks for the great patch. As mentioned, it works for several people, however, I applied the latest complete patch to commerce_physical module, but still I am unable to see the rules here : http://mywebsite.com/admin/config/workflow/rules
I even uninstalled and re-installed the module, cleared cache, did everything that I understood should work but still no clue. I am stuck, can you please give me a few lines of quick help? Would be appreciated.
Thanks
Comment #24
duaelfrThis patch provide Rules conditions which are in the Components tab.
They are usable in common rules but the patch does not provide any of them.
Comment #25
muhammad.tanweer commentedThanks for responding back. However, I am still unclear on the use of it. Here is the scenario I want to accomplish. Can you please give me some instructions? Would be appreciated
I have a boolean field in the commerce product types named "non shippable" . I want that if the boolean field is checked (i.e. its value is 1) , it will mean that the item is non-shippable. Now, if an order contains all the products which are non-shippable, I need to skip the shipping step. Can you please instruct me specifically about this? Would be highly appreciated.
Muhammad.
Comment #26
duaelfrYour use case is a bit complicated. By keeping your boolean field you will have to build a really complex rule to meet your needs.
However, Commerce Physical, with this patch, gives you a new condition which name is "Order is shippable". An order is shippable when the sum of its weights or dimensions is not null.
Comment #27
andyf commentedI needed to check if the weight was less than or equal to a value, which isn't possible with #18. The attached patch is #18 with an extra parameter for the total weight and maximum dimension conditions that lets you specify the operator to use (eg. <=). I'm afraid I only tested the weight condition (that's the one I need), but I also modified the dimension condition as I presume someone at some point might find that useful.
Thanks
Comment #28
ponies@bradhawkins: Sorry it took me a while to get back to this. I lost my machine and the patch along with it. I'm rerolling now.
Here's the error I'm getting with #18
Call to undefined function commerce_physical_order_volume() in /var/www/drupal/sites/all/modules/commerce_ups/commerce_ups.xml.inc on line 31
Comment #29
poniesHere's #18 with commerce_physical_order_volume() added back in.
Comment #30
poniesDisregard #29. This one did the trick for me. There was a dimensional misalignment with the $dimension array.
Comment #31
andyf commentedImho #30 should be rerolled with the operator changes from #27. This is in line with other Commerce Rules, eg. Order contains a particular product, Total product quantity comparison, Order balance comparison.
Comment #32
nandotinoco commented#18 worked a treat for me. Looking forward to see this patch rolled out. Thanks
Comment #33
la.carvalho commentedCould someone please give details on how to work with this? I'm quite new to Drupal and can't seem to understand how this works...
I have a shipping service setup and want to be able to set different prices depending on weight ranges. How can I do that?
Thanks a lot
Comment #34
muhammad.tanweer commentedYour scenario is pretty straight forward. You ll need to create a rule and add the case in it to check order's weight. You can set different shipping price depending on weight.
Muhammad.
Comment #35
scarr commentedmuhammad.tanweer: did you get your situation in #25 figured out, or did you just use the "Order is shippable" suggested by #26? I have a similar situation using a boolean to indicate whether an item ships freight, and am wondering what your rules solution was if you did go that route?
Thank you!
Steve
Comment #36
tbenice commentedhad some trouble applying #30, #29 applied and I changed the 1 line 29->30. FYI.
Comment #37
tbenice commentedHey, can we make this feature compatible with commerce_product_bundle? It would just be a couple more lines to dig into bundle line items....
I'm going to make it so on my version and am happy to contrib back if you like the idea.
Comment #38
ioanmar commented1. I had installed physical module. Then created a weight field for my product type and then I also installed commerce_physical. To begin with, should I recreate the weight product type field?
2. I have applied the #18 patch succesfully and I can see these three new rules in the conditions selection list, under commerce physical. I have created some test products with a value in their weight field. Here's is what I want to succeed:
I have a flat rate for shippable products and I want to charge extra shipping costs to customers according to order's total weight (note that some products won't have anything in the weight field and in this case I don't want to take them into consideration for the shipping cost calculation).
I want to charge €3 (flat rate's base charge) if the order is 0-2 kilos (that happens already with that flat rate above).
BUT: In case the total order's weight is more than 2 kilos, I want to add €1.2 to shipping cost for every additional kilo.
e.g. order has 3 products:
> Product 1 - 2 kilos
> Product 2 - 0 kilos (or no value in weight field)
> Product 3 - 2.5 kilos
>> Total order's weight: 2+0+2.5 = 4.5 kilos (ie 2.5 kilos to be charged extra)
So the shipping charge should be €3 (flat rate's base) + €1.2*2.5 = €6.
I would really appreciate any suggestions on this.
Comment #39
duaelfrI think the simplest way to acheive this is to create one flat rate per weight slice and to select the good rate using the given conditions.
For this module to be complete we should add attributes to the order for the total weight and the maximum size which could be used as a token to make some calculation as you need. If someone have some time to spend on it, I think it would be really great.
[Note for the one who will do this : please take the time to merge the patches #30 and #27, thanks]
Comment #40
ioanmar commentedThank you for your quich response.
Do you suggest to create a flat rate for order weight < 2 kilos and additional ones for total order weight
- <3 ¬ <2
- <4 ¬ <3
- <5 ¬ <4
etc etc ??
That would work well but I need too many flat rates for all these cases. To play it safe, I could create let's say 10 of these flat rates but customers could order even heavier stuff from my store from time to time...
Isn't there a way to have the 1st rule for order weight <2 kilos and as weight increases, multiply the order weight to €1.20 for every extra kilo?
Comment #41
duaelfrI think you currently cannot find a better solution.
With a bit of code you could have a token for the order's total weight which could be used in a calculation rule but not with the actual patch.
Comment #42
ioanmar commentedI created these flat-rate & rules pairs but ALL flat rates are available to customer during checkout (also see #11 by rogermcallen).
I got it finally - the correct way to implement this is NOT to create new calculation rules for each flat rate but to edit each flat rate's component by adding the proper condition for order's weight.
Comment #43
muhammad.tanweer commentedInfact I used the way that if an order's weight is 0, its shipping is skipped. Otherwise, shipping is applied. My requirement was fulfilled with that.
Comment #44
tbenice commentedneed to change line 320 from:
to:
Comment #45
divan commentedError: Unknown condition commerce_physical_rules_order_is_under_weight_limit
Everything worked perfectly ... but today this error here :(
I tried to reinstall the module and put the patch but nothing works anymore.
This happened after update Flat Rate module ... any ideas?
Can anyone give me scheme commerce_flat_rate_service table from the database?
Comment #46
divan commentedError: Unknown condition commerce_physical_rules_order_is_under_weight_limit
Comment #47
star-szr@divan - Please don't change the category or priority, the issue was correctly filed before.
Comment #48
rupertj commentedIt'd be nice to get this issue back on track actually - it's looking slightly at risk of descending into nothing but support requests.
Which patch are people having success with? If we could get it committed, then people could file their own issues for support. If someone who's set this up recently could submit a re-rolled patch with any of the mentioned tweaks, that'd be even better.
Comment #49
divan commentedComment #50
divan commentedCan someone explain everything in one post step by step? Please. This will help all people who have a problem with this module.
... or if someone can put a module whith patch on zip file?
Comment #51
ponies#30 works for me.
Comment #52
divan commentedponies How? I already try the third day ... in components of the Rule does not appear Order's total weight is under a chosen limit ... Please put your folder commerce_physical in a zip to check where my error
Comment #53
joecanti commentedHi all,
#27 works great for me - I suggest it gets committed as I think it covers most possible options.
Thanks everyone! Joe
Comment #54
joecanti commentedOk step by step:
- Download the module. Download the patch file #27.
- Use a program to patch the files. the .module file gets changed, and commerce_physical.rules.inc gets created.
- Upload. overwrite. Clear all caches.
- Now add a flat rate shipping service. Give a name and save.
- Edit the component, and add a condition. You can now choose a weight condition (under a new section called 'commerce physical'.
- To show this shipping option for weights under 5kg, just choose >, and then add 5 as a weight.
- To show this shipping option for weights between 5 and 10, first add a condition for '>5', then add an 'AND' condition, then add a final condition for '<10' - this gives you a weight band between 5 and 10.
- Now add a new weight field to your products - edit some products and add the weight values in - and now test by adding to your basket, and proceeding to the checkout.
Hope this helps!
(ps patching is sometimes annoying - attached is the .module file and the new commerce_physical.rules.inc - put them both in the module folder after renaming to look like: (This is for #27)
commerce_physical.module
commerce_physical.rules.inc
Joe
Comment #55
shaneonabike commentedI can confirm that #27 works for me :)
Comment #56
shaneonabike commentedI started experiencing some issues where the shipping page was not being skipped and throwing errors. I realized that was a result of the physical weight now being zero and not being passed properly into the physical weight conversion. Patched that here: #1887116: Physical Weight conversion
Comment #57
torgospizzaI'd also like to see #30 and #27 combined and re-rolled; they overlap in too many places to make a clean patch possible, but at this point it looks like #27 is the preferred patch.
Can someone from the Commerce team look at this and share their thoughts? This is our #1 requirement for our store (99.9% of our products are downloads) and so this is fundamental for us.
Setting to "Needs review" for now, so someone (maybe @rszrama or @googletorp?) can take a look, but NB that this should probably be "Needs work".
Thanks!
Comment #58
nodecode commentedJust checking if anyone has managed to combine #27 and #30 as seems to be the consensus? I agree it looks pretty messy of a job and fear this is where the issue may stall. IMHO the rules features introduced here will go a long way toward resolving a number of deficiencies with commerce shipping.
Comment #59
svouthi commentedThank you, joecanti, for posting the files. I haven't any experience with patches yet (though I'm sure I will get some), so I appreciated having quick access to this feature. However, after loading the files, I found that commerce_physical.module did not play nicely with my UPS shipping setup, which threw errors (I'm running Commerce Physical v.7.x-1.x-dev and Commerce UPS v.7.x-2.0.) So, I switched back to my original commerce_physical.module file (as if I just added commerce_physical.rules_.inc to my setup) and all seems fine. Will see how this goes. Thanks again!
Comment #60
DrMicky commented#27 Works great for me too thanks.
When I applied I got the following result (not sure if it's important but thought I should report it)
Comment #61
chrisNtampa commented#30 worked for my project and was critical in making the client happy. Thanks to everyone that contributed.
Comment #62
summit commentedHi, will #27 in combination with #30 be committed please?
I do not know what to do to get this working?
Greetings, Martijn
Comment #63
poniesI'll see if I can combine the two this week.
Comment #64
zmove commentedSubscribing and I hope it will be commited soon as it's a so basic function to test order total weight to calculate shipping and this is so complicated ATM with rules, loops and components.
It should be as easy as ubercart that provided a weight condition to test without the necessity to create loop, components etc...
Comment #65
summit commentedIt would be great if also Dimensions would be implemented in the rules. Lots of shipping prices consists of weight and dimensions volume...
greetings, Martijn
Comment #66
deggertsen commentedI have attempted to combine the patch in #27 and #30 in this patch. I have not yet tested so someone might want to double check to see if I missed anything.
The main difference I was able to find in #27 to what is in #30 is in commerce_physical.rules.inc.
#30 also brought back the commerce_physical_order_volume() function.
I discarded the rules in #30 commerce_physical_rules_order_is_under_weight_limit and commerce_physical_rules_order_maximum_dimension_is_under_size in favor of what appear to be better rules in #27 commerce_physical_rules_order_weight_comparison and commerce_physical_rules_order_max_dimension_comparison.
Most everything else appeared to be the same.
Comment #67
calmforce commentedThere is a bug in the patch #66: the function commerce_physical_rules_order_max_dimension_comparison() uses unidentified variable $limit. It should use $value provided as a function argument.
Comment #68
deggertsen commentedGood catch. Should be fixed now in this patch version.
Comment #69
deggertsen commentedI've been testing this on a few sites including a production site for the last month (almost) and I have had no problems. Moving to RTBC.
Comment #70
Tor Arne Thune commentedI've had this running on a production environment for about 3 weeks now so, +1 to the RTBC of #68.
Comment #71
torgospizzaWould love to see the patch committed as well.
Failing that, perhaps the Rules could be added to Commerce Rules Extra?
Comment #72
Anonymous (not verified) commented+1 for me too, can someone submit this patch please? Thank you so much!
Comment #73
blueblade commentedFor those who are trying to get total weight, please see this post http://www.drupalcommerce.org/discussions/817/shipping-prices-depending-.... Much appreciated if you can tell me what Fonant means in his post "This can be output using the Debug value action, if the Devel module is enabled." Everything looks fine in the rule he sets up but I just cant get the variable to show up at all....help, please.
Comment #74
deggertsen commented@torgosPizza, adding these Rules to Commerce Rules Extra may be good seeing as this module has not had a commit since October of 2011 and rszrama is next to impossible to reach.
Comment #75
deggertsen commentedRemoved some old patches from the issue display to make it more clear to those first visiting this issue.
Also, lets move any discussion about having these rules as part of Commerce Rules Extra to this issue: #2139413: commerce_physical Rules conditions
Comment #76
deggertsen commentedMore file cleanup...
Comment #77
blueblade commentedWhen I load "Maximum Product Dimension Comparison", I dont see any options that let me choose any of the dimensions, all I see is the Data Selector which I have tried many ways to access the width/length/height with no success, the comparison operator, value and dimension unit dropdown. Can someone please tell me how to specify the width/length/height to compare, please?
Comment #78
deggertsen commented@blueblade. Based on the function commerce_physical_rules_order_max_dimension_comparison() it appears that it is setup to check each dimension and use the largest dimension as the Maximum. So it will check the width, length, and height and compare it against what you put in as the value. If any of those do not match your value and operator it will return False. For example if you entered 5 for your value and < for your operator and the dimensions being compared are height:4, width:3, and length:5, it will return false because the length is not less than 5. Hopefully that makes sense.
So this works as designed. I think your concept probably makes more sense for this rule, but it would take some modification. Seeing as none of my clients need this rule I will probably not be the one to make those modifications.
Comment #79
blueblade commented@deggertsen Many thanks to your quick reply. I was just wondering how to get this figured out but now I think I can go to bed and have a good night sleep. Thanks so much!! Have a wonderful day yourself.
Comment #80
macman911 commentedI've applied patch 66 but I can't see "commerce physical" rules listed in the conditions lists. I've spent two days on it with no luck. I could really do with this feature working, so would greatly appreciate some suggestions.
Comment #81
deggertsen commentedHopefully you mean the patch in #68 as there was a problem with the one in #66. Be sure that you have cleared all your caches as the new rules conditions will not appear until you do.
Comment #82
macman911 commentedThank you for your fast reply @deggertsen - it finally worked, weight is working beautifully :) However, if I use the Maximum product dimension comparison I get this error: "Fatal error: Cannot use assign-op operators with overloaded objects nor string offsets in /home/public_html/profiles/commerce_kickstart/modules/contrib/physical/physical.module on line 855" and "Notice: Undefined variable: multiplier in physical_dimensions_convert() (line 855 of /home/public_html/profiles/commerce_kickstart/modules/contrib/physical/physical.module).".
Comment #83
deggertsen commented@macman911, could you post your rule here so that I can see what operator you are using and the context of the rule? I'm not sure I'll be able to help you with this one, but I'm willing to look at it.
Comment #84
deggertsen commentedComment #85
macman911 commentedThank you @deggertsen for the kind offer. Here is an export of the rule (I hope this is what you meant).
{ "commerce_shipping_service_royal_mail_signed_for_1st_class_" : {
"LABEL" : "Rate Royal Mail Signed For 1st Class - Small Parcel",
"PLUGIN" : "rule",
"REQUIRES" : [ "commerce_order", "commerce_physical", "commerce_shipping" ],
"USES VARIABLES" : { "commerce_order" : { "label" : "Order", "type" : "commerce_order" } },
"IF" : [
{ "commerce_order_compare_address" : {
"commerce_order" : [ "commerce_order" ],
"address_field" : "commerce_customer_shipping|commerce_customer_address",
"address_component" : "country",
"value" : "GB"
}
},
{ "commerce_physical_rules_order_weight_comparison" : {
"commerce_order" : [ "commerce_order" ],
"operator" : "\u003C",
"value" : "2",
"unit" : "kg"
}
},
{ "commerce_physical_rules_order_max_dimension_comparison" : {
"commerce_order" : [ "commerce_order" ],
"operator" : "\u003C",
"value" : "20",
"unit" : "cm"
}
}
],
"DO" : [
{ "commerce_shipping_service_rate_order" : {
"shipping_service_name" : "royal_mail_signed_for_1st_class_",
"commerce_order" : [ "commerce-order" ]
}
}
]
}
}
Comment #86
deggertsen commented@macman911, Sorry it took me so long to get back. Unfortunately, I can't see any problems with your rule and have no idea why you would be getting that error. Did you ever figure it out?
Comment #87
Open Pixel commentedHi all,
I wanted to thanks all contributors for this patch. I made some tests and it works perfectly ...
It's a pity that this kind of features aren't included in DC package.
Thanks for all
Comment #88
jpmelguizo commentedThanks for the patch. It was really usefull for an specific weight shipping tariff system that I needed for a shop.
The system itself had 5 different weight intervals with fixed shipping rates and then a price-per-kilo addition for everything above that. e.g.:
I added an action in order to get the total order weight so I could use it later in a calculation rule:
I'm sure there would be many different ways to implement what I needed, but this was the easiest for me. Also, having the total order weight as a variable might be useful for other cases too.
Comment #89
Exigo commented#68 worked for me. After several days of struggle and seaching of a solution I found it. If it had been commited to the module I would have solved my problems with shipping weight calculations so much easier. Many thanks to everyone who have contributed to this patch. If the original authors of the physical module do not have the time for review, should someone else take it over?
Comment #90
zmove commentedAny news about an officiel commit ? It's marked as reviewed and tested, and it's quite annoying now to take care of the module updates because you have to apply patch each time you update.
Comment #91
deggertsen commentedrszrama is probably the one who would need to commit this and he is one busy dude. Maybe someone would want to PM him and offer to co-maintain this module?
Comment #92
drupal4u.org commented@deggertsen: I can follow up to #68. Weights work perfectly. This saved my day :-)
Then I start with the dimensions. Before applying #68 I get the following error for dimensions which I think is „normal“, because #68 is not yet applied:
„Notice: Undefined variable: limit in commerce_physical_rules_order_max_dimension_comparison() (Zeile 113 von /home/httpd/vhosts/immodeal.ch/wood3/sites/all/modules/commerce_physical/commerce_physical.rules.inc).
After applying #68 I get the same fatal error as macman911 in #82:
Fatal error: Cannot use assign-op operators with overloaded objects nor string offsets in /home/httpd/vhosts/immodeal.ch/wood3/sites/all/modules/physical/physical.module on line 855
Comment #93
perignon commentedWould there be any interest in adding this to Commerce Rules Extra? Someone brought it up in the issue queue almost a year ago #2139413: commerce_physical Rules conditions
Comment #94
rcodinaThe patch on #68 doesn't work for me because new options doesn't appear for me. However if I take all the code inside commerce_physical.api.php and I put it adapted into a new module (inside .module file) it works like a charm. Maybe a new patch has to be done with a hook in .module file informing in this module there are plugins for rules module.
Comment #95
deggertsen commented@rcodina, perhaps you could help write a patch for #2139413: commerce_physical Rules conditions. I'm having a hard time finding time to do it.
Comment #96
rcodina@deggertsen I don't have much time right now. However I have uploaded what I've done here. Maybe helps someone to do the patch. I'm testing the code and probably commit fixes soon.
This may be interesting too for somebody.
Comment #97
das-peter commentedHi there,
these changes look really promising. As I've lately got maintainer permissions for this module I'll have look into this soon.
Comment #98
deggertsen commented@das-peter, that would be awesome. Let me know if you have any questions. I would way rather see these committed to Commerce Physical Product than to Commerce Rules Extra. I don't really like those "extra" modules when I can avoid them. I think the patch in #68 is ready to go. I've been using it for awhile now.
Comment #100
das-peter commentedHere we go.
Thanks to all that worked on this!
I just had some minor cleanups to do -> coding standards (I recommend using the CodeSniffer in the coder module).
The one change I didn't commit is this:
As much as I'm in favour of using the metric system by default, this would be an API change since it could break modules that use the function and simply expect "in" as unit.
Comment #101
deggertsen commentedThank you! So glad to have this committed!
Also thanks for the Code Sniffer recommendation. Some of us just don't know about these sorts of things yet so it's nice to be educated =).
Comment #103
lalbright commentedSo based on das-peter's comment #100, regarding the metric system, does that mean that this code now works for US Standard? Not metric? Or both?
Comment #104
perignon commentedRemember that the world over, Metric is the standard of measure. Even in the US the meter is the standard length of measure and has been since 1893. People in the US don't realize this fact that an inch is standardized against the meter.
Comment #105
Jeremy Scott commentedWhen I click on the commerce_physical_rules_conditions-1344962-68.patch in #68 it opens the file in another tab. What do I have to do to get this to work on my website. Any help will be greatly appreciated. Thanks.
Comment #106
torgospizza@Jeremy Scott, since this is listed as "committed" the easiest way would be to download the -dev version of the module.
Comment #107
Jeremy Scott commented@torgosPizza, Thanks for your response. That is the version I have installed. Could you tell me how to enable the shipping rate selection by weight?
Comment #108
torgospizzaIt should be part of Rules, available as a new Rules condition. I haven't used that one in particular yet, though, so I don't think I can be of much help there.
Comment #109
Jeremy Scott commentedOk, thank you very much. Maybe someone who has used it will be able to offer me some guidance.
Comment #110
Jeremy Scott commentedI found the answer in #54. Apparently I overlooked it the first time I read through the comments.